From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752875AbbAaLh0 (ORCPT ); Sat, 31 Jan 2015 06:37:26 -0500 Received: from pandora.arm.linux.org.uk ([78.32.30.218]:38666 "EHLO pandora.arm.linux.org.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751144AbbAaLhX (ORCPT ); Sat, 31 Jan 2015 06:37:23 -0500 Date: Sat, 31 Jan 2015 11:36:59 +0000 From: Russell King - ARM Linux To: Yakir Yang Cc: Liam Girdwood , Mark Brown , Jaroslav Kysela , Takashi Iwai , Lars-Peter Clausen , Brian Austin , Bard Liao , Max Filippov , Oder Chiou , Arnd Bergmann , Sean Cross , Jyri Sarha , Ben Zhang , linux-kernel@vger.kernel.org, alsa-devel@alsa-project.org, Heiko Stuebner , linux-arm-kernel@lists.infradead.org, Rob Herring , Pawel Moll , Mark Rutland , Ian Campbell , Kumar Gala , devicetree@vger.kernel.org, djkurtz@chromium.org, dbehr@chromoum.org, mmind00@googlemail.com, dianders@chromium.org, marcheu@chromium.org, mark.yao@rock-chips.com, rockchip-discuss@chromium.org Subject: Re: [PATCH v2 12/12] dt-bindings: Add documentation for Rockchip dw-hdmi-audio Message-ID: <20150131113659.GA26493@n2100.arm.linux.org.uk> References: <1422617031-25098-1-git-send-email-ykk@rock-chips.com> <1422618253-27312-1-git-send-email-ykk@rock-chips.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1422618253-27312-1-git-send-email-ykk@rock-chips.com> User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, Jan 30, 2015 at 06:44:13AM -0500, Yakir Yang wrote: > Required properties: > - compatible: platform specific > - cpu-of-node: the device node of cpu_dai > > Signed-off-by: Yakir Yang > --- > Changes in v2: > - remove codec-name and codec-dai-name > - rename rockchip,rockchip-hdmi-audio.txt to rockchip,rockchip-dw-hdmi-audio.txt > > .../bindings/sound/rockchip,rockchip-dw-hdmi-audio.txt | 12 ++++++++++++ > 1 file changed, 12 insertions(+) > create mode 100644 Documentation/devicetree/bindings/sound/rockchip,rockchip-dw-hdmi-audio.txt > > diff --git a/Documentation/devicetree/bindings/sound/rockchip,rockchip-dw-hdmi-audio.txt b/Documentation/devicetree/bindings/sound/rockchip,rockchip-dw-hdmi-audio.txt > new file mode 100644 > index 0000000..5b86eed > --- /dev/null > +++ b/Documentation/devicetree/bindings/sound/rockchip,rockchip-dw-hdmi-audio.txt > @@ -0,0 +1,12 @@ > +Rockchip hdmi audio bindings > + > +Required properties: > +- compatible: platform specific > +- cpu-of-node: the device node of cpu_dai > + > +Example: > + > +sound { > + compatible = "rockchip,rk3288-hdmi-audio"; > + cpu-of-node = <&i2s>; > +}; In patch 11, it looks like you parse a property called i2s-controller. This doesn't appear to be documented. Maybe it's what you call "cpu-of-node" above? -- FTTC broadband for 0.8mile line: currently at 10.5Mbps down 400kbps up according to speedtest.net.