Thread (9 messages) 9 messages, 5 authors, 2008-08-29

Re: [PATCH] [8xx] powerpc: Add support for the MPC852 based mgsuvd board from keymile.

From: Stephen Rothwell <hidden>
Date: 2008-06-14 07:57:39

Hi Heiko,

On Fri, 13 Jun 2008 21:56:00 +0200 Heiko Schocher [off-list ref] wrote:
quoted hunk ↗ jump to hunk
+++ 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                    sfr@canb.auug.org.au
http://www.canb.auug.org.au/~sfr/

Attachments

  • (unnamed) [application/pgp-signature] 197 bytes
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help