[1/2] arm64: dts: qcom: sa8450p-pmics: add missing interrupt include

Message ID 20230111082331.20641-2-johan+linaro@kernel.org
State New
Headers
Series arm64: dts: qcom: sa8450p-pmics: rename pmic labels |

Commit Message

Johan Hovold Jan. 11, 2023, 8:23 a.m. UTC
  Add the missing interrupt-controller include which is needed by the RTC
node.

Signed-off-by: Johan Hovold <johan+linaro@kernel.org>
---
 arch/arm64/boot/dts/qcom/sa8540p-pmics.dtsi | 1 +
 1 file changed, 1 insertion(+)
  

Comments

Brian Masney Jan. 11, 2023, 10:49 a.m. UTC | #1
On Wed, Jan 11, 2023 at 09:23:30AM +0100, Johan Hovold wrote:
> Add the missing interrupt-controller include which is needed by the RTC
> node.
> 
> Signed-off-by: Johan Hovold <johan+linaro@kernel.org>

Reviewed-by: Brian Masney <bmasney@redhat.com>
  
Eric Chanudet Jan. 11, 2023, 3:06 p.m. UTC | #2
On Wed, Jan 11, 2023 at 09:23:30AM +0100, Johan Hovold wrote:
> Add the missing interrupt-controller include which is needed by the RTC
> node.
> 
> Signed-off-by: Johan Hovold <johan+linaro@kernel.org>

Reviewed-by: Eric Chanudet <echanude@redhat.com>

Nit: sa8450p-pmics -> sa8540p-pmics in the subject. It was mistyped in
the commit renaming the file unfortunately.
  
Johan Hovold Jan. 11, 2023, 3:53 p.m. UTC | #3
On Wed, Jan 11, 2023 at 10:06:19AM -0500, Eric Chanudet wrote:
> On Wed, Jan 11, 2023 at 09:23:30AM +0100, Johan Hovold wrote:
> > Add the missing interrupt-controller include which is needed by the RTC
> > node.
> > 
> > Signed-off-by: Johan Hovold <johan+linaro@kernel.org>
> 
> Reviewed-by: Eric Chanudet <echanude@redhat.com>
> 
> Nit: sa8450p-pmics -> sa8540p-pmics in the subject. It was mistyped in
> the commit renaming the file unfortunately.

Ah, good catch. I'll resend with that fixed. Thanks.

Johan
  

Patch

diff --git a/arch/arm64/boot/dts/qcom/sa8540p-pmics.dtsi b/arch/arm64/boot/dts/qcom/sa8540p-pmics.dtsi
index c9b8da43b237..8c393f0bd6a8 100644
--- a/arch/arm64/boot/dts/qcom/sa8540p-pmics.dtsi
+++ b/arch/arm64/boot/dts/qcom/sa8540p-pmics.dtsi
@@ -4,6 +4,7 @@ 
  * Copyright (c) 2022, Linaro Limited
  */
 
+#include <dt-bindings/interrupt-controller/irq.h>
 #include <dt-bindings/spmi/spmi.h>
 
 &spmi_bus {