[PATCH] arm: mvebu: a38x: Fix 5200000 baudrate

Stefan Roese sr at denx.de
Thu Sep 2 14:18:45 CEST 2021


On 02.09.21 12:00, Pali Rohár wrote:
> Fix baudrate value 5150000 which was added in commit ead4864fa6cc ("arm:
> mvebu: a38x: Define supported UART baudrates").
> 
> Exact value for divisor 3 with 250 MHz TCLK is 5208333 baudrate. In above
> commit I incorrectly rounded it to 5150000 value due to testing with
> USB-UART hw which incorrectly reported exact value and divisor configured
> on other other end of UART link.
> 
> Fix this value to 520000 baudrate which is more close to the exact hardware
> value and also has less fraction parts.
> 
> Signed-off-by: Pali Rohár <pali at kernel.org>
> Fixes: ead4864fa6cc ("arm: mvebu: a38x: Define supported UART baudrates")

Reviewed-by: Stefan Roese <sr at denx.de>

Thanks,
Stefan

> ---
>   include/configs/mv-common.h | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/include/configs/mv-common.h b/include/configs/mv-common.h
> index 53d7acbb1034..6036bf4fa5d7 100644
> --- a/include/configs/mv-common.h
> +++ b/include/configs/mv-common.h
> @@ -45,7 +45,7 @@
>   					  230400, 460800, 500000, 576000, \
>   					  921600, 1000000, 1152000, 1500000, \
>   					  2000000, 2500000, 3125000, 4000000, \
> -					  5150000 }
> +					  5200000 }
>   #endif
>   
>   /* auto boot */
> 


Viele Grüße,
Stefan

-- 
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-51 Fax: (+49)-8142-66989-80 Email: sr at denx.de


More information about the U-Boot mailing list