Hi John, Thank you for the patch.
On ven., déc. 20, 2024 at 15:53, Jonathan Humphreys <[email protected]> wrote: > OSPI flash writing was broken on am62x SKs as the OSPI flash device has a > 256k sector size but was set to 20k. This breaks OSPI flash writes because > the current implementation of the SF DFU backend requires that the DFU data > buffer size be the same as the sector erase size, and the USB DFU config > fragment (for both A53 and R5) reduced the size of the DFU data buffer, > presumably so that it can fit into SPL's much smaller heap. > > Change the SYS_DFU_DATA_BUF_SIZE to the erase sector size, which still fits > within the post-relocation heap size in SPL. > > Fixes: dfc2dff5a844 ("configs: am62x_evm_*: Enable USB and DFU support") > > Signed-off-by: Jonathan Humphreys <[email protected]> Reviewed-by: Mattijs Korpershoek <[email protected]> > --- > configs/am62x_a53_usbdfu.config | 2 +- > configs/am62x_r5_usbdfu.config | 2 +- > 2 files changed, 2 insertions(+), 2 deletions(-) > > diff --git a/configs/am62x_a53_usbdfu.config b/configs/am62x_a53_usbdfu.config > index 0d3c6df1e73..812f99ee70b 100644 > --- a/configs/am62x_a53_usbdfu.config > +++ b/configs/am62x_a53_usbdfu.config > @@ -10,7 +10,7 @@ CONFIG_SYSCON=y > CONFIG_SPL_SYSCON=y > CONFIG_DFU_MMC=y > CONFIG_DFU_RAM=y > -CONFIG_SYS_DFU_DATA_BUF_SIZE=0x5000 > +CONFIG_SYS_DFU_DATA_BUF_SIZE=0x40000 > CONFIG_SYS_DFU_MAX_FILE_SIZE=0x800000 > CONFIG_USB=y > CONFIG_DM_USB_GADGET=y > diff --git a/configs/am62x_r5_usbdfu.config b/configs/am62x_r5_usbdfu.config > index 772bb2ab935..efaae504c1a 100644 > --- a/configs/am62x_r5_usbdfu.config > +++ b/configs/am62x_r5_usbdfu.config > @@ -1,7 +1,7 @@ > CONFIG_SPL_ENV_SUPPORT=y > CONFIG_SYSCON=y > CONFIG_SPL_SYSCON=y > -CONFIG_SYS_DFU_DATA_BUF_SIZE=0x5000 > +CONFIG_SYS_DFU_DATA_BUF_SIZE=0x40000 > CONFIG_MISC=y > CONFIG_USB=y > CONFIG_DM_USB_GADGET=y > -- > 2.34.1
