Hi Tony, On 09/20/2012 12:05 AM, Tony Lindgren wrote: > This can be local to mach-omap2 after we rip out the messed up > extmute stuff from board-zoom-peripherals.c that's all over > the place and probably does not even work because of all the > hardcoded GPIO numbers. > > I also suggest ASoC gusy remove all extmute handling from > sound/soc/codecs/twl4030.c unless it's been tested to work > and implemented in a saner way. I just wanted to note that I also have pending patch for zoom2.c as part of the twl4030 audio module DT support series: http://lkml.indiana.edu/hypermail/linux/kernel/1209.1/00711.html > > Cc: Peter Ujfalusi <peter.ujfalusi@xxxxxx> > Cc: Jarkko Nikula <jarkko.nikula@xxxxxxxxxx> > Cc: Liam Girdwood <lrg@xxxxxx> > Cc: Mark Brown <broonie@xxxxxxxxxxxxxxxxxxxxxxxxxxx> > Cc: alsa-devel@xxxxxxxxxxxxxxxx > Signed-off-by: Tony Lindgren <tony@xxxxxxxxxxx> > --- > arch/arm/mach-omap2/board-3630sdp.c | 2 +- > arch/arm/mach-omap2/board-ldp.c | 2 +- > arch/arm/mach-omap2/board-zoom-debugboard.c | 2 +- > arch/arm/mach-omap2/board-zoom-display.c | 2 +- > arch/arm/mach-omap2/board-zoom-peripherals.c | 16 +--------------- > arch/arm/mach-omap2/board-zoom.c | 2 +- > arch/arm/mach-omap2/board-zoom.h | 0 > sound/soc/omap/zoom2.c | 2 +- > 8 files changed, 7 insertions(+), 21 deletions(-) > rename arch/arm/mach-omap2/{include/mach/board-zoom.h => board-zoom.h} (100%) > > diff --git a/arch/arm/mach-omap2/board-3630sdp.c b/arch/arm/mach-omap2/board-3630sdp.c > index 8518b13..c7d6ea2 100644 > --- a/arch/arm/mach-omap2/board-3630sdp.c > +++ b/arch/arm/mach-omap2/board-3630sdp.c > @@ -20,7 +20,7 @@ > #include <plat/gpmc-smc91x.h> > #include <plat/usb.h> > > -#include <mach/board-zoom.h> > +#include "board-zoom.h" > > #include "board-flash.h" > #include "mux.h" > diff --git a/arch/arm/mach-omap2/board-ldp.c b/arch/arm/mach-omap2/board-ldp.c > index 3f3a552..d0372a6 100644 > --- a/arch/arm/mach-omap2/board-ldp.c > +++ b/arch/arm/mach-omap2/board-ldp.c > @@ -36,7 +36,7 @@ > #include <plat/mcspi.h> > #include "common.h" > #include <plat/gpmc.h> > -#include <mach/board-zoom.h> > +#include "board-zoom.h" > #include <plat/usb.h> > #include <plat/gpmc-smsc911x.h> > > diff --git a/arch/arm/mach-omap2/board-zoom-debugboard.c b/arch/arm/mach-omap2/board-zoom-debugboard.c > index 0d8d919..42c97c2 100644 > --- a/arch/arm/mach-omap2/board-zoom-debugboard.c > +++ b/arch/arm/mach-omap2/board-zoom-debugboard.c > @@ -20,7 +20,7 @@ > #include <plat/gpmc.h> > #include <plat/gpmc-smsc911x.h> > > -#include <mach/board-zoom.h> > +#include "board-zoom.h" > > #include "soc.h" > #include "common.h" > diff --git a/arch/arm/mach-omap2/board-zoom-display.c b/arch/arm/mach-omap2/board-zoom-display.c > index ea79bc2..6a162fb 100644 > --- a/arch/arm/mach-omap2/board-zoom-display.c > +++ b/arch/arm/mach-omap2/board-zoom-display.c > @@ -16,7 +16,7 @@ > #include <linux/spi/spi.h> > #include <plat/mcspi.h> > #include <video/omapdss.h> > -#include <mach/board-zoom.h> > +#include "board-zoom.h" > > #include "common.h" > > diff --git a/arch/arm/mach-omap2/board-zoom-peripherals.c b/arch/arm/mach-omap2/board-zoom-peripherals.c > index 6bcc107..c8377b71 100644 > --- a/arch/arm/mach-omap2/board-zoom-peripherals.c > +++ b/arch/arm/mach-omap2/board-zoom-peripherals.c > @@ -28,7 +28,7 @@ > #include "common.h" > #include <plat/usb.h> > > -#include <mach/board-zoom.h> > +#include "board-zoom.h" > > #include "mux.h" > #include "hsmmc.h" > @@ -245,12 +245,6 @@ static int zoom_twl_gpio_setup(struct device *dev, > return ret; > } > > -/* EXTMUTE callback function */ > -static void zoom2_set_hs_extmute(int mute) > -{ > - gpio_set_value(ZOOM2_HEADSET_EXTMUTE_GPIO, mute); > -} > - > static struct twl4030_gpio_platform_data zoom_gpio_data = { > .setup = zoom_twl_gpio_setup, > }; > @@ -271,14 +265,6 @@ static int __init omap_i2c_init(void) > TWL_COMMON_PDATA_MADC | TWL_COMMON_PDATA_AUDIO, > TWL_COMMON_REGULATOR_VDAC | TWL_COMMON_REGULATOR_VPLL2); > > - if (machine_is_omap_zoom2()) { > - struct twl4030_codec_data *codec_data; > - codec_data = zoom_twldata.audio->codec; > - > - codec_data->ramp_delay_value = 3; /* 161 ms */ > - codec_data->hs_extmute = 1; > - codec_data->set_hs_extmute = zoom2_set_hs_extmute; > - } > omap_pmic_init(1, 2400, "twl5030", 7 + OMAP_INTC_START, &zoom_twldata); > omap_register_i2c_bus(2, 400, NULL, 0); > omap_register_i2c_bus(3, 400, NULL, 0); > diff --git a/arch/arm/mach-omap2/board-zoom.c b/arch/arm/mach-omap2/board-zoom.c > index 4994438..cbfd9df 100644 > --- a/arch/arm/mach-omap2/board-zoom.c > +++ b/arch/arm/mach-omap2/board-zoom.c > @@ -24,7 +24,7 @@ > #include "common.h" > #include <plat/usb.h> > > -#include <mach/board-zoom.h> > +#include "board-zoom.h" > > #include "board-flash.h" > #include "mux.h" > diff --git a/arch/arm/mach-omap2/include/mach/board-zoom.h b/arch/arm/mach-omap2/board-zoom.h > similarity index 100% > rename from arch/arm/mach-omap2/include/mach/board-zoom.h > rename to arch/arm/mach-omap2/board-zoom.h > diff --git a/sound/soc/omap/zoom2.c b/sound/soc/omap/zoom2.c > index 920e0d9..187cb0c 100644 > --- a/sound/soc/omap/zoom2.c > +++ b/sound/soc/omap/zoom2.c > @@ -28,7 +28,6 @@ > #include <asm/mach-types.h> > #include <mach/hardware.h> > #include <mach/gpio.h> > -#include <mach/board-zoom.h> > #include <plat/mcbsp.h> > > /* Register descriptions for twl4030 codec part */ > @@ -39,6 +38,7 @@ > #include "omap-pcm.h" > > #define ZOOM2_HEADSET_MUX_GPIO (OMAP_MAX_GPIO_LINES + 15) > +#define ZOOM2_HEADSET_EXTMUTE_GPIO 153 > > static int zoom2_hw_params(struct snd_pcm_substream *substream, > struct snd_pcm_hw_params *params) > -- Péter -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
