Re: [PATCH 1/1] clk: exynos4: Add clock entries for TMU

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

 



Hi Tomasz,

On 19 April 2013 20:00, Tomasz Figa <t.figa@xxxxxxxxxxx> wrote:
> Hi Sachin,
>
> On Friday 19 of April 2013 11:46:06 Sachin Kamat wrote:
>> Added clock entries for thermal management unit (TMU) for
>> Exynos4 SoCs.
>>
>> Signed-off-by: Sachin Kamat <sachin.kamat@xxxxxxxxxx>
>> Cc: Thomas Abraham <thomas.abraham@xxxxxxxxxx>
>> Cc: Mike Turquette <mturquette@xxxxxxxxxx>
>> ---
>> Should be applied on top of the below patches:
>> https://patchwork.kernel.org/patch/2448711/
>> https://patchwork.kernel.org/patch/2459831/
>> ---
>>  .../devicetree/bindings/clock/exynos4-clock.txt    |    1 +
>>  drivers/clk/samsung/clk-exynos4.c                  |    4 +++-
>>  2 files changed, 4 insertions(+), 1 deletion(-)
>>
>> diff --git a/Documentation/devicetree/bindings/clock/exynos4-clock.txt
>> b/Documentation/devicetree/bindings/clock/exynos4-clock.txt index
>> 14d5c2a..5b17c4d 100644
>> --- a/Documentation/devicetree/bindings/clock/exynos4-clock.txt
>> +++ b/Documentation/devicetree/bindings/clock/exynos4-clock.txt
>> @@ -236,6 +236,7 @@ Exynos4 SoC and this is specified where applicable.
>>    spi0_isp_sclk       380     Exynos4x12
>>    spi1_isp_sclk       381     Exynos4x12
>>    uart_isp_sclk       382     Exynos4x12
>> +  tmu                      383
>
> Documentation specifies this clock as "tmu_apbif". I think we should keep our
> clock names consistent with documentation.

Exynos5 clock file listed this as 'tmu'. Hence I kept it the same.
We can probably update this to match the documentation.

>
>>
>>               [Mux Clocks]
>>
>> diff --git a/drivers/clk/samsung/clk-exynos4.c
>> b/drivers/clk/samsung/clk-exynos4.c index 09cf161..fc4f662 100644
>> --- a/drivers/clk/samsung/clk-exynos4.c
>> +++ b/drivers/clk/samsung/clk-exynos4.c
>> @@ -170,7 +170,7 @@ enum exynos4_clks {
>>       gicisp, smmu_isp, smmu_drc, smmu_fd, smmu_lite0, smmu_lite1, mcuctl_isp,
>>       mpwm_isp, i2c0_isp, i2c1_isp, mtcadc_isp, pwm_isp, wdt_isp, uart_isp,
>>       asyncaxim, smmu_ispcx, spi0_isp, spi1_isp, pwm_isp_sclk, spi0_isp_sclk,
>> -     spi1_isp_sclk, uart_isp_sclk,
>> +     spi1_isp_sclk, uart_isp_sclk, tmu,
>>
>>       /* mux clocks */
>>       mout_fimc0 = 384, mout_fimc1, mout_fimc2, mout_fimc3, mout_cam0,
>> @@ -815,6 +815,7 @@ static struct samsung_gate_clock exynos4210_gate_clks[]
>> __initdata = { GATE_A(keyif, "keyif", "aclk100", E4210_GATE_IP_PERIR, 16,
>> 0, 0, "keypad"), GATE_DA(sclk_fimd1, "exynos4-fb.1", "sclk_fimd1",
>> "div_fimd1",
>>                       E4210_SRC_MASK_LCD1, 0, CLK_SET_RATE_PARENT, 0, "sclk_fimd"),
>> +     GATE(tmu, "tmu", "aclk100", E4210_GATE_IP_PERIR, 17, 0, 0),
>>  };
>>
>>  /* list of gate clocks supported in exynos4x12 soc */
>> @@ -915,6 +916,7 @@ static struct samsung_gate_clock exynos4x12_gate_clks[]
>> __initdata = { GATE(spi1_isp, "spi1_isp", "aclk200", E4X12_GATE_ISP1, 13,
>>                       CLK_IGNORE_UNUSED, 0),
>>       GATE(g2d, "g2d", "aclk200", GATE_IP_DMC, 23, 0, 0),
>> +     GATE(tmu, "tmu", "aclk100", E4X12_GATE_IP_PERIR, 17, 0, 0),
>>  };
>>
>>  #ifdef CONFIG_OF
>
> Btw. Lukasz Majewski have a complete series of patches to fix device tree
> support of TMU on Exynos4x12 and it includes changes to Exynos clock driver as
> well, with proper clock name. He will be sending it soon.
>
> Would you mind if we use his patch instead?

I have already sent other patches to add missing support for clocks
and Exynos4x12 in Exynos thermal driver. I also have the dts patches
ready for 4x12 boards which I was hoping to send shortly.


-- 
With warm regards,
Sachin
--
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