On 2022-03-03 01:54:06, Dmitry Baryshkov wrote: > The number of interrupt cells for the mdss interrupt controller is 1, > meaning there should only be one cell for the interrupt number, not two. > Drop the second cell containing (unused) irq flags. > > Reviewed-by: Stephen Boyd <[email protected]> > Fixes: 12d540375736 ("arm64: dts: qcom: msm8996: Add DSI0 nodes") > Fixes: 3a4547c1fc2f ("arm64: qcom: msm8996.dtsi: Add Display nodes") > Signed-off-by: Dmitry Baryshkov <[email protected]>
Thanks for adding the Fixes: tag. Reviewed-by: Marijn Suijten <[email protected]> > --- > arch/arm64/boot/dts/qcom/msm8996.dtsi | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/arch/arm64/boot/dts/qcom/msm8996.dtsi > b/arch/arm64/boot/dts/qcom/msm8996.dtsi > index f0f81c23c16f..0597d865a4a6 100644 > --- a/arch/arm64/boot/dts/qcom/msm8996.dtsi > +++ b/arch/arm64/boot/dts/qcom/msm8996.dtsi > @@ -788,7 +788,7 @@ mdp: mdp@901000 { > reg-names = "mdp_phys"; > > interrupt-parent = <&mdss>; > - interrupts = <0 IRQ_TYPE_LEVEL_HIGH>; > + interrupts = <0>; > > clocks = <&mmcc MDSS_AHB_CLK>, > <&mmcc MDSS_AXI_CLK>, > @@ -834,7 +834,7 @@ dsi0: dsi@994000 { > reg-names = "dsi_ctrl"; > > interrupt-parent = <&mdss>; > - interrupts = <4 IRQ_TYPE_LEVEL_HIGH>; > + interrupts = <4>; > > clocks = <&mmcc MDSS_MDP_CLK>, > <&mmcc MDSS_BYTE0_CLK>, > @@ -904,7 +904,7 @@ hdmi: hdmi-tx@9a0000 { > "hdcp_physical"; > > interrupt-parent = <&mdss>; > - interrupts = <8 IRQ_TYPE_LEVEL_HIGH>; > + interrupts = <8>; > > clocks = <&mmcc MDSS_MDP_CLK>, > <&mmcc MDSS_AHB_CLK>, > -- > 2.34.1 >
