Messages in this thread | | | Subject | Re: [PATCH v2 2/3] ASoC: amd: dma driver changes for BT I2S instance | From | "Mukunda,Vijendar" <> | Date | Thu, 19 Apr 2018 10:07:18 +0530 |
| |
On Thursday 19 April 2018 05:57 AM, Daniel Kurtz wrote: > Hi Vijendar, > > On Wed, Apr 18, 2018 at 5:02 AM Vijendar Mukunda <Vijendar.Mukunda@amd.com> > 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 BT >> controller Instance. > > I like the direction this patch is taking, but I think it would be easier > to review if you could split it into 2 parts: > (1) the cleanup of the existing driver to use a simplified flow for > playback vs capture paths. > (2) adding the BT I2S channel. > > > Thanks, > -Dan >
Hi Dan,
I am fine with splitting the patch into two parts.
Thanks, Vijendar
| |