Re: [PATCH v6] clk: Exynos5250: Add clocks for G3D

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Tomasz,

On Thu, May 1, 2014 at 9:37 PM, Tomasz Figa <tomasz.figa@xxxxxxxxx> wrote:
> Hi Arun,
>
>
> On 01.05.2014 00:48, Tomasz Figa wrote:
>>
>> On 28.04.2014 11:37, Arun Kumar K wrote:
>>>
>>> This patch adds the required clocks for ARM Mali IP
>>> in Exynos5250.
>>>
>>> Signed-off-by: Arun Kumar K <arun.kk@xxxxxxxxxxx>
>>> ---
>>> Changes from v5
>>> - Addressed comments from Tomasz Figa
>>>    http://www.spinics.net/lists/arm-kernel/msg326118.html
>>> Changes from v4
>>> - Rebased on latest kernel
>>> - Added macros
>>> Changes from v3
>>> - Renamed some clocks as per Tomasz Figa's comments
>>> Changes from v2
>>> - Rebased on clk-next
>>> Changes from v1
>>> - Removed exporting of parent DIV clock for g3d
>>>    as per Tomsz Figa's comment.
>>> ---
>>>   drivers/clk/samsung/clk-exynos5250.c   |   15 ++++++++++++++-
>>>   include/dt-bindings/clock/exynos5250.h |    4 +++-
>>>   2 files changed, 17 insertions(+), 2 deletions(-)
>>>
>>> diff --git a/drivers/clk/samsung/clk-exynos5250.c
>>> b/drivers/clk/samsung/clk-exynos5250.c
>>> index e7ee442..df02526 100644
>>> --- a/drivers/clk/samsung/clk-exynos5250.c
>>> +++ b/drivers/clk/samsung/clk-exynos5250.c
>>> @@ -37,6 +37,7 @@
>>>   #define VPLL_CON0        0x10140
>>>   #define GPLL_CON0        0x10150
>>>   #define SRC_TOP0        0x10210
>>> +#define SRC_TOP1        0x10214
>>>   #define SRC_TOP2        0x10218
>>>   #define SRC_TOP3        0x1021c
>>>   #define SRC_GSCL        0x10220
>>> @@ -71,6 +72,7 @@
>>>   #define GATE_IP_GSCL        0x10920
>>>   #define GATE_IP_DISP1        0x10928
>>>   #define GATE_IP_MFC        0x1092c
>>> +#define GATE_IP_G3D        0x10930
>>>   #define GATE_IP_GEN        0x10934
>>>   #define GATE_IP_FSYS        0x10944
>>>   #define GATE_IP_PERIC        0x10950
>>> @@ -100,6 +102,7 @@ static unsigned long exynos5250_clk_regs[]
>>> __initdata = {
>>>       DIV_CPU0,
>>>       SRC_CORE1,
>>>       SRC_TOP0,
>>> +    SRC_TOP1,
>>>       SRC_TOP2,
>>>       SRC_TOP3,
>>>       SRC_GSCL,
>>> @@ -133,6 +136,7 @@ static unsigned long exynos5250_clk_regs[]
>>> __initdata = {
>>>       DIV_PERIC5,
>>>       GATE_IP_GSCL,
>>>       GATE_IP_MFC,
>>> +    GATE_IP_G3D,
>>>       GATE_IP_GEN,
>>>       GATE_IP_FSYS,
>>>       GATE_IP_PERIC,
>>> @@ -189,10 +193,12 @@ PNAME(mout_vpllsrc_p)    = { "fin_pll",
>>> "sclk_hdmi27m" };
>>>   PNAME(mout_vpll_p)    = { "mout_vpllsrc", "fout_vpll" };
>>>   PNAME(mout_cpll_p)    = { "fin_pll", "fout_cpll" };
>>>   PNAME(mout_epll_p)    = { "fin_pll", "fout_epll" };
>>> +PNAME(mout_gpll_p)    = { "fin_pll", "fout_gpll" };
>>>   PNAME(mout_mpll_user_p)    = { "fin_pll", "mout_mpll" };
>>>   PNAME(mout_bpll_user_p)    = { "fin_pll", "mout_bpll" };
>>>   PNAME(mout_aclk166_p)    = { "mout_cpll", "mout_mpll_user" };
>>>   PNAME(mout_aclk200_p)    = { "mout_mpll_user", "mout_bpll_user" };
>>> +PNAME(mout_aclk400_p)    = { "mout_aclk400_g3d_mid", "mout_gpll" };
>>>   PNAME(mout_aclk200_sub_p) = { "fin_pll", "div_aclk200" };
>>>   PNAME(mout_aclk266_sub_p) = { "fin_pll", "div_aclk266" };
>>>   PNAME(mout_aclk333_sub_p) = { "fin_pll", "div_aclk333" };
>>> @@ -273,12 +279,16 @@ static struct samsung_mux_clock
>>> exynos5250_mux_clks[] __initdata = {
>>>       MUX(0, "mout_aclk166", mout_aclk166_p, SRC_TOP0, 8, 1),
>>>       MUX(0, "mout_aclk200", mout_aclk200_p, SRC_TOP0, 12, 1),
>>>       MUX(0, "mout_aclk333", mout_aclk166_p, SRC_TOP0, 16, 1),
>>> +    MUX(0, "mout_aclk400_g3d_mid", mout_aclk200_p, SRC_TOP0, 20, 1),
>>> +
>>> +    MUX(0, "mout_aclk400_g3d", mout_aclk400_p, SRC_TOP1, 28, 1),
>>>
>>>       MUX(0, "mout_cpll", mout_cpll_p, SRC_TOP2, 8, 1),
>>>       MUX(0, "mout_epll", mout_epll_p, SRC_TOP2, 12, 1),
>>>       MUX(0, "mout_vpll", mout_vpll_p, SRC_TOP2, 16, 1),
>>>       MUX(0, "mout_mpll_user", mout_mpll_user_p, SRC_TOP2, 20, 1),
>>>       MUX(0, "mout_bpll_user", mout_bpll_user_p, SRC_TOP2, 24, 1),
>>> +    MUX(CLK_MOUT_GPLL, "mout_gpll", mout_gpll_p, SRC_TOP2, 28, 1),
>>>
>>>       MUX(0, "mout_aclk200_disp1_sub", mout_aclk200_sub_p, SRC_TOP3,
>>> 4, 1),
>>>       MUX(0, "mout_aclk266_gscl_sub", mout_aclk266_sub_p, SRC_TOP3, 8,
>>> 1),
>>> @@ -351,6 +361,8 @@ static struct samsung_div_clock
>>> exynos5250_div_clks[] __initdata = {
>>>       DIV(0, "div_aclk200", "mout_aclk200", DIV_TOP0, 12, 3),
>>>       DIV(0, "div_aclk266", "mout_mpll_user", DIV_TOP0, 16, 3),
>>>       DIV(0, "div_aclk333", "mout_aclk333", DIV_TOP0, 20, 3),
>>> +    DIV(0, "div_aclk400_g3d", "mout_aclk400_g3d", DIV_TOP0,
>>> +                            24, 3),
>>>
>>>       DIV(0, "div_aclk66_pre", "mout_mpll_user", DIV_TOP1, 24, 3),
>>>
>>> @@ -533,7 +545,8 @@ static struct samsung_gate_clock
>>> exynos5250_gate_clks[] __initdata = {
>>>           0),
>>>       GATE(CLK_SMMU_MFCL, "smmu_mfcl", "mout_aclk333_sub",
>>> GATE_IP_MFC, 2, 0,
>>>           0),
>>> -
>>> +    GATE(CLK_G3D, "g3d", "div_aclk400_g3d", GATE_IP_G3D, 0,
>>> +                    CLK_SET_RATE_PARENT, 0),
>>>       GATE(CLK_ROTATOR, "rotator", "div_aclk266", GATE_IP_GEN, 1, 0, 0),
>>>       GATE(CLK_JPEG, "jpeg", "div_aclk166", GATE_IP_GEN, 2, 0, 0),
>>>       GATE(CLK_MDMA1, "mdma1", "div_aclk266", GATE_IP_GEN, 4, 0, 0),
>>> diff --git a/include/dt-bindings/clock/exynos5250.h
>>> b/include/dt-bindings/clock/exynos5250.h
>>> index 922f2dc..a3c6777 100644
>>> --- a/include/dt-bindings/clock/exynos5250.h
>>> +++ b/include/dt-bindings/clock/exynos5250.h
>>> @@ -150,11 +150,13 @@
>>>   #define CLK_G2D            345
>>>   #define CLK_MDMA0        346
>>>   #define CLK_SMMU_MDMA0        347
>>> +#define CLK_G3D            348
>>>
>>>   /* mux clocks */
>>>   #define CLK_MOUT_HDMI        1024
>>> +#define CLK_MOUT_GPLL        1025
>>>
>>>   /* must be greater than maximal clock id */
>>> -#define CLK_NR_CLKS        1025
>>> +#define CLK_NR_CLKS        1026
>>>
>>>   #endif /* _DT_BINDINGS_CLOCK_EXYNOS_5250_H */
>>>
>>
>> Applied.
>
>
> Uhm, actually not applied, as we seem to have a numbering clash with
> previous patch adding gate clock for SSS block
> (clk: samsung exynos5250/5420: Add gate clock for SSS module).
>
> I can change this to 349 when applying, but you would need to make sure that
> you reflect this in your internal trees and rebuild your DTBs. Is it fine
> for you?
>

Yes it is fine with me. You can change the number to 349.

Thanks & Regards
Arun

> Best regards,
> Tomasz
--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux SoC Development]     [Linux Rockchip Development]     [Linux USB Development]     [Video for Linux]     [Linux Audio Users]     [Linux SCSI]     [Yosemite News]

  Powered by Linux