mbox series

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

Message ID 20230322174148.810938-1-krzysztof.kozlowski@linaro.org (mailing list archive)
Headers show
Series mailbox/arm64/ qcom: rework compatibles for fallback | expand

Message

Krzysztof Kozlowski March 22, 2023, 5:41 p.m. UTC 
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(-)

Comments

Bjorn Andersson April 7, 2023, 6:51 p.m. UTC | #1 
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,
Konrad Dybcio Aug. 5, 2023, 12:21 p.m. UTC | #2 
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
Krzysztof Kozlowski Aug. 5, 2023, 6:46 p.m. UTC | #3 
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
Krzysztof Kozlowski Aug. 27, 2023, 12:17 p.m. UTC | #4 
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