Re: [PATCH 1/3] staging: wlags49_h2: ap_h2: fixes coding style issues

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

 



Hi Greg,

Thanks for the feedback.  Lesson learned, will remember to do things
in smaller increments and also wrap lines better.

Kindly disregard all three patches.

I'll redo and resend soon :)

Johan

On 16 November 2012 19:05, Greg KH <gregkh@xxxxxxxxxxxxxxxxxxx> wrote:
> On Fri, Nov 16, 2012 at 06:52:39PM +0200, Johan Meiring wrote:
>> This corrects several instances of spaces before tabs and incorrect spacing around "*" characters.  It also fixes some 80+ character line length issues, but the majority of such cases have been left as is, as this was probably the author's intention.
>
> Please wrap your changelog lines at the properly length (i.e. 72
> columns)
>
> Also, please only do one thing per patch.  You are doing multiple things
> here, which makes it hard to review.
>
> Same goes for the other patches in this series, please break it up into
> smaller steps, which makes for more patches, but it makes it easier to
> review to ensure you didn't do something wrong.
>
> Care to redo this and resend it?
>
> thanks,
>
> greg k-h
_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/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