19 Apr
2018
19 Apr
'18
6:37 a.m.
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