Message ID | 1579145331-78633-2-git-send-email-zhangliguang@linux.alibaba.com (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
Series | [V2,1/3] firmware: arm_sdei: fix possible deadlock | expand |
Hi Luanshi, On 16/01/2020 03:28, luanshi wrote: > Remove one unnecessary white line. > > Signed-off-by: Liguang Zhang <zhangliguang@linux.alibaba.com> > --- > drivers/firmware/arm_sdei.c | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/drivers/firmware/arm_sdei.c b/drivers/firmware/arm_sdei.c > index 37e9bf0..f81c09e 100644 > --- a/drivers/firmware/arm_sdei.c > +++ b/drivers/firmware/arm_sdei.c > @@ -599,7 +599,6 @@ static int _sdei_event_register(struct sdei_event *event) > event->registered, > SDEI_EVENT_REGISTER_RM_ANY, 0); > > - > err = sdei_do_cross_call(_local_event_register, event); > if (err) { > spin_lock(&event->sdei_event_lock); I'm afraid these whitespace-only patches aren't worth sending. If its not caught at review, it gets to annoy the reader until someone can do a drive-by fix when they are changing adjacent code. I've merged this with the first patch in the eventual series. Thanks, James
diff --git a/drivers/firmware/arm_sdei.c b/drivers/firmware/arm_sdei.c index 37e9bf0..f81c09e 100644 --- a/drivers/firmware/arm_sdei.c +++ b/drivers/firmware/arm_sdei.c @@ -599,7 +599,6 @@ static int _sdei_event_register(struct sdei_event *event) event->registered, SDEI_EVENT_REGISTER_RM_ANY, 0); - err = sdei_do_cross_call(_local_event_register, event); if (err) { spin_lock(&event->sdei_event_lock);
Remove one unnecessary white line. Signed-off-by: Liguang Zhang <zhangliguang@linux.alibaba.com> --- drivers/firmware/arm_sdei.c | 1 - 1 file changed, 1 deletion(-)