Hi Dan, On 2018/9/7 17:17, Dan Carpenter wrote: > On Fri, Sep 07, 2018 at 04:48:44PM +0800, Gao Xiang wrote: >> Hi Thomas, >> >> On 2018/9/3 9:53, Chao Yu wrote: >>> Hi, >>> >>> It looks like there is another patch from Pavel Zemlyanoy changing the same >>> place, I think it needs to rebase this patch on that one. >>> >>> [PATCH 1/6] staging: erofs: formatting fix in unzip_vle_lz4.c >>> >> >> Thanks for your patch. >> >> Do you have some interest to fix this patch as Chao's suggestion? >> Or I could fix this patch based on your work, if you have no time to do...Thanks in advance. > > Gao, don't worry about it... It's not a bug fix. These kinds of > patches are straight forward to review. You either apply it instantly > or you respond and delete it from your brain. > > We don't try to co-ordinate patches. We just apply whatever comes in > the order that it arrives. > > If it's a bugfix then it's another story. In that case we do ask people > to redo it. Sometimes for security patches, I have fixed patches for > people and given them authorship credit with the From: header in git. > Other times maintainers write their own patch and give Reported-by > credit instead. > > For checkpatch.pl patches, if you decide to do the work yourself then > there is no Reported-by tag required. > > If you try to keep track of everyone sending patches then it's too much > work... Forget about it. > OK, Thanks for your kindly detailed explanation. ;) I will fix these style issues by myself later if Thomas has no response. But anyway, any patches helping to erofs are welcome. :) Thanks, Gao Xiang > regards, > dan carpenter > _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel