[PATCH 16/16] board: rzg2l: Add RZ/G2L SMARC EVK board

Paul Barker paul.barker.ct at bp.renesas.com
Wed Oct 4 15:56:05 CEST 2023


On Wed, Oct 04, 2023 at 02:33:07PM +0200, Marek Vasut wrote:
> On 10/4/23 11:36, Paul Barker wrote:
> 
> [...]
> 
> > > > +int dram_init(void)
> > > > +{
> > > > +	return fdtdec_setup_mem_size_base();
> > > > +}
> > > > +
> > > > +int dram_init_banksize(void)
> > > > +{
> > > > +	fdtdec_setup_memory_banksize();
> > > 
> > > This and the one function above can likely go into arch/mach-rmobile/
> > 
> > We're not able to test every supported mach-rmobile board so we've
> > preferred to keep changes to RZ/G2L specific code where possible. I'm
> > happy to merge this into mach-rmobile (along with the similar code in
> > board/renesas/rcar-common/common.c), but it will only be tested on the
> > boards we have in our team.
> 
> Don't you already have a G2L specific file in arch/arm/mach-rmobile/ ?
> 
> (I can test whatever changes on R-Car if needed)

We could move these into arch/arm/mach-rmobile/memmap-rzg2l.c I guess.

> 
> > > > +	return 0;
> > > > +}
> > > > +
> > > > +void __weak reset_cpu(void)
> > > > +{
> > > 
> > > Is this platform missing a way to reset the system ?
> > > Please fix.
> > 
> > Adding reset support is on my todo list. I'm prioritising Ethernet
> > support above this right now, but we don't mind too much what order we
> > do things. Let me know if you'd prefer us to prioritise reset support.
> 
> Isn't the reset like really really simple ? Enable SYSRESET config option
> and do TFA/SMC reset I guess ?

We don't have support for that in TF-A yet.

Thanks,
Paul
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 228 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20231004/750ae4aa/attachment.sig>


More information about the U-Boot mailing list