port from 2.4.21 to 2.6.8 for mpc5200 - part 1
sam at ravnborg.org
Mon Aug 23 09:04:36 EST 2004
On Sun, Aug 22, 2004 at 10:31:08PM +0400, Andrey Volkov wrote:
> Hi all,
> Below path of kernel-188.8.131.52 for Motorola MPC5200.
> This patch was based on MontaVista 2.4.21 kernel, and
> * Motorola BestComm API support
> * little correction in Kconfig/Makefile to support it
A few random comments.
1) Please remove the following comment from the Makefiles:
+# Note! Dependencies are done automagically by 'make dep', which also
+# removes any old dependencies. DON'T put your own dependencies here
+# unless it's something special (ie not a .c file).
+# Note 2! The CFLAGS definition is now in the main makefile...
It is no longer relevant for 2.6
2) -objs -> -y
Preferred style is:
+bestcomm-y := bestcomm_api.o \
Same functionality though.
3) Comment style does not look like kernel-doc??
4) Coding style not followed
- Mixed casing
- if ( xx ) => if (xx)
- if( => if (
5) This code should be deleted:
+# define NULL ((void *)0)
+#endif /* NULL */
6) Linux style is to avoid typedefs
+typedef sint8 TaskId;
+typedef sint8 BDIdx;
7) if MPC52xx_SDMA_DEDUG
Hide this check in a function - not at all file
8) In general lot of preprocessor conditionals that should be minimized
** Sent via the linuxppc-embedded mail list. See http://lists.linuxppc.org/
More information about the Linuxppc-embedded