[U-Boot] [PATCH] net: Fix faulty definition of uec_initialize()
Joakim Tjernlund
joakim.tjernlund at transmode.se
Tue Aug 10 14:10:14 CEST 2010
vapierfilter at gmail.com wrote on 2010/08/10 08:04:02:
>
> On Tue, Aug 10, 2010 at 1:52 AM, Joakim Tjernlund wrote:
> > Ben Warren wrote on 2010/08/09 22:54:10:
> >> On 8/9/2010 1:18 PM, Mike Frysinger wrote:
> >> > On Mon, Aug 9, 2010 at 3:15 PM, Joakim Tjernlund wrote:
> >> >> The correct definition is in drivers/qe/uec.h so just
> >> >> include that one instead.
> >> > ... except that bleeds a ton of crap from uec.h into the global
> >> > namespace for everyone. better would be to move the prototype out of
> >> > uec.h and into netdev.h and make sure uec.h includes netdev.h.
> >
> > hmm, then the public include file for UEC users become "../drivers/uec.h"
> > while every one else uses netdev.h, doesn't seem the right way go
>
> which isnt uncommon today. simply grep the tree for
> "include.*drivers/net/". while i agree it isnt exactly a great idea,
> there is no clean standard today in u-boot for drivers to expose their
> internals to random consumers. some drivers dump their header files
> into include/, so personally, i prefer the include ../driver/net/..
> style over that.
I see, but that defeats the purpose of netdev.h I think. Oh well,
just gonna have to live with that then.
Jocke
More information about the U-Boot
mailing list