[U-Boot] [PATCH v10] Marvell MV88F6281GTW_GE Board support
Prafulla Wadaskar
prafulla at marvell.com
Fri May 22 13:20:27 CEST 2009
> -----Original Message-----
> From: Wolfgang Denk [mailto:wd at denx.de]
> Sent: Friday, May 22, 2009 4:37 PM
> To: Prafulla Wadaskar
> Cc: u-boot at lists.denx.de; Ashish Karkare; Prabhanjan Sarnaik;
> Ronen Shitrit
> Subject: Re: [U-Boot] [PATCH v10] Marvell MV88F6281GTW_GE
> Board support
>
> Dear Prafulla Wadaskar,
>
> In message
> <1242865967-7179-1-git-send-email-prafulla at marvell.com> you wrote:
> >
> > This is Marvell's 88F6281_A0 based custom board developed
> for wireless
> > access point product
>
> > --- /dev/null
> > +++ b/board/Marvell/mv88f6281gtw_ge/Makefile
> > @@ -0,0 +1,51 @@
> > +#
> > +# (C) Copyright 2009
> > +# Marvell Semiconductor <www.marvell.com> # Prafulla Wadaskar
> > +<prafulla at marvell.com>
>
> As mentioned before, this makes no sense. Either Marvell
> holds the copyright, or you do.
Marvell holds copyright, I will remove "prafulla at marvell.com"
This is there from first submission in all the patches
such comments in early feedback is more helpful....
I will correct and re-submit all patches...
> > +#define CONFIG_KIRKWOOD_RGMII_PAD_1V8 /* Set RGMII
> Pad voltage to 1.8V */
>
> Line too long.
BTW what is max allowed line size? All line are below 80 chars.
Regards..
Prafulla . .
More information about the U-Boot
mailing list