On Fri, Sep 20, 2024 at 11:02 PM Jeff Johnson <quic_jjohnson@xxxxxxxxxxx> wrote: > > > > > Let me give you an analogy: we don't really need to have always-on, fixed > > regulators in DTS. The drivers don't really need them. We do it for > > completeness of the HW description. > > Again, since I'm a DT n00b: > Just to make sure I understand, you are saying that with this change any > existing .dts/.dtb files will still work with an updated driver, so the new > properties are not required to be populated on existing devices. > There are no driver updates. No functional change. > However a new driver with support for these properties will utilize them when > they are present, and the current ath11k .dts files will need to be updated to > include these properties for pci17cb,1103, i.e. the following needs updating: > arch/arm64/boot/dts/qcom/sc8280xp-lenovo-thinkpad-x13s.dts What new driver? The dts is being updated in a separate series[1]. It makes this platform use the new power sequencing subsystem for wcn6855. All other changes required to make it work are already upstream. There's no change to ath11k. Bart [1] https://lore.kernel.org/all/20240905122023.47251-1-brgl@xxxxxxxx/