[U-Boot] [PATCH V3 2/5] mv_egiga: support SoCs other than kirkwood
Prafulla Wadaskar
prafulla at marvell.com
Mon Jul 12 08:53:52 CEST 2010
> -----Original Message-----
> From: Ben Warren [mailto:biggerbadderben at gmail.com]
> Sent: Monday, July 12, 2010 11:54 AM
> To: Prafulla Wadaskar
> Cc: Albert Aribaud; u-boot at lists.denx.de
> Subject: Re: [U-Boot] [PATCH V3 2/5] mv_egiga: support SoCs
> other than kirkwood
>
> On 7/11/2010 10:45 PM, Prafulla Wadaskar wrote:
> >
> >
> >> -----Original Message-----
> >> From: u-boot-bounces at lists.denx.de
> >> [mailto:u-boot-bounces at lists.denx.de] On Behalf Of Albert Aribaud
> >> Sent: Sunday, July 11, 2010 1:32 PM
> >> To: u-boot at lists.denx.de
> >> Subject: [U-Boot] [PATCH V3 2/5] mv_egiga: support SoCs other
> >> than kirkwood
...snip...
> >> struct eth_device *dev = eth_get_dev_by_name(devname);
> >> - struct kwgbe_device *dkwgbe = to_dkwgbe(dev);
> >> - struct kwgbe_registers *regs = dkwgbe->regs;
> >> + struct mv_egiga_device *dmvegiga = to_mv_egiga(dev);
> >> + struct mv_egiga_registers *regs = dmvegiga->regs;
> > I suggest to keep name as mvgbe here instead of mv_egiga, 3
> additional chars, increases overall code size
> huh? The name is consistent with the rest of his work, and *if* the
> code really increases in size, I can't imagine that 3 chars really
> matters...
That's true.
But if we can do it why to avoid it? again it helps to keep same indentation (keeping them below 80char size)
Regards..
Prafulla . .
More information about the U-Boot
mailing list