Unverified Commit 55c209cd authored by Claudiu Beznea's avatar Claudiu Beznea Committed by Mark Brown
Browse files

ASoC: renesas: rz-ssi: Use only the proper amount of dividers 



There is no need to populate the ckdv[] with invalid dividers as that
part will not be indexed anyway. The ssi->audio_mck/bclk_rate should
always be >= 0. While at it, change the ckdv type as u8, as the divider
128 was previously using the s8 sign bit.

Signed-off-by: default avatarClaudiu Beznea <claudiu.beznea.uj@bp.renesas.com>
Fixes: 03e786bd ("ASoC: sh: Add RZ/G2L SSIF-2 driver")
Reviewed-by: default avatarGeert Uytterhoeven <geert+renesas@glider.be>
Link: https://patch.msgid.link/20241210170953.2936724-6-claudiu.beznea.uj@bp.renesas.com


Signed-off-by: default avatarMark Brown <broonie@kernel.org>
parent 541011dc

sound/soc/renesas/rz-ssi.c

+1 −2
Original line number Diff line number Diff line
@@ -258,8 +258,7 @@ static void rz_ssi_stream_quit(struct rz_ssi_priv *ssi, 
static int rz_ssi_clk_setup(struct rz_ssi_priv *ssi, unsigned int rate,

			    unsigned int channels)

{

	static s8 ckdv[16] = { 1,  2,  4,  8, 16, 32, 64, 128,

			       6, 12, 24, 48, 96, -1, -1, -1 };

	static u8 ckdv[] = { 1,  2,  4,  8, 16, 32, 64, 128, 6, 12, 24, 48, 96 };

	unsigned int channel_bits = 32;	/* System Word Length */

	unsigned long bclk_rate = rate * channels * channel_bits;

	unsigned int div;