devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
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

      parent reply	other threads:[~2013-05-10  0:24 UTC|newest]

Thread overview: 10+ 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 ` [PATCH V2 1/3] clk: samsung: register audio subsystem clocks using common clock framework Padmavathi Venna
2013-05-10  0:21   ` Tomasz Figa
2013-05-11 10:13     ` Padma Venkat
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  9:54   ` Tomasz Figa
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-10  0:24 ` Tomasz Figa [this message]

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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).