| Message ID | 20230626-topic-bindingsfixups-v1-3-254ae8642e69@linaro.org |
|---|---|
| State | Accepted |
| Commit | 7e1f024ef0d1da456f61d00f01dc3287ede915b3 |
| Headers | show |
| Series | random msm bindings fixes | expand |
On Montag, 26. Juni 2023 22:00:25 CEST Konrad Dybcio wrote: > Add the missing short interrupt. This fixes the schema warning: > > wled@d800: interrupt-names: ['ovp'] is too short > > Fixes: fe508ced49dd ("arm64: dts: qcom: pm6150l: Add wled node") > Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org> > --- > arch/arm64/boot/dts/qcom/pm6150l.dtsi | 5 +++-- > 1 file changed, 3 insertions(+), 2 deletions(-) > > diff --git a/arch/arm64/boot/dts/qcom/pm6150l.dtsi > b/arch/arm64/boot/dts/qcom/pm6150l.dtsi index 6a7fe1e59f15..d13a1ab7c20b > 100644 > --- a/arch/arm64/boot/dts/qcom/pm6150l.dtsi > +++ b/arch/arm64/boot/dts/qcom/pm6150l.dtsi > @@ -121,8 +121,9 @@ pm6150l_flash: led-controller@d300 { > pm6150l_wled: leds@d800 { > compatible = "qcom,pm6150l-wled"; > reg = <0xd800>, <0xd900>; > - interrupts = <0x5 0xd8 0x1 IRQ_TYPE_EDGE_RISING>; > - interrupt-names = "ovp"; > + interrupts = <0x5 0xd8 0x1 IRQ_TYPE_EDGE_RISING>, > + <0x5 0xd8 0x2 IRQ_TYPE_EDGE_RISING>; > + interrupt-names = "ovp", "short"; Are you sure this interrupt really exists? It's not a thing used downstream at least: https://android.googlesource.com/kernel/msm-extra/devicetree/+/refs/heads/android-msm-bramble-4.19-android11-qpr1/qcom/pm6150l.dtsi#293 Regards Luca > label = "backlight"; > > status = "disabled";
diff --git a/arch/arm64/boot/dts/qcom/pm6150l.dtsi b/arch/arm64/boot/dts/qcom/pm6150l.dtsi index 6a7fe1e59f15..d13a1ab7c20b 100644 --- a/arch/arm64/boot/dts/qcom/pm6150l.dtsi +++ b/arch/arm64/boot/dts/qcom/pm6150l.dtsi @@ -121,8 +121,9 @@ pm6150l_flash: led-controller@d300 { pm6150l_wled: leds@d800 { compatible = "qcom,pm6150l-wled"; reg = <0xd800>, <0xd900>; - interrupts = <0x5 0xd8 0x1 IRQ_TYPE_EDGE_RISING>; - interrupt-names = "ovp"; + interrupts = <0x5 0xd8 0x1 IRQ_TYPE_EDGE_RISING>, + <0x5 0xd8 0x2 IRQ_TYPE_EDGE_RISING>; + interrupt-names = "ovp", "short"; label = "backlight"; status = "disabled";
Add the missing short interrupt. This fixes the schema warning: wled@d800: interrupt-names: ['ovp'] is too short Fixes: fe508ced49dd ("arm64: dts: qcom: pm6150l: Add wled node") Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org> --- arch/arm64/boot/dts/qcom/pm6150l.dtsi | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-)