Re: [PATCH 3/5] arm: initial support for Marvell Dove SoCs

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

On Mon, May 13, 2013 at 03:06:55PM +0200, Sebastian Hesselbarth wrote:
> On 05/13/13 12:57, Sascha Hauer wrote:
> >On Mon, May 13, 2013 at 11:17:21AM +0200, Sebastian Hesselbarth wrote:
> >>The regbase pointer is required as early debug _will_ access register
> >>before and after remap and there is no way around it. But
> >>mvreadl/mvwritel will be limited to code that sits in mach-mvebu, all
> >>drivers will depend on DT.
> >
> >Having the register base as variable sounds like a good idea. This
> >should give us some more flexibility, no matter what we do later.
> >
> >Please note that normally barebox images are expected to be runnable
> >second stage (bootm barebox.bin). Though not really mandatory this still
> >is a nice feature for development. This becomes difficult to support if
> >the initial code expects the registers at 0xd0000000, hence I suggested
> >remapping it in the kwb image so that all second stage code can already
> >work on the remapped registers.
> 
> Yeah, this is bugging Thomas and me for some time. The tricky part in
> this is, that the register for setting the internal register base is in
> the internal registers itself. You don't know the base address, you
> cannot remap it - you can't even read it. Thomas is working on
> something, but he will have to comment on that.

Dosenöffner innenliegend - Can opener inside ;)

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