[U-Boot-Users] [PATCH] ML507: New board support
Wolfgang Denk
wd at denx.de
Fri Jul 11 15:59:26 CEST 2008
In message <1215778964-6771-1-git-send-email-ricardo.ribalda at uam.es> you wrote:
> -Suport for the Xilinx ML507 Development Board
There are some coding stylke issues: trailing white space, aliglnment
not by TAB, etc.
> +++ b/board/xilinx/ml507/init.S
> @@ -0,0 +1,45 @@
> +/*
> + (C) Copyright 2008
> + Ricado Ribalda, Universidad Autonoma de Madrid, ricardo.ribalda<at>uam.es , ricardo.ribalda<at>gmail.com
Line too long.
> +int checkboard(void)
> +{
> +
> + printf
> + ("ML507 Board ported by Ricardo Ribalda ricardo.ribalda at uam.es (http://qtec.com) \n");
Line too long. And please print only SHORT messages!
> +phys_size_t initdram(int board_type)
> +{
> + return 256 * 1024 * 1024;
Please use memsize() to autodetect the size of your RAM instead of
hard-wiring it.
> +int testdram(void)
> +{
> + printf("test: xxx MB - ok\n");
> +
> + return (0);
> +}
Please remove.
> new file mode 100644
> index 0000000..054661c
> --- /dev/null
> +++ b/board/xilinx/ml507/u-boot.lds
> @@ -0,0 +1,149 @@
> +/*
> + * (C) Copyright 2000
> + * Wolfgang Denk, DENX Software Engineering, wd at denx.de.
> + * Modified by: Ricardo Ribalda Delgado ricardo.ribalda<at>gmail.com ricardo.ribalda<at>uam.es
Line too long.
Best regards,
Wolfgang Denk
--
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
It became apparent that one reason why the Ice Giants were known as
the Ice Giants was because they were, well, giants. The other was
that they were made of ice. -Terry Pratchett, _Sourcery_
More information about the U-Boot
mailing list