port from 2.4.21 to 2.6.8 for mpc5200 - part 1

port from 2.4.21 to 2.6.8 for mpc5200 - part 1

[Andrey Volkov]

[-- Attachment #1: Type: text/plain, Size: 323 bytes --]

Hi all,

Below path of kernel-2.6.8.1 for Motorola MPC5200.
This patch was based on MontaVista 2.4.21 kernel, and
included:

* Motorola BestComm API support
* little correction in Kconfig/Makefile to support it

--
Best regards,
 Andrey Volkov

 P.S. Sorry for bzipped tar, but patch is too big for send as plain
 text :(.

[-- Attachment #2: base_bestcommapi.diff.tar.bz2 --]
[-- Type: application/octet-stream, Size: 23505 bytes --]

Re: port from 2.4.21 to 2.6.8 for mpc5200 - part 1

[Sam Ravnborg]

On Sun, Aug 22, 2004 at 10:31:08PM +0400, Andrey Volkov wrote:
> Hi all,
>
> Below path of kernel-2.6.8.1 for Motorola MPC5200.
> This patch was based on MontaVista 2.4.21 kernel, and
> included:
>
> * Motorola BestComm API support
> * little correction in Kconfig/Makefile to support it

A few random comments.

	Sam

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 (
- int
  mpc52xx_sdma_fec =>
  int mpc52xx_sdma_fec


5) This code should be deleted:
+#ifndef NULL
+# define NULL ((void *)0)
+#endif	/* NULL */

6) Linux style is to avoid typedefs
+typedef sint8 TaskId;
+typedef sint8 BDIdx;
Kill those

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


	Sam

** Sent via the linuxppc-embedded mail list. See http://lists.linuxppc.org/

Re: port from 2.4.21 to 2.6.8 for mpc5200 - part 1

[Wolfgang Denk]

In message <679770249.20040822223108@varma-el.com> you wrote:
>
> Below path of kernel-2.6.8.1 for Motorola MPC5200.
> This patch was based on MontaVista 2.4.21 kernel, and
> included:
>
> * Motorola BestComm API support
> * little correction in Kconfig/Makefile to support it

This patch is based on a very old version of the BestComm code.  This
version  of  the code contains several well known bugs, and it is NOT
available under a GPL compatible license.

Please make sure to use a current and GPLed version of  the  BC  code
instead.

Best regards,

Wolfgang Denk

--
Software Engineering:  Embedded and Realtime Systems,  Embedded Linux
Phone: (+49)-8142-4596-87  Fax: (+49)-8142-4596-88  Email: wd@denx.de
Heavier than air flying machines are impossible.
                    -- Lord Kelvin, President, Royal Society, c. 1895

** Sent via the linuxppc-embedded mail list. See http://lists.linuxppc.org/