Re: [PATCH v2] staging: dgap: fix kernel oops on port open

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

 



On Fri, Feb 28, 2014 at 08:56:37AM -0500, Mark Hounschell wrote:
> Thanks Dan. I see what should be done. I like and can work on this.
> But is it OK to save all the 80 char problems until the end of this
> next series or more likely a separate patch all together?

What ever you want to do is ok.  :P  Greg applies them on a first come
first serve basis.

In theory, you should sent bug fixes first.  Then my hint would be to
send the patches in the least controversial to most controversial.  That
way if you have to redo one it will be at the end and we can apply the
earlier ones.

Otherwise send them in whatever order you want.

> Since I'm trying to make individual patches that address specific
> checkpatch problems I am running into a chicken and egg sort of thing.
> Some of the next series will have checkpatch warnings/errors that are
> corrected in later patches.

It's fine for your patch to have checkpatch warnings if they were there
in the original code.  It's also fine if you change indentation and it
makes you go over the 80 character mark.

Common sense applies.

> Will this be OK? I think the review process will be much easier this
> way?

Fine fine.

Just send us what you have so far.  It's better to not get too big of
pile of patches until you get a feel for which things we are going to
complain about.

regards,
dan carpenter

_______________________________________________
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