Re: [PATCH] staging: rtl8712: Fix for long lines in Kconfig help

[Greg Kroah-Hartman <gregkh@linuxfoundation.org>]

On Thu, Mar 12, 2020 at 05:38:18PM +0530, R Veera Kumar wrote:
> On Thu, Mar 12, 2020 at 12:54:09PM +0100, Greg Kroah-Hartman wrote:
> > On Thu, Mar 12, 2020 at 01:40:55PM +0530, R Veera Kumar wrote:
> > > Fixes two long line in Kconfig help.
> > > Found using checkpatch.pl
> > > 
> > > Signed-off-by: R Veera Kumar <vkor@vkten.in>
> > > ---
> > >  drivers/staging/rtl8712/Kconfig | 7 +++++--
> > >  1 file changed, 5 insertions(+), 2 deletions(-)
> > > 
> > > diff --git a/drivers/staging/rtl8712/Kconfig b/drivers/staging/rtl8712/Kconfig
> > > index b6dcda77db1f..678bbeace823 100644
> > > --- a/drivers/staging/rtl8712/Kconfig
> > > +++ b/drivers/staging/rtl8712/Kconfig
> > > @@ -6,13 +6,16 @@ config R8712U
> > >  	select WEXT_PRIV
> > >  	select FW_LOADER
> > >  	help
> > > -	    This option adds the Realtek RTL8712 USB device such as the D-Link DWA-130.
> > > +	    This option adds the Realtek RTL8712 USB device such as the
> > > +	    D-Link DWA-130.
> > > +
> > >  	    If built as a module, it will be called r8712u.
> > >  
> > >  config R8712_TX_AGGR
> > >  	bool "Realtek RTL8712U Transmit Aggregation code"
> > >  	depends on R8712U && BROKEN
> > >  	help
> > > -	    This option provides transmit aggregation for the Realtek RTL8712 USB device.
> > > +	    This option provides transmit aggregation for the
> > > +	    Realtek RTL8712 USB device.
> > >  
> > 
> > Why did you pick two different line lengths to wrap at for these two
> > options?
> > 
> > Please be consistent.
> > 
> 
> So should I resend the patch.

I can not take it as-is, so why wouldn't you fix it up and resend it?
:)

thanks,

greg k-h