diff mbox

ASoC: arizona: Fix bclk for sample rates that are multiple of 4kHz

Message ID 1450880285-23865-1-git-send-email-ckeepax@opensource.wolfsonmicro.com (mailing list archive)
State Accepted
Commit e73694d871867cae8471d2350ce89acb38bc2b63
Headers show

Commit Message

Charles Keepax Dec. 23, 2015, 2:18 p.m. UTC
From: Nikesh Oswal <Nikesh.Oswal@cirrus.com>

For a sample rate of 12kHz the bclk was taken from the 44.1kHz table as
we test for a multiple of 8kHz. This patch fixes this issue by testing
for multiples of 4kHz instead.

Signed-off-by: Nikesh Oswal <Nikesh.Oswal@cirrus.com>
Signed-off-by: Charles Keepax <ckeepax@opensource.wolfsonmicro.com>
---
 sound/soc/codecs/arizona.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Charles Keepax Dec. 23, 2015, 2:31 p.m. UTC | #1
On Wed, Dec 23, 2015 at 02:18:05PM +0000, Charles Keepax wrote:
> From: Nikesh Oswal <Nikesh.Oswal@cirrus.com>
> 
> For a sample rate of 12kHz the bclk was taken from the 44.1kHz table as
> we test for a multiple of 8kHz. This patch fixes this issue by testing
> for multiples of 4kHz instead.
> 
> Signed-off-by: Nikesh Oswal <Nikesh.Oswal@cirrus.com>
> Signed-off-by: Charles Keepax <ckeepax@opensource.wolfsonmicro.com>
> ---

And... I forgot to CC Nikesh, sorry added now.

Thanks,
Charles

>  sound/soc/codecs/arizona.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/sound/soc/codecs/arizona.c b/sound/soc/codecs/arizona.c
> index 88e2c74..33143fe 100644
> --- a/sound/soc/codecs/arizona.c
> +++ b/sound/soc/codecs/arizona.c
> @@ -1659,7 +1659,7 @@ static int arizona_hw_params(struct snd_pcm_substream *substream,
>  	bool reconfig;
>  	unsigned int aif_tx_state, aif_rx_state;
>  
> -	if (params_rate(params) % 8000)
> +	if (params_rate(params) % 4000)
>  		rates = &arizona_44k1_bclk_rates[0];
>  	else
>  		rates = &arizona_48k_bclk_rates[0];
> -- 
> 2.1.4
diff mbox

Patch

diff --git a/sound/soc/codecs/arizona.c b/sound/soc/codecs/arizona.c
index 88e2c74..33143fe 100644
--- a/sound/soc/codecs/arizona.c
+++ b/sound/soc/codecs/arizona.c
@@ -1659,7 +1659,7 @@  static int arizona_hw_params(struct snd_pcm_substream *substream,
 	bool reconfig;
 	unsigned int aif_tx_state, aif_rx_state;
 
-	if (params_rate(params) % 8000)
+	if (params_rate(params) % 4000)
 		rates = &arizona_44k1_bclk_rates[0];
 	else
 		rates = &arizona_48k_bclk_rates[0];