From mboxrd@z Thu Jan  1 00:00:00 1970
From: Jagan Teki <jagan@amarulasolutions.com>
Date: Wed, 31 Oct 2018 12:05:05 +0530
Subject: [U-Boot] [PATCH v2] sf: remove spi-flash chip names from SPL
	binary
In-Reply-To: <20181007071601.12302-1-simon.k.r.goldschmidt@gmail.com>
References: <20181007071601.12302-1-simon.k.r.goldschmidt@gmail.com>
Message-ID: <CAMty3ZAJC2e6rMdKOKC8Lxbw4jYmNLOLoEZU9i-HJYrjp8A6Vg@mail.gmail.com>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

On Sun, Oct 7, 2018 at 12:46 PM Simon Goldschmidt
<simon.k.r.goldschmidt@gmail.com> 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?