lkml.org 
[lkml]   [2018]   [Apr]   [19]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH v2 2/3] ASoC: amd: dma driver changes for BT I2S instance
From
Date


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

\
 
 \ /
  Last update: 2018-04-19 06:35    [W:0.048 / U:0.056 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site