Re: [PATCH 2/2 V2] ASoC: fix pxa-ssp compiling issue under mach-mmp

* Re: [PATCH 2/2 V2] ASoC: fix pxa-ssp compiling issue under mach-mmp
       [not found] <1337915528-11887-1-git-send-email-zhouqiao@marvell.com>
@ 2012-05-25  4:20 ` Eric Miao
  2012-05-30 10:18   ` Mark Brown
  2012-05-30 10:22 ` Mark Brown
  1 sibling, 1 reply; 3+ messages in thread
From: Eric Miao @ 2012-05-25  4:20 UTC (permalink / raw)
  To: alsa-devel; +Cc: Mark Brown, Liam Girdwood, Haojian Zhuang, alsa-devel

On Fri, May 25, 2012 at 11:12 AM, Qiao Zhou <zhouqiao@marvell.com> wrote:
> pxa-ssp.c uses API like cpu_is_pxa3xx(), cpu_is_pxa2xx(), which is
> defined under arch-pxa architecture, and drivers under mach-mmp
> can't find it. so just use ssp->type to replace that API.
>
> Signed-off-by: Qiao Zhou <zhouqiao@marvell.com>

Part of job of #ifdef ... #endif, and cpu_is_pxa*() is to reduce code size when
the support for these SoCs are not compiled, but providing the overall
maintenance burden and ugliness, I'm fine with the changes here.

Acked-by: Eric Miao <eric.y.miao@gmail.com>

> ---
>  include/linux/pxa2xx_ssp.h |    1 +
>  sound/soc/pxa/pxa-ssp.c    |   38 +++++++++++---------------------------
>  2 files changed, 12 insertions(+), 27 deletions(-)
>
> diff --git a/include/linux/pxa2xx_ssp.h b/include/linux/pxa2xx_ssp.h
> index f9fe15e..f366320 100644
> --- a/include/linux/pxa2xx_ssp.h
> +++ b/include/linux/pxa2xx_ssp.h
> @@ -160,6 +160,7 @@ enum pxa_ssp_type {
>        PXA25x_SSP,  /* pxa 210, 250, 255, 26x */
>        PXA25x_NSSP, /* pxa 255, 26x (including ASSP) */
>        PXA27x_SSP,
> +       PXA3xx_SSP,
>        PXA168_SSP,
>        PXA910_SSP,
>        CE4100_SSP,
> diff --git a/sound/soc/pxa/pxa-ssp.c b/sound/soc/pxa/pxa-ssp.c
> index fd04ce1..3c89162 100644
> --- a/sound/soc/pxa/pxa-ssp.c
> +++ b/sound/soc/pxa/pxa-ssp.c
> @@ -33,7 +33,6 @@
>
>  #include <mach/hardware.h>
>  #include <mach/dma.h>
> -#include <mach/audio.h>
>
>  #include "../../arm/pxa2xx-pcm.h"
>  #include "pxa-ssp.h"
> @@ -195,7 +194,7 @@ static void pxa_ssp_set_scr(struct ssp_device *ssp, u32 div)
>  {
>        u32 sscr0 = pxa_ssp_read_reg(ssp, SSCR0);
>
> -       if (cpu_is_pxa25x() && ssp->type == PXA25x_SSP) {
> +       if (ssp->type == PXA25x_SSP) {
>                sscr0 &= ~0x0000ff00;
>                sscr0 |= ((div - 2)/2) << 8; /* 2..512 */
>        } else {
> @@ -213,7 +212,7 @@ static u32 pxa_ssp_get_scr(struct ssp_device *ssp)
>        u32 sscr0 = pxa_ssp_read_reg(ssp, SSCR0);
>        u32 div;
>
> -       if (cpu_is_pxa25x() && ssp->type == PXA25x_SSP)
> +       if (ssp->type == PXA25x_SSP)
>                div = ((sscr0 >> 8) & 0xff) * 2 + 2;
>        else
>                div = ((sscr0 >> 8) & 0xfff) + 1;
> @@ -243,7 +242,7 @@ static int pxa_ssp_set_dai_sysclk(struct snd_soc_dai *cpu_dai,
>                break;
>        case PXA_SSP_CLK_PLL:
>                /* Internal PLL is fixed */
> -               if (cpu_is_pxa25x())
> +               if (ssp->type == PXA25x_SSP)
>                        priv->sysclk = 1843200;
>                else
>                        priv->sysclk = 13000000;
> @@ -267,11 +266,11 @@ static int pxa_ssp_set_dai_sysclk(struct snd_soc_dai *cpu_dai,
>
>        /* The SSP clock must be disabled when changing SSP clock mode
>         * on PXA2xx.  On PXA3xx it must be enabled when doing so. */
> -       if (!cpu_is_pxa3xx())
> +       if (ssp->type != PXA3xx_SSP)
>                clk_disable(ssp->clk);
>        val = pxa_ssp_read_reg(ssp, SSCR0) | sscr0;
>        pxa_ssp_write_reg(ssp, SSCR0, val);
> -       if (!cpu_is_pxa3xx())
> +       if (ssp->type != PXA3xx_SSP)
>                clk_enable(ssp->clk);
>
>        return 0;
> @@ -295,24 +294,20 @@ static int pxa_ssp_set_dai_clkdiv(struct snd_soc_dai *cpu_dai,
>        case PXA_SSP_AUDIO_DIV_SCDB:
>                val = pxa_ssp_read_reg(ssp, SSACD);
>                val &= ~SSACD_SCDB;
> -#if defined(CONFIG_PXA3xx)
> -               if (cpu_is_pxa3xx())
> +               if (ssp->type == PXA3xx_SSP)
>                        val &= ~SSACD_SCDX8;
> -#endif
>                switch (div) {
>                case PXA_SSP_CLK_SCDB_1:
>                        val |= SSACD_SCDB;
>                        break;
>                case PXA_SSP_CLK_SCDB_4:
>                        break;
> -#if defined(CONFIG_PXA3xx)
>                case PXA_SSP_CLK_SCDB_8:
> -                       if (cpu_is_pxa3xx())
> +                       if (ssp->type == PXA3xx_SSP)
>                                val |= SSACD_SCDX8;
>                        else
>                                return -EINVAL;
>                        break;
> -#endif
>                default:
>                        return -EINVAL;
>                }
> @@ -338,10 +333,8 @@ static int pxa_ssp_set_dai_pll(struct snd_soc_dai *cpu_dai, int pll_id,
>        struct ssp_device *ssp = priv->ssp;
>        u32 ssacd = pxa_ssp_read_reg(ssp, SSACD) & ~0x70;
>
> -#if defined(CONFIG_PXA3xx)
> -       if (cpu_is_pxa3xx())
> +       if (ssp->type == PXA3xx_SSP)
>                pxa_ssp_write_reg(ssp, SSACDD, 0);
> -#endif
>
>        switch (freq_out) {
>        case 5622000:
> @@ -366,11 +359,10 @@ static int pxa_ssp_set_dai_pll(struct snd_soc_dai *cpu_dai, int pll_id,
>                break;
>
>        default:
> -#ifdef CONFIG_PXA3xx
>                /* PXA3xx has a clock ditherer which can be used to generate
>                 * a wider range of frequencies - calculate a value for it.
>                 */
> -               if (cpu_is_pxa3xx()) {
> +               if (ssp->type == PXA3xx_SSP) {
>                        u32 val;
>                        u64 tmp = 19968;
>                        tmp *= 1000000;
> @@ -387,7 +379,6 @@ static int pxa_ssp_set_dai_pll(struct snd_soc_dai *cpu_dai, int pll_id,
>                                val, freq_out);
>                        break;
>                }
> -#endif
>
>                return -EINVAL;
>        }
> @@ -596,10 +587,8 @@ static int pxa_ssp_hw_params(struct snd_pcm_substream *substream,
>        /* bit size */
>        switch (params_format(params)) {
>        case SNDRV_PCM_FORMAT_S16_LE:
> -#ifdef CONFIG_PXA3xx
> -               if (cpu_is_pxa3xx())
> +               if (ssp->type == PXA3xx_SSP)
>                        sscr0 |= SSCR0_FPCKE;
> -#endif
>                sscr0 |= SSCR0_DataSize(16);
>                break;
>        case SNDRV_PCM_FORMAT_S24_LE:
> @@ -624,9 +613,7 @@ static int pxa_ssp_hw_params(struct snd_pcm_substream *substream,
>                        * trying and failing a lot; some of the registers
>                        * needed for that mode are only available on PXA3xx.
>                        */
> -
> -#ifdef CONFIG_PXA3xx
> -                       if (!cpu_is_pxa3xx())
> +                       if (ssp->type != PXA3xx_SSP)
>                                return -EINVAL;
>
>                        sspsp |= SSPSP_SFRMWDTH(width * 2);
> @@ -634,9 +621,6 @@ static int pxa_ssp_hw_params(struct snd_pcm_substream *substream,
>                        sspsp |= SSPSP_EDMYSTOP(3);
>                        sspsp |= SSPSP_DMYSTOP(3);
>                        sspsp |= SSPSP_DMYSTRT(1);
> -#else
> -                       return -EINVAL;
> -#endif
>                } else {
>                        /* The frame width is the width the LRCLK is
>                         * asserted for; the delay is expressed in
> --
> 1.7.4.1
>
_______________________________________________
Alsa-devel mailing list
Alsa-devel@alsa-project.org
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel

^ permalink raw reply	[flat|nested] 3+ messages in thread