From mboxrd@z Thu Jan 1 00:00:00 1970 From: Lukasz Majewski Date: Sun, 26 May 2019 10:22:00 +0200 Subject: [U-Boot] [PATCH] configs: Remove CONFIG_SPL_FS_EXT4 for all MX6 platforms In-Reply-To: References: <20190525164934.13008-1-ezequiel@collabora.com> <176e687d6410b404948389e290ba98f3583aefcf.camel@collabora.com> <830f4dcc-590a-c1c0-e786-2389622b6f1a@denx.de> <20190525230821.GA20781@bill-the-cat> <87b35bb5-3061-5dcc-e7ad-0f43046598f8@denx.de> <20190525232318.GB20781@bill-the-cat> Message-ID: <20190526102200.796afadc@jawa> 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 Dear Marek, Tom, > On 5/26/19 1:23 AM, Tom Rini wrote: > > On Sun, May 26, 2019 at 01:20:34AM +0200, Marek Vasut wrote: =20 > >> On 5/26/19 1:08 AM, Tom Rini wrote: =20 > >>> On Sun, May 26, 2019 at 12:57:08AM +0200, Marek Vasut wrote: =20 > >>>> On 5/26/19 12:45 AM, Ezequiel Garcia wrote: =20 > >>>>> On Sun, 2019-05-26 at 00:24 +0200, Marek Vasut wrote: =20 > >>>>>> On 5/25/19 11:47 PM, Ezequiel Garcia wrote: =20 > >>>>>>> On Sat, 2019-05-25 at 22:15 +0200, Marek Vasut wrote: =20 > >>>>>>>> On 5/25/19 6:49 PM, Ezequiel Garcia wrote: =20 > >>>>>>>>> i.MX6 platforms boot U-Boot second-stage from unformatted > >>>>>>>>> space, and should not need Ext filesystem support on SPL. > >>>>>>>>> > >>>>>>>>> The commit was generated with: > >>>>>>>>> > >>>>>>>>> git grep -l MX6 -- configs/ | xargs grep -l SPL_FS_EXT4 | > >>>>>>>>> xargs sed -i -e '/CONFIG_SPL_FS_EXT4=3Dy/d' > >>>>>>>>> > >>>>>>>>> This change has a dramatic impact on SPL size: > >>>>>>>>> > >>>>>>>>> ./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% > >>>>>>>>> > >>>>>>>>> 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! > >>>>>>>>> > >>>>>>>>> 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. > >>>>>>>> > >>>>>>>> And also, NAK, this removes functionality from SPL which > >>>>>>>> worked fine before.=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 problems ?=20 > >>>>> > >>>>> Sorry for not being clear. I am asking why turn on a feature > >>>>> that is so heavy, on a system that won't need it (such as > >>>>> Sabre* boards, Wandboard and similar)? =20 > >>>> > >>>> Two reasons: > >>>> 1) It was enabled, disabling it means removing functionality for > >>>> no good reason (oops, bloat, is not a good reason), and that is > >>>> not desired. 2) Booting from block device implies booting from a > >>>> filesystem, otherwise you might overwrite various things on the > >>>> block device when updating the file (u-boot image). =20 > >>> > >>> So, are you using SPL to load something from ext4 or not? =20 > >> > >> Yes, I think that's what I said. > >> =20 > >>> There are > >>> setups where people have configured the system such that SPL loads > >>> something from ext4 and that's why we have it available. Is > >>> anyone doing that on Novena? Or any iMX system? =20 > >> > >> Quoting my first response in this thread: > >> " > >> NAK, I boot my Novena from ext4 and this just broke it. > >> " =20 > >=20 > > Actually, I wasn't sure from your first response if you're using > > SPL to load u-boot from EXT4 or not. So, Novena is a no and we > > should wait for more board maintainers to speak up to see if they > > use it or not, thanks! =20 >=20 > Novena is certainly a no. Since I use a couple of wandboards, those > are no as well. >=20 > But I do not want to see useful functionality removed from SPL only to > make space for useless DM/DT bloat. Temporarily band-aiding this real > problem by removing functionality is a no-go, no matter how you slice > it. The real fix is to fix the DM/DT and figure out a way to reduce > it's size and _retain_ _all_ the functionality. >=20 I fully agree with Marek here - the DM/DT SPL support (in the form as it is now) is a bloat. The SPL size increases considerably. The _real_ issue is the lack of OF_PLATDATA support for DM/DTS drivers, which are used when we convert u-boot to DM/DTS. For boards, which have enough internal on-chip RAM (OCRAM in case of imx6q) it is doable to convert them to DM/DTS in SPL. However, nobody wants to say it loud, but the footprint considerably increases in SPL - for example: SPL (and I only use eMMC there - no SPI, no I2C): ------------------------------------------------ Before conversion: 35 KiB DM conversion: 47 KiB (around ~25% increase) For u-boot proper: ---------------------------- Footprint changes (u-boot.imx): Before conversion: 345 KiB DM conversion: 415 KiB (around + 17% increase) This has several implications: 1. We replace small, robust SPL (I speak only for i.MX) with something which is XX% larger and hence boot time increases. In short - customers are/will not be happy. We do introduce regression here. 2. To make it usable (with the size comparable to what we do have now) we need to add OF_PLATDATA support to eMMC, SPI, I2C, etc. drivers. Test them and validate. That is why we now "just" convert only U-Boot proper to DM/DTS.=20 As a follow up - it seems pragmatic to not touch SPL for now, and start the conversion ONLY when necessary drivers gain support for OF_PLATDATA (so we don't need DTS support in SPL). Unfortunately, it would take some time. Best regards, Lukasz Majewski -- DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-59 Fax: (+49)-8142-66989-80 Email: lukma at denx.de -------------- next part -------------- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 488 bytes Desc: OpenPGP digital signature URL: