qcom,hawi-cdsp-pas was initially grouped as a fallback to qcom,sm8550-cdsp-pas on the assumption of hardware-level compatibility. Bringup revealed that the NSP proxy power domain on Hawi requires a specific RPMH performance level that the sm8550 driver data does not provide, breaking the compatibility contract the fallback string implies. Maili CDSP shares the same NSP power domain topology as Hawi and must fall back to qcom,hawi-cdsp-pas rather than qcom,sm8550-cdsp-pas to inherit the correct cx/mxc/nsp power-domain constraint.
Remove qcom,hawi-cdsp-pas from the sm8550-cdsp-pas fallback items block, add it to the standalone compatible enum, and extend the existing cx/mxc/nsp power-domain constraint to cover it explicitly. Move qcom,maili-cdsp-pas from the sm8550-cdsp-pas fallback to a new hawi-cdsp-pas fallback items block; it is covered by the hawi cx/mxc/nsp constraint implicitly via the contains check. --- Changes in v4: - Dropped 1/3 patches from the series as that seems unnecessary. - Added maili cdsp pas as fallback to hawi cdsp pas. - Added Tested-by tag. - Link to v3: https://lore.kernel.org/lkml/[email protected]/ Changes in v3: - Converted warn into failure in 1/3 as per review. - Added R-b tag. - Link to v2: https://lore.kernel.org/lkml/[email protected]/ Changes in v2: - Propagating the err dev_pm_genpd_set_performance_state() as a separate commit. - Added a binding correction for hawi as it should be standalone one instead of falling back to sm8550. - Add num_proxy_pd_performance_states count field with a WARN_ON in probe to catch mismatch between the array size and the actual proxy PD count at boot time, returning -EINVAL if they diverge. - [v1] https://lore.kernel.org/lkml/[email protected]/ Mukesh Ojha (2): dt-bindings: remoteproc: qcom,sm8550-pas: make qcom,hawi-cdsp-pas standalone remoteproc: qcom_q6v5_pas: add per-PD proxy performance states for Hawi CDSP .../bindings/remoteproc/qcom,sm8550-pas.yaml | 8 +++- drivers/remoteproc/qcom_q6v5_pas.c | 48 ++++++++++++++++++- 2 files changed, 53 insertions(+), 3 deletions(-) -- 2.55.0
