Re: [PATCH 1/2 v8] ASoC: dwc: Add PIO PCM extension

2016-06-07 Thread Mark Brown
On Mon, May 30, 2016 at 10:53:15AM +0100, Jose Abreu wrote: A few small things but this looks basically fine. > + if (isr[i] & 0x33) > + irq_valid = true; This checks for a mask of 0x33 but... > + > + /* > + * Check if TX fifo is empty.

Re: [PATCH 1/2 v8] ASoC: dwc: Add PIO PCM extension

2016-06-07 Thread Mark Brown
On Mon, May 30, 2016 at 10:53:15AM +0100, Jose Abreu wrote: A few small things but this looks basically fine. > + if (isr[i] & 0x33) > + irq_valid = true; This checks for a mask of 0x33 but... > + > + /* > + * Check if TX fifo is empty.