Re: [PATCH 1/1] highbank: use the provided dtb by the firmware to probe barebox device and mem size

[Sascha Hauer <s.hauer@pengutronix.de>]

On Tue, Feb 12, 2013 at 10:11:00PM +0100, Jean-Christophe PLAGNIOL-VILLARD wrote:
> the dtb is at 0x1000
> 
> if no dtb present use C code device
> 
> kepp in C the timer/gpio/uart
> 
> Cc: Rob Herring <rob.herring@calxeda.com>
> Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com>
> ---
>  arch/arm/boards/highbank/env/bin/init_board |    7 +++++
>  arch/arm/boards/highbank/init.c             |   44 ++++++++++++++++++++++++---
>  2 files changed, 47 insertions(+), 4 deletions(-)
>  create mode 100644 arch/arm/boards/highbank/env/bin/init_board
> 
> diff --git a/arch/arm/boards/highbank/env/bin/init_board b/arch/arm/boards/highbank/env/bin/init_board

We already have:

| commit f59f5e86256f82daf4af3bf1922805592b780613
| Author: Sascha Hauer <s.hauer@pengutronix.de>
| Date:   Sat Oct 6 19:51:47 2012 +0200
| 
|     of: find and register memory during probe
|     
|     This automatically registers the memory nodes in the devicetree.
|     
|     Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>

I suspect there is something missing since this commit checks for the
'device_type' property, but does not actually test if it matches
'memory'. So there might be some work necessary, but this code already
handles 64bit cells correctly, at least it tries to ;)

Sascha

-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox