[PATCH 2/3] bsp/versal: Support more than 4G of RAM

Gedare Bloom gedare at rtems.org
Wed Jun 15 14:48:14 UTC 2022


This was copied from the ZynqMP, should that also use a 64-bit value
for the ram length?

Probably any 64-bit architecture should do so?

On Tue, Jun 14, 2022 at 6:40 PM Chris Johns <chrisj at rtems.org> wrote:
>
> On 15/6/2022 2:43 am, Joel Sherrill wrote:
> > OK but the commit message could be a bit more enlightening.
>
> What about:
>
>  bsp/versal: Make RAM base 64bit to support more than 4G of RAM
>
> > It took me a minute to realize that all this did was change the type
> > of the memory length parameter. It actually does not change any
> > lengths to > 4 GB.
>
> Yes it is a constraint on the value and not the value.
>
> Chris
> _______________________________________________
> devel mailing list
> devel at rtems.org
> http://lists.rtems.org/mailman/listinfo/devel


More information about the devel mailing list