[U-Boot] [PATCH] arm64: versal: Fix for OCM overwriting issue

[U-Boot] [PATCH] arm64: versal: Fix for OCM overwriting issue

[Michal Simek]

From: Ashok Reddy Soma <ashok.reddy.soma@xilinx.com>

This patch reduces the console buffer size from 2048 to 1024, thus fixes
OCM overwriting issue.

Differences are in bss section as is shown from output:
xilinx_versal_mini: all -2056 bss -2048 text -8
   u-boot: add: 0/0, grow: 0/-3 bytes: 0/-2056 (-2056)
     function                                   old     new   delta
     cli_simple_run_command                     340     332      -8
     static.lastcommand                        2049    1025   -1024
     console_buffer                            2049    1025   -1024

Better would be to use PPU RAM but this change is also aligned with changes
done in ZynqMP mini configurations.

Signed-off-by: Ashok Reddy Soma <ashok.reddy.soma@xilinx.com>
Signed-off-by: Michal Simek <michal.simek@xilinx.com>
---

 include/configs/xilinx_versal_mini.h | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/include/configs/xilinx_versal_mini.h b/include/configs/xilinx_versal_mini.h
index 4b3691b40733..2195bf0e1064 100644
--- a/include/configs/xilinx_versal_mini.h
+++ b/include/configs/xilinx_versal_mini.h
@@ -29,4 +29,7 @@
 #undef CONFIG_BOOTP_BOOTFILESIZE
 #undef CONFIG_BOOTP_MAY_FAIL
 
+#undef CONFIG_SYS_CBSIZE
+#define CONFIG_SYS_CBSIZE		1024
+
 #endif /* __CONFIG_VERSAL_MINI_H */
-- 
2.17.1

[U-Boot] [PATCH] arm64: versal: Fix for OCM overwriting issue

[Michal Simek]

st 11. 9. 2019 v 10:52 odesílatel Michal Simek <michal.simek@xilinx.com> napsal:
>
> From: Ashok Reddy Soma <ashok.reddy.soma@xilinx.com>
>
> This patch reduces the console buffer size from 2048 to 1024, thus fixes
> OCM overwriting issue.
>
> Differences are in bss section as is shown from output:
> xilinx_versal_mini: all -2056 bss -2048 text -8
>    u-boot: add: 0/0, grow: 0/-3 bytes: 0/-2056 (-2056)
>      function                                   old     new   delta
>      cli_simple_run_command                     340     332      -8
>      static.lastcommand                        2049    1025   -1024
>      console_buffer                            2049    1025   -1024
>
> Better would be to use PPU RAM but this change is also aligned with changes
> done in ZynqMP mini configurations.
>
> Signed-off-by: Ashok Reddy Soma <ashok.reddy.soma@xilinx.com>
> Signed-off-by: Michal Simek <michal.simek@xilinx.com>
> ---
>
>  include/configs/xilinx_versal_mini.h | 3 +++
>  1 file changed, 3 insertions(+)
>
> diff --git a/include/configs/xilinx_versal_mini.h b/include/configs/xilinx_versal_mini.h
> index 4b3691b40733..2195bf0e1064 100644
> --- a/include/configs/xilinx_versal_mini.h
> +++ b/include/configs/xilinx_versal_mini.h
> @@ -29,4 +29,7 @@
>  #undef CONFIG_BOOTP_BOOTFILESIZE
>  #undef CONFIG_BOOTP_MAY_FAIL
>
> +#undef CONFIG_SYS_CBSIZE
> +#define CONFIG_SYS_CBSIZE              1024
> +
>  #endif /* __CONFIG_VERSAL_MINI_H */
> --
> 2.17.1
>

Applied.
M

-- 
Michal Simek, Ing. (M.Eng), OpenPGP -> KeyID: FE3D1F91
w: www.monstr.eu p: +42-0-721842854
Maintainer of Linux kernel - Xilinx Microblaze
Maintainer of Linux kernel - Xilinx Zynq ARM and ZynqMP ARM64 SoCs
U-Boot custodian - Xilinx Microblaze/Zynq/ZynqMP/Versal SoCs