mbox series

[v2,0/6] media: dt-bindings: media: camss: Fix interrupt types

Message ID 20240923072827.3772504-1-vladimir.zapolskiy@linaro.org (mailing list archive)
Headers show
Series media: dt-bindings: media: camss: Fix interrupt types | expand

Message

Vladimir Zapolskiy Sept. 23, 2024, 7:28 a.m. UTC
It was discovered that on a few Qualcomm platforms types of interrupts
do not match both downstream code and a type requested by the CAMSS driver.

The mismatched interrupt type between firmware and the correspondent CAMSS
driver leads to known problems, which were discussed previously:

  https://lore.kernel.org/lkml/20220530080842.37024-4-manivannan.sadhasivam@linaro.org/

Here the situation is right the same, namely a repeated bind of camss device
is not possible due to a wrongly specified interrupt type, and it may lead
to an issue in runtime manifested like this:

  irq: type mismatch, failed to map hwirq-509 for interrupt-controller@17a00000!

Changes from v1 to v2:
* added gained Acked-by, Tested-by and Reviewed-by tags,
* per patch review requests from Krzysztof deduplicated "media:" from subjects.

Link to v1 of the changeset:

  https://lore.kernel.org/all/20240905164142.3475873-1-vladimir.zapolskiy@linaro.org/

Vladimir Zapolskiy (6):
  dt-bindings: media: qcom,sc8280xp-camss: Fix interrupt types
  dt-bindings: media: qcom,sdm845-camss: Fix interrupt types
  dt-bindings: media: qcom,sm8250-camss: Fix interrupt types
  arm64: dts: qcom: sc8280xp: Fix interrupt type of camss interrupts
  arm64: dts: qcom: sdm845: Fix interrupt types of camss interrupts
  arm64: dts: qcom: sm8250: Fix interrupt types of camss interrupts

 .../bindings/media/qcom,sc8280xp-camss.yaml   | 40 +++++++++----------
 .../bindings/media/qcom,sdm845-camss.yaml     | 20 +++++-----
 .../bindings/media/qcom,sm8250-camss.yaml     | 28 ++++++-------
 arch/arm64/boot/dts/qcom/sc8280xp.dtsi        | 40 +++++++++----------
 arch/arm64/boot/dts/qcom/sdm845.dtsi          | 20 +++++-----
 arch/arm64/boot/dts/qcom/sm8250.dtsi          | 28 ++++++-------
 6 files changed, 88 insertions(+), 88 deletions(-)

Comments

Bryan O'Donoghue Sept. 23, 2024, 12:43 p.m. UTC | #1
On 23/09/2024 08:28, Vladimir Zapolskiy wrote:
> It was discovered that on a few Qualcomm platforms types of interrupts
> do not match both downstream code and a type requested by the CAMSS driver.
> 
> The mismatched interrupt type between firmware and the correspondent CAMSS
> driver leads to known problems, which were discussed previously:
> 
>    https://lore.kernel.org/lkml/20220530080842.37024-4-manivannan.sadhasivam@linaro.org/
> 
> Here the situation is right the same, namely a repeated bind of camss device
> is not possible due to a wrongly specified interrupt type, and it may lead
> to an issue in runtime manifested like this:
> 
>    irq: type mismatch, failed to map hwirq-509 for interrupt-controller@17a00000!
> 
> Changes from v1 to v2:
> * added gained Acked-by, Tested-by and Reviewed-by tags,
> * per patch review requests from Krzysztof deduplicated "media:" from subjects.
> 
> Link to v1 of the changeset:
> 
>    https://lore.kernel.org/all/20240905164142.3475873-1-vladimir.zapolskiy@linaro.org/
> 
> Vladimir Zapolskiy (6):
>    dt-bindings: media: qcom,sc8280xp-camss: Fix interrupt types
>    dt-bindings: media: qcom,sdm845-camss: Fix interrupt types
>    dt-bindings: media: qcom,sm8250-camss: Fix interrupt types
>    arm64: dts: qcom: sc8280xp: Fix interrupt type of camss interrupts
>    arm64: dts: qcom: sdm845: Fix interrupt types of camss interrupts
>    arm64: dts: qcom: sm8250: Fix interrupt types of camss interrupts
> 
>   .../bindings/media/qcom,sc8280xp-camss.yaml   | 40 +++++++++----------
>   .../bindings/media/qcom,sdm845-camss.yaml     | 20 +++++-----
>   .../bindings/media/qcom,sm8250-camss.yaml     | 28 ++++++-------
>   arch/arm64/boot/dts/qcom/sc8280xp.dtsi        | 40 +++++++++----------
>   arch/arm64/boot/dts/qcom/sdm845.dtsi          | 20 +++++-----
>   arch/arm64/boot/dts/qcom/sm8250.dtsi          | 28 ++++++-------
>   6 files changed, 88 insertions(+), 88 deletions(-)
> 

Reviewed-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>

For the series.