[U-Boot] [PATCH v5] Marvell MV88F6281GTW_GE Board support

Ben Warren biggerbadderben at gmail.com
Mon May 4 05:33:41 CEST 2009


Hi Prafulla,

On Sun, May 3, 2009 at 10:27 AM, Prafulla Wadaskar <prafulla at marvell.com>wrote:

>
> >       + * along with this program; if not, write to the Free Software
> >       + * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston,
> >       + * MA 02110-1301 USA
> >       + */
> >       +
> >
> >       +#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?

>
> >
> >       +/*
> >       + * Ethernet Driver configuration
> >       + */
> >       +#ifdef CONFIG_CMD_NET
> >       +#define CONFIG_NETCONSOLE      /* include NetConsole
> > support   */
> >       +#define CONFIG_NET_MULTI       /* specify more that
> > one ports available */
> >       +#define        CONFIG_MII              /* expose smi
> > ove miiphy interface */
> >       +#define CONFIG_KIRKWOOD_EGIGA  /* Enable kirkwood Gbe
> > Controller Driver */
> >
> > This EGIGA driver hasn't been accepted yet.  In fact, I think
> > you've only sent one spin.  It needs to be in before this board...
> You are correct,
> I am ready with the latest spin, but it has close dependency with kirkwood
> SoC layer, once this gets through I had a plan to release egiga driver
> patch. BTW code builds cleanly without egiga driver so I do not want to
> remove this :-) since it will be required afterward once kirkwood_egiga
> driver gets in.
> Any suggestion welcomed ???
>
No, this seems OK.  Like you say, it builds cleanly without.  As long as
they make it into the same release everything should be cool.

>
> Regards..
> Prafulla . .
>

Thanks for all your hard work.

regards,
Ben


More information about the U-Boot mailing list