All of lore.kernel.org
 help / color / mirror / Atom feed
From: Mike Turquette <mturquette@linaro.org>
To: Sylwester Nawrocki <s.nawrocki@samsung.com>,
	Javier Martinez Canillas <javier.martinez@collabora.co.uk>,
	Kukjin Kim <kgene@kernel.org>
Cc: Tomasz Figa <tomasz.figa@gmail.com>,
	Rahul Sharma <rahul.sharma@samsung.com>,
	Arun Kumar K <arun.kk@samsung.com>,
	Doug Anderson <dianders@chromium.org>,
	Olof Johansson <olof@lixom.net>,
	Marek Szyprowski <m.szyprowski@samsung.com>,
	Tobias Jakobi <liquid.acid@gmx.net>,
	Andrzej Hajda <a.hajda@samsung.com>,
	Krzysztof Kozlowski <k.kozlowski@samsung.com>,
	Kevin Hilman <khilman@linaro.org>,
	Joonyoung Shim <jy0922.shim@samsung.com>,
	linux-samsung-soc@vger.kernel.org,
	linux-arm-kernel@lists.infradead.org,
	linux-kernel@vger.kernel.org
Subject: Re: [PATCH 1/2] clk: exynos5420: Add IDs for clocks used in DISP1 power domain
Date: Tue, 20 Jan 2015 09:54:02 -0800	[thread overview]
Message-ID: <20150120175402.22722.84342@quantum> (raw)
In-Reply-To: <54BE6050.1090205@samsung.com>

Quoting Sylwester Nawrocki (2015-01-20 06:04:00)
> Hi,
> 
> On 20/01/15 11:35, Javier Martinez Canillas wrote:
> > When a power domain is powered off on Exynos5420 SoC, the input clocks of
> > the devices attached to this power domain are re-parented to oscclk and
> > restored to the original parent after powering on the power domain.
> > 
> > So a reference to the input and parent clocks for the devices attached to
> > a power domain are needed to be able to do the re-parenting. The DISP1 pd
> > includes modules which uses the following clocks:
> > 
> > ACLK_200_DISP1 (MIXER and HDMILINK)
> > ACLK_300_DISP1 (FIMD1)
> > ACLK_400_DISP1 (Internal Buses)
> > 
> > Each of these clocks are generated as the output of a clock mux so add an
> > ID for all of these clock muxes and their parents to be referenced in the
> > DISP1 power domain device node.
> > 
> > Signed-off-by: Javier Martinez Canillas <javier.martinez@collabora.co.uk>
> 
> The patch looks OK to me, I'm fine with it being merged via Kukjin's tree
> due to the dts dependencies (including other pending dts patches touching
> the arch/arm/boot/dts/exynos5420.dtsi file).
> I think we need also Mike ACK for that, I could also queue the patch for
> the clk tree and create a topic branch, but merging both patches via
> arm-soc seems a more sane option in this case.
> 
> Acked-by: Sylwester Nawrocki <s.nawrocki@samsung.com>

Acked-by: Michael Turquette <mturquette@linaro.org>

> 
> --
> Thanks,
> Sylwester
> 

WARNING: multiple messages have this Message-ID (diff)
From: mturquette@linaro.org (Mike Turquette)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 1/2] clk: exynos5420: Add IDs for clocks used in DISP1 power domain
Date: Tue, 20 Jan 2015 09:54:02 -0800	[thread overview]
Message-ID: <20150120175402.22722.84342@quantum> (raw)
In-Reply-To: <54BE6050.1090205@samsung.com>

Quoting Sylwester Nawrocki (2015-01-20 06:04:00)
> Hi,
> 
> On 20/01/15 11:35, Javier Martinez Canillas wrote:
> > When a power domain is powered off on Exynos5420 SoC, the input clocks of
> > the devices attached to this power domain are re-parented to oscclk and
> > restored to the original parent after powering on the power domain.
> > 
> > So a reference to the input and parent clocks for the devices attached to
> > a power domain are needed to be able to do the re-parenting. The DISP1 pd
> > includes modules which uses the following clocks:
> > 
> > ACLK_200_DISP1 (MIXER and HDMILINK)
> > ACLK_300_DISP1 (FIMD1)
> > ACLK_400_DISP1 (Internal Buses)
> > 
> > Each of these clocks are generated as the output of a clock mux so add an
> > ID for all of these clock muxes and their parents to be referenced in the
> > DISP1 power domain device node.
> > 
> > Signed-off-by: Javier Martinez Canillas <javier.martinez@collabora.co.uk>
> 
> The patch looks OK to me, I'm fine with it being merged via Kukjin's tree
> due to the dts dependencies (including other pending dts patches touching
> the arch/arm/boot/dts/exynos5420.dtsi file).
> I think we need also Mike ACK for that, I could also queue the patch for
> the clk tree and create a topic branch, but merging both patches via
> arm-soc seems a more sane option in this case.
> 
> Acked-by: Sylwester Nawrocki <s.nawrocki@samsung.com>

Acked-by: Michael Turquette <mturquette@linaro.org>

> 
> --
> Thanks,
> Sylwester
> 

  reply	other threads:[~2015-01-20 17:54 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-01-20 10:35 [PATCH 0/2] Add HDMI support for Exynos5420 platform Javier Martinez Canillas
2015-01-20 10:35 ` Javier Martinez Canillas
2015-01-20 10:35 ` [PATCH 1/2] clk: exynos5420: Add IDs for clocks used in DISP1 power domain Javier Martinez Canillas
2015-01-20 10:35   ` Javier Martinez Canillas
2015-01-20 14:04   ` Sylwester Nawrocki
2015-01-20 14:04     ` Sylwester Nawrocki
2015-01-20 17:54     ` Mike Turquette [this message]
2015-01-20 17:54       ` Mike Turquette
2015-01-21 10:38       ` Javier Martinez Canillas
2015-01-21 10:38         ` Javier Martinez Canillas
2015-01-23  6:51         ` Kukjin Kim
2015-01-23  6:51           ` Kukjin Kim
2015-01-23  8:48           ` Javier Martinez Canillas
2015-01-23  8:48             ` Javier Martinez Canillas
2015-01-20 10:35 ` [PATCH 2/2] ARM: dts: Add DISP1 power domain for exynos5420 Javier Martinez Canillas
2015-01-20 10:35   ` Javier Martinez Canillas

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=20150120175402.22722.84342@quantum \
    --to=mturquette@linaro.org \
    --cc=a.hajda@samsung.com \
    --cc=arun.kk@samsung.com \
    --cc=dianders@chromium.org \
    --cc=javier.martinez@collabora.co.uk \
    --cc=jy0922.shim@samsung.com \
    --cc=k.kozlowski@samsung.com \
    --cc=kgene@kernel.org \
    --cc=khilman@linaro.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-samsung-soc@vger.kernel.org \
    --cc=liquid.acid@gmx.net \
    --cc=m.szyprowski@samsung.com \
    --cc=olof@lixom.net \
    --cc=rahul.sharma@samsung.com \
    --cc=s.nawrocki@samsung.com \
    --cc=tomasz.figa@gmail.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 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.