On 7/17/21 1:22 AM, Pierre-Louis Bossart wrote: > > > On 7/16/21 11:00 PM, Vijendar Mukunda wrote: >> This adds an ASoC driver for the ACP (Audio CoProcessor) >> block on AMD Vangogh APU. >> >> Vijendar Mukunda (12): >> ASoC: amd: add Vangogh ACP5x IP register header >> ASoC: amd: add Vangogh ACP PCI driver >> ASoc: amd: add acp5x init/de-init functions >> ASoC: amd: create acp5x platform devices >> ASoC: amd: add ACP5x PCM platform driver >> ASoC: amd: irq handler changes for ACP5x PCM dma driver >> ASoC: amd: add ACP5x pcm dma driver ops >> ASoC: amd: add vangogh i2s controller driver >> ASoC: amd: add vangogh i2s dai driver ops >> ASoC: amd: add vangogh pci driver pm ops >> ASoC: amd: add vangogh i2s dma driver pm ops >> ASoC: amd: enable vangogh acp5x driver build >> --- >> v1 -> v2: fixed review comments > > usually the version number is at the patchset level, here you included v1, v2 and v3 patches and all but the last one use the RESEND prefix? > > is this intentional? As We have respin the patch series, we have updated cover letter with version as V2. Got your point. Cover letter shouldn't have any version number. We have updated patches as v2 version with fixes. It's my bad. For last patch, We have removed extra stuff which we have added earlier and marked patch version as V3. > >> --- >> sound/soc/amd/Kconfig | 9 + >> sound/soc/amd/Makefile | 1 + >> sound/soc/amd/vangogh/Makefile | 9 + >> sound/soc/amd/vangogh/acp5x-i2s.c | 429 ++++++++++++++++ >> sound/soc/amd/vangogh/acp5x-pcm-dma.c | 525 ++++++++++++++++++++ >> sound/soc/amd/vangogh/acp5x.h | 192 +++++++ >> sound/soc/amd/vangogh/pci-acp5x.c | 327 ++++++++++++ >> sound/soc/amd/vangogh/vg_chip_offset_byte.h | 337 +++++++++++++ >> 8 files changed, 1829 insertions(+) >> create mode 100644 sound/soc/amd/vangogh/Makefile >> create mode 100644 sound/soc/amd/vangogh/acp5x-i2s.c >> create mode 100644 sound/soc/amd/vangogh/acp5x-pcm-dma.c >> create mode 100644 sound/soc/amd/vangogh/acp5x.h >> create mode 100644 sound/soc/amd/vangogh/pci-acp5x.c >> create mode 100644 sound/soc/amd/vangogh/vg_chip_offset_byte.h >>