From mboxrd@z Thu Jan 1 00:00:00 1970 From: Lukasz Majewski Date: Wed, 3 May 2017 22:48:43 +0200 Subject: [U-Boot] [PATCH 3/3] spl: dfu: reduce spl-dfu MLO size In-Reply-To: <20170503124215.GS12511@bill-the-cat> References: <1493212449-26863-1-git-send-email-ravibabu@ti.com> <1493212449-26863-4-git-send-email-ravibabu@ti.com> <20170426133556.GX12511@bill-the-cat> <6C6B28D4DC342643927BEAFCE8707BF6C0B6E052@DBDE04.ent.ti.com> <20170427123129.GP12511@bill-the-cat> <6C6B28D4DC342643927BEAFCE8707BF6C0B75B5C@DBDE04.ent.ti.com> <20170503124215.GS12511@bill-the-cat> Message-ID: <20170503224843.122ae80d@jawa> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hi Tom, > On Wed, May 03, 2017 at 08:36:31AM +0000, B, Ravi wrote: > > Tom > > > > >>> > > >>> Yes you are correct. > > >>> But what is happening here is, the > > >>> CONFIG_DFU_ selected through > > >>> Kconfig/Menuconfig is applicable for both SPL and U-Boot. Hence > > >>> CONFIG_DFU_MMC/NA > > >ND/SF gets compiled for SPL as well, which needs run_command(). > > >Actually CONFIG_DFU_MMC/NAND/etc is not scoped for SPL-DFU. > > >>> As we have aligned, not to increase the SPL size, user shall > > >>> use SPL-DFU feature to boot to u-boot, then utilize the full > > >>> featured DFU to flash MMC/NAND/SF. > > >>> > > >>> I get undefined reference to common function run_command(), > > >>> "dfu_fill_entitiy_" in driver/dfu/dfu.c. The dfu.c > > >>> is common for both SPL-DFU and U-boot. > > > > >>OK. I think we need to introduce SPL_DFU_xxx Kconfig options, > > >>and use CONFIG_IS_ENABLED(DFU_xxx) so that we will get things > > >>enabled/disabled (and discarded) as needed. > > > > >Ok, will do. > > > > Correct me if I am wrong, I need understand if we introduce say > > SPL_DFU_MMC Kconfig options, then whether need to support DFU_MMC in > > SPL ? > > Again this will increase the SPL-size, and also DFU_MMC uses > > run_command() again, there is dependency of cli.c, hush etc. > > SPL_DFU_MMC will only increase the size of SPL if it's enabled. Being > able to switch to testing with CONFIG_IS_ENABLED(DFU_xxx) means that > we'll be able to keep the space savings while also not making various > parts of the code harder to read with more #ifdef tests. > +1 (and I would like to see the code soon :-) ) 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-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de -------------- next part -------------- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 181 bytes Desc: OpenPGP digital signature URL: