From: Russell King - ARM Linux <linux@arm.linux.org.uk>
To: Yakir Yang <ykk@rock-chips.com>
Cc: "Heiko Stuebner" <heiko@sntech.de>,
"Rob Herring" <robh+dt@kernel.org>,
"Pawel Moll" <pawel.moll@arm.com>,
"Mark Rutland" <mark.rutland@arm.com>,
"Ian Campbell" <ijc+devicetree@hellion.org.uk>,
"Kumar Gala" <galak@codeaurora.org>,
"Philipp Zabel" <p.zabel@pengutronix.de>,
Zubair.Kakakhel@imgtec.com,
"Fabio Estevam" <fabio.estevam@freescale.com>,
"Shawn Guo" <shawn.guo@linaro.org>,
dri-devel@lists.freedesktop.org,
"Daniel Kurtz" <djkurtz@chromium.org>,
"Sonny Rao" <sonnyrao@chromium.org>,
"Dylan Reid" <dgreid@chromium.org>,
"Xu Jianqun" <jay.xu@rock-chips.com>,
"St��phane Marchesin" <marcheu@chromium.org>,
"Lars-Peter Clausen" <lars@metafoo.de>,
"Brian Austin" <brian.austin@cirrus.com>,
"Bard Liao" <bardliao@realtek.com>,
"Max Filippov" <jcmvbkbc@gmail.com>,
"Oder Chiou" <oder_chiou@realtek.com>,
devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
linux-arm-kernel@lists.infradead.org,
linux-rockchip@lists.infradead.org, alsa-devel@alsa-project.org,
"Arnd Bergmann" <arnd@arndb.de>,
"David Airlie" <airlied@linux.ie>, "Sean Cross" <xobs@kosagi.com>,
"Grant Likely" <grant.likely@linaro.org>,
"Greg Kroah-Hartman" <gregkh@linuxfoundation.org>,
"Ben Zhang" <benzh@chromium.org>,
"Liam Girdwood" <lgirdwood@gmail.com>,
"Jaroslav Kysela" <perex@perex.cz>,
"Mark Brown" <broonie@kernel.org>, "Takashi Iwai" <tiwai@suse.de>,
"Jyri Sarha" <jsarha@ti.com>
Subject: Re: [PATCH 5/6] dt-bindings: Add documentation for Rockchip hdmi-audio
Date: Mon, 15 Dec 2014 10:40:29 +0000 [thread overview]
Message-ID: <20141215104029.GE11285@n2100.arm.linux.org.uk> (raw)
In-Reply-To: <1418612114-19692-1-git-send-email-ykk@rock-chips.com>
On Mon, Dec 15, 2014 at 10:55:14AM +0800, Yakir Yang wrote:
> Required properties:
> - compatible: platform specific
> - cpu-of-node: the device node of cpu_dai
> - codec-name: the dw-hdmi codec's device name
> - codec-dai-name: the dw-hdmi codec's dai name
This is not actually a separate physical device from the main dw-hdmi
device. What you are doing is merely describing the Linux kernel
internals in DT, which is not what DT is supposed to be about. DT is
supposed to describe the hardware in an operating system independent
manner.
Including details like this (because ASoC needs a separate DT node) is
the wrong approach.
--
FTTC broadband for 0.8mile line: currently at 9.5Mbps down 400kbps up
according to speedtest.net.
next prev parent reply other threads:[~2014-12-15 10:40 UTC|newest]
Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-12-15 2:11 [RFC PATCH 0/6] Those patches is used for dw_hdmi audio Yakir Yang
2014-12-15 2:11 ` [PATCH 1/6] drm: bridge/dw_hdmi: add hdmi audio config interfaces Yakir Yang
2014-12-15 2:11 ` [PATCH 2/6] drm: bridge/bridge: add support for rk3288 hdmi-audio Yakir Yang
2014-12-15 2:11 ` [PATCH 3/6] ASoC: dw-hdmi-audio: add codec driver for dw hdmi audio Yakir Yang
2014-12-15 2:11 ` [PATCH 4/6] ASoC: rockchip-hdmi-audio: add sound driver for " Yakir Yang
2014-12-15 2:55 ` [PATCH 5/6] dt-bindings: Add documentation for Rockchip hdmi-audio Yakir Yang
2014-12-15 9:02 ` Lars-Peter Clausen
2014-12-15 9:46 ` Kuankuan.Yang
2014-12-15 10:40 ` Russell King - ARM Linux [this message]
2014-12-15 12:02 ` Mark Brown
2014-12-15 13:10 ` Kuankuan.Yang
2014-12-15 16:18 ` Mark Brown
2014-12-16 2:14 ` Kuankuan.Yang
2014-12-16 2:38 ` Kuankuan.Yang
2014-12-16 6:17 ` Kuankuan.Yang
2014-12-16 9:34 ` Russell King - ARM Linux
2014-12-16 9:48 ` Kuankuan.Yang
2014-12-15 2:56 ` [PATCH 6/6] ARM: dts: rockchip: add hdmi audio nodes Yakir Yang
2014-12-15 10:38 ` [RFC PATCH 0/6] Those patches is used for dw_hdmi audio Russell King - ARM Linux
[not found] ` <548EC9E1.5040206@rock-chips.com>
2014-12-15 11:52 ` Kuankuan.Yang
2014-12-15 12:00 ` Russell King - ARM Linux
2014-12-15 12:46 ` Kuankuan.Yang
2014-12-15 15:48 ` Jean-Francois Moine
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20141215104029.GE11285@n2100.arm.linux.org.uk \
--to=linux@arm.linux.org.uk \
--cc=Zubair.Kakakhel@imgtec.com \
--cc=airlied@linux.ie \
--cc=alsa-devel@alsa-project.org \
--cc=arnd@arndb.de \
--cc=bardliao@realtek.com \
--cc=benzh@chromium.org \
--cc=brian.austin@cirrus.com \
--cc=broonie@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=dgreid@chromium.org \
--cc=djkurtz@chromium.org \
--cc=dri-devel@lists.freedesktop.org \
--cc=fabio.estevam@freescale.com \
--cc=galak@codeaurora.org \
--cc=grant.likely@linaro.org \
--cc=gregkh@linuxfoundation.org \
--cc=heiko@sntech.de \
--cc=ijc+devicetree@hellion.org.uk \
--cc=jay.xu@rock-chips.com \
--cc=jcmvbkbc@gmail.com \
--cc=jsarha@ti.com \
--cc=lars@metafoo.de \
--cc=lgirdwood@gmail.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-rockchip@lists.infradead.org \
--cc=marcheu@chromium.org \
--cc=mark.rutland@arm.com \
--cc=oder_chiou@realtek.com \
--cc=p.zabel@pengutronix.de \
--cc=pawel.moll@arm.com \
--cc=perex@perex.cz \
--cc=robh+dt@kernel.org \
--cc=shawn.guo@linaro.org \
--cc=sonnyrao@chromium.org \
--cc=tiwai@suse.de \
--cc=xobs@kosagi.com \
--cc=ykk@rock-chips.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®