Re: [U-Boot] [PATCH 1/7] rockchip: rk3399: update PPLL and pmu_pclk frequency

2016-09-06 Thread Simon Glass 
Hi Kever,

On 6 September 2016 at 03:52, Kever Yang  wrote:
> Hi Simon,
>
> On 09/06/2016 09:03 AM, Simon Glass wrote:
>>
>> Hi Kever,
>>
>> On 29 August 2016 at 21:02, Kever Yang  wrote:
>>>
>>> This patch update PPLL to 676MHz and PMU_PCLK to 48MHz.
>>
>> Why?
>>
>
> 1. 48MHz can make sure the pwm can get exact 50% duty ratio, but 99MHz can
> not,
> 2. We think 48MHz is fast enough for pmu pclk and it is lower power cost
> than 99MHz,
> 3. PPLL 676 MHz and PMU_PCLK 48MHz are the clock rate we are using
> internally for kernel,
> it suppose not to change the bus clock like pmu_pclk in kernel, so we want
> to change it in uboot.

OK, thank you. Please can you add this info to the commit message in
v2? My point is that commits should explain why they are needed, if
not obvious.

>
> Thanks,
> - Kever
>
>>> Signed-off-by: Kever Yang 
>>> ---
>>>
>>>   arch/arm/include/asm/arch-rockchip/cru_rk3399.h | 4 ++--
>>>   1 file changed, 2 insertions(+), 2 deletions(-)
>>>
>>> diff --git a/arch/arm/include/asm/arch-rockchip/cru_rk3399.h
>>> b/arch/arm/include/asm/arch-rockchip/cru_rk3399.h
>>> index c919f47..6776e48 100644
>>> --- a/arch/arm/include/asm/arch-rockchip/cru_rk3399.h
>>> +++ b/arch/arm/include/asm/arch-rockchip/cru_rk3399.h
>>> @@ -64,9 +64,9 @@ check_member(rk3399_cru, sdio1_con[1], 0x594);
>>>   #define APLL_HZ(600*MHz)
>>>   #define GPLL_HZ(594*MHz)
>>>   #define CPLL_HZ(384*MHz)
>>> -#define PPLL_HZ(594*MHz)
>>> +#define PPLL_HZ(676*MHz)
>>>
>>> -#define PMU_PCLK_HZ(99*MHz)
>>> +#define PMU_PCLK_HZ(48*MHz)
>>>
>>>   #define ACLKM_CORE_HZ  (300*MHz)
>>>   #define ATCLK_CORE_HZ  (300*MHz)
>>> --
>>> 1.9.1
>>>
>> Regards,
>> Simon
>>
>>
>>
>
>

Regards,
Simon
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH 1/7] rockchip: rk3399: update PPLL and pmu_pclk frequency

2016-09-06 Thread Kever Yang 

Hi Simon,

On 09/06/2016 09:03 AM, Simon Glass wrote:

Hi Kever,

On 29 August 2016 at 21:02, Kever Yang  wrote:

This patch update PPLL to 676MHz and PMU_PCLK to 48MHz.

Why?



1. 48MHz can make sure the pwm can get exact 50% duty ratio, but 99MHz 
can not,
2. We think 48MHz is fast enough for pmu pclk and it is lower power cost 
than 99MHz,
3. PPLL 676 MHz and PMU_PCLK 48MHz are the clock rate we are using 
internally for kernel,
it suppose not to change the bus clock like pmu_pclk in kernel, so we 
want to change it in uboot.


Thanks,
- Kever

Signed-off-by: Kever Yang 
---

  arch/arm/include/asm/arch-rockchip/cru_rk3399.h | 4 ++--
  1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/arm/include/asm/arch-rockchip/cru_rk3399.h 
b/arch/arm/include/asm/arch-rockchip/cru_rk3399.h
index c919f47..6776e48 100644
--- a/arch/arm/include/asm/arch-rockchip/cru_rk3399.h
+++ b/arch/arm/include/asm/arch-rockchip/cru_rk3399.h
@@ -64,9 +64,9 @@ check_member(rk3399_cru, sdio1_con[1], 0x594);
  #define APLL_HZ(600*MHz)
  #define GPLL_HZ(594*MHz)
  #define CPLL_HZ(384*MHz)
-#define PPLL_HZ(594*MHz)
+#define PPLL_HZ(676*MHz)

-#define PMU_PCLK_HZ(99*MHz)
+#define PMU_PCLK_HZ(48*MHz)

  #define ACLKM_CORE_HZ  (300*MHz)
  #define ATCLK_CORE_HZ  (300*MHz)
--
1.9.1


Regards,
Simon






___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH 1/7] rockchip: rk3399: update PPLL and pmu_pclk frequency

2016-09-05 Thread Simon Glass 
Hi Kever,

On 29 August 2016 at 21:02, Kever Yang  wrote:
> This patch update PPLL to 676MHz and PMU_PCLK to 48MHz.

Why?

>
> Signed-off-by: Kever Yang 
> ---
>
>  arch/arm/include/asm/arch-rockchip/cru_rk3399.h | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/arch/arm/include/asm/arch-rockchip/cru_rk3399.h 
> b/arch/arm/include/asm/arch-rockchip/cru_rk3399.h
> index c919f47..6776e48 100644
> --- a/arch/arm/include/asm/arch-rockchip/cru_rk3399.h
> +++ b/arch/arm/include/asm/arch-rockchip/cru_rk3399.h
> @@ -64,9 +64,9 @@ check_member(rk3399_cru, sdio1_con[1], 0x594);
>  #define APLL_HZ(600*MHz)
>  #define GPLL_HZ(594*MHz)
>  #define CPLL_HZ(384*MHz)
> -#define PPLL_HZ(594*MHz)
> +#define PPLL_HZ(676*MHz)
>
> -#define PMU_PCLK_HZ(99*MHz)
> +#define PMU_PCLK_HZ(48*MHz)
>
>  #define ACLKM_CORE_HZ  (300*MHz)
>  #define ATCLK_CORE_HZ  (300*MHz)
> --
> 1.9.1
>

Regards,
Simon
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] [PATCH 1/7] rockchip: rk3399: update PPLL and pmu_pclk frequency

2016-08-29 Thread Kever Yang 
This patch update PPLL to 676MHz and PMU_PCLK to 48MHz.

Signed-off-by: Kever Yang 
---

 arch/arm/include/asm/arch-rockchip/cru_rk3399.h | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/arm/include/asm/arch-rockchip/cru_rk3399.h 
b/arch/arm/include/asm/arch-rockchip/cru_rk3399.h
index c919f47..6776e48 100644
--- a/arch/arm/include/asm/arch-rockchip/cru_rk3399.h
+++ b/arch/arm/include/asm/arch-rockchip/cru_rk3399.h
@@ -64,9 +64,9 @@ check_member(rk3399_cru, sdio1_con[1], 0x594);
 #define APLL_HZ(600*MHz)
 #define GPLL_HZ(594*MHz)
 #define CPLL_HZ(384*MHz)
-#define PPLL_HZ(594*MHz)
+#define PPLL_HZ(676*MHz)
 
-#define PMU_PCLK_HZ(99*MHz)
+#define PMU_PCLK_HZ(48*MHz)
 
 #define ACLKM_CORE_HZ  (300*MHz)
 #define ATCLK_CORE_HZ  (300*MHz)
-- 
1.9.1

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot