evbppc/gt64260 if_gfe config change

["Doug Fraser" <dwfraser%onebox.com@localhost>]

We have an evbppc port to a custom gt64260/PPC750CXe running here that is 
largely
compatible with the EV6420 port. (which makes sense, given the common bridge).

In the function gfe_attach, in the file sys/dev/marvell/if_gfe.c, around line 
263,
there is a block of code that sets the interface to MII or RMII based on the 
config flags.
I would like to see a change that will accept the config as provided by the 
boot monitor
either based on a different flag (2 as opposed to 1 ?) or a config option to 
bypass
the entire block. Any feelings on which method would be best?

Currently I use flag bit 2 to signal this as follows.

        /*
         * This block sets up the MAC/PHY interface to be MII or RMII.
         * Only do this if the config flag bit 2 is NOT set.
         * If flag bit 2 is set, then use the config set by the boot monitor.
         * If flag bit 1 is set, then the interface is RMII, else the
         * interface is MII
         */
        if (!(sc->sc_dev.dv_cfdata->cf_flags & 2)) {
                if (sc->sc_dev.dv_cfdata->cf_flags & 1) {
                        sc->sc_pcxr |= ETH_EPCXR_RMIIEn;
                } else {
                        sc->sc_pcxr &= ~ETH_EPCXR_RMIIEn;
                }
        }

        /* display what we have set.... */
        if(sc->sc_pcxr & ETH_EPCXR_RMIIEn) {
                aprint_normal(", phy %d (rmii)", phyaddr);
        } else {
                aprint_normal(", phy %d (mii)", phyaddr);
        }


Also, being new to the NetBSD community, how would I present
this change back to the community for consideration?

Thanks

-- 
Douglas Fraser