[PATCH v3 00/11] mailbox/arm64/ qcom: rework compatibles for fallback

Krzysztof Kozlowski posted 11 patches 2 years, 5 months ago
.../mailbox/qcom,apcs-kpss-global.yaml        | 58 ++++++++++---------
arch/arm64/boot/dts/qcom/ipq8074.dtsi         |  3 +-
arch/arm64/boot/dts/qcom/msm8976.dtsi         |  3 +-
arch/arm64/boot/dts/qcom/msm8998.dtsi         |  3 +-
arch/arm64/boot/dts/qcom/qcs404.dtsi          |  3 +-
arch/arm64/boot/dts/qcom/sc7180.dtsi          |  3 +-
arch/arm64/boot/dts/qcom/sdm630.dtsi          |  3 +-
arch/arm64/boot/dts/qcom/sm6115.dtsi          |  3 +-
arch/arm64/boot/dts/qcom/sm6125.dtsi          |  3 +-
arch/arm64/boot/dts/qcom/sm8150.dtsi          |  3 +-
drivers/mailbox/qcom-apcs-ipc-mailbox.c       | 11 ++--
11 files changed, 55 insertions(+), 41 deletions(-)
[PATCH v3 00/11] mailbox/arm64/ qcom: rework compatibles for fallback
Posted by Krzysztof Kozlowski 2 years, 5 months ago
Hi,

Changes since v2
================
1. Split fixes to separate patchset which is now dependency:
   https://lore.kernel.org/linux-arm-msm/20230322173559.809805-1-krzysztof.kozlowski@linaro.org/T/#t
2. Add Ack
3. No other changes, as discussion with Dmitry did not reach conclusion on incompatibility.

Changes since v1
================
1. Rebase
2. Make msm8994 fallback for several variants, not msm8953, because the latter
   actually might take some clocks.
3. Two new patches for SDX55.
4. Minor corrections in bindings style.
v1: https://lore.kernel.org/all/20230202161856.385825-1-krzysztof.kozlowski@linaro.org/

Description
===========

If entire approach is accepted (and correct), there are no dependencies and
patches can be picked independently.  Although the best in the same cycle, so
there will be no new `dtbs_check` warnings.

Best regards,
Krzysztof

Krzysztof Kozlowski (11):
  dt-bindings: mailbox: qcom,apcs-kpss-global: use fallbacks
  mailbox: qcom-apcs-ipc: do not grow the of_device_id
  arm64: dts: qcom: ipq8074: add compatible fallback to mailbox
  arm64: dts: qcom: msm8976: add compatible fallback to mailbox
  arm64: dts: qcom: msm8998: add compatible fallback to mailbox
  arm64: dts: qcom: sdm630: add compatible fallback to mailbox
  arm64: dts: qcom: sm6115: add compatible fallback to mailbox
  arm64: dts: qcom: sm6125: add compatible fallback to mailbox
  arm64: dts: qcom: qcs404: add compatible fallback to mailbox
  arm64: dts: qcom: sc7180: add compatible fallback to mailbox
  arm64: dts: qcom: sm8150: add compatible fallback to mailbox

 .../mailbox/qcom,apcs-kpss-global.yaml        | 58 ++++++++++---------
 arch/arm64/boot/dts/qcom/ipq8074.dtsi         |  3 +-
 arch/arm64/boot/dts/qcom/msm8976.dtsi         |  3 +-
 arch/arm64/boot/dts/qcom/msm8998.dtsi         |  3 +-
 arch/arm64/boot/dts/qcom/qcs404.dtsi          |  3 +-
 arch/arm64/boot/dts/qcom/sc7180.dtsi          |  3 +-
 arch/arm64/boot/dts/qcom/sdm630.dtsi          |  3 +-
 arch/arm64/boot/dts/qcom/sm6115.dtsi          |  3 +-
 arch/arm64/boot/dts/qcom/sm6125.dtsi          |  3 +-
 arch/arm64/boot/dts/qcom/sm8150.dtsi          |  3 +-
 drivers/mailbox/qcom-apcs-ipc-mailbox.c       | 11 ++--
 11 files changed, 55 insertions(+), 41 deletions(-)

-- 
2.34.1
Re: [PATCH v3 00/11] mailbox/arm64/ qcom: rework compatibles for fallback
Posted by Konrad Dybcio 2 years, 1 month ago
On 22.03.2023 18:41, Krzysztof Kozlowski wrote:
> Hi,
> 
> Changes since v2
> ================
> 1. Split fixes to separate patchset which is now dependency:
>    https://lore.kernel.org/linux-arm-msm/20230322173559.809805-1-krzysztof.kozlowski@linaro.org/T/#t
> 2. Add Ack
> 3. No other changes, as discussion with Dmitry did not reach conclusion on incompatibility.
> 
> Changes since v1
> ================
> 1. Rebase
> 2. Make msm8994 fallback for several variants, not msm8953, because the latter
>    actually might take some clocks.
> 3. Two new patches for SDX55.
> 4. Minor corrections in bindings style.
> v1: https://lore.kernel.org/all/20230202161856.385825-1-krzysztof.kozlowski@linaro.org/
> 
> Description
> ===========
> 
> If entire approach is accepted (and correct), there are no dependencies and
> patches can be picked independently.  Although the best in the same cycle, so
> there will be no new `dtbs_check` warnings.
> 
> Best regards,
> Krzysztof
Looks like this was only partially merged, resulting in schema warnings

Konrad
Re: [PATCH v3 00/11] mailbox/arm64/ qcom: rework compatibles for fallback
Posted by Krzysztof Kozlowski 2 years, 1 month ago
On 05/08/2023 14:21, Konrad Dybcio wrote:
> On 22.03.2023 18:41, Krzysztof Kozlowski wrote:
>> Hi,
>>
>> Changes since v2
>> ================
>> 1. Split fixes to separate patchset which is now dependency:
>>    https://lore.kernel.org/linux-arm-msm/20230322173559.809805-1-krzysztof.kozlowski@linaro.org/T/#t
>> 2. Add Ack
>> 3. No other changes, as discussion with Dmitry did not reach conclusion on incompatibility.
>>
>> Changes since v1
>> ================
>> 1. Rebase
>> 2. Make msm8994 fallback for several variants, not msm8953, because the latter
>>    actually might take some clocks.
>> 3. Two new patches for SDX55.
>> 4. Minor corrections in bindings style.
>> v1: https://lore.kernel.org/all/20230202161856.385825-1-krzysztof.kozlowski@linaro.org/
>>
>> Description
>> ===========
>>
>> If entire approach is accepted (and correct), there are no dependencies and
>> patches can be picked independently.  Although the best in the same cycle, so
>> there will be no new `dtbs_check` warnings.
>>
>> Best regards,
>> Krzysztof
> Looks like this was only partially merged, resulting in schema warnings

There was discussion/disagreement about the bindings. DTS was applied,
thus it's partially limbo state...

Best regards,
Krzysztof
Re: [PATCH v3 00/11] mailbox/arm64/ qcom: rework compatibles for fallback
Posted by Krzysztof Kozlowski 2 years ago
On 05/08/2023 20:46, Krzysztof Kozlowski wrote:
> On 05/08/2023 14:21, Konrad Dybcio wrote:
>> On 22.03.2023 18:41, Krzysztof Kozlowski wrote:
>>> Hi,
>>>
>>> Changes since v2
>>> ================
>>> 1. Split fixes to separate patchset which is now dependency:
>>>    https://lore.kernel.org/linux-arm-msm/20230322173559.809805-1-krzysztof.kozlowski@linaro.org/T/#t
>>> 2. Add Ack
>>> 3. No other changes, as discussion with Dmitry did not reach conclusion on incompatibility.
>>>
>>> Changes since v1
>>> ================
>>> 1. Rebase
>>> 2. Make msm8994 fallback for several variants, not msm8953, because the latter
>>>    actually might take some clocks.
>>> 3. Two new patches for SDX55.
>>> 4. Minor corrections in bindings style.
>>> v1: https://lore.kernel.org/all/20230202161856.385825-1-krzysztof.kozlowski@linaro.org/
>>>
>>> Description
>>> ===========
>>>
>>> If entire approach is accepted (and correct), there are no dependencies and
>>> patches can be picked independently.  Although the best in the same cycle, so
>>> there will be no new `dtbs_check` warnings.
>>>
>>> Best regards,
>>> Krzysztof
>> Looks like this was only partially merged, resulting in schema warnings
> 
> There was discussion/disagreement about the bindings. DTS was applied,
> thus it's partially limbo state...

Anyone has any ideas what to do with this stuff? DTS was applied, even
though bindings were discussed. Since some time we have tons of warnings
because of this.

Maybe let's just go with my original patchset?

Best regards,
Krzysztof
Re: (subset) [PATCH v3 00/11] mailbox/arm64/ qcom: rework compatibles for fallback
Posted by Bjorn Andersson 2 years, 5 months ago
On Wed, 22 Mar 2023 18:41:37 +0100, Krzysztof Kozlowski wrote:
> Changes since v2
> ================
> 1. Split fixes to separate patchset which is now dependency:
>    https://lore.kernel.org/linux-arm-msm/20230322173559.809805-1-krzysztof.kozlowski@linaro.org/T/#t
> 2. Add Ack
> 3. No other changes, as discussion with Dmitry did not reach conclusion on incompatibility.
> 
> [...]

Applied, thanks!

[03/11] arm64: dts: qcom: ipq8074: add compatible fallback to mailbox
        commit: d93bd4630ce163f3761aedc0b342b072bee6db6b
[04/11] arm64: dts: qcom: msm8976: add compatible fallback to mailbox
        commit: 81cd4883b1a2d7c612df9f526598e6a841ab3881
[05/11] arm64: dts: qcom: msm8998: add compatible fallback to mailbox
        commit: 112f33b369e5e38119910a3365537bcc5d3d4135
[06/11] arm64: dts: qcom: sdm630: add compatible fallback to mailbox
        commit: 2d034324fdba6555fb642b127167ae5ba0e30158
[07/11] arm64: dts: qcom: sm6115: add compatible fallback to mailbox
        commit: fb6198bb4980cd4a7a89affe6bef4e6e81e97316
[08/11] arm64: dts: qcom: sm6125: add compatible fallback to mailbox
        commit: 61799f9d573fe31222b0224acbb545fae259366a
[09/11] arm64: dts: qcom: qcs404: add compatible fallback to mailbox
        commit: 4c90ceae6ec093ee9a73a48c8dacaa6d1404770b
[10/11] arm64: dts: qcom: sc7180: add compatible fallback to mailbox
        commit: c1393bdf181143b21cb1ad8dbad6f3840b171855
[11/11] arm64: dts: qcom: sm8150: add compatible fallback to mailbox
        commit: 9b2e284a5dc2d23a7102ff87fac3aa9d6b88118e

Best regards,
-- 
Bjorn Andersson <andersson@kernel.org>