From: "Heiko Stübner" <heiko@sntech.de>
To: Rahul Sharma <rahul.sharma@samsung.com>
Cc: linux-samsung-soc@vger.kernel.org,
devicetree-discuss@lists.ozlabs.org, kgene.kim@samsung.com,
inki.dae@samsung.com, s.nawrocki@samsung.com,
thomas.abraham@linaro.org, joshi@samsung.com,
r.sh.open@gmail.com
Subject: Re: [RFC 0/2] clk: samsung: add composite clocks
Date: Mon, 20 May 2013 16:14:30 +0200 [thread overview]
Message-ID: <201305201614.31103.heiko@sntech.de> (raw)
In-Reply-To: <1369059428-26820-1-git-send-email-rahul.sharma@samsung.com>
Am Montag, 20. Mai 2013, 16:17:06 schrieb Rahul Sharma:
> This patch adds support for composite clocks for samsung SoCs.
> Many drivers need access to a common clock which support gating
> and/or muxing and/or rate control operations. For example hdmi
> which needs to switch between parents and call enable/disable for
> "sclk_hdmi".
>
> This patch set also adds composite clock for exyno5250 hdmi. Based
> on the review comment, I will extended this to other exynos SoCs
> clocks files.
I think I remember reading somewhere that the target of the common clock
framework was to prevent every SoC from introducing their own special clock
types and instead create these structures from separate clocks (mux clk +
gate clk) and not to have every SoC create their own custom clock types.
The Samsung clock drivers at the moment follow this paradigm of combining the
existing "simple" clocks and only introduce new clock types for the pll
clocks, that really need special handling.
So it would probably good to keep it this way and define your clocks from
their individual components, as all the other Samsung clocks currently do.
Heiko
next prev parent reply other threads:[~2013-05-20 14:14 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-05-20 14:17 [RFC 0/2] clk: samsung: add composite clocks Rahul Sharma
2013-05-20 14:14 ` Heiko Stübner [this message]
2013-05-20 18:19 ` Rahul Sharma
2013-05-20 18:54 ` Heiko Stübner
2013-05-20 14:17 ` [RFC 1/2] clk: samsung: add support for " Rahul Sharma
2013-05-20 14:17 ` [RFC 2/2] clk: samsung: add exynos5250 composite clock for hdmi Rahul Sharma
2013-05-20 18:57 ` Tomasz Figa
2013-05-20 20:02 ` Saravana Kannan
2013-05-21 4:12 ` Rahul Sharma
2013-05-21 23:15 ` 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=201305201614.31103.heiko@sntech.de \
--to=heiko@sntech.de \
--cc=devicetree-discuss@lists.ozlabs.org \
--cc=inki.dae@samsung.com \
--cc=joshi@samsung.com \
--cc=kgene.kim@samsung.com \
--cc=linux-samsung-soc@vger.kernel.org \
--cc=r.sh.open@gmail.com \
--cc=rahul.sharma@samsung.com \
--cc=s.nawrocki@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).