From mboxrd@z Thu Jan 1 00:00:00 1970 From: Simon Goldschmidt Date: Wed, 31 Oct 2018 08:00:55 +0100 Subject: [U-Boot] [PATCH v2] sf: remove spi-flash chip names from SPL binary In-Reply-To: References: <20181007071601.12302-1-simon.k.r.goldschmidt@gmail.com> Message-ID: List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Wed, Oct 31, 2018 at 7:35 AM Jagan Teki wrote: > > On Sun, Oct 7, 2018 at 12:46 PM Simon Goldschmidt > wrote: > > > > The table of spi flash chips 'spi_flash_ids' currently > > includes chip names. The only usage of these is to > > print the name when the chip is probed. > > > > Since this message is not shown in SPL, we can remove > > the names from the SPL binary. > > > > Removing the chip names saves ~890 Byte from the SPL > > binary in my configuration (socfpga_socrates_defconfig, > > MACRONIX, SPANSION and STMICRO enabled). > > I'm not still quite happy with this change, other than reducing the > size which introduces many ifdef's what would you benefits with this > 890bytes? do you have any room to add any key features in SPL? Well, SPL is always tight, isn't it? Currently, I have ~7.5 kB free on my socfpga gen5 board. And to get the features required by management, I will have to add suppport for loading U-Boot from a FIT image and checking its signature. Because of that, I figured saving RAM in SPL would be good. I don't know my required features fit in after applying this patch, though. But if you don't like it, feel free to drop it. I don't have an immediate need to apply this. Simon