linux-kernel - Re: [PATCH] ASoC: codecs: Add support for AK4458 DAC driver

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20180205060834.7fwbqnyzrfze4cli@rob-hp-laptop>
Date:   Mon, 5 Feb 2018 00:08:34 -0600
From:   Rob Herring <robh@...nel.org>
To:     Cosmin-Gabriel Samoila <cosmin.samoila@....com>
Cc:     broonie@...nel.org, alsa-devel@...a-project.org,
        devicetree@...r.kernel.org, linux-kernel@...r.kernel.org,
        wakasugi.jb@...asahi-kasei.co.jp, shengjiu.wang@....com,
        daniel.baluta@....com, mihai.serban@...il.com, linux-imx@....com,
        Mihai Serban <mihai.serban@....com>
Subject: Re: [PATCH] ASoC: codecs: Add support for AK4458 DAC driver

On Wed, Jan 31, 2018 at 03:20:09PM +0200, Cosmin-Gabriel Samoila wrote:
> The AK4458 is a 32-bit 8ch Premium DAC that corresponds
> to a 768kHz PCM input and an 11.2MHz DSD input at maximum.
> It supports I2S, DSD and TDM modes with 24 or 32 bit MSB
> or 16, 24, 32 LSB formats. Its datasheet is available here:
> https://www.akm.com/akm/en/file/datasheet/AK4458VN.pdf
> 
> Signed-off-by: Junichi Wakasugi <wakasugi.jb@...asahi-kasei.co.jp>
> Signed-off-by: Mihai Serban <mihai.serban@....com>
> Signed-off-by: Shengjiu Wang <shengjiu.wang@....com>
> Signed-off-by: Cosmin-Gabriel Samoila <cosmin.samoila@....com>
> ---
>  Documentation/devicetree/bindings/sound/ak4458.txt |   23 +

Separate patch please.

>  sound/soc/codecs/Kconfig                           |   18 +
>  sound/soc/codecs/Makefile                          |    6 +
>  sound/soc/codecs/ak4458-i2c.c                      |   79 ++
>  sound/soc/codecs/ak4458-spi.c                      |   61 ++
>  sound/soc/codecs/ak4458.c                          | 1132 ++++++++++++++++++++
>  sound/soc/codecs/ak4458.h                          |  127 +++
>  7 files changed, 1446 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/sound/ak4458.txt
>  create mode 100644 sound/soc/codecs/ak4458-i2c.c
>  create mode 100644 sound/soc/codecs/ak4458-spi.c
>  create mode 100644 sound/soc/codecs/ak4458.c
>  create mode 100644 sound/soc/codecs/ak4458.h
> 
> diff --git a/Documentation/devicetree/bindings/sound/ak4458.txt b/Documentation/devicetree/bindings/sound/ak4458.txt
> new file mode 100644
> index 0000000..b9e6eb3
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/sound/ak4458.txt
> @@ -0,0 +1,23 @@
> +AK4458 audio DAC
> +
> +This device supports both I2C and SPI modes.
> +
> +Required properties:
> +
> +- compatible : "asahi-kasei,ak4458"
> +- reg : The I2C address of the device for I2C, the chip select number for SPI.
> +- asahi-kasei,pdn-gpios: A GPIO specifier for the GPIO controlling
> +	the power down & reset pin.

reset-gpios like the Asahi Kasei ADC under review.
 
> +- asahi-kasei,mute-gpios: A GPIO specifier for the GPIO controlling
> +	the soft mute pin.

State the active state.

> +
> +Example:
> +
> +&i2c {
> +	ak4458: ak4458@10 {

dac@10

> +		compatible = "asahi-kasei,ak4458";
> +		reg = <0x10>;
> +		asahi-kasei,pdn-gpios = <&gpio1 10 GPIO_ACTIVE_HIGH>
> +		asahi-kasei,mute-gpios = <&gpio1 11 GPIO_ACTIVE_HIGH>
> +	};
> +};