linux-kernel - Re: [PATCH 0/3] ASoC: add dio2125 amplifier support

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: <1488903977.28627.4.camel@baylibre.com>
Date:   Tue, 07 Mar 2017 17:26:17 +0100
From:   Jerome Brunet <jbrunet@...libre.com>
To:     Mark Brown <broonie@...nel.org>,
        Liam Girdwood <lgirdwood@...il.com>
Cc:     alsa-devel@...a-project.org, devicetree@...r.kernel.org,
        linux-kernel@...r.kernel.org, Kevin Hilman <khilman@...libre.com>,
        Neil Armstrong <narmstrong@...libre.com>
Subject: Re: [PATCH 0/3] ASoC: add dio2125 amplifier support

Hi Mark,

In this series, you applied patch 2 and 3 but not patch 1.
(so the dioo prefix is used but not documented)

Is it on purpose ?
The patch does not have the "ASoC" prefix, this maybe why it was
missed. Since, the file does no belong to alsa, I wasn't sure whether
to put the prefix or not.

Regards

Jerome

On Mon, 2017-03-06 at 18:44 +0100, Jerome Brunet wrote:
> This patchset adds the support for the dio2125 from Dioo
> microcircuits.
> The dio2125 is audio driver with adjustable gain. The EN pad allows
> to
> enable or disable the outputs of the driver.
> 
> The link to the datasheet is not provided as part of the commit. It
> is not
> provided the manufacturer and it is doubtful it will still be
> available a
> few years from now. Still provided here in case anyone is interested
> [0].
> 
> This patchset has been tested on Amlogic's meson-gxbb-p200 board.
> 
> [0]: http://www.winine.com/Line%E9%9F%B3/DIO2125XM10%20V1.0.pdf
> 
> Jerome Brunet (3):
>   dt-bindings: add vendor prefix for Dioo
>   ASoC: dio2125: add dio2125 amp driver
>   ASoC: dio2125: add dt-bindings
> 
>  .../devicetree/bindings/sound/dioo,dio2125.txt     |  12 +++
>  .../devicetree/bindings/vendor-prefixes.txt        |   1 +
>  sound/soc/codecs/Kconfig                           |   5 +
>  sound/soc/codecs/Makefile                          |   2 +
>  sound/soc/codecs/dio2125.c                         | 120
> +++++++++++++++++++++
>  5 files changed, 140 insertions(+)
>  create mode 100644
> Documentation/devicetree/bindings/sound/dioo,dio2125.txt
>  create mode 100644 sound/soc/codecs/dio2125.c
>