Re: [PATCH 08/29] ARM: OMAP2+: Make board-zoom.h local

[Peter Ujfalusi <peter.ujfalusi@ti.com>]

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@ti.com>
> Cc: Jarkko Nikula <jarkko.nikula@bitmer.com>
> Cc: Liam Girdwood <lrg@ti.com>
> Cc: Mark Brown <broonie@opensource.wolfsonmicro.com>
> Cc: alsa-devel@alsa-project.org
> Signed-off-by: Tony Lindgren <tony@atomide.com>
> ---
>  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
_______________________________________________
Alsa-devel mailing list
Alsa-devel@alsa-project.org
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel