diff mbox

[2/2] arm64: dts: set UART1 clock frequency to 150MHz

Message ID 1467965467-28180-2-git-send-email-jorge.ramirez-ortiz@linaro.org (mailing list archive)
State New, archived
Headers show

Commit Message

Jorge Ramirez-Ortiz July 8, 2016, 8:11 a.m. UTC 
Enable support for higher baud rates (up to 3Mbps) in UART1 - required
for bluetooth transfers.

Signed-off-by: Jorge Ramirez-Ortiz <jorge.ramirez-ortiz@linaro.org>
Tested-by: Jorge Ramirez-Ortiz <jorge.ramirez-ortiz@linaro.org>
---
 arch/arm64/boot/dts/hisilicon/hi6220-hikey.dts | 2 ++
 1 file changed, 2 insertions(+)

Comments

Michael Turquette July 8, 2016, 5:23 p.m. UTC | #1 
Quoting Jorge Ramirez-Ortiz (2016-07-08 01:11:07)
> Enable support for higher baud rates (up to 3Mbps) in UART1 - required
> for bluetooth transfers.
> 
> Signed-off-by: Jorge Ramirez-Ortiz <jorge.ramirez-ortiz@linaro.org>
> Tested-by: Jorge Ramirez-Ortiz <jorge.ramirez-ortiz@linaro.org>

Reviewed-by: Michael Turquette <mturquette@baylibre.com>

> ---
>  arch/arm64/boot/dts/hisilicon/hi6220-hikey.dts | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/arch/arm64/boot/dts/hisilicon/hi6220-hikey.dts b/arch/arm64/boot/dts/hisilicon/hi6220-hikey.dts
> index e92a30c..27be812 100644
> --- a/arch/arm64/boot/dts/hisilicon/hi6220-hikey.dts
> +++ b/arch/arm64/boot/dts/hisilicon/hi6220-hikey.dts
> @@ -55,6 +55,8 @@
>                 };
>  
>                 uart1: uart@f7111000 {
> +                       assigned-clocks = <&sys_ctrl HI6220_UART1_SRC>;
> +                       assigned-clock-rates = <150000000>;
>                         status = "ok";
>                 };
>  
> -- 
> 2.7.4
>
Wei Xu July 11, 2016, 9:53 a.m. UTC | #2 
Hi Jorge,

On 08/07/2016 09:11, Jorge Ramirez-Ortiz wrote:
> Enable support for higher baud rates (up to 3Mbps) in UART1 - required
> for bluetooth transfers.
> 
> Signed-off-by: Jorge Ramirez-Ortiz <jorge.ramirez-ortiz@linaro.org>
> Tested-by: Jorge Ramirez-Ortiz <jorge.ramirez-ortiz@linaro.org>

Fine to me.
Thanks!

Acked-by: Wei Xu <xuwei5@hisilicon.com>

Best Regards,
Wei

> ---
>  arch/arm64/boot/dts/hisilicon/hi6220-hikey.dts | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/arch/arm64/boot/dts/hisilicon/hi6220-hikey.dts b/arch/arm64/boot/dts/hisilicon/hi6220-hikey.dts
> index e92a30c..27be812 100644
> --- a/arch/arm64/boot/dts/hisilicon/hi6220-hikey.dts
> +++ b/arch/arm64/boot/dts/hisilicon/hi6220-hikey.dts
> @@ -55,6 +55,8 @@
>  		};
>  
>  		uart1: uart@f7111000 {
> +			assigned-clocks = <&sys_ctrl HI6220_UART1_SRC>;
> +			assigned-clock-rates = <150000000>;
>  			status = "ok";
>  		};
>  
>
Jorge Ramirez-Ortiz Aug. 19, 2016, 6:57 a.m. UTC | #3 
On 07/11/2016 11:53 AM, Wei Xu wrote:
> Hi Jorge,
>
> On 08/07/2016 09:11, Jorge Ramirez-Ortiz wrote:
>> Enable support for higher baud rates (up to 3Mbps) in UART1 - required
>> for bluetooth transfers.
>>
>> Signed-off-by: Jorge Ramirez-Ortiz <jorge.ramirez-ortiz@linaro.org>
>> Tested-by: Jorge Ramirez-Ortiz <jorge.ramirez-ortiz@linaro.org>
> Fine to me.
> Thanks!
>
> Acked-by: Wei Xu <xuwei5@hisilicon.com>
>
> Best Regards,
> Wei
>
>> ---
>>   arch/arm64/boot/dts/hisilicon/hi6220-hikey.dts | 2 ++
>>   1 file changed, 2 insertions(+)
>>
>> diff --git a/arch/arm64/boot/dts/hisilicon/hi6220-hikey.dts b/arch/arm64/boot/dts/hisilicon/hi6220-hikey.dts
>> index e92a30c..27be812 100644
>> --- a/arch/arm64/boot/dts/hisilicon/hi6220-hikey.dts
>> +++ b/arch/arm64/boot/dts/hisilicon/hi6220-hikey.dts
>> @@ -55,6 +55,8 @@
>>   		};
>>   
>>   		uart1: uart@f7111000 {
>> +			assigned-clocks = <&sys_ctrl HI6220_UART1_SRC>;
>> +			assigned-clock-rates = <150000000>;
>>   			status = "ok";
>>   		};
>>   
>>
Wei Xu,

I couldn't find the change upstream. has it been merged?
Wei Xu Aug. 19, 2016, 8:29 a.m. UTC | #4 
On 19/08/2016 07:57, Jorge Ramirez wrote:
> On 07/11/2016 11:53 AM, Wei Xu wrote:
>> Hi Jorge,
>>
>> On 08/07/2016 09:11, Jorge Ramirez-Ortiz wrote:
>>> Enable support for higher baud rates (up to 3Mbps) in UART1 - required
>>> for bluetooth transfers.
>>>
>>> Signed-off-by: Jorge Ramirez-Ortiz <jorge.ramirez-ortiz@linaro.org>
>>> Tested-by: Jorge Ramirez-Ortiz <jorge.ramirez-ortiz@linaro.org>
>> Fine to me.
>> Thanks!
>>
>> Acked-by: Wei Xu <xuwei5@hisilicon.com>
>>
>> Best Regards,
>> Wei
>>
>>> ---
>>>   arch/arm64/boot/dts/hisilicon/hi6220-hikey.dts | 2 ++
>>>   1 file changed, 2 insertions(+)
>>>
>>> diff --git a/arch/arm64/boot/dts/hisilicon/hi6220-hikey.dts b/arch/arm64/boot/dts/hisilicon/hi6220-hikey.dts
>>> index e92a30c..27be812 100644
>>> --- a/arch/arm64/boot/dts/hisilicon/hi6220-hikey.dts
>>> +++ b/arch/arm64/boot/dts/hisilicon/hi6220-hikey.dts
>>> @@ -55,6 +55,8 @@
>>>           };
>>>             uart1: uart@f7111000 {
>>> +            assigned-clocks = <&sys_ctrl HI6220_UART1_SRC>;
>>> +            assigned-clock-rates = <150000000>;
>>>               status = "ok";
>>>           };
>>>  

Hi Jorge,

> Wei Xu,
> 
> I couldn't find the change upstream. has it been merged?

No.
I thought Mike will pick them.
If nothing need to be change, I will pick up it.

Best Regards,
Wei

> 
> 
> .
>
Jorge Ramirez-Ortiz Aug. 19, 2016, 9:30 a.m. UTC | #5 
On 08/19/2016 10:29 AM, Wei Xu wrote:
>
> On 19/08/2016 07:57, Jorge Ramirez wrote:
>> On 07/11/2016 11:53 AM, Wei Xu wrote:
>>> Hi Jorge,
>>>
>>> On 08/07/2016 09:11, Jorge Ramirez-Ortiz wrote:
>>>> Enable support for higher baud rates (up to 3Mbps) in UART1 - required
>>>> for bluetooth transfers.
>>>>
>>>> Signed-off-by: Jorge Ramirez-Ortiz <jorge.ramirez-ortiz@linaro.org>
>>>> Tested-by: Jorge Ramirez-Ortiz <jorge.ramirez-ortiz@linaro.org>
>>> Fine to me.
>>> Thanks!
>>>
>>> Acked-by: Wei Xu <xuwei5@hisilicon.com>
>>>
>>> Best Regards,
>>> Wei
>>>
>>>> ---
>>>>    arch/arm64/boot/dts/hisilicon/hi6220-hikey.dts | 2 ++
>>>>    1 file changed, 2 insertions(+)
>>>>
>>>> diff --git a/arch/arm64/boot/dts/hisilicon/hi6220-hikey.dts b/arch/arm64/boot/dts/hisilicon/hi6220-hikey.dts
>>>> index e92a30c..27be812 100644
>>>> --- a/arch/arm64/boot/dts/hisilicon/hi6220-hikey.dts
>>>> +++ b/arch/arm64/boot/dts/hisilicon/hi6220-hikey.dts
>>>> @@ -55,6 +55,8 @@
>>>>            };
>>>>              uart1: uart@f7111000 {
>>>> +            assigned-clocks = <&sys_ctrl HI6220_UART1_SRC>;
>>>> +            assigned-clock-rates = <150000000>;
>>>>                status = "ok";
>>>>            };
>>>>   
> Hi Jorge,
>
>> Wei Xu,
>>
>> I couldn't find the change upstream. has it been merged?
> No.
> I thought Mike will pick them.
> If nothing need to be change, I will pick up it.

yes please, AFAIK nothing needs to be changed.
Wei Xu Aug. 24, 2016, 3:24 p.m. UTC | #6 
Hi Jorge,

On 08/07/2016 09:11, Jorge Ramirez-Ortiz wrote:
> Enable support for higher baud rates (up to 3Mbps) in UART1 - required
> for bluetooth transfers.
> 
> Signed-off-by: Jorge Ramirez-Ortiz <jorge.ramirez-ortiz@linaro.org>
> Tested-by: Jorge Ramirez-Ortiz <jorge.ramirez-ortiz@linaro.org>
> ---

Applied to the hisi-dt-4.9 branch.
Thanks!

Best Regards,
Wei

>  arch/arm64/boot/dts/hisilicon/hi6220-hikey.dts | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/arch/arm64/boot/dts/hisilicon/hi6220-hikey.dts b/arch/arm64/boot/dts/hisilicon/hi6220-hikey.dts
> index e92a30c..27be812 100644
> --- a/arch/arm64/boot/dts/hisilicon/hi6220-hikey.dts
> +++ b/arch/arm64/boot/dts/hisilicon/hi6220-hikey.dts
> @@ -55,6 +55,8 @@
>  		};
>  
>  		uart1: uart@f7111000 {
> +			assigned-clocks = <&sys_ctrl HI6220_UART1_SRC>;
> +			assigned-clock-rates = <150000000>;
>  			status = "ok";
>  		};
>  
>
diff mbox

Patch

diff --git a/arch/arm64/boot/dts/hisilicon/hi6220-hikey.dts b/arch/arm64/boot/dts/hisilicon/hi6220-hikey.dts
index e92a30c..27be812 100644
--- a/arch/arm64/boot/dts/hisilicon/hi6220-hikey.dts
+++ b/arch/arm64/boot/dts/hisilicon/hi6220-hikey.dts
@@ -55,6 +55,8 @@ 
 		};
 
 		uart1: uart@f7111000 {
+			assigned-clocks = <&sys_ctrl HI6220_UART1_SRC>;
+			assigned-clock-rates = <150000000>;
 			status = "ok";
 		};