On Wed, Jul 17, 2024 at 09:09:13AM -0500, Nishanth Menon wrote: > On 09:40-20240717, Kumar, Udit wrote: > > On 7/17/2024 1:35 AM, Francesco Dolcini wrote: > > > On Mon, Jul 15, 2024 at 11:13:01PM +0530, Udit Kumar wrote: > > > > Increase malloc size to 32 MB to align with other J7 > > > > family devices. > > > Is this needed for any specific reason? I am asking to understand if > > > other boards using the same SOC should be updated to prevent > > > $random_issue. > > > > In our downstream version, we are seeing an issue , while using DFU to MMC > > > > Please see > > > > https://gist.github.com/uditkumarti/b8209f61eff3154f6eb288ba63715263#file-gistfile1-txt-L82 > > > > > > > > Since DFU is not supported yet in mainline u-boot, so I avoided to mention > > that failure. > > > > Also this change is generic, so i thought of pushing it now instead of > > waiting for DFU series. > > if we are going to repeat this over and over - does it make sense to > have this in include? so that *other* board manufacturers do not need to > re-discover this?
The reason we (our board) might be fine, is that we have default 0x2000000 if ARCH_ROCKCHIP || ARCH_OMAP2PLUS || ARCH_MESON || ARCH_K3 from the downstream TI branch (the specific board we have with J784S4 is not in mainline yet). Francesco
