> > > > 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@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 -- To unsubscribe from this list: send the line "unsubscribe linux-i2c" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html