[PATCH 4/6] arm64/ls1043a: add DTS for Freescale LS1043A SoC

[Li Leo]

> >
> > Hi,
> >
> > > +/memreserve/ 0x80000000 0x00010000;
> >
> > Why is this necessary?
> 
> This memory region is pre-reserved for the spin-table/psci, although didn't add
> Enable method of secondary cores.

Can this be reserved by bootloader?

> 
> > If this is necessary, please add a comment stating what this is for.
> >
> > > +               cpu at 3 {
> > > +                       device_type = "cpu";
> > > +                       compatible = "arm,cortex-a53";
> > > +                       reg = <0x0 0x3>;
> > > +                       clocks = <&clockgen 1 0>;
> > > +               };
> >
> > Missing enable-method properties on all the secondary CPUs.
> >
> 
> There are two methods (spin-table and psci) to bring up secondary cores, which
> one do you think is better?

Do we have the PSCI support ready in both UEFI and u-boot?  If not, we should use spin-table for now.

Regards,
Leo