2023-06-12 09:06:31

by YingKun Meng

[permalink] [raw]
Subject: [ PATCH v2 0/3] Add Loongson I2S controller support

Hi all,

This patchset adds support for Loongson I2S controller, and
introduce a ASoC machine driver for loongson platform.

The Loongson I2S controller is available on Loongson
7a2000/2k2000 chips, works as a PCI device. It has two
private DMA controllers used to playback and capture.
Each DMA controller has one channel.

The ASoC machine driver adds support for audio device which
using loongson I2S controller to tranfser the audio data.
The audio device uses "PRP0001" as its ACPI device ID, which
provides a means to use the existing DT-compatible device
identification in ACPI.


Thanks.

---
v2:
* Use ACPI bindings to reference I2S and codec nodes.
* Add 'required' restriction for sound-dai property in
dt-bindings.
* Fix build errors from [email protected].
* Replace pci_xxx() APIs with pcim_xxx() APIs.
* mirror changes in log printing.

v1:
* Add support for Loongson I2S controller.

Yingkun Meng (3):
ASoC: Add support for Loongson I2S controller
ASoC: loongson: Add Loongson ASoC Sound Card Support
ASoC: dt-bindings: Add support for Loongson audio card

.../sound/loongson,ls-audio-card.yaml | 70 +++
sound/soc/Kconfig | 1 +
sound/soc/Makefile | 1 +
sound/soc/loongson/Kconfig | 26 +
sound/soc/loongson/Makefile | 8 +
sound/soc/loongson/loongson_card.c | 237 +++++++++
sound/soc/loongson/loongson_i2s.c | 213 ++++++++
sound/soc/loongson/loongson_i2s.h | 70 +++
sound/soc/loongson/loongson_i2s_pci.c | 500 ++++++++++++++++++
9 files changed, 1126 insertions(+)
create mode 100644 Documentation/devicetree/bindings/sound/loongson,ls-audio-card.yaml
create mode 100644 sound/soc/loongson/Kconfig
create mode 100644 sound/soc/loongson/Makefile
create mode 100644 sound/soc/loongson/loongson_card.c
create mode 100644 sound/soc/loongson/loongson_i2s.c
create mode 100644 sound/soc/loongson/loongson_i2s.h
create mode 100644 sound/soc/loongson/loongson_i2s_pci.c


base-commit: 62a97bea5cce5317d6d7630f7bcf0cdf5333e269
--
2.33.0



2023-06-12 19:23:23

by Mark Brown

[permalink] [raw]
Subject: Re: [ PATCH v2 0/3] Add Loongson I2S controller support

On Mon, Jun 12, 2023 at 04:50:48PM +0800, YingKun Meng wrote:
> Hi all,
>
> This patchset adds support for Loongson I2S controller, and
> introduce a ASoC machine driver for loongson platform.
>
> The Loongson I2S controller is available on Loongson
> 7a2000/2k2000 chips, works as a PCI device. It has two
> private DMA controllers used to playback and capture.
> Each DMA controller has one channel.
>
> The ASoC machine driver adds support for audio device which
> using loongson I2S controller to tranfser the audio data.
> The audio device uses "PRP0001" as its ACPI device ID, which
> provides a means to use the existing DT-compatible device
> identification in ACPI.

The code here all looks broadly good - I did have a few review comments
that I sent in reply to the individual patches but they're more
stylistic than anything too substantial.


Attachments:
(No filename) (876.00 B)
signature.asc (499.00 B)
Download all attachments

2023-06-13 12:58:39

by YingKun Meng

[permalink] [raw]
Subject: Re: [ PATCH v2 0/3] Add Loongson I2S controller support


On 2023/6/13 03:07, Mark Brown wrote:
> On Mon, Jun 12, 2023 at 04:50:48PM +0800, YingKun Meng wrote:
>> Hi all,
>>
>> This patchset adds support for Loongson I2S controller, and
>> introduce a ASoC machine driver for loongson platform.
>>
>> The Loongson I2S controller is available on Loongson
>> 7a2000/2k2000 chips, works as a PCI device. It has two
>> private DMA controllers used to playback and capture.
>> Each DMA controller has one channel.
>>
>> The ASoC machine driver adds support for audio device which
>> using loongson I2S controller to tranfser the audio data.
>> The audio device uses "PRP0001" as its ACPI device ID, which
>> provides a means to use the existing DT-compatible device
>> identification in ACPI.
> The code here all looks broadly good - I did have a few review comments
> that I sent in reply to the individual patches but they're more
> stylistic than anything too substantial.


Thanks