[PATCH 03/17] ARM: pxa/raumfeld: add SMSC9220 ethernet support

Daniel Mack daniel at caiaq.de
Wed Nov 25 10:49:47 EST 2009


On Wed, Nov 25, 2009 at 05:26:36PM +0200, Mike Rapoport wrote:
> Daniel Mack wrote:
> > +static mfp_cfg_t raumfeld_eth_pin_config[] __initdata = {
> > +	GPIO1_nCS2,			/* CS */
> > +	GPIO40_GPIO | MFP_PULL_HIGH,	/* IRQ */
> > +};
> 
> Wouldn't it be better to keep mfp tables for common pins and each of board
> variants rather than configure mfp for particular interface? This applies for
> other patches that have
> +	pxa3xx_mfp_config(ARRAY_AND_SIZE(raumfeld_<something>_pin_config));
> as well. :)

I had it that way, but the idea is to prevent double definitions. What
we have now is a config for all pins needed by the ethernet device, and
each machine which needs that just references it. It looks more straight
forward this way, but eventually that might be a matter of taste :)

Unless anyone has a strong opinion abouth that, I'd like to keep it as
it is now.

Daniel



More information about the linux-arm-kernel mailing list