All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Kuankuan.Yang" <ykk-TNX95d0MmH7DzftRWevZcw@public.gmane.org>
To: Mark Brown <broonie-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>,
	Russell King - ARM Linux
	<linux-lFZ/pmaqli7XmaaqVzeoHQ@public.gmane.org>
Cc: "Heiko Stuebner" <heiko-4mtYJXux2i+zQB+pC5nmwQ@public.gmane.org>,
	"Rob Herring" <robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>,
	"Pawel Moll" <pawel.moll-5wv7dgnIgG8@public.gmane.org>,
	"Mark Rutland" <mark.rutland-5wv7dgnIgG8@public.gmane.org>,
	"Ian Campbell"
	<ijc+devicetree-KcIKpvwj1kUDXYZnReoRVg@public.gmane.org>,
	"Kumar Gala" <galak-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org>,
	"Philipp Zabel" <p.zabel-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org>,
	Zubair.Kakakhel-1AXoQHu6uovQT0dZR+AlfA@public.gmane.org,
	"Fabio Estevam"
	<fabio.estevam-KZfg59tc24xl57MIdRCFDg@public.gmane.org>,
	"Shawn Guo" <shawn.guo-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>,
	dri-devel-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org,
	"Daniel Kurtz" <djkurtz-F7+t8E8rja9g9hUCZPvPmw@public.gmane.org>,
	"Sonny Rao" <sonnyrao-F7+t8E8rja9g9hUCZPvPmw@public.gmane.org>,
	"Dylan Reid" <dgreid-F7+t8E8rja9g9hUCZPvPmw@public.gmane.org>,
	"Xu Jianqun" <jay.xu-TNX95d0MmH7DzftRWevZcw@public.gmane.org>,
	"St��phane Marchesin"
	<marcheu-F7+t8E8rja9g9hUCZPvPmw@public.gmane.org>,
	"Lars-Peter Clausen"
	<lars-Qo5EllUWu/uELgA04lAiVw@public.gmane.org>,
	"Brian Austin"
	<brian.austin-jGc1dHjMKG3QT0dZR+AlfA@public.gmane.org>,
	"Bard Liao" <bardliao-Rasf1IRRPZFBDgjK7y7TUQ@public.gmane.org>,
	"Max Filippov" <jcmvbkbc-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>,
	"Oder Chiou" <oder_chiou-Rasf1IRRPZFBDgjK7y7TUQ@public.gmane.org>,
	devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	linux-arm-kernel@lists.
Subject: Re: [PATCH 5/6] dt-bindings: Add documentation for Rockchip hdmi-audio
Date: Mon, 15 Dec 2014 21:10:26 +0800	[thread overview]
Message-ID: <548EDDC2.9050104@rock-chips.com> (raw)
In-Reply-To: <20141215120205.GZ11764-GFdadSzt00ze9xe1eoZjHA@public.gmane.org>

Hi Mark & Russell:

thanks for your replay, what your say is right, those dt is merely 
describing the Linux kernel internals.
Instead of transmit cpu-of-node & codec-name & codec-dai-name through 
dt, i can give those an const
name to rockchip-hdmi-audio.c .

In that way, dt will only need compatible for creating sound device. is 
it ok  ?
sound {
     compatible = "rockchip,rk3288-hdmi-audio";
     status = "okay";
}
could you give me some advises.


Best Regards.

在 2014年12月15日 20:02, Mark Brown 写道:
> On Mon, Dec 15, 2014 at 10:40:29AM +0000, Russell King - ARM Linux wrote:
>
>> Including details like this (because ASoC needs a separate DT node) is
>> the wrong approach.
> And indeed there should be no Linux-internal reason for that - we should
> be able to use whatever DT node makes sense, if there's some reason
> that's not possible we should fix that (we do have several drivers doing
> this sort of thing already though).


--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

WARNING: multiple messages have this Message-ID (diff)
From: ykk@rock-chips.com (Kuankuan.Yang)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 5/6] dt-bindings: Add documentation for Rockchip hdmi-audio
Date: Mon, 15 Dec 2014 21:10:26 +0800	[thread overview]
Message-ID: <548EDDC2.9050104@rock-chips.com> (raw)
In-Reply-To: <20141215120205.GZ11764@sirena.org.uk>

Hi Mark & Russell:

thanks for your replay, what your say is right, those dt is merely 
describing the Linux kernel internals.
Instead of transmit cpu-of-node & codec-name & codec-dai-name through 
dt, i can give those an const
name to rockchip-hdmi-audio.c .

In that way, dt will only need compatible for creating sound device. is 
it ok  ?
sound {
     compatible = "rockchip,rk3288-hdmi-audio";
     status = "okay";
}
could you give me some advises.


Best Regards.

? 2014?12?15? 20:02, Mark Brown ??:
> On Mon, Dec 15, 2014 at 10:40:29AM +0000, Russell King - ARM Linux wrote:
>
>> Including details like this (because ASoC needs a separate DT node) is
>> the wrong approach.
> And indeed there should be no Linux-internal reason for that - we should
> be able to use whatever DT node makes sense, if there's some reason
> that's not possible we should fix that (we do have several drivers doing
> this sort of thing already though).

WARNING: multiple messages have this Message-ID (diff)
From: "Kuankuan.Yang" <ykk@rock-chips.com>
To: Mark Brown <broonie@kernel.org>,
	Russell King - ARM Linux <linux@arm.linux.org.uk>
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>,
	"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 21:10:26 +0800	[thread overview]
Message-ID: <548EDDC2.9050104@rock-chips.com> (raw)
In-Reply-To: <20141215120205.GZ11764@sirena.org.uk>

Hi Mark & Russell:

thanks for your replay, what your say is right, those dt is merely 
describing the Linux kernel internals.
Instead of transmit cpu-of-node & codec-name & codec-dai-name through 
dt, i can give those an const
name to rockchip-hdmi-audio.c .

In that way, dt will only need compatible for creating sound device. is 
it ok  ?
sound {
     compatible = "rockchip,rk3288-hdmi-audio";
     status = "okay";
}
could you give me some advises.


Best Regards.

在 2014年12月15日 20:02, Mark Brown 写道:
> On Mon, Dec 15, 2014 at 10:40:29AM +0000, Russell King - ARM Linux wrote:
>
>> Including details like this (because ASoC needs a separate DT node) is
>> the wrong approach.
> And indeed there should be no Linux-internal reason for that - we should
> be able to use whatever DT node makes sense, if there's some reason
> that's not possible we should fix that (we do have several drivers doing
> this sort of thing already though).



  parent reply	other threads:[~2014-12-15 13:10 UTC|newest]

Thread overview: 66+ 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 ` Yakir Yang
2014-12-15  2:11 ` 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 4/6] ASoC: rockchip-hdmi-audio: add sound driver for hdmi audio Yakir Yang
2014-12-15  2:11   ` Yakir Yang
2014-12-15  2:11   ` Yakir Yang
2014-12-15  2:55 ` [PATCH 5/6] dt-bindings: Add documentation for Rockchip hdmi-audio Yakir Yang
2014-12-15  2:55   ` Yakir Yang
2014-12-15  2:55   ` Yakir Yang
     [not found]   ` <1418612114-19692-1-git-send-email-ykk-TNX95d0MmH7DzftRWevZcw@public.gmane.org>
2014-12-15  9:02     ` Lars-Peter Clausen
2014-12-15  9:02       ` Lars-Peter Clausen
2014-12-15  9:02       ` Lars-Peter Clausen
2014-12-15  9:28       ` ykk
     [not found]       ` <548EA3B9.5080407-Qo5EllUWu/uELgA04lAiVw@public.gmane.org>
2014-12-15  9:46         ` Kuankuan.Yang
2014-12-15  9:46           ` Kuankuan.Yang
2014-12-15  9:46           ` Kuankuan.Yang
2014-12-15 10:40     ` Russell King - ARM Linux
2014-12-15 10:40       ` Russell King - ARM Linux
2014-12-15 10:40       ` Russell King - ARM Linux
2014-12-15 12:02       ` Mark Brown
2014-12-15 12:02         ` Mark Brown
2014-12-15 12:02         ` Mark Brown
     [not found]         ` <20141215120205.GZ11764-GFdadSzt00ze9xe1eoZjHA@public.gmane.org>
2014-12-15 13:10           ` Kuankuan.Yang [this message]
2014-12-15 13:10             ` Kuankuan.Yang
2014-12-15 13:10             ` Kuankuan.Yang
     [not found]             ` <548EDDC2.9050104-TNX95d0MmH7DzftRWevZcw@public.gmane.org>
2014-12-15 16:18               ` Mark Brown
2014-12-15 16:18                 ` Mark Brown
2014-12-15 16:18                 ` Mark Brown
     [not found]                 ` <20141215161827.GE11764-GFdadSzt00ze9xe1eoZjHA@public.gmane.org>
2014-12-16  2:14                   ` Kuankuan.Yang
2014-12-16  2:14                     ` Kuankuan.Yang
2014-12-16  2:14                     ` Kuankuan.Yang
2014-12-16  2:38                   ` Kuankuan.Yang
2014-12-16  2:38                     ` Kuankuan.Yang
2014-12-16  2:38                     ` Kuankuan.Yang
2014-12-16  6:17                   ` Kuankuan.Yang
2014-12-16  6:17                     ` Kuankuan.Yang
2014-12-16  6:17                     ` Kuankuan.Yang
     [not found]                     ` <548FCE8F.7090106-TNX95d0MmH7DzftRWevZcw@public.gmane.org>
2014-12-16  9:34                       ` Russell King - ARM Linux
2014-12-16  9:34                         ` Russell King - ARM Linux
2014-12-16  9:34                         ` Russell King - ARM Linux
     [not found]                         ` <20141216093443.GN11285-l+eeeJia6m9vn6HldHNs0ANdhmdF6hFW@public.gmane.org>
2014-12-16  9:48                           ` Kuankuan.Yang
2014-12-16  9:48                             ` Kuankuan.Yang
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  2:56   ` Yakir Yang
2014-12-15  2:56   ` Yakir Yang
     [not found] ` <1418609494-15820-1-git-send-email-ykk-TNX95d0MmH7DzftRWevZcw@public.gmane.org>
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     ` Yakir Yang
2014-12-15 10:38   ` [RFC PATCH 0/6] Those patches is used for dw_hdmi audio Russell King - ARM Linux
2014-12-15 10:38     ` Russell King - ARM Linux
2014-12-15 10:38     ` Russell King - ARM Linux
2014-12-15 11:45     ` Kuankuan.Yang
2014-12-15 11:52       ` Kuankuan.Yang
2014-12-15 11:52         ` Kuankuan.Yang
2014-12-15 11:52         ` Kuankuan.Yang
2014-12-15 12:00         ` Russell King - ARM Linux
2014-12-15 12:00           ` Russell King - ARM Linux
2014-12-15 12:00           ` Russell King - ARM Linux
2014-12-15 12:46           ` Kuankuan.Yang
2014-12-15 12:46             ` Kuankuan.Yang
2014-12-15 12:46             ` Kuankuan.Yang
2014-12-15 15:48           ` Jean-Francois Moine
2014-12-15 15:48             ` Jean-Francois Moine
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=548EDDC2.9050104@rock-chips.com \
    --to=ykk-tnx95d0mmh7dzftrwevzcw@public.gmane.org \
    --cc=Zubair.Kakakhel-1AXoQHu6uovQT0dZR+AlfA@public.gmane.org \
    --cc=bardliao-Rasf1IRRPZFBDgjK7y7TUQ@public.gmane.org \
    --cc=brian.austin-jGc1dHjMKG3QT0dZR+AlfA@public.gmane.org \
    --cc=broonie-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org \
    --cc=devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=dgreid-F7+t8E8rja9g9hUCZPvPmw@public.gmane.org \
    --cc=djkurtz-F7+t8E8rja9g9hUCZPvPmw@public.gmane.org \
    --cc=dri-devel-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org \
    --cc=fabio.estevam-KZfg59tc24xl57MIdRCFDg@public.gmane.org \
    --cc=galak-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org \
    --cc=heiko-4mtYJXux2i+zQB+pC5nmwQ@public.gmane.org \
    --cc=ijc+devicetree-KcIKpvwj1kUDXYZnReoRVg@public.gmane.org \
    --cc=jay.xu-TNX95d0MmH7DzftRWevZcw@public.gmane.org \
    --cc=jcmvbkbc-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org \
    --cc=lars-Qo5EllUWu/uELgA04lAiVw@public.gmane.org \
    --cc=linux-arm-kernel@lists. \
    --cc=linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=linux-lFZ/pmaqli7XmaaqVzeoHQ@public.gmane.org \
    --cc=marcheu-F7+t8E8rja9g9hUCZPvPmw@public.gmane.org \
    --cc=mark.rutland-5wv7dgnIgG8@public.gmane.org \
    --cc=oder_chiou-Rasf1IRRPZFBDgjK7y7TUQ@public.gmane.org \
    --cc=p.zabel-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org \
    --cc=pawel.moll-5wv7dgnIgG8@public.gmane.org \
    --cc=robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org \
    --cc=shawn.guo-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org \
    --cc=sonnyrao-F7+t8E8rja9g9hUCZPvPmw@public.gmane.org \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.