[U-Boot] [PATCH v3 1/1] ARM: Add support for Seagate GoFlexHome
Wolfgang Denk
wd at denx.de
Thu Apr 11 09:34:41 CEST 2013
Dear Suriyan Ramasami,
In message <1365638673-7815-1-git-send-email-suriyan.r at gmail.com> you wrote:
> This is yet another attempt with patch v3.
> Thanks to Wolfgang, Prafulla and Albert for comments, corrections,
> and suggestions.
On addition to the comments already made about how the commit message
should be organized and where comments go, here some additional
remarks:
> + static const u32 kwmpp_config[] = {
...
> + };
> + kirkwood_mpp_conf(kwmpp_config, NULL);
Please always insert a blank line between declarations and code.
> +static void set_leds(u32 leds, u32 blinking)
> +{
> + struct kwgpio_registers *r = (struct kwgpio_registers *)KW_GPIO1_BASE;
> + u32 oe = readl(&r->oe) | BOTH_LEDS;
> + writel(oe & ~leds, &r->oe); /* active low */
Ditto. Please fix globally.
> + u32 bl = readl(&r->blink_en) & ~BOTH_LEDS;
Please never mix declarations and code. Please fix globally.
> diff --git a/board/Seagate/goflexhome/goflexhome.h b/board/Seagate/goflexhome/goflexhome.h
> new file mode 100644
> index 0000000..2af2596
> --- /dev/null
> +++ b/board/Seagate/goflexhome/goflexhome.h
What is this file needed for? It appears all this could go into
include/configs/goflexhome.h instead?
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
"Science makes godlike -- it is all over with priests and gods when
man becomes scientific. Moral: science is the forbidden as such -- it
alone is forbidden. Science is the *first* sin, the *original* sin.
*This alone is morality.* ``Thou shalt not know'' -- the rest
follows." - Friedrich Nietzsche
More information about the U-Boot
mailing list