Re: [PATCH V3 10/10] ASoC: amd: dma driver changes for bt i2s instance

2018-05-06 Thread Daniel Kurtz
On Thu, May 3, 2018 at 10:35 PM Agrawal, Akshu wrote: > On 5/3/2018 10:10 PM, Daniel Kurtz wrote: > > On Thu, May 3, 2018 at 1:33 AM Mukunda,Vijendar < vijendar.muku...@amd.com> > > wrote: > > > > > > > >> On Thursday 03 May 2018 11:13 AM, Daniel Kurtz wrote: > >>> Some checkpatch nits below...

Re: [PATCH V3 10/10] ASoC: amd: dma driver changes for bt i2s instance

2018-05-03 Thread Agrawal, Akshu
On 5/3/2018 10:10 PM, Daniel Kurtz wrote: On Thu, May 3, 2018 at 1:33 AM Mukunda,Vijendar wrote: On Thursday 03 May 2018 11:13 AM, Daniel Kurtz wrote: Some checkpatch nits below... On Tue, May 1, 2018 at 2:53 PM Vijendar Mukunda < vijendar.muku...@amd.com> wrote: With in ACP, There a

Re: [PATCH V3 10/10] ASoC: amd: dma driver changes for bt i2s instance

2018-05-03 Thread Daniel Kurtz
On Thu, May 3, 2018 at 1:33 AM Mukunda,Vijendar wrote: > On Thursday 03 May 2018 11:13 AM, Daniel Kurtz wrote: > > Some checkpatch nits below... > > > > On Tue, May 1, 2018 at 2:53 PM Vijendar Mukunda < vijendar.muku...@amd.com> > > wrote: > > > >> With in ACP, There are three I2S controllers c

Re: [PATCH V3 10/10] ASoC: amd: dma driver changes for bt i2s instance

2018-05-03 Thread Mukunda,Vijendar
On Thursday 03 May 2018 11:13 AM, Daniel Kurtz wrote: Some checkpatch nits below... On Tue, May 1, 2018 at 2:53 PM Vijendar Mukunda wrote: With in ACP, There are three I2S controllers can be configured/enabled ( I2S SP, I2S MICSP, I2S BT). Default enabled I2S controller instance is I2S SP.

Re: [PATCH V3 10/10] ASoC: amd: dma driver changes for bt i2s instance

2018-05-02 Thread Daniel Kurtz
Some checkpatch nits below... On Tue, May 1, 2018 at 2:53 PM Vijendar Mukunda wrote: > With in ACP, There are three I2S controllers can be > configured/enabled ( I2S SP, I2S MICSP, I2S BT). > Default enabled I2S controller instance is I2S SP. > This patch provides required changes to support I2S