RE: [PATCH v3 0/3] ASoC: Intel: Add KeemBay ASoC platform driver

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




-----Original Message-----
From: Rojewski, Cezary <cezary.rojewski@xxxxxxxxx> 
Sent: Tuesday, June 2, 2020 5:31 AM
To: Sia, Jee Heng <jee.heng.sia@xxxxxxxxx>
Cc: alsa-devel@xxxxxxxxxxxxxxxx; liam.r.girdwood@xxxxxxxxxxxxxxx; broonie@xxxxxxxxxx; tiwai@xxxxxxxx; pierre-louis.bossart@xxxxxxxxxxxxxxx; Shevchenko, Andriy <andriy.shevchenko@xxxxxxxxx>
Subject: Re: [PATCH v3 0/3] ASoC: Intel: Add KeemBay ASoC platform driver

On 2020-06-01 11:53 AM, Sia Jee Heng wrote:
> The below series of patches support the KeemBay ASoC platform driver.
> The platform driver initialize the i2s to capture and playback the pcm 
> data on the ARM. The i2s is running in polling mode.
> 
> There is no DSP in the KeemBay SoC. Users are rely on the Gstreamer 
> plugin to perform Audio preprocessing.
> 
> Audio graph card is used to connect the platform driver with the
> tlv320aic3204 codec.
> 
> Change History:
> v3:
> - Adjusted header format.
> - Use Audio graph card instead of custom sound card.
> - Use if-else instead of conditional operator.
> - Enabled .set_fmt to configure master clock.
> 
> v2:
> - Corrected I2S naming for DT binding.
> 
> v1:
> - Initial version.
> 
> Sia Jee Heng (3):
>    ASoC: Intel: Add KeemBay platform driver
>    ASoC: Intel: Add makefiles and kconfig changes for KeemBay
>    dt-bindings: sound: Add documentation for KeemBay i2s
> 

As these series has been given plenty of comments internally from Andy, Pierre and me, I'd be best to have all reviewers CC'ed here.
[>>]  Noted.
Czarek




[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Pulse Audio]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux