On Fri, Apr 4, 2025 at 7:04 AM Stephan Gerhold <stephan.gerhold@xxxxxxxxxx> wrote: > > On Thu, Apr 03, 2025 at 09:59:27PM -0500, Rob Herring (Arm) wrote: > > The "qcom,acc" and "qcom,saw" properties aren't valid with "spin-table" > > enable-method nor are they used on 64-bit kernels, so they can be > > dropped. > > > > The bootloader we currently use on these devices reads these properties > to set up the spin-table, so removing these will break booting secondary > CPU cores. > > The motivation for implementing it that way was that 32-bit vs 64-bit > kernel shouldn't be relevant for the describing the hardware blocks in > the device tree. The code in the bootloader is generic and handles > different SoCs (e.g. msm8916 with 4 cores and msm8939 with 8 cores, the > enable sequences are identical). > > Can we keep this in somehow? To be fair, I'm not sure what property we > could match on to check if these properties are allowed ... Yes, we can keep them. We'll have to allow them with "spin-table" and "psci" I guess. Rob
