On Saturday 18 April 2009 23:13:58 Peter Tyser wrote:
> On Sat, 2009-04-18 at 14:23 -0400, Mike Frysinger wrote:
> > On Thursday 16 April 2009 15:41:22 Peter Tyser wrote:
> > > +unsigned int i2c_get_bus_speed(void)
> > > + __attribute((weak, alias("__def_i2c_get_bus_speed")));
> >
> > __attribute__ ?
>
> Interesting, thanks for catching that.  I blindly copied the weak
> functions in net/eth.c which also use the __attribute declaration.  It
> looks like gcc treats __attribute and __attribute__ the same, but I
> couldn't find any definitive info on __attribute.  I'll resubmit using
> __attribute__.
>
> Anyone have a reason that we shouldn't replace all u-boot references to
> __attribute with __attribute__?

not that i know of ... using __XXX__ with gcc keywords tends to be preferred.
-mike

Attachment: signature.asc
Description: This is a digitally signed message part.

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to