From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S965913AbdCXP1T (ORCPT ); Fri, 24 Mar 2017 11:27:19 -0400 Received: from mail-ot0-f196.google.com ([74.125.82.196]:34248 "EHLO mail-ot0-f196.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S936389AbdCXP0T (ORCPT ); Fri, 24 Mar 2017 11:26:19 -0400 Date: Fri, 24 Mar 2017 10:26:13 -0500 From: Rob Herring To: Ryan Lee Cc: lgirdwood@gmail.com, broonie@kernel.org, mark.rutland@arm.com, perex@perex.cz, tiwai@suse.com, kuninori.morimoto.gx@renesas.com, arnd@arndb.de, ckeepax@opensource.wolfsonmicro.com, lars@metafoo.de, bardliao@realtek.com, nh6z@nh6z.net, KCHSU0@nuvoton.com, axel.lin@ingics.com, romain.perier@collabora.com, petr@barix.com, srinivas.kandagatla@linaro.org, oder_chiou@realtek.com, Paul.Handrigan@cirrus.com, alsa-devel@alsa-project.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, dgreid@google.com Subject: Re: [PATCH] ASoC: Add support for Maxim Integrated MAX98927 Amplifier Message-ID: <20170324152613.qzvsviznhiz5f3sj@rob-hp-laptop> References: <1489970423-29353-1-git-send-email-ryans.lee@maximintegrated.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1489970423-29353-1-git-send-email-ryans.lee@maximintegrated.com> User-Agent: Mutt/1.6.2-neo (2016-08-21) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, Mar 20, 2017 at 09:40:23AM +0900, Ryan Lee wrote: > Signed-off-by: Ryan Lee > --- > Resubmit the intial version of MAX98927 driver. Added all fixes into the initial patch. > > .../devicetree/bindings/sound/max98927.txt | 39 + > sound/soc/codecs/Kconfig | 5 + > sound/soc/codecs/Makefile | 2 + > sound/soc/codecs/max98927.c | 1040 ++++++++++++++++++++ > sound/soc/codecs/max98927.h | 267 +++++ > 5 files changed, 1353 insertions(+) > create mode 100755 Documentation/devicetree/bindings/sound/max98927.txt > mode change 100644 => 100755 sound/soc/codecs/Kconfig > mode change 100644 => 100755 sound/soc/codecs/Makefile > create mode 100755 sound/soc/codecs/max98927.c > create mode 100755 sound/soc/codecs/max98927.h > > diff --git a/Documentation/devicetree/bindings/sound/max98927.txt b/Documentation/devicetree/bindings/sound/max98927.txt > new file mode 100755 > index 0000000..d610879 > --- /dev/null > +++ b/Documentation/devicetree/bindings/sound/max98927.txt > @@ -0,0 +1,39 @@ > +max98927 audio CODEC > + > +This device supports I2C. Can we combine this and the other max9892x bindings. They all look similar. > + > +Required properties: > + > + - compatible : "maxim,max98927" > + > + - maxim,vmon-slot-no : slot number used to send voltage information While vendor prefix is preferred, so is using the same property names. Either use the existing names or deprecate the old ones in favor of these. > + or in inteleave mode this will be used as > + interleave slot. > + This property can be set to values from 0 to 15 for slot 0 to 15. > + Default value is 0. > + > + - maxim,imon-slot-no : slot number used to send current information > + This property can be set to values from 0 to 15 for slot 0 to 15. > + Default value is 0. > + > + - maxim,interleave-mode : When using two MAX98927 in a system it is > + possible to create ADC data that that will > + overflow the frame size. Digital Audio Interleave > + mode provides a means to output VMON and IMON data > + from two devices on a single DOUT line when running > + smaller frames sizes such as 32 BCLKS per LRCLK or > + 48 BCLKS per LRCLK. > + This property can be set to values from 0 to 1 for off and on. > + Default value is 0. > + > + - reg : the I2C address of the device for I2C > + > +Example: > + > +codec: max98927@3a { > + compatible = "maxim,max98927"; > + maxim,vmon-slot-no = <0>; > + maxim,imon-slot-no = <1>; > + maxim,interleave-mode = <0>; > + reg = <0x3a>; > +}; From mboxrd@z Thu Jan 1 00:00:00 1970 From: Rob Herring Subject: Re: [PATCH] ASoC: Add support for Maxim Integrated MAX98927 Amplifier Date: Fri, 24 Mar 2017 10:26:13 -0500 Message-ID: <20170324152613.qzvsviznhiz5f3sj@rob-hp-laptop> References: <1489970423-29353-1-git-send-email-ryans.lee@maximintegrated.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline In-Reply-To: <1489970423-29353-1-git-send-email-ryans.lee@maximintegrated.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: alsa-devel-bounces@alsa-project.org Sender: alsa-devel-bounces@alsa-project.org To: Ryan Lee Cc: mark.rutland@arm.com, alsa-devel@alsa-project.org, kuninori.morimoto.gx@renesas.com, lgirdwood@gmail.com, tiwai@suse.com, srinivas.kandagatla@linaro.org, romain.perier@collabora.com, bardliao@realtek.com, lars@metafoo.de, axel.lin@ingics.com, Paul.Handrigan@cirrus.com, devicetree@vger.kernel.org, arnd@arndb.de, nh6z@nh6z.net, broonie@kernel.org, ckeepax@opensource.wolfsonmicro.com, dgreid@google.com, oder_chiou@realtek.com, KCHSU0@nuvoton.com, linux-kernel@vger.kernel.org, petr@barix.com List-Id: devicetree@vger.kernel.org On Mon, Mar 20, 2017 at 09:40:23AM +0900, Ryan Lee wrote: > Signed-off-by: Ryan Lee > --- > Resubmit the intial version of MAX98927 driver. Added all fixes into the initial patch. > > .../devicetree/bindings/sound/max98927.txt | 39 + > sound/soc/codecs/Kconfig | 5 + > sound/soc/codecs/Makefile | 2 + > sound/soc/codecs/max98927.c | 1040 ++++++++++++++++++++ > sound/soc/codecs/max98927.h | 267 +++++ > 5 files changed, 1353 insertions(+) > create mode 100755 Documentation/devicetree/bindings/sound/max98927.txt > mode change 100644 => 100755 sound/soc/codecs/Kconfig > mode change 100644 => 100755 sound/soc/codecs/Makefile > create mode 100755 sound/soc/codecs/max98927.c > create mode 100755 sound/soc/codecs/max98927.h > > diff --git a/Documentation/devicetree/bindings/sound/max98927.txt b/Documentation/devicetree/bindings/sound/max98927.txt > new file mode 100755 > index 0000000..d610879 > --- /dev/null > +++ b/Documentation/devicetree/bindings/sound/max98927.txt > @@ -0,0 +1,39 @@ > +max98927 audio CODEC > + > +This device supports I2C. Can we combine this and the other max9892x bindings. They all look similar. > + > +Required properties: > + > + - compatible : "maxim,max98927" > + > + - maxim,vmon-slot-no : slot number used to send voltage information While vendor prefix is preferred, so is using the same property names. Either use the existing names or deprecate the old ones in favor of these. > + or in inteleave mode this will be used as > + interleave slot. > + This property can be set to values from 0 to 15 for slot 0 to 15. > + Default value is 0. > + > + - maxim,imon-slot-no : slot number used to send current information > + This property can be set to values from 0 to 15 for slot 0 to 15. > + Default value is 0. > + > + - maxim,interleave-mode : When using two MAX98927 in a system it is > + possible to create ADC data that that will > + overflow the frame size. Digital Audio Interleave > + mode provides a means to output VMON and IMON data > + from two devices on a single DOUT line when running > + smaller frames sizes such as 32 BCLKS per LRCLK or > + 48 BCLKS per LRCLK. > + This property can be set to values from 0 to 1 for off and on. > + Default value is 0. > + > + - reg : the I2C address of the device for I2C > + > +Example: > + > +codec: max98927@3a { > + compatible = "maxim,max98927"; > + maxim,vmon-slot-no = <0>; > + maxim,imon-slot-no = <1>; > + maxim,interleave-mode = <0>; > + reg = <0x3a>; > +};