[alsa-devel] [PATCH] pxa2xx-lib: support building for several pxa's

Mark Brown broonie at opensource.wolfsonmicro.com
Thu Jul 31 11:37:57 CEST 2008


On Thu, Jul 31, 2008 at 02:41:21AM +0400, Dmitry wrote:

> > #ifdef's. Just a quick question for Dmitry, why all the if (!
> > cpu_is_pxa3xx())'s around the pxa_gpio_mode's? Also, there was a build
> > error when just building the sound/arm pxa2xx-ac97.

> AFAIR pxa_gpio_mode isn't valid for the the pxa3xx, so cpu_is_pxa3xx().

Not really.  The thing is that historically drivers set the MFP mappings
up themselves rather than letting the board code do it.  Going forward
that should be being done by the platform code but simply removing the
configuration for PXA2xx would break existing users.

Aside from that the PXA3xx series provide multiple MFP options for the
AC97 interface so we can't do an unconditional setup for them anyway -
the configuration is always board specific.


More information about the Alsa-devel mailing list