From: Tomasz Figa <tomasz.figa@gmail.com>
To: linux-arm-kernel@lists.infradead.org
Cc: Padmavathi Venna <padma.v@samsung.com>,
linux-samsung-soc@vger.kernel.org,
devicetree-discuss@lists.ozlabs.org, padma.kvr@gmail.com,
sbkim73@samsung.com, kgene.kim@samsung.com,
broonie@opensource.wolfsonmicro.com, thomas.abraham@linaro.org,
s.nawrocki@samsung.com, Mike Turquette <mturquette@linaro.org>
Subject: Re: [PATCH V2 0/3] clk: Samsung: audss: Register audio subsytem clocks using common clk framework
Date: Fri, 10 May 2013 02:24:38 +0200 [thread overview]
Message-ID: <1561518.UR9S1ZDj1n@flatron> (raw)
In-Reply-To: <1367909016-19657-1-git-send-email-padma.v@samsung.com>
Hi Padmavathi,
On Tuesday 07 of May 2013 12:13:33 Padmavathi Venna wrote:
> Samsung S5PV210 and Exynos SoC has a separate subsystem for audio. This
> subsystem has a internal clock controller which controls i2s0 and pcm0
> clocks. This patch series adds the Samsung audio subsytem clock to the
> common clock framework and provides the I2S controllers clock
> information in the dtsi file.
>
> This patch series is dependent on
> https://patchwork.kernel.org/patch/2532091/
> http://www.spinics.net/lists/linux-kbuild/msg07616.html
>
> This patch series is made based on Kukjin Kim for-next branch
>
> Changes since V1:
> - Reworked on all review comments by Sylwester Nawrocki
> - Added a header file for all clock indexes as requested by
Sylwester
> - Added different compatible names for s5pv210, exynos4 and
exynos5
> - Registered the pcm clocks with common clock framework
>
> Padmavathi Venna (3):
> clk: samsung: register audio subsystem clocks using common clock
> framework
> ARM: dts: add Exynos audio subsystem clock controller node
> ARM: dts: add clock provider information for i2s controllers in
> Exynos5250
>
> .../bindings/clock/clk-samsung-audss.txt | 64 +++++++++
> arch/arm/boot/dts/exynos5250.dtsi | 16 +++
> drivers/clk/samsung/Makefile | 1 +
> drivers/clk/samsung/clk-samsung-audss.c | 137
> ++++++++++++++++++++ include/dt-bindings/clk/samsung-audss-clk.h
> | 25 ++++ 5 files changed, 243 insertions(+), 0 deletions(-)
> create mode 100644
> Documentation/devicetree/bindings/clock/clk-samsung-audss.txt create
> mode 100644 drivers/clk/samsung/clk-samsung-audss.c
> create mode 100644 include/dt-bindings/clk/samsung-audss-clk.h
Also, I think that this series lacks
Cc: Mike Turquette <mturquette@linaro.org>
as he's the maintainer of the clk subsystem.
Best regards,
Tomasz
WARNING: multiple messages have this Message-ID (diff)
From: tomasz.figa@gmail.com (Tomasz Figa)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH V2 0/3] clk: Samsung: audss: Register audio subsytem clocks using common clk framework
Date: Fri, 10 May 2013 02:24:38 +0200 [thread overview]
Message-ID: <1561518.UR9S1ZDj1n@flatron> (raw)
In-Reply-To: <1367909016-19657-1-git-send-email-padma.v@samsung.com>
Hi Padmavathi,
On Tuesday 07 of May 2013 12:13:33 Padmavathi Venna wrote:
> Samsung S5PV210 and Exynos SoC has a separate subsystem for audio. This
> subsystem has a internal clock controller which controls i2s0 and pcm0
> clocks. This patch series adds the Samsung audio subsytem clock to the
> common clock framework and provides the I2S controllers clock
> information in the dtsi file.
>
> This patch series is dependent on
> https://patchwork.kernel.org/patch/2532091/
> http://www.spinics.net/lists/linux-kbuild/msg07616.html
>
> This patch series is made based on Kukjin Kim for-next branch
>
> Changes since V1:
> - Reworked on all review comments by Sylwester Nawrocki
> - Added a header file for all clock indexes as requested by
Sylwester
> - Added different compatible names for s5pv210, exynos4 and
exynos5
> - Registered the pcm clocks with common clock framework
>
> Padmavathi Venna (3):
> clk: samsung: register audio subsystem clocks using common clock
> framework
> ARM: dts: add Exynos audio subsystem clock controller node
> ARM: dts: add clock provider information for i2s controllers in
> Exynos5250
>
> .../bindings/clock/clk-samsung-audss.txt | 64 +++++++++
> arch/arm/boot/dts/exynos5250.dtsi | 16 +++
> drivers/clk/samsung/Makefile | 1 +
> drivers/clk/samsung/clk-samsung-audss.c | 137
> ++++++++++++++++++++ include/dt-bindings/clk/samsung-audss-clk.h
> | 25 ++++ 5 files changed, 243 insertions(+), 0 deletions(-)
> create mode 100644
> Documentation/devicetree/bindings/clock/clk-samsung-audss.txt create
> mode 100644 drivers/clk/samsung/clk-samsung-audss.c
> create mode 100644 include/dt-bindings/clk/samsung-audss-clk.h
Also, I think that this series lacks
Cc: Mike Turquette <mturquette@linaro.org>
as he's the maintainer of the clk subsystem.
Best regards,
Tomasz
next prev parent reply other threads:[~2013-05-10 0:24 UTC|newest]
Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-05-07 6:43 [PATCH V2 0/3] clk: Samsung: audss: Register audio subsytem clocks using common clk framework Padmavathi Venna
2013-05-07 6:43 ` Padmavathi Venna
2013-05-07 6:43 ` [PATCH V2 1/3] clk: samsung: register audio subsystem clocks using common clock framework Padmavathi Venna
2013-05-07 6:43 ` Padmavathi Venna
2013-05-10 0:21 ` Tomasz Figa
2013-05-10 0:21 ` Tomasz Figa
2013-05-11 10:13 ` Padma Venkat
2013-05-11 10:13 ` Padma Venkat
2013-05-11 11:42 ` Sylwester Nawrocki
2013-05-11 11:42 ` Sylwester Nawrocki
2013-05-07 6:43 ` [PATCH V2 2/3] ARM: dts: add Exynos audio subsystem clock controller node Padmavathi Venna
2013-05-07 6:43 ` Padmavathi Venna
2013-05-07 9:54 ` Tomasz Figa
2013-05-07 9:54 ` Tomasz Figa
2013-05-07 11:14 ` Padma Venkat
2013-05-07 11:14 ` Padma Venkat
2013-05-07 6:43 ` [PATCH V2 3/3] ARM: dts: add clock provider information for i2s controllers in Exynos5250 Padmavathi Venna
2013-05-07 6:43 ` Padmavathi Venna
2013-05-10 0:24 ` Tomasz Figa [this message]
2013-05-10 0:24 ` [PATCH V2 0/3] clk: Samsung: audss: Register audio subsytem clocks using common clk framework Tomasz Figa
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=1561518.UR9S1ZDj1n@flatron \
--to=tomasz.figa@gmail.com \
--cc=broonie@opensource.wolfsonmicro.com \
--cc=devicetree-discuss@lists.ozlabs.org \
--cc=kgene.kim@samsung.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-samsung-soc@vger.kernel.org \
--cc=mturquette@linaro.org \
--cc=padma.kvr@gmail.com \
--cc=padma.v@samsung.com \
--cc=s.nawrocki@samsung.com \
--cc=sbkim73@samsung.com \
--cc=thomas.abraham@linaro.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.