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

Message ID 1467965467-28180-2-git-send-email-jorge.ramirez-ortiz@linaro.org
State Accepted
Commit 1b9c7b2d6339cb1353292a25c052f2b86efbb1b8
Headers show

Commit Message

Jorge Ramirez-Ortiz July 8, 2016, 8:11 a.m.
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(+)

-- 
2.7.4

Comments

Wei Xu July 11, 2016, 9:53 a.m. | #1
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. | #2
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. | #3
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. | #4
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. | #5
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";

>  		};

>  

>

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";
 		};