Re: [PATCH] staging: greybus: Fix line ending with a '('

["Frank A. Cancio Bello" <frank@generalsoftwareinc.com>]

On Sun, Oct 27, 2019 at 08:12:40AM +0100, Greg KH wrote:
> On Sun, Oct 27, 2019 at 06:33:38AM +0000, Frank A. Cancio Bello wrote:
> > checkpatch.pl message:
> > "CHECK:OPEN_ENDED_LINE: Lines should not end with a '('"
> > 
> > Signed-off-by: Frank A. Cancio Bello <frank@generalsoftwareinc.com>
> > ---
> >  .../staging/greybus/audio_manager_module.c    | 32 ++++++++-----------
> >  .../staging/greybus/audio_manager_private.h   |  7 ++--
> >  2 files changed, 16 insertions(+), 23 deletions(-)
> > 
> > diff --git a/drivers/staging/greybus/audio_manager_module.c b/drivers/staging/greybus/audio_manager_module.c
> > index 2bfd804183c4..1b52dd116e38 100644
> > --- a/drivers/staging/greybus/audio_manager_module.c
> > +++ b/drivers/staging/greybus/audio_manager_module.c
> > @@ -70,9 +70,8 @@ static void gb_audio_module_release(struct kobject *kobj)
> >  	kfree(module);
> >  }
> >  
> > -static ssize_t gb_audio_module_name_show(
> > -	struct gb_audio_manager_module *module,
> > -	struct gb_audio_manager_module_attribute *attr, char *buf)
> > +static ssize_t gb_audio_module_name_show(struct gb_audio_manager_module *module,
> > +		struct gb_audio_manager_module_attribute *attr, char *buf)
> 
> Same thing here, your line is too long now, it's fine as-is.
>

Thank you one more time for your quick response.

frank a.

> thanks,
> 
> greg k-h