Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1593593 > unrolled thread

[PATCH 0/3] ASoC: add dio2125 amplifier support

Back to article view | Back to linux.kernel

Contents

  [PATCH 0/3] ASoC: add dio2125 amplifier support Jerome Brunet <jbrunet@baylibre.com> - 2017-03-06 20:00 +0100
    [PATCH 1/3] dt-bindings: add vendor prefix for Dioo Jerome Brunet <jbrunet@baylibre.com> - 2017-03-06 20:00 +0100
    [PATCH 3/3] ASoC: dio2125: add dt-bindings Jerome Brunet <jbrunet@baylibre.com> - 2017-03-06 20:50 +0100
      Applied "ASoC: dio2125: add dt-bindings" to the asoc tree Mark Brown <broonie@kernel.org> - 2017-03-07 14:10 +0100
    Re: [PATCH 0/3] ASoC: add dio2125 amplifier support Jerome Brunet <jbrunet@baylibre.com> - 2017-03-07 17:30 +0100
      Re: [PATCH 0/3] ASoC: add dio2125 amplifier support Mark Brown <broonie@kernel.org> - 2017-03-08 12:00 +0100

#1593593 — [PATCH 0/3] ASoC: add dio2125 amplifier support

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

-- 
2.9.3

[toc] | [next] | [standalone]

#1593594 — [PATCH 1/3] dt-bindings: add vendor prefix for Dioo

Dioo (http://http://www.dioo.com) design and produce power analog and
mixed signal IC.

Signed-off-by: Jerome Brunet <jbrunet@baylibre.com>
---
 Documentation/devicetree/bindings/vendor-prefixes.txt | 1 +
 1 file changed, 1 insertion(+)

diff --git a/Documentation/devicetree/bindings/vendor-prefixes.txt b/Documentation/devicetree/bindings/vendor-prefixes.txt
index ec0bfb9bbebd..bb0edca6ec05 100644
--- a/Documentation/devicetree/bindings/vendor-prefixes.txt
+++ b/Documentation/devicetree/bindings/vendor-prefixes.txt
@@ -79,6 +79,7 @@ denx	Denx Software Engineering
 devantech	Devantech, Ltd.
 digi	Digi International Inc.
 digilent	Diglent, Inc.
+dioo	Dioo Microcircuit Co., Ltd
 dlg	Dialog Semiconductor
 dlink	D-Link Corporation
 dmo	Data Modul AG
-- 
2.9.3

[toc] | [prev] | [next] | [standalone]

#1593660 — [PATCH 3/3] ASoC: dio2125: add dt-bindings

Signed-off-by: Jerome Brunet <jbrunet@baylibre.com>
---
 Documentation/devicetree/bindings/sound/dioo,dio2125.txt | 12 ++++++++++++
 1 file changed, 12 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/sound/dioo,dio2125.txt

diff --git a/Documentation/devicetree/bindings/sound/dioo,dio2125.txt b/Documentation/devicetree/bindings/sound/dioo,dio2125.txt
new file mode 100644
index 000000000000..63dbfe0f11d0
--- /dev/null
+++ b/Documentation/devicetree/bindings/sound/dioo,dio2125.txt
@@ -0,0 +1,12 @@
+DIO2125 Audio Driver
+
+Required properties:
+- compatible : "dioo,dio2125"
+- enable-gpios : the gpio connected to the enable pin of the dio2125
+
+Example:
+
+amp: analog-amplifier {
+	compatible = "dioo,dio2125";
+	enable-gpios = <&gpio GPIOH_3 0>;
+};
-- 
2.9.3

[toc] | [prev] | [next] | [standalone]

#1594219 — Applied "ASoC: dio2125: add dt-bindings" to the asoc tree

The patch

   ASoC: dio2125: add dt-bindings

has been applied to the asoc tree at

   git://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git 

All being well this means that it will be integrated into the linux-next
tree (usually sometime in the next 24 hours) and sent to Linus during
the next merge window (or sooner if it is a bug fix), however if
problems are discovered then the patch may be dropped or reverted.  

You may get further e-mails resulting from automated or manual testing
and review of the tree, please engage with people reporting problems and
send followup patches addressing any issues that are reported if needed.

If any updates are required or you are submitting further changes they
should be sent as incremental updates against current git, existing
patches will not be replaced.

Please add any relevant lists and maintainers to the CCs when replying
to this mail.

Thanks,
Mark

From 1e03580b3a7213292fae9a6677edfaa83843fa4e Mon Sep 17 00:00:00 2001
From: Jerome Brunet <jbrunet@baylibre.com>
Date: Mon, 6 Mar 2017 18:44:51 +0100
Subject: [PATCH] ASoC: dio2125: add dt-bindings

Signed-off-by: Jerome Brunet <jbrunet@baylibre.com>
Signed-off-by: Mark Brown <broonie@kernel.org>
---
 Documentation/devicetree/bindings/sound/dioo,dio2125.txt | 12 ++++++++++++
 1 file changed, 12 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/sound/dioo,dio2125.txt

diff --git a/Documentation/devicetree/bindings/sound/dioo,dio2125.txt b/Documentation/devicetree/bindings/sound/dioo,dio2125.txt
new file mode 100644
index 000000000000..63dbfe0f11d0
--- /dev/null
+++ b/Documentation/devicetree/bindings/sound/dioo,dio2125.txt
@@ -0,0 +1,12 @@
+DIO2125 Audio Driver
+
+Required properties:
+- compatible : "dioo,dio2125"
+- enable-gpios : the gpio connected to the enable pin of the dio2125
+
+Example:
+
+amp: analog-amplifier {
+	compatible = "dioo,dio2125";
+	enable-gpios = <&gpio GPIOH_3 0>;
+};
-- 
2.11.0

[toc] | [prev] | [next] | [standalone]

#1594399

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
> 

[toc] | [prev] | [next] | [standalone]

#1595049

[Multipart message — attachments visible in raw view] — view raw

On Tue, Mar 07, 2017 at 05:26:17PM +0100, Jerome Brunet wrote:
> Hi Mark,

Please don't top post, reply in line with needed context.  This allows
readers to readily follow the flow of conversation and understand what
you are talking about and also helps ensure that everything in the
discussion is being addressed.

> 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.

This is a generic DT patch, not part of the bindings, so I'd expect the
DT maintainers to take it.

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web