[PATCH 07/10] ARM: mx28: add the second fec device registration

Shawn Guo shawn.guo at freescale.com
Wed Dec 29 07:05:09 EST 2010


Hi Uwe,

On Wed, Dec 29, 2010 at 11:50:01AM +0100, Uwe Kleine-König wrote:
> Hello Shawn,
> 
> > -static const struct fec_platform_data mx28_fec_pdata __initconst = {
> > -	.phy = PHY_INTERFACE_MODE_RMII,
> > +static struct fec_platform_data mx28_fec_pdata[] = {
> > +	{
> > +		/* fec0 */
> > +		.phy = PHY_INTERFACE_MODE_RMII,
> > +	}, {
> > +		/* fec1 */
> > +		.phy = PHY_INTERFACE_MODE_RMII,
> > +	},
> >  };
> I think there is no need to duplicate the platform_data as mx28_add_fec
> makes a copy anyhow.  Did you remove __initconst and const on purpose?
> 
The mac field is different for fec0 and fec1 and needs to be
assigned in mx28evk_fec_get_mac.

-- 
Regards,
Shawn




More information about the linux-arm-kernel mailing list