Re: [PATCH 2/5] ARM: S3C24XX: move plat-s3c24xx/irq.c into common.c

[Arnd Bergmann <arnd@arndb.de>]

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@sntech.de>

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.

The other four patches look fine to me.

	Arnd