Unverified Commit d059cd40 authored by Kuninori Morimoto's avatar Kuninori Morimoto Committed by Mark Brown
Browse files

ASoC: rsnd: setup clock-out only when all conditions are right 



Current adg.c doesn't assume that requested clock out divide condition
doesn't match. In such case, it will indicate strange message, and will
register NULL clock, etc. It is just a DT setting miss, but is
confusable. This patch check all conditions for it.

Reported-by: default avatarVincenzo De Michele <vincenzo.michele@davinci.de>
Signed-off-by: default avatarKuninori Morimoto <kuninori.morimoto.gx@renesas.com>
Link: https://lore.kernel.org/r/87lee2zlf7.wl-kuninori.morimoto.gx@renesas.com


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

sound/soc/sh/rcar/adg.c

+4 −0
Original line number Diff line number Diff line
@@ -607,6 +607,10 @@ static int rsnd_adg_get_clkout(struct rsnd_priv *priv) 
		}

	}



	if (!(adg->brg_rate[ADG_HZ_48]  && req_Hz[ADG_HZ_48]) &&

	    !(adg->brg_rate[ADG_HZ_441] && req_Hz[ADG_HZ_441]))

		goto rsnd_adg_get_clkout_end;



	if (approximate)

		dev_info(dev, "It uses CLK_I as approximate rate");