From mboxrd@z Thu Jan 1 00:00:00 1970 From: Lucas Stach Date: Mon, 01 Oct 2012 19:23:32 +0200 Subject: [U-Boot] [PATCH v2 1/2] tegra: clean up board include hell In-Reply-To: References: <1348948929-29928-1-git-send-email-dev@lynxeye.de> <5069C3C4.9080701@wwwdotorg.org> Message-ID: <1349112212.1520.16.camel@tellur> 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, Am Montag, den 01.10.2012, 10:12 -0700 schrieb Tom Warren: > Lucas, Stephen, et al. > > On Mon, Oct 1, 2012 at 9:39 AM, Tom Warren wrote: > > Lucas, > > > > On Mon, Oct 1, 2012 at 9:24 AM, Stephen Warren wrote: > >> On 09/29/2012 02:02 PM, Lucas Stach wrote: > >>> The prototypes used in board files were all scattered out, which lead to > >>> code duplication between SPL and normal U-Boot and some prototypes not actually > >>> being used. Consolidate this in a common board header. > >> > >> Seems reasonable enough to me. This series or anything similar that's > >> also OK with Tom is: > >> Acked-by: Stephen Warren > >> > > > > I don't think this hurts anything. I'll see if it applies cleanly to > > u-boot-tegra/next and let you know. > > > > Tom > > I've applied Lucas' include file cleanup patches, and done a .MAKEALL > -s tegra20 and a checkpatch run w/o error, so I've pushed a new > u-boot-tegra/next to denx.de. > > Here are the latest commits in that repo since Albert's last ARM update: > > 0edfb9c tegra: nand: add board pinmux > a05feff tegra: clean up board include hell > 9c53038 tegra: add funcmux entry for NAND attached to KBC > 08eb412 tegra20: rework UART GPIO handling > 55da8d1 tegra20: add clock_set_pllout function > 20e825f tegra20: complete periph_id enum > b1ae54d tegra: enable CONFIG_CMD_PART > 0e1b95c Tegra20: Move some include files to arch-tegra for sharing with Tegra30 > bfba961 Tegra20: Move some code files to common directories for upcoming Tegra30 > 4910a5c tegra: Rename Medcom to Medcom-Wide > 81637cd tegra: Update Avionic Design vendor prefix > > This is what I'm going to base my imminent T30 patches on. Hope to > have something for review in a day or two. > Can you do a P-R to Albert to include this in the u-boot-arm/next tree? The reason I'm prodding is that we need the following two patches to propagate to the u-boot-usb/next tree, to make the the USB driver move feasible. 0e1b95c Tegra20: Move some include files to arch-tegra for sharing with Tegra30 bfba961 Tegra20: Move some code files to common directories for upcoming Tegra30 As the u-boot/next tree is already open for merging I think the best way to achieve this is to let it trickle upstream, so that Marek can just base his next tree on an updated version of u-boot/next. There is no reason to to wait for the Tegra 3 work to land, except if you plan to rework the two mentioned commits along the way. Thanks, Lucas