From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tom Rini Date: Thu, 21 Nov 2019 17:45:14 -0500 Subject: [U-Boot] Maximum size of u-boot.imx for TBS2910 board In-Reply-To: <3647be08-dd89-ba9a-51a7-09cd08176816@denx.de> References: <20191121201253.GN971@bill-the-cat> <5e69c39f-a36a-bcd3-8020-e752ded84166@gmx.de> <3647be08-dd89-ba9a-51a7-09cd08176816@denx.de> Message-ID: <20191121224514.GS971@bill-the-cat> 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 Thu, Nov 21, 2019 at 11:01:43PM +0100, Marek Vasut wrote: > On 11/21/19 10:59 PM, Heinrich Schuchardt wrote: > > On 11/21/19 9:12 PM, Tom Rini wrote: > >> On Thu, Nov 21, 2019 at 09:09:29PM +0100, Heinrich Schuchardt wrote: > >>> Hello Soeren, > >>> > >>> when trying to add support for function key support in the USB keyboa= rd > >>> driver u-boot.imx for the TBS2910 surpassed the maximum size for > >>> u-boot.imx. > >>> https://travis-ci.org/marex/u-boot-usb/builds/614059004 > >>> > >>> Do you remember why on the TBS2910 board this size is limited to > >>> 0x5fc00? Other i.MX6 boards like the Wandboard allow a much larger > >>> u-boot.imx. > >>> > >>> The limit is defined here: > >>> include/configs/tbs2910.h:80: > >>> #define CONFIG_BOARD_SIZE_LIMIT 392192 /* (CONFIG_ENV_OFFSET - 1024) = */ > >>> > >>> Could the value CONFIG_ENV_OFFSET=3D0x60000 be enlarged? > >>> > >>> Many i.MX6 defconfigs use CONFIG_ENV_OFFSET=3D0xC0000. > >> > >> The nature of these boards (aimed at end users) means that we just do > >> not want to / cannot really move the stored environment.=C2=A0 Thanks! > >=20 > > Another possibility would be to reduce the image size by using > > CONFIG_REGEX=3Dn which should be fine for a board with only one support= ed > > network interface. >=20 > But the board was fine before your patchset got applied and this is just > a workaround for added bloat, which reduces functionality. I dislike > trading functionality for bloat, sorry. One persons "bloat" is another persons "added functionality". I believe the specific changes in question that once again push this board over fall in to that grey area. Whatever size-trimming the board maintainer is fine with next is fine with me, but needs to get ack'd by someone. --=20 Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 833 bytes Desc: not available URL: