Hi Arnd, Am Montag 07 Mai 2012, 21:59:21 schrieb Arnd Bergmann: > On Monday 07 May 2012, Heiko Stübner wrote: > > Again following the model set out by s3c64xx. > > > > As checkpatch showed a lot of style errors and warning, these got fixed > > too. > > > > Signed-off-by: Heiko Stueber <heiko@xxxxxxxxx> > > I object on two grounds: > > 1. there are plans to move irq chip drivers to drivers/irqchip eventually, > so I think it makes little sense to merge the current implementation into > another file. That just makes it harder to split it out again. Better leave > the file named irq.c and just move it to the mach-s3c24xx for now. > > 2. You should *never* move code around and change it at the same time > if you have a choice. Instead if you discover coding style issues with > code you want to move, fix those first and then do a second patch to > move it without further changes. Otherwise it becomes very hard to > check that the coding style changes are actually correct. thanks for the two pointers, you are of course right with them. I will do another round of the patches. Thanks Heiko -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
 |