[U-Boot] [PATCH v5] Marvell MV88F6281GTW_GE Board support
Prafulla Wadaskar
prafulla at marvell.com
Mon May 4 05:45:36 CEST 2009
> >
> > +#include <common.h>
> > +#include <../drivers/net/phy/mv88e61xx.h>
> >
> > If this header is meant to be public it should be in
> > "include". Sorry I didn't catch that earlier.
> > <snip>
>
> This header has some public and some driver specific information
> Only a configuration structure in this file is needed here,
> I can put it in netdev.h but I didn't find such
> implementation there.
> How about creating a new header file for phy/switch?
>
>
> I think it's appropriate to put the struct definition and
> public prototypes in netdev.h. There's a lot of work in
> progress regarding PHYs, and I may move the prototypes later,
> but for now netdev is it. Unless the driver-specific stuff
> will be used in the future by other drivers, we may as well
> move it back to the .c file. I know, I know, I made you
> create the header file in the first place :)
> Would you believe that some people have called me indecisive?
Dear Ben, that's okay,
For evolving projects like u-boot, the positive progress happens in this direction only.
How do we improve if we don't experiment... :-)
Currently there is only switch_init function header in netdev.h I will put a struct definition too.
Thus we can remove mv88361xx.h dependency in board specific files
Just let me know - shall I send a delta for this or clean patch v9 for MV88E61XX driver?
If I send you clean patch, you may need to revert earlier patch on your branch
Regards..
Prafulla . .
>
More information about the U-Boot
mailing list