From: Tom Rini <trini@ti.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 1/5] am33xx: move ti i2c baseboard header handling to board/ti/am335x/evm.{c, h}
Date: Tue, 16 Oct 2012 08:35:11 -0700 [thread overview]
Message-ID: <507D7EAF.5080000@ti.com> (raw)
In-Reply-To: <1350400526-21714-2-git-send-email-peter.korsgaard@barco.com>
-----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-----
next prev parent reply other threads:[~2012-10-16 15:35 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-10-16 15:15 [U-Boot] [PATCH 0/5] am33xx: support non-ti boards Peter Korsgaard
2012-10-16 15:15 ` [U-Boot] [PATCH 1/5] am33xx: move ti i2c baseboard header handling to board/ti/am335x/evm.{c, h} Peter Korsgaard
2012-10-16 15:35 ` Tom Rini [this message]
2012-10-17 15:46 ` Peter Korsgaard
2012-10-17 15:55 ` Tom Rini
2012-10-17 19:24 ` Peter Korsgaard
2012-10-16 15:15 ` [U-Boot] [PATCH 2/5] am335x/evm.c: don't make wdtimer/uart_base global Peter Korsgaard
2012-10-16 15:15 ` [U-Boot] [PATCH 3/5] am33xx/board: use cpu_mmc_init() for default mmc initialization Peter Korsgaard
2012-10-16 15:36 ` Tom Rini
2012-10-16 17:37 ` Peter Korsgaard
2012-10-16 15:15 ` [U-Boot] [PATCH 4/5] am33xx: move generic parts of pinmux handling out from board/ti/am335x Peter Korsgaard
2012-10-16 15:15 ` [U-Boot] [PATCH 5/5] am33xx: support board specific ddr settings Peter Korsgaard
2012-10-16 15:31 ` Tom Rini
2012-10-16 17:43 ` Peter Korsgaard
2012-10-16 21:13 ` Tom Rini
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=507D7EAF.5080000@ti.com \
--to=trini@ti.com \
--cc=u-boot@lists.denx.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox