Thread (13 messages) 13 messages, 5 authors, 2021-08-24

Re: [PATCH v2 1/4] ASoC: rockchip: add support for i2s-tdm controller

From: Mark Brown <broonie@kernel.org>
Date: 2021-08-23 11:20:08
Also in: alsa-devel, linux-rockchip, lkml

On Sat, Aug 21, 2021 at 10:45:52PM +0200, Nicolas Frattaroli wrote:
On Freitag, 20. August 2021 21:02:16 CEST Pierre-Louis Bossart wrote:
quoted
quoted
+	regmap_read(i2s_tdm->regmap, I2S_CLR, &val);
+	/* Wait on the clear operation to finish */
+	while (val) {
quoted
delay needed here?
The rockchip_i2s.c code doesn't have a delay here either, but I can
add one of 1 usec for good measure, it seems weird to retry the
read as fast as it can.
You should probably have a cpu_relax() in there at least.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help