diff mbox series

[2/3] arm64: dts: ls208xa: use defines for timer interrupts

Message ID 20240605153020.104717-2-krzysztof.kozlowski@linaro.org (mailing list archive)
State In Next, archived
Headers show
Series [1/3] arm64: dts: freescale: use defines for interrupts | expand

Commit Message

Krzysztof Kozlowski June 5, 2024, 3:30 p.m. UTC
Replace hard-coded interrupt parts (GIC, flags) with standard defines
for readability.  No changes in resulting DTBs.  The comment was saying
interrupt was active low, but the actual used value was active high, so
assume that the code, not the comment, is correct.

Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
 arch/arm64/boot/dts/freescale/fsl-ls208xa.dtsi | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

Comments

Peng Fan June 6, 2024, 1:35 a.m. UTC | #1
> Subject: [PATCH 2/3] arm64: dts: ls208xa: use defines for timer interrupts
> 
> Replace hard-coded interrupt parts (GIC, flags) with standard defines for
> readability.  No changes in resulting DTBs.  The comment was saying interrupt
> was active low, but the actual used value was active high, so assume that the
> code, not the comment, is correct.
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>

Reviewed-by: Peng Fan <peng.fan@nxp.com>
> ---
>  arch/arm64/boot/dts/freescale/fsl-ls208xa.dtsi | 8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/arch/arm64/boot/dts/freescale/fsl-ls208xa.dtsi
> b/arch/arm64/boot/dts/freescale/fsl-ls208xa.dtsi
> index cc305e629bdc..040a48c88fab 100644
> --- a/arch/arm64/boot/dts/freescale/fsl-ls208xa.dtsi
> +++ b/arch/arm64/boot/dts/freescale/fsl-ls208xa.dtsi
> @@ -241,10 +241,10 @@ map0 {
> 
>  	timer: timer {
>  		compatible = "arm,armv8-timer";
> -		interrupts = <1 13 4>, /* Physical Secure PPI, active-low */
> -			     <1 14 4>, /* Physical Non-Secure PPI, active-low
> */
> -			     <1 11 4>, /* Virtual PPI, active-low */
> -			     <1 10 4>; /* Hypervisor PPI, active-low */
> +		interrupts = <GIC_PPI 13 IRQ_TYPE_LEVEL_HIGH>, /* Physical
> Secure PPI */
> +			     <GIC_PPI 14 IRQ_TYPE_LEVEL_HIGH>, /* Physical
> Non-Secure PPI */
> +			     <GIC_PPI 11 IRQ_TYPE_LEVEL_HIGH>, /* Virtual
> PPI */
> +			     <GIC_PPI 10 IRQ_TYPE_LEVEL_HIGH>; /*
> Hypervisor PPI */
>  	};
> 
>  	psci {
> --
> 2.43.0
>
diff mbox series

Patch

diff --git a/arch/arm64/boot/dts/freescale/fsl-ls208xa.dtsi b/arch/arm64/boot/dts/freescale/fsl-ls208xa.dtsi
index cc305e629bdc..040a48c88fab 100644
--- a/arch/arm64/boot/dts/freescale/fsl-ls208xa.dtsi
+++ b/arch/arm64/boot/dts/freescale/fsl-ls208xa.dtsi
@@ -241,10 +241,10 @@  map0 {
 
 	timer: timer {
 		compatible = "arm,armv8-timer";
-		interrupts = <1 13 4>, /* Physical Secure PPI, active-low */
-			     <1 14 4>, /* Physical Non-Secure PPI, active-low */
-			     <1 11 4>, /* Virtual PPI, active-low */
-			     <1 10 4>; /* Hypervisor PPI, active-low */
+		interrupts = <GIC_PPI 13 IRQ_TYPE_LEVEL_HIGH>, /* Physical Secure PPI */
+			     <GIC_PPI 14 IRQ_TYPE_LEVEL_HIGH>, /* Physical Non-Secure PPI */
+			     <GIC_PPI 11 IRQ_TYPE_LEVEL_HIGH>, /* Virtual PPI */
+			     <GIC_PPI 10 IRQ_TYPE_LEVEL_HIGH>; /* Hypervisor PPI */
 	};
 
 	psci {