Re: [RESEND PATCH] staging: rtl8192u: Fix indentation for cleaner code

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Sep 24, 2019 at 8:47 PM Greg KH <gregkh@xxxxxxxxxxxxxxxxxxx> wrote:
>
> On Fri, Sep 13, 2019 at 11:31:01PM +0530, Sumera Priyadarsini wrote:
> > Fixes indentation for if condition in the file r8190_rtl8256.c for better readability as suggested by Dan Carpenter.
>
> Please wrap your lines at 72 columns.
>
>
I will keep this in mind. I was under the impression that the line
length must be 80 columns
but will make the change immediately. To be able to wrap the lines,
maybe code implementation
needs to be changed slightly as there is a lot of nesting present in
the current code?
>
>
> >
> > Reported-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx>
> > Signed-off-by: Sumera Priyadarsini <sylphrenadin@xxxxxxxxx>
> > ---
> >  drivers/staging/rtl8192u/r8190_rtl8256.c | 4 ++--
> >  1 file changed, 2 insertions(+), 2 deletions(-)
>
> Why was this resent?  What changed from the 1st one that caused this to
> be required to be resent?
>
> Now you owe me a v2 of this patch with the changelog text fixed up :)
>
> thanks,
>
> greg k-h

I actually did not know that the merge window was closed, and thought
this patch may have been
missed as I did not get any reply.  So I resent it as it was following
the instructions here(https://kernelnewbies.org/Outreachyfirstpatch)
Should I send a v2 with the changelog stating the same?

regards,
Sumera
_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel



[Index of Archives]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux