From mboxrd@z Thu Jan 1 00:00:00 1970 From: Dinh Nguyen Date: Tue, 18 Aug 2015 22:09:23 -0500 Subject: [U-Boot] [PATCH 8/8] arm: socfpga: Make the pinmux table const u8 In-Reply-To: <1439248246-9701-9-git-send-email-marex@denx.de> References: <1439248246-9701-1-git-send-email-marex@denx.de> <1439248246-9701-9-git-send-email-marex@denx.de> Message-ID: <55D3F363.4070109@opensource.altera.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On 8/10/15 6:10 PM, Marek Vasut wrote: > Now that we're actually converting the QTS-generated header files, > we can even adjust their data types. A good candidate for this is > the pinmux table, where each entry can have value in the range of > 0..3, but each element is declared as unsigned long. By changing > the type to u8, we can save over 600 Bytes from the SPL, so do it. > This patch also constifies the array. > > Signed-off-by: Marek Vasut > --- > arch/arm/mach-socfpga/include/mach/system_manager.h | 3 +-- > arch/arm/mach-socfpga/qts-filter.sh | 2 +- > arch/arm/mach-socfpga/system_manager.c | 2 +- > arch/arm/mach-socfpga/wrap_pinmux_config.c | 3 +-- > board/altera/arria5-socdk/qts/pinmux_config.h | 2 +- > board/altera/cyclone5-socdk/qts/pinmux_config.h | 2 +- > 6 files changed, 6 insertions(+), 8 deletions(-) > > diff --git a/arch/arm/mach-socfpga/include/mach/system_manager.h b/arch/arm/mach-socfpga/include/mach/system_manager.h > index 46af30b..8712f8e 100644 > --- a/arch/arm/mach-socfpga/include/mach/system_manager.h > +++ b/arch/arm/mach-socfpga/include/mach/system_manager.h > @@ -12,8 +12,7 @@ > void sysmgr_pinmux_init(void); > void sysmgr_config_warmrstcfgio(int enable); > > -void sysmgr_get_pinmux_table(const unsigned long **table, > - unsigned int *table_len); > +void sysmgr_get_pinmux_table(const u8 **table, unsigned int *table_len); > #endif > > struct socfpga_system_manager { > diff --git a/arch/arm/mach-socfpga/qts-filter.sh b/arch/arm/mach-socfpga/qts-filter.sh > index 16d3a2a..fc41d99 100755 > --- a/arch/arm/mach-socfpga/qts-filter.sh > +++ b/arch/arm/mach-socfpga/qts-filter.sh > @@ -66,7 +66,7 @@ process_pinmux_config() { > EOF > > # Retrieve the pinmux config and zap the ad-hoc length encoding > - sed -n '/^unsigned/ !b; :next {/^unsigned/ s/\[.*\]/[]/;p;n;b next}' \ > + sed -n '/^unsigned/ !b; :next {/^unsigned/ {s/\[.*\]/[]/s/unsigned long/const u8/};p;n;b next}' \ > ${in_dir}/generated/pinmux_config_${soc}.c > Should this change be in it's own patch? Thanks, Dinh