[v3,3/7] arm: juno: Fix UART clock rate

Message ID 20200427181804.15787-4-andre.przywara@arm.com
State Superseded
Headers show
Series
  • Arm Juno board OF_CONTROL upgrade
Related show

Commit Message

Andre Przywara April 27, 2020, 6:18 p.m.
The UART base clock rate was typo-ed in the header file, probably because
the reference (the Linux .dts) was also wrong[1].

Fix the number to make the baud rate more correct.

Signed-off-by: Andre Przywara <andre.przywara at arm.com>
Reviewed-by: Linus Walleij <linus.walleij at linaro.org>

[1] https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=39a1a8941b2
---
 include/configs/vexpress_aemv8a.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Simon Glass April 28, 2020, 5:57 p.m. | #1
On Mon, 27 Apr 2020 at 12:18, Andre Przywara <andre.przywara at arm.com> wrote:
>
> The UART base clock rate was typo-ed in the header file, probably because
> the reference (the Linux .dts) was also wrong[1].
>
> Fix the number to make the baud rate more correct.
>
> Signed-off-by: Andre Przywara <andre.przywara at arm.com>
> Reviewed-by: Linus Walleij <linus.walleij at linaro.org>
>
> [1] https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=39a1a8941b2
> ---
>  include/configs/vexpress_aemv8a.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Reviewed-by: Simon Glass <sjg at chromium.org>
Tom Rini May 7, 2020, 1:03 p.m. | #2
On Mon, Apr 27, 2020 at 07:18:00PM +0100, Andre Przywara wrote:

> The UART base clock rate was typo-ed in the header file, probably because
> the reference (the Linux .dts) was also wrong[1].
> 
> Fix the number to make the baud rate more correct.
> 
> Signed-off-by: Andre Przywara <andre.przywara at arm.com>
> Reviewed-by: Linus Walleij <linus.walleij at linaro.org>
> 
> [1] https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=39a1a8941b2
> Reviewed-by: Simon Glass <sjg at chromium.org>

Applied to u-boot/master, thanks!

Patch

diff --git a/include/configs/vexpress_aemv8a.h b/include/configs/vexpress_aemv8a.h
index 6f81760612..3c85c93a5c 100644
--- a/include/configs/vexpress_aemv8a.h
+++ b/include/configs/vexpress_aemv8a.h
@@ -102,7 +102,7 @@ 
 
 /* PL011 Serial Configuration */
 #ifdef CONFIG_TARGET_VEXPRESS64_JUNO
-#define CONFIG_PL011_CLOCK		7273800
+#define CONFIG_PL011_CLOCK		7372800
 #else
 #define CONFIG_PL011_CLOCK		24000000
 #endif