From mboxrd@z Thu Jan  1 00:00:00 1970
From: Tom Rini <trini@ti.com>
Date: Tue, 16 Oct 2012 08:35:11 -0700
Subject: [U-Boot] [PATCH 1/5] am33xx: move ti i2c baseboard header
 handling to board/ti/am335x/evm.{c, h}
In-Reply-To: <1350400526-21714-2-git-send-email-peter.korsgaard@barco.com>
References: <1350400526-21714-1-git-send-email-peter.korsgaard@barco.com>
	<1350400526-21714-2-git-send-email-peter.korsgaard@barco.com>
Message-ID: <507D7EAF.5080000@ti.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

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On 10/16/12 08:15, Peter Korsgaard wrote:
> The i2c header is specific to ti(-derived) boards, and not generic
> for all am335x boards.
> 
> Signed-off-by: Peter Korsgaard <peter.korsgaard@barco.com> --- 
> arch/arm/cpu/armv7/am33xx/board.c                  |  235
> -------------------- arch/arm/include/asm/arch-am33xx/sys_proto.h
> |   27 --- board/ti/am335x/Makefile                           |
> 1 + .../armv7/am33xx/board.c => board/ti/am335x/evm.c  |   33 +-- 
> .../sys_proto.h => board/ti/am335x/evm.h           |   24 +- 
> board/ti/am335x/mux.c                              |    1 + 6 files
> changed, 10 insertions(+), 311 deletions(-) copy
> arch/arm/cpu/armv7/am33xx/board.c => board/ti/am335x/evm.c (90%) 
> copy arch/arm/include/asm/arch-am33xx/sys_proto.h =>
> board/ti/am335x/evm.h (75%)

I'd like to see it as board/ti/am33xx/board.[ch] but other than that I
can see where you're coming from.  Can you post a WIP version of your
s_init / board_init?  I'd like to see what maybe we should call __weak
or try and restructure a bit more.  Thanks!

- -- 
Tom
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.11 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://www.enigmail.net/

iQIcBAEBAgAGBQJQfX6vAAoJENk4IS6UOR1W6h0QAIhAoLVANPWvrtH+MP5S6WBD
G4kqyaefd0AJmCNX9CXMT+Qx/rjMqM/bPr047K80xj0XpC3GLaGMiZJbnjXoFwcg
bm0hd3ogK9AeSReONsMjk5qndLtBEfVniGDPi33atzimEUjsHYZdc9zR0SbP7+il
7n8YlZQoCH6KytZNn1/REXQQfs1ZY+zRA5uUxgqIpejCBTjoQXItQ1zURXUftD76
Rufk4vzEEvSVeQ/RIW3xhSLPzUTLjI1dBfZOgK5tzCb5ltxE9C8bOpIqt+ISuh73
1nLsJnxDQF5TR1WHyyX8yIN8FVow/eglaHCfi3y0NZdkRiFWlkN+rhL+XqNIOfiP
/ddZ41Vo5oA5t4wxWktd+GN6gSEivfjPw8m8EoJCSNdBgZi1yDD8YPshVcVQm29f
EiHOi1ZOzCkWk5m12XVRl32LxtikEOjb2Icw2tjZ187s4Ow3S3H6+dseRKkgqxk8
WrnVt64mr8LimdeEZShWOcm+0iFMAdYzmmHmtGL+8lqyeAPoHIQ0LGRnjNarbtXJ
EAcremyRzIgtnMGMDwDpWczXoH+sYfPaNSzT9tJ39quWkn5f7KLU5zC40BnxkAjh
0fxmdec6TN2a29sfPL3fXiD1e7Nvirem8jJrOcNyP90fGx3cwfRkNrQgLitYpY/w
VtncBdacDWaqaTtSGDlR
=4apt
-----END PGP SIGNATURE-----