Hi Heiko,

On Fri, 13 Jun 2008 21:56:00 +0200 Heiko Schocher <[EMAIL PROTECTED]> wrote:
>
> +++ b/arch/powerpc/platforms/8xx/mgsuvd.c
> @@ -0,0 +1,93 @@
> +/* arch/powerpc/platforms/8xx/mgsuvd.c

We have stopped putting file names in files.

> +#include <linux/ioport.h>
> +#include <linux/device.h>
> +#include <linux/of_platform.h>
> +
> +#include <asm/io.h>
> +#include <asm/machdep.h>
> +#include <asm/processor.h>
> +#include <asm/system.h>
> +#include <asm/cpm1.h>
> +#include <asm/fs_pd.h>

I am petty sure you don't need to include linux/device.h, but you should
include asm/prom.h for the flat device tree accessors.

Also, we don't put a space between a function name and the opening
parenthesis.

-- 
Cheers,
Stephen Rothwell                    [EMAIL PROTECTED]
http://www.canb.auug.org.au/~sfr/

Attachment: pgps1Y8AIqYaT.pgp
Description: PGP signature

_______________________________________________
Linuxppc-dev mailing list
Linuxppc-dev@ozlabs.org
https://ozlabs.org/mailman/listinfo/linuxppc-dev

Reply via email to