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.
