On Tue, Sep 09, 2014 at 08:16:47AM +0800, Barry Song wrote: > > > On 14-9-9 上午7:15, "Greg KH" <gregkh@xxxxxxxxxxxxxxxxxxx> wrote: > > >On Mon, Aug 18, 2014 at 05:32:52PM +0800, Barry Song wrote: > >> From: Qipan Li <Qipan.Li@xxxxxxx> > >> > >> According to key customer's requirement, fix "line over 80 > >> characters". > > > >Someone is forcing you to fix all upstream kernel.org code for 80 > >columns? Who? > > we were asked to fix csr platform codes some days ago. They have been > fixed locally. What is "locally"? And you really are forced to do foolish cleanups for no reason in mainline kernel code for no reason? > You know, the difficulty is maintaining the difference between > mainline and local codes. > So I thought this is better if you can merge it. I'm not taking stuff that isn't correct. Or is foolish. Neither should you. > >Sorry, not going to take this, if your customer blames you, point them > >at me. > > I understand you are very helpful. But pointing them to you will actually > make things worse as I think we have no chance to convince them as we have > tried our best to do that. > The benefit of this patch is making people happy. But if you are not > happy, I can make myself unhappy to maintain two versions. Try fixing the code to look better, your changes were not valid ones, you are quieting some automatic tool checker, you aren't doing the correct thing and following the "spirit" of the rule here. Make the code look cleaner, and I'll be glad to take the change. greg k-h -- To unsubscribe from this list: send the line "unsubscribe linux-serial" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
