Re: [PATCH RESEND 00/31] AMD SOF based generic SoundWire machine driver

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

 



On 01/08/24 16:27, Charles Keepax wrote:
> On Thu, Aug 01, 2024 at 02:44:15PM +0530, Vijendar Mukunda wrote:
>> This patch series majorly consists of below changes.
>> - Rename structures, macros and codec helper names used in Intel
>> SoundWire generic driver to make it generic.
>> - Move Intel SoundWire driver common structures, macros and codec
>> helper functions to common placeholder so that it can be used by
>> other platform machine driver.
>> - Refactor few SoundWire common codec helper functions.
>> - AMD SOF based generic SoundWire machine driver for ACP 6.3 variant.
>>
>> This work started a couple of months ago to avoid duplication of code
>> that wasn't really Intel-specific in the "sof_sdw" machine driver.
>> The code went through multiple iterations, was tested for multiple weeks
>> and a couple of build issues reported by the Intel kbuild bots were
>> corrected.
>>
>> This is the initial version of SoundWire machine driver for AMD
>> platforms. Additional code refactoring will be done in the next step on
>> the AMD side.
>>
>> Link: https://github.com/thesofproject/linux/pull/5068
>> Reviewed-by: Bard Liao <yung-chuan.liao@xxxxxxxxxxxxxxx>
>> Reviewed-by: Pierre-Louis Bossart <pierre-louis.bossart@xxxxxxxxxxxxxxx>
>>
> Would be good to also include the linux-sound mailing list on the
> cover letter, if not the individuals given they are basically the
> same on each patch.
>
> Also assuming your mail timed out again, you should be able to do
> a separate git send-email for the later patches using the
> --in-reply-to option to give the Message-ID of the cover letter
> and they should append to the same chain.
>
> Thanks,
> Charles

Thanks, Charles, for your suggestion. I have shared the rest of the
Patch series.

-
Vijendar



[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