From: Kukjin Kim <kgene@kernel.org>
To: 'Javier Martinez Canillas' <javier.martinez@collabora.co.uk>,
'Mike Turquette' <mturquette@linaro.org>,
'Sylwester Nawrocki' <s.nawrocki@samsung.com>,
'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: Fri, 23 Jan 2015 15:51:53 +0900 [thread overview]
Message-ID: <0c2601d036d9$1276e7d0$3764b770$@kernel.org> (raw)
In-Reply-To: <54BF81C2.3020000@collabora.co.uk>
Javier Martinez Canillas wrote:
>
> Hello,
>
Hi,
> On 01/20/2015 06:54 PM, Mike Turquette wrote:
> > 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 a lot Sylwester and Mike for your acks.
>
> Kukjin, could you please pick $subject and "Patch 2/2 ARM: dts: Add DISP1
> power domain for exynos5420" through your tree?
>
Sure, I will after applying Marek's generic power domain patch.
Thanks,
Kukjin
WARNING: multiple messages have this Message-ID (diff)
From: kgene@kernel.org (Kukjin Kim)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 1/2] clk: exynos5420: Add IDs for clocks used in DISP1 power domain
Date: Fri, 23 Jan 2015 15:51:53 +0900 [thread overview]
Message-ID: <0c2601d036d9$1276e7d0$3764b770$@kernel.org> (raw)
In-Reply-To: <54BF81C2.3020000@collabora.co.uk>
Javier Martinez Canillas wrote:
>
> Hello,
>
Hi,
> On 01/20/2015 06:54 PM, Mike Turquette wrote:
> > 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 a lot Sylwester and Mike for your acks.
>
> Kukjin, could you please pick $subject and "Patch 2/2 ARM: dts: Add DISP1
> power domain for exynos5420" through your tree?
>
Sure, I will after applying Marek's generic power domain patch.
Thanks,
Kukjin
next prev parent reply other threads:[~2015-01-23 6:51 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
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 [this message]
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='0c2601d036d9$1276e7d0$3764b770$@kernel.org' \
--to=kgene@kernel.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=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=mturquette@linaro.org \
--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.