From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tom Rini Date: Mon, 6 Jan 2014 12:36:37 -0500 Subject: [U-Boot] [RFC][PATCH 0/7] TI: OMAP3: Use common config file. In-Reply-To: References: <1386361824-21900-1-git-send-email-eballetbo@gmail.com> Message-ID: <20140106173637.GP420@bill-the-cat> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Mon, Dec 23, 2013 at 12:11:25PM +0100, Enric Balletbo Serra wrote: > 2013/12/6 Enric Balletbo i Serra : > > Hi all, > > > > Most of the boards based on TI processors uses common configuration files > > (ti_armv7_common.h, ti__common.h) to avoid duplication of code. > > This is right except for OMAP3-based boards. In order to use the same schema > > as used on am33xx, omap4, omap5 and dra7 TI processors these patches create > > a new ti_omap3_common.h (that include ti_armv7_common.h) with the purpose that > > all OMAP3 board can use it. > > > > Patches 1 and 2 just renames current omap4|omap5_common.h to > > ti_omap4|omap5_common.h to be coherent with current ti_am33xx_common.h, > > ti_armv7_common.h and the new ti_omap3_common.h. It's just a cosmetic change so > > if people don't like it I don't have any inconvenient to remove from these > > series. > > > > Patches 3 and 4 modifies the ti_armv7_common.h to be more compatible with OMAP3 > > boards. For example, patch 3 removes the assumption that all ti_armv7 have an > > ELM hardware engine and patch 4 handles the case that the number of DRAM banks > > is defined at board level. The patch 5 is also required to integrate the use > > of ti_armv7_common.h on OMAP3 boards. > > > > Patch 6 creates the new ti_omap3_common.h to be used for any OMAP3-based board. > > > > And finally, patch 7 moves the IGEP boards to use the new common file. As I > > only have IGEP hardware to test these patches I decided only implement the use > > case for these boards. I don't have any inconvenient to move other OMAP3 boards > > to use this schema but I prefer leave the decision to the board maintainers. > > > > Any comments, improvements, fixes are welcome. > > > > Best regards, > > > > Enric Balletbo i Serra (7): > > ARM: OMAP4: Rename to ti_omap4_common.h > > ARM: OMAP5: Rename to ti_omap5_common.h > > TI: armv7: Move ELM support to SoC configuration file. > > TI: armv7: Do not define the number DRAM banks if is already defined. > > ARM: OMAP3: Rename OMAP3_PUBLIC_SRAM_* to NON_SECURE_SRAM_* > > TI: OMAP3: Create common config files for TI OMAP3 platforms. > > OMAP3: igep00x0: Convert to ti_omap3_common.h. > > > > arch/arm/include/asm/arch-omap3/omap3.h | 6 +- > > include/configs/dra7xx_evm.h | 4 +- > > include/configs/omap3_igep00x0.h | 190 +-------------------- > > include/configs/omap4_panda.h | 4 +- > > include/configs/omap4_sdp4430.h | 4 +- > > include/configs/omap5_uevm.h | 4 +- > > include/configs/ti_am335x_common.h | 4 + > > include/configs/ti_armv7_common.h | 11 +- > > include/configs/ti_omap3_common.h | 73 ++++++++ > > .../configs/{omap4_common.h => ti_omap4_common.h} | 10 +- > > .../configs/{omap5_common.h => ti_omap5_common.h} | 10 +- > > 11 files changed, 118 insertions(+), 202 deletions(-) > > create mode 100644 include/configs/ti_omap3_common.h > > rename include/configs/{omap4_common.h => ti_omap4_common.h} (95%) > > rename include/configs/{omap5_common.h => ti_omap5_common.h} (95%) > > Ping, any comment on this patch series ? I intend to pick this up after v2014.01. Thanks! -- Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 836 bytes Desc: Digital signature URL: