On Thu, Jan 26, 2012 at 00:23, Stephen Warren <swarren@xxxxxxxxxx> wrote: > Leon Romanovsky wrote at Wednesday, January 25, 2012 11:50 AM: >> Signed-off-by: Leon Romanovsky <leon@xxxxxxx> > >> diff --git a/arch/arm/mach-tegra/board-paz00-pinmux.c b/arch/arm/mach-tegra/board-paz00-pinmux.c > ... >> @@ -152,6 +152,7 @@ static struct tegra_gpio_table gpio_table[] = { > ... >> + { .gpio = TEGRA_HP_DET, .enable = true }, > > There's no need to do this at all if you're only supporting audio when > booting use device tree. If you intend for audio to work when booting > using a board file, then the previous patch needs a bunch of rework > to make some of the code conditional on device tree; the same structure > as tegra_wm8903.c uses. > > Talking of which, in the previous patch, you can remove a number of > things from tegra_alc5632.c if you are only supporting device tree, like > the DAPM route map, since that's parsed from device tree now. Thanks for the input As you mentioned we are going support only device tree. > > -- > nvpublic > -- Leon Romanovsky | Independent Linux Consultant www.leon.nu | leon@xxxxxxx -- 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