On Mon, 8 Jan 2018 15:26:41 +0000, Mark Brown wrote: > On Mon, Jan 08, 2018 at 12:23:01PM +0000, John Keeping wrote: > > > @@ -527,6 +531,7 @@ static bool rockchip_i2s_volatile_reg(struct device *dev, unsigned int reg) > > static bool rockchip_i2s_precious_reg(struct device *dev, unsigned int reg) > > { > > switch (reg) { > > + case I2S_RXDR: > > default: > > return false; > > } > > This doesn't look right? No, it's not, it should be "return true;" for I2S_RXDR. _______________________________________________ Alsa-devel mailing list Alsa-devel@xxxxxxxxxxxxxxxx http://mailman.alsa-project.org/mailman/listinfo/alsa-devel