On Wed, Feb 23, 2011 at 1:16 PM, Stephen Warren <swarren@xxxxxxxxxx> wrote: > Colin Cross wrote at Wednesday, February 23, 2011 2:08 PM: >> >> On Wed, Feb 23, 2011 at 10:58 AM, Stephen Warren <swarren@xxxxxxxxxx> wrote: >> > * Set up platform data required by I2C, and ASoC machine & codec drivers. >> > * Enable required GPIO pins as GPIOs. >> > * Initialize audio-related clocks. >> > * Correctly configure pinmux for audio-related GPIOs. >> > >> > Signed-off-by: Stephen Warren <swarren@xxxxxxxxxx> >> > diff --git a/arch/arm/mach-tegra/board-harmony.h b/arch/arm/mach-tegra/board-harmony.h >> > ... >> > +#define GPIO_WM8903(_x_) (TEGRA_NR_GPIOS + (_x_)) >> Can you name this HARMONY_GPIO_WM8903? > > Sure. But then, wouldn't it make sense to also rename all the TEGRA_GPIO_* > below too? True, but no point in adding churn. Those can be cleaned up later. I just want it to be clear that the numbering of gpios above TEGRA_NR_GPIOS is specific to the board, so the next board author doesn't get it wrong. >> > + >> > #define TEGRA_GPIO_SD2_CD TEGRA_GPIO_PI5 >> > #define TEGRA_GPIO_SD2_WP TEGRA_GPIO_PH1 >> > #define TEGRA_GPIO_SD2_POWER TEGRA_GPIO_PT3 >> > #define TEGRA_GPIO_SD4_CD TEGRA_GPIO_PH2 >> > #define TEGRA_GPIO_SD4_WP TEGRA_GPIO_PH3 >> > #define TEGRA_GPIO_SD4_POWER TEGRA_GPIO_PI6 >> > +#define TEGRA_GPIO_CDC_IRQ TEGRA_GPIO_PX3 >> > +#define TEGRA_GPIO_SPKR_EN GPIO_WM8903(2) >> > +#define TEGRA_GPIO_HP_DET TEGRA_GPIO_PW2 >> > +#define TEGRA_GPIO_INT_MIC_EN TEGRA_GPIO_PX0 >> > +#define TEGRA_GPIO_EXT_MIC_EN TEGRA_GPIO_PX1 > > -- > nvpublic > > -- To unsubscribe from this list: send the line "unsubscribe linux-tegra" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html