[PATCH] configs: am65x_evm: Define the maximum file size for DFU

[PATCH] configs: am65x_evm: Define the maximum file size for DFU

[Aswath Govindraju]

In include/dfu.h, if CONFIG_SYS_DFU_MAX_FILE_SIZE is not defined then it is
defined as CONFIG_SYS_DFU_DATA_BUF_SIZE. This is 128 KiB for a53 core and
20 KiB for r5 core. If a larger file is transferred using dfu then it
fails.

CONFIG_SYS_DFU_DATA_BUF_SIZE can not be increased as there is not enough
heap memory to be allocated for the buffer in case of R5 spl.

Fix this by defining CONFIG_SYS_DFU_MAX_FILE_SIZE as the default
CONFIG_SYS_DFU_DATA_BUF_SIZE value.

Signed-off-by: Aswath Govindraju <a-govindraju@ti.com>
---
 include/configs/am65x_evm.h | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/include/configs/am65x_evm.h b/include/configs/am65x_evm.h
index 41c62785e438..9d9d0c93a206 100644
--- a/include/configs/am65x_evm.h
+++ b/include/configs/am65x_evm.h
@@ -56,6 +56,12 @@
 #define CONFIG_SKIP_LOWLEVEL_INIT
 #endif
 
+/*
+ * If the maximum size is not declared then it is defined as
+ * CONFIG_SYS_DFU_DATA_BUF_SIZE.
+ */
+#define CONFIG_SYS_DFU_MAX_FILE_SIZE	(1024 * 1024 * 8)   /* 8 MiB */
+
 #define CONFIG_SPL_MAX_SIZE		CONFIG_SYS_K3_MAX_DOWNLODABLE_IMAGE_SIZE
 
 #define CONFIG_SYS_BOOTM_LEN		SZ_64M
-- 
2.17.1

[PATCH] configs: am65x_evm: Define the maximum file size for DFU

[Lokesh Vutla]

On 24/11/20 3:59 pm, Aswath Govindraju wrote:
> In include/dfu.h, if CONFIG_SYS_DFU_MAX_FILE_SIZE is not defined then it is
> defined as CONFIG_SYS_DFU_DATA_BUF_SIZE. This is 128 KiB for a53 core and
> 20 KiB for r5 core. If a larger file is transferred using dfu then it
> fails.
> 
> CONFIG_SYS_DFU_DATA_BUF_SIZE can not be increased as there is not enough
> heap memory to be allocated for the buffer in case of R5 spl.
> 
> Fix this by defining CONFIG_SYS_DFU_MAX_FILE_SIZE as the default
> CONFIG_SYS_DFU_DATA_BUF_SIZE value.
> 
> Signed-off-by: Aswath Govindraju <a-govindraju@ti.com>

Applied to u-boot-ti/for-next

Thanks and regards,
Lokesh

> ---
>  include/configs/am65x_evm.h | 6 ++++++
>  1 file changed, 6 insertions(+)
> 
> diff --git a/include/configs/am65x_evm.h b/include/configs/am65x_evm.h
> index 41c62785e438..9d9d0c93a206 100644
> --- a/include/configs/am65x_evm.h
> +++ b/include/configs/am65x_evm.h
> @@ -56,6 +56,12 @@
>  #define CONFIG_SKIP_LOWLEVEL_INIT
>  #endif
>  
> +/*
> + * If the maximum size is not declared then it is defined as
> + * CONFIG_SYS_DFU_DATA_BUF_SIZE.
> + */
> +#define CONFIG_SYS_DFU_MAX_FILE_SIZE	(1024 * 1024 * 8)   /* 8 MiB */
> +
>  #define CONFIG_SPL_MAX_SIZE		CONFIG_SYS_K3_MAX_DOWNLODABLE_IMAGE_SIZE
>  
>  #define CONFIG_SYS_BOOTM_LEN		SZ_64M
>