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

Jean-Christophe PLAGNIOL-VILLARD plagnioj at jcrosoft.com
Wed Feb 13 04:53:41 EST 2013


On 09:18 Wed 13 Feb     , Sascha Hauer wrote:
> 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 at calxeda.com>
> > Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD <plagnioj at 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 at 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 at 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 ;)
ok
> 
> 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 |



More information about the barebox mailing list