From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ezequiel Garcia Date: Sat, 25 May 2019 19:45:53 -0300 Subject: [U-Boot] [PATCH] configs: Remove CONFIG_SPL_FS_EXT4 for all MX6 platforms In-Reply-To: References: <20190525164934.13008-1-ezequiel@collabora.com> Message-ID: <176e687d6410b404948389e290ba98f3583aefcf.camel@collabora.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable To: u-boot@lists.denx.de On Sun, 2019-05-26 at 00:24 +0200, Marek Vasut wrote: > On 5/25/19 11:47 PM, Ezequiel Garcia wrote: > > On Sat, 2019-05-25 at 22:15 +0200, Marek Vasut wrote: > > > On 5/25/19 6:49 PM, Ezequiel Garcia wrote: > > > > i.MX6 platforms boot U-Boot second-stage from unformatted space, > > > > and should not need Ext filesystem support on SPL. > > > >=20 > > > > The commit was generated with: > > > >=20 > > > > git grep -l MX6 -- configs/ | xargs grep -l SPL_FS_EXT4 | xargs sed= -i -e '/CONFIG_SPL_FS_EXT4=3Dy/d' > > > >=20 > > > > This change has a dramatic impact on SPL size: > > > >=20 > > > > ./scripts/bloat-o-meter old new > > > > add/remove: 0/59 grow/shrink: 0/3 up/down: 0/-8674 (-8674) > > > > [..] > > > > Total: Before=3D38320, After=3D29646, chg -22.64% > > > >=20 > > > > Cc: Otavio Salvador > > > > Cc: Fabio Estevam > > > > Cc: Peng Fan > > > > Cc: Marek Vasut > > > > Cc: Stefano Babic > > > > Cc: Stefan Roese > > > > Cc: "Eric B=C3=A9nard" > > > > Cc: Breno Lima > > > > Cc: Francesco Montefoschi > > > > Signed-off-by: Ezequiel Garcia > > > > --- > > > > Tested on Wandboard only. Maintainers, please ACK or NAK! > > > >=20 > > > > configs/cgtqmx6eval_defconfig | 1 - > > > > configs/mx6cuboxi_defconfig | 1 - > > > > configs/mx6sabreauto_defconfig | 1 - > > > > configs/mx6sabresd_defconfig | 1 - > > > > configs/mx6slevk_spl_defconfig | 1 - > > > > configs/mx6sxsabresd_spl_defconfig | 1 - > > > > configs/mx6ul_14x14_evk_defconfig | 1 - > > > > configs/mx6ul_9x9_evk_defconfig | 1 - > > > > configs/novena_defconfig | 1 - > > >=20 > > > NAK, I boot my Novena from ext4 and this just broke it. > > >=20 > > > And also, NAK, this removes functionality from SPL which worked fine = before. > > >=20 > >=20 > > I'll drop from Novena, but I think the patch still makes some sense, > > why do you want Ext4 on SPL? >=20 > What other filesystem is available in SPL and doesn't have patent problem= s ? >=20 Sorry for not being clear. I am asking why turn on a feature that is so hea= vy, on a system that won't need it (such as Sabre* boards, Wandboard and simila= r)?