From: t.figa@samsung•com (Tomasz Figa)
To: linux-arm-kernel@lists•infradead.org
Subject: [PATCH 04/14] clk: samsung: Add clock description for basic CMU blocks
Date: Wed, 27 Aug 2014 14:35:53 +0200 [thread overview]
Message-ID: <53FDD0A9.3040109@samsung.com> (raw)
In-Reply-To: <1409132889-2080-4-git-send-email-ch.naveen@samsung.com>
On 27.08.2014 11:48, Naveen Krishna Chatradhi wrote:
> This patch adds clock description for MUX, DIV, GATE and PLL
> clocks available in TOPC, TOP0, TOP1, BUS0, BUS1, FSYS0, FSYS1,
> CCORE, PERIC0, PERIC1, PERIS, DISP, G3D, MSCL and MFC blocks.
[snip]
> +Phy clocks:
> +
> +There are several clocks which are generated by specific PHYs.
> +These clocks are fed into the clock controller and then routed to
> +the hardware blocks. These clocks are defined as fixed clocks in the
> +driver with following names:
> +
The names are missing?
> +Required Properties for Clock Controller:
> +
> + - compatible: should be one of the following.
> + 1) "samsung,exynos7-clock-topc"
> + 2) "samsung,exynos7-clock-top0"
> + 3) "samsung,exynos7-clock-top1"
> + 4) "samsung,exynos7-clock-atlas"
> + 5) "samsung,exynos7-clock-ccore"
[snip]
> + - clocks: list of clock identifiers which are fed as the input to
> + the given clock controller. Please refer the next section to find
> + the input clocks for a given controller.
> +
> + - clock-names: list of names of clocks which are fed as the input
> + to the given clock controller.
In the dtsi file added by patch 11/14, I don't see those being specified.
> +
> +Input clocks for topc clock controller:
> + - fin_pll
> + - fout_aud_pll
[snip]
> +/* PMS values for PLL 1451x */
> +static const struct samsung_pll_rate_table pll1451x_24mhz_tbl[] = {
> + /* rate, m, p, s */
> + PLL_35XX_RATE(660000000, 165, 3, 1),
This array needs to be sorted by output rate. However you might want to
rebase this series onto series [1] to eliminate this requirement. Be
aware that mentioned series might need a respin, though.
Also the rates seem to be too nicely rounded. Are they the real values
that can be obtained using the PLL equation (what is required by the PLL
framework) or a copy paste from the documentation? Similarly, series [1]
lessens the requirement a bit, due to recalculation of rates from
specified coefficients and warning about incorrect entries.
[1]
https://www.mail-archive.com/linux-samsung-soc at vger.kernel.org/msg35188.html
> + PLL_35XX_RATE(1800000000, 150, 2, 0),
> + PLL_35XX_RATE(1700000000, 425, 6, 0),
> + PLL_35XX_RATE(1600000000, 200, 3, 0),
> + PLL_35XX_RATE(1500000000, 125, 2, 0),
> + PLL_35XX_RATE(1400000000, 175, 3, 0),
> + PLL_35XX_RATE(1380000000, 115, 2, 0),
> + PLL_35XX_RATE(1300000000, 325, 6, 0),
> + PLL_35XX_RATE(1200000000, 100, 2, 0),
> + PLL_35XX_RATE(1180000000, 295, 6, 0),
> + PLL_35XX_RATE(1104000000, 276, 6, 0),
[snip]
> + DIV(0, "dout_sclk_aud_pll", "mout_aud_pll_ctrl", DIV_TOPC3, 28, 3),
> +};
> +
> +static struct samsung_gate_clock topc_gate_clks[] __initdata = {
> + GATE(ACLK_CCORE_532, "aclk_ccore_532", "dout_aclk_ccore_532",
> + ENABLE_ACLK_TOPC0, 0, CLK_IGNORE_UNUSED, 0),
Why CLK_IGNORE_UNUSED? (The same question for all the gate clocks
defined with it in the driver.)
Best regards,
Tomasz
next prev parent reply other threads:[~2014-08-27 12:35 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-08-27 9:48 [PATCH 01/14] clk: samsung: add support for 145xx and 1460x PLLs Naveen Krishna Chatradhi
2014-08-27 9:48 ` [PATCH 02/14] clk: samsung: Factor out the common code to clk.c Naveen Krishna Chatradhi
2014-08-27 10:15 ` Naveen Krishna Ch
2014-08-27 9:48 ` [PATCH 03/14] clk: samsung: Add fixed_factor_clocks field to struct exynos_cmu_info Naveen Krishna Chatradhi
2014-08-27 9:48 ` [PATCH 04/14] clk: samsung: Add clock description for basic CMU blocks Naveen Krishna Chatradhi
2014-08-27 12:35 ` Tomasz Figa [this message]
2014-09-03 7:40 ` Naveen Krishna Ch
2014-08-27 9:48 ` [PATCH 05/14] pinctrl: exynos: Add driver data for Exynos7 Naveen Krishna Chatradhi
2014-09-02 12:38 ` Linus Walleij
2014-09-02 12:41 ` Tomasz Figa
2014-08-27 9:48 ` [PATCH 06/14] pinctrl: samsung: use CONFIG_PINCTRL_SAMSUNG symbol in makefile Naveen Krishna Chatradhi
2014-08-29 12:37 ` Linus Walleij
2014-09-03 7:41 ` Naveen Krishna Ch
2014-08-27 12:10 ` [PATCH 01/14] clk: samsung: add support for 145xx and 1460x PLLs Tomasz Figa
2014-09-03 7:37 ` Naveen Krishna Ch
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=53FDD0A9.3040109@samsung.com \
--to=t.figa@samsung$(echo .)com \
--cc=linux-arm-kernel@lists$(echo .)infradead.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
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