diff mbox series

[v2] arm64: dts: qcom: timer should use only 32-bit size

Message ID 20220515221758.88723-1-david@ixit.cz
State Superseded
Headers show
Series [v2] arm64: dts: qcom: timer should use only 32-bit size | expand

Commit Message

David Heidelberg May 15, 2022, 10:17 p.m. UTC
There's no reason the timer needs > 32-bits of size.
Since we using 32-bit size, we need to define ranges properly.

Fixes warnings as:
```
arch/arm64/boot/dts/qcom/sdm845-oneplus-fajita.dt.yaml: timer@17c90000: #size-cells:0:0: 1 was expected
        From schema: Documentation/devicetree/bindings/timer/arm,arch_timer_mmio.yaml
```

Signed-off-by: David Heidelberg <david@ixit.cz>

---
Replaces "dt-bindings: timer: add #size-cells 2 for ARM"
v2: fix accidental change of value 0x0 to 0

 arch/arm64/boot/dts/qcom/ipq6018.dtsi | 20 ++++++++++----------
 arch/arm64/boot/dts/qcom/sc7180.dtsi  | 20 ++++++++++----------
 arch/arm64/boot/dts/qcom/sc7280.dtsi  | 20 ++++++++++----------
 arch/arm64/boot/dts/qcom/sdm845.dtsi  | 20 ++++++++++----------
 arch/arm64/boot/dts/qcom/sm6350.dtsi  | 20 ++++++++++----------
 arch/arm64/boot/dts/qcom/sm8150.dtsi  | 20 ++++++++++----------
 arch/arm64/boot/dts/qcom/sm8250.dtsi  | 20 ++++++++++----------
 arch/arm64/boot/dts/qcom/sm8350.dtsi  | 20 ++++++++++----------
 arch/arm64/boot/dts/qcom/sm8450.dtsi  | 20 ++++++++++----------
 9 files changed, 90 insertions(+), 90 deletions(-)

Comments

Bjorn Andersson June 26, 2022, 3:12 a.m. UTC | #1
On Sun 15 May 17:17 CDT 2022, David Heidelberg wrote:

> There's no reason the timer needs > 32-bits of size.
> Since we using 32-bit size, we need to define ranges properly.
> 
> Fixes warnings as:
> ```
> arch/arm64/boot/dts/qcom/sdm845-oneplus-fajita.dt.yaml: timer@17c90000: #size-cells:0:0: 1 was expected
>         From schema: Documentation/devicetree/bindings/timer/arm,arch_timer_mmio.yaml
> ```
> 
> Signed-off-by: David Heidelberg <david@ixit.cz>
> 
> ---
> Replaces "dt-bindings: timer: add #size-cells 2 for ARM"

I'm not able to find this patch, but I'm guessing that you tried to make
it possible to allow #size-cells to be 2 in the binding?

> v2: fix accidental change of value 0x0 to 0
> 
>  arch/arm64/boot/dts/qcom/ipq6018.dtsi | 20 ++++++++++----------
>  arch/arm64/boot/dts/qcom/sc7180.dtsi  | 20 ++++++++++----------
>  arch/arm64/boot/dts/qcom/sc7280.dtsi  | 20 ++++++++++----------
>  arch/arm64/boot/dts/qcom/sdm845.dtsi  | 20 ++++++++++----------
>  arch/arm64/boot/dts/qcom/sm6350.dtsi  | 20 ++++++++++----------
>  arch/arm64/boot/dts/qcom/sm8150.dtsi  | 20 ++++++++++----------
>  arch/arm64/boot/dts/qcom/sm8250.dtsi  | 20 ++++++++++----------
>  arch/arm64/boot/dts/qcom/sm8350.dtsi  | 20 ++++++++++----------
>  arch/arm64/boot/dts/qcom/sm8450.dtsi  | 20 ++++++++++----------
>  9 files changed, 90 insertions(+), 90 deletions(-)
> 
> diff --git a/arch/arm64/boot/dts/qcom/ipq6018.dtsi b/arch/arm64/boot/dts/qcom/ipq6018.dtsi
> index c89499e366d3..8638231dbd2f 100644
> --- a/arch/arm64/boot/dts/qcom/ipq6018.dtsi
> +++ b/arch/arm64/boot/dts/qcom/ipq6018.dtsi
> @@ -526,8 +526,8 @@ timer {
>  
>  		timer@b120000 {
>  			#address-cells = <2>;
> -			#size-cells = <2>;
> -			ranges;
> +			#size-cells = <1>;

While perfectly valid, I don't like the asymmetry of #address-cells = 2
and #size-cells = 1.

If the binding maintainers don't approve of allowing #size-cells = 2,
can you please make #address-cells = 1 as well?

Regards,
Bjorn

> +			ranges = <0 0 0 0 0x10000000>;
>  			compatible = "arm,armv7-timer-mem";
>  			reg = <0x0 0x0b120000 0x0 0x1000>;
>  
> @@ -535,49 +535,49 @@ frame@b120000 {
>  				frame-number = <0>;
>  				interrupts = <GIC_SPI 8 IRQ_TYPE_LEVEL_HIGH>,
>  					     <GIC_SPI 7 IRQ_TYPE_LEVEL_HIGH>;
> -				reg = <0x0 0x0b121000 0x0 0x1000>,
> -				      <0x0 0x0b122000 0x0 0x1000>;
> +				reg = <0x0 0x0b121000 0x1000>,
> +				      <0x0 0x0b122000 0x1000>;
>  			};
>  
>  			frame@b123000 {
>  				frame-number = <1>;
>  				interrupts = <GIC_SPI 9 IRQ_TYPE_LEVEL_HIGH>;
> -				reg = <0x0 0xb123000 0x0 0x1000>;
> +				reg = <0x0 0x0b123000 0x1000>;
>  				status = "disabled";
>  			};
>  
>  			frame@b124000 {
>  				frame-number = <2>;
>  				interrupts = <GIC_SPI 10 IRQ_TYPE_LEVEL_HIGH>;
> -				reg = <0x0 0x0b124000 0x0 0x1000>;
> +				reg = <0x0 0x0b124000 0x1000>;
>  				status = "disabled";
>  			};
>  
>  			frame@b125000 {
>  				frame-number = <3>;
>  				interrupts = <GIC_SPI 11 IRQ_TYPE_LEVEL_HIGH>;
> -				reg = <0x0 0x0b125000 0x0 0x1000>;
> +				reg = <0x0 0x0b125000 0x1000>;
>  				status = "disabled";
>  			};
>  
>  			frame@b126000 {
>  				frame-number = <4>;
>  				interrupts = <GIC_SPI 12 IRQ_TYPE_LEVEL_HIGH>;
> -				reg = <0x0 0x0b126000 0x0 0x1000>;
> +				reg = <0x0 0x0b126000 0x1000>;
>  				status = "disabled";
>  			};
>  
>  			frame@b127000 {
>  				frame-number = <5>;
>  				interrupts = <GIC_SPI 13 IRQ_TYPE_LEVEL_HIGH>;
> -				reg = <0x0 0x0b127000 0x0 0x1000>;
> +				reg = <0x0 0x0b127000 0x1000>;
>  				status = "disabled";
>  			};
>  
>  			frame@b128000 {
>  				frame-number = <6>;
>  				interrupts = <GIC_SPI 14 IRQ_TYPE_LEVEL_HIGH>;
> -				reg = <0x0 0x0b128000 0x0 0x1000>;
> +				reg = <0x0 0x0b128000 0x1000>;
>  				status = "disabled";
>  			};
>  		};
> diff --git a/arch/arm64/boot/dts/qcom/sc7180.dtsi b/arch/arm64/boot/dts/qcom/sc7180.dtsi
> index 5dcaac23a138..4b7be66e04df 100644
> --- a/arch/arm64/boot/dts/qcom/sc7180.dtsi
> +++ b/arch/arm64/boot/dts/qcom/sc7180.dtsi
> @@ -3385,8 +3385,8 @@ watchdog@17c10000 {
>  
>  		timer@17c20000{
>  			#address-cells = <2>;
> -			#size-cells = <2>;
> -			ranges;
> +			#size-cells = <1>;
> +			ranges = <0 0 0 0 0x20000000>;
>  			compatible = "arm,armv7-timer-mem";
>  			reg = <0 0x17c20000 0 0x1000>;
>  
> @@ -3394,49 +3394,49 @@ frame@17c21000 {
>  				frame-number = <0>;
>  				interrupts = <GIC_SPI 8 IRQ_TYPE_LEVEL_HIGH>,
>  					     <GIC_SPI 6 IRQ_TYPE_LEVEL_HIGH>;
> -				reg = <0 0x17c21000 0 0x1000>,
> -				      <0 0x17c22000 0 0x1000>;
> +				reg = <0 0x17c21000 0x1000>,
> +				      <0 0x17c22000 0x1000>;
>  			};
>  
>  			frame@17c23000 {
>  				frame-number = <1>;
>  				interrupts = <GIC_SPI 9 IRQ_TYPE_LEVEL_HIGH>;
> -				reg = <0 0x17c23000 0 0x1000>;
> +				reg = <0 0x17c23000 0x1000>;
>  				status = "disabled";
>  			};
>  
>  			frame@17c25000 {
>  				frame-number = <2>;
>  				interrupts = <GIC_SPI 10 IRQ_TYPE_LEVEL_HIGH>;
> -				reg = <0 0x17c25000 0 0x1000>;
> +				reg = <0 0x17c25000 0x1000>;
>  				status = "disabled";
>  			};
>  
>  			frame@17c27000 {
>  				frame-number = <3>;
>  				interrupts = <GIC_SPI 11 IRQ_TYPE_LEVEL_HIGH>;
> -				reg = <0 0x17c27000 0 0x1000>;
> +				reg = <0 0x17c27000 0x1000>;
>  				status = "disabled";
>  			};
>  
>  			frame@17c29000 {
>  				frame-number = <4>;
>  				interrupts = <GIC_SPI 12 IRQ_TYPE_LEVEL_HIGH>;
> -				reg = <0 0x17c29000 0 0x1000>;
> +				reg = <0 0x17c29000 0x1000>;
>  				status = "disabled";
>  			};
>  
>  			frame@17c2b000 {
>  				frame-number = <5>;
>  				interrupts = <GIC_SPI 13 IRQ_TYPE_LEVEL_HIGH>;
> -				reg = <0 0x17c2b000 0 0x1000>;
> +				reg = <0 0x17c2b000 0x1000>;
>  				status = "disabled";
>  			};
>  
>  			frame@17c2d000 {
>  				frame-number = <6>;
>  				interrupts = <GIC_SPI 14 IRQ_TYPE_LEVEL_HIGH>;
> -				reg = <0 0x17c2d000 0 0x1000>;
> +				reg = <0 0x17c2d000 0x1000>;
>  				status = "disabled";
>  			};
>  		};
> diff --git a/arch/arm64/boot/dts/qcom/sc7280.dtsi b/arch/arm64/boot/dts/qcom/sc7280.dtsi
> index e66fc67de206..1b81cd1e84e9 100644
> --- a/arch/arm64/boot/dts/qcom/sc7280.dtsi
> +++ b/arch/arm64/boot/dts/qcom/sc7280.dtsi
> @@ -4772,8 +4772,8 @@ watchdog@17c10000 {
>  
>  		timer@17c20000 {
>  			#address-cells = <2>;
> -			#size-cells = <2>;
> -			ranges;
> +			#size-cells = <1>;
> +			ranges = <0 0 0 0 0x20000000>;
>  			compatible = "arm,armv7-timer-mem";
>  			reg = <0 0x17c20000 0 0x1000>;
>  
> @@ -4781,49 +4781,49 @@ frame@17c21000 {
>  				frame-number = <0>;
>  				interrupts = <GIC_SPI 8 IRQ_TYPE_LEVEL_HIGH>,
>  					     <GIC_SPI 6 IRQ_TYPE_LEVEL_HIGH>;
> -				reg = <0 0x17c21000 0 0x1000>,
> -				      <0 0x17c22000 0 0x1000>;
> +				reg = <0 0x17c21000 0x1000>,
> +				      <0 0x17c22000 0x1000>;
>  			};
>  
>  			frame@17c23000 {
>  				frame-number = <1>;
>  				interrupts = <GIC_SPI 9 IRQ_TYPE_LEVEL_HIGH>;
> -				reg = <0 0x17c23000 0 0x1000>;
> +				reg = <0 0x17c23000 0x1000>;
>  				status = "disabled";
>  			};
>  
>  			frame@17c25000 {
>  				frame-number = <2>;
>  				interrupts = <GIC_SPI 10 IRQ_TYPE_LEVEL_HIGH>;
> -				reg = <0 0x17c25000 0 0x1000>;
> +				reg = <0 0x17c25000 0x1000>;
>  				status = "disabled";
>  			};
>  
>  			frame@17c27000 {
>  				frame-number = <3>;
>  				interrupts = <GIC_SPI 11 IRQ_TYPE_LEVEL_HIGH>;
> -				reg = <0 0x17c27000 0 0x1000>;
> +				reg = <0 0x17c27000 0x1000>;
>  				status = "disabled";
>  			};
>  
>  			frame@17c29000 {
>  				frame-number = <4>;
>  				interrupts = <GIC_SPI 12 IRQ_TYPE_LEVEL_HIGH>;
> -				reg = <0 0x17c29000 0 0x1000>;
> +				reg = <0 0x17c29000 0x1000>;
>  				status = "disabled";
>  			};
>  
>  			frame@17c2b000 {
>  				frame-number = <5>;
>  				interrupts = <GIC_SPI 13 IRQ_TYPE_LEVEL_HIGH>;
> -				reg = <0 0x17c2b000 0 0x1000>;
> +				reg = <0 0x17c2b000 0x1000>;
>  				status = "disabled";
>  			};
>  
>  			frame@17c2d000 {
>  				frame-number = <6>;
>  				interrupts = <GIC_SPI 14 IRQ_TYPE_LEVEL_HIGH>;
> -				reg = <0 0x17c2d000 0 0x1000>;
> +				reg = <0 0x17c2d000 0x1000>;
>  				status = "disabled";
>  			};
>  		};
> diff --git a/arch/arm64/boot/dts/qcom/sdm845.dtsi b/arch/arm64/boot/dts/qcom/sdm845.dtsi
> index 0692ae0e60a4..9a1705bfa425 100644
> --- a/arch/arm64/boot/dts/qcom/sdm845.dtsi
> +++ b/arch/arm64/boot/dts/qcom/sdm845.dtsi
> @@ -4949,8 +4949,8 @@ slimbam: dma-controller@17184000 {
>  
>  		timer@17c90000 {
>  			#address-cells = <2>;
> -			#size-cells = <2>;
> -			ranges;
> +			#size-cells = <1>;
> +			ranges = <0 0 0 0 0x20000000>;
>  			compatible = "arm,armv7-timer-mem";
>  			reg = <0 0x17c90000 0 0x1000>;
>  
> @@ -4958,49 +4958,49 @@ frame@17ca0000 {
>  				frame-number = <0>;
>  				interrupts = <GIC_SPI 7 IRQ_TYPE_LEVEL_HIGH>,
>  					     <GIC_SPI 6 IRQ_TYPE_LEVEL_HIGH>;
> -				reg = <0 0x17ca0000 0 0x1000>,
> -				      <0 0x17cb0000 0 0x1000>;
> +				reg = <0 0x17ca0000 0x1000>,
> +				      <0 0x17cb0000 0x1000>;
>  			};
>  
>  			frame@17cc0000 {
>  				frame-number = <1>;
>  				interrupts = <GIC_SPI 8 IRQ_TYPE_LEVEL_HIGH>;
> -				reg = <0 0x17cc0000 0 0x1000>;
> +				reg = <0 0x17cc0000 0x1000>;
>  				status = "disabled";
>  			};
>  
>  			frame@17cd0000 {
>  				frame-number = <2>;
>  				interrupts = <GIC_SPI 9 IRQ_TYPE_LEVEL_HIGH>;
> -				reg = <0 0x17cd0000 0 0x1000>;
> +				reg = <0 0x17cd0000 0x1000>;
>  				status = "disabled";
>  			};
>  
>  			frame@17ce0000 {
>  				frame-number = <3>;
>  				interrupts = <GIC_SPI 10 IRQ_TYPE_LEVEL_HIGH>;
> -				reg = <0 0x17ce0000 0 0x1000>;
> +				reg = <0 0x17ce0000 0x1000>;
>  				status = "disabled";
>  			};
>  
>  			frame@17cf0000 {
>  				frame-number = <4>;
>  				interrupts = <GIC_SPI 11 IRQ_TYPE_LEVEL_HIGH>;
> -				reg = <0 0x17cf0000 0 0x1000>;
> +				reg = <0 0x17cf0000 0x1000>;
>  				status = "disabled";
>  			};
>  
>  			frame@17d00000 {
>  				frame-number = <5>;
>  				interrupts = <GIC_SPI 12 IRQ_TYPE_LEVEL_HIGH>;
> -				reg = <0 0x17d00000 0 0x1000>;
> +				reg = <0 0x17d00000 0x1000>;
>  				status = "disabled";
>  			};
>  
>  			frame@17d10000 {
>  				frame-number = <6>;
>  				interrupts = <GIC_SPI 13 IRQ_TYPE_LEVEL_HIGH>;
> -				reg = <0 0x17d10000 0 0x1000>;
> +				reg = <0 0x17d10000 0x1000>;
>  				status = "disabled";
>  			};
>  		};
> diff --git a/arch/arm64/boot/dts/qcom/sm6350.dtsi b/arch/arm64/boot/dts/qcom/sm6350.dtsi
> index d4f8f33f3f0c..fc473cb8c93b 100644
> --- a/arch/arm64/boot/dts/qcom/sm6350.dtsi
> +++ b/arch/arm64/boot/dts/qcom/sm6350.dtsi
> @@ -1305,56 +1305,56 @@ timer@17c20000 {
>  			reg = <0x0 0x17c20000 0x0 0x1000>;
>  			clock-frequency = <19200000>;
>  			#address-cells = <2>;
> -			#size-cells = <2>;
> -			ranges;
> +			#size-cells = <1>;
> +			ranges = <0 0 0 0 0x20000000>;
>  
>  			frame@17c21000 {
>  				frame-number = <0>;
>  				interrupts = <GIC_SPI 8 IRQ_TYPE_LEVEL_HIGH>,
>  					     <GIC_SPI 6 IRQ_TYPE_LEVEL_HIGH>;
> -				reg = <0x0 0x17c21000 0x0 0x1000>,
> -				      <0x0 0x17c22000 0x0 0x1000>;
> +				reg = <0x0 0x17c21000 0x1000>,
> +				      <0x0 0x17c22000 0x1000>;
>  			};
>  
>  			frame@17c23000 {
>  				frame-number = <1>;
>  				interrupts = <GIC_SPI 9 IRQ_TYPE_LEVEL_HIGH>;
> -				reg = <0x0 0x17c23000 0x0 0x1000>;
> +				reg = <0x0 0x17c23000 0x1000>;
>  				status = "disabled";
>  			};
>  
>  			frame@17c25000 {
>  				frame-number = <2>;
>  				interrupts = <GIC_SPI 10 IRQ_TYPE_LEVEL_HIGH>;
> -				reg = <0x0 0x17c25000 0x0 0x1000>;
> +				reg = <0x0 0x17c25000 0x1000>;
>  				status = "disabled";
>  			};
>  
>  			frame@17c27000 {
>  				frame-number = <3>;
>  				interrupts = <GIC_SPI 11 IRQ_TYPE_LEVEL_HIGH>;
> -				reg = <0x0 0x17c27000 0x0 0x1000>;
> +				reg = <0x0 0x17c27000 0x1000>;
>  				status = "disabled";
>  			};
>  
>  			frame@17c29000 {
>  				frame-number = <4>;
>  				interrupts = <GIC_SPI 12 IRQ_TYPE_LEVEL_HIGH>;
> -				reg = <0x0 0x17c29000 0x0 0x1000>;
> +				reg = <0x0 0x17c29000 0x1000>;
>  				status = "disabled";
>  			};
>  
>  			frame@17c2b000 {
>  				frame-number = <5>;
>  				interrupts = <GIC_SPI 13 IRQ_TYPE_LEVEL_HIGH>;
> -				reg = <0x0 0x17c2b000 0x0 0x1000>;
> +				reg = <0x0 0x17c2b000 0x1000>;
>  				status = "disabled";
>  			};
>  
>  			frame@17c2d000 {
>  				frame-number = <6>;
>  				interrupts = <GIC_SPI 14 IRQ_TYPE_LEVEL_HIGH>;
> -				reg = <0x0 0x17c2d000 0x0 0x1000>;
> +				reg = <0x0 0x17c2d000 0x1000>;
>  				status = "disabled";
>  			};
>  		};
> diff --git a/arch/arm64/boot/dts/qcom/sm8150.dtsi b/arch/arm64/boot/dts/qcom/sm8150.dtsi
> index 8ea44c4b56b4..066cc00191b2 100644
> --- a/arch/arm64/boot/dts/qcom/sm8150.dtsi
> +++ b/arch/arm64/boot/dts/qcom/sm8150.dtsi
> @@ -3945,8 +3945,8 @@ watchdog@17c10000 {
>  
>  		timer@17c20000 {
>  			#address-cells = <2>;
> -			#size-cells = <2>;
> -			ranges;
> +			#size-cells = <1>;
> +			ranges = <0 0 0 0 0x20000000>;
>  			compatible = "arm,armv7-timer-mem";
>  			reg = <0x0 0x17c20000 0x0 0x1000>;
>  			clock-frequency = <19200000>;
> @@ -3955,49 +3955,49 @@ frame@17c21000{
>  				frame-number = <0>;
>  				interrupts = <GIC_SPI 8 IRQ_TYPE_LEVEL_HIGH>,
>  					     <GIC_SPI 6 IRQ_TYPE_LEVEL_HIGH>;
> -				reg = <0x0 0x17c21000 0x0 0x1000>,
> -				      <0x0 0x17c22000 0x0 0x1000>;
> +				reg = <0x0 0x17c21000 0x1000>,
> +				      <0x0 0x17c22000 0x1000>;
>  			};
>  
>  			frame@17c23000 {
>  				frame-number = <1>;
>  				interrupts = <GIC_SPI 9 IRQ_TYPE_LEVEL_HIGH>;
> -				reg = <0x0 0x17c23000 0x0 0x1000>;
> +				reg = <0x0 0x17c23000 0x1000>;
>  				status = "disabled";
>  			};
>  
>  			frame@17c25000 {
>  				frame-number = <2>;
>  				interrupts = <GIC_SPI 10 IRQ_TYPE_LEVEL_HIGH>;
> -				reg = <0x0 0x17c25000 0x0 0x1000>;
> +				reg = <0x0 0x17c25000 0x1000>;
>  				status = "disabled";
>  			};
>  
>  			frame@17c27000 {
>  				frame-number = <3>;
>  				interrupts = <GIC_SPI 11 IRQ_TYPE_LEVEL_HIGH>;
> -				reg = <0x0 0x17c26000 0x0 0x1000>;
> +				reg = <0x0 0x17c26000 0x1000>;
>  				status = "disabled";
>  			};
>  
>  			frame@17c29000 {
>  				frame-number = <4>;
>  				interrupts = <GIC_SPI 12 IRQ_TYPE_LEVEL_HIGH>;
> -				reg = <0x0 0x17c29000 0x0 0x1000>;
> +				reg = <0x0 0x17c29000 0x1000>;
>  				status = "disabled";
>  			};
>  
>  			frame@17c2b000 {
>  				frame-number = <5>;
>  				interrupts = <GIC_SPI 13 IRQ_TYPE_LEVEL_HIGH>;
> -				reg = <0x0 0x17c2b000 0x0 0x1000>;
> +				reg = <0x0 0x17c2b000 0x1000>;
>  				status = "disabled";
>  			};
>  
>  			frame@17c2d000 {
>  				frame-number = <6>;
>  				interrupts = <GIC_SPI 14 IRQ_TYPE_LEVEL_HIGH>;
> -				reg = <0x0 0x17c2d000 0x0 0x1000>;
> +				reg = <0x0 0x17c2d000 0x1000>;
>  				status = "disabled";
>  			};
>  		};
> diff --git a/arch/arm64/boot/dts/qcom/sm8250.dtsi b/arch/arm64/boot/dts/qcom/sm8250.dtsi
> index cf0c97bd5ad3..ae7d0573e1dc 100644
> --- a/arch/arm64/boot/dts/qcom/sm8250.dtsi
> +++ b/arch/arm64/boot/dts/qcom/sm8250.dtsi
> @@ -4868,8 +4868,8 @@ watchdog@17c10000 {
>  
>  		timer@17c20000 {
>  			#address-cells = <2>;
> -			#size-cells = <2>;
> -			ranges;
> +			#size-cells = <1>;
> +			ranges = <0 0 0 0 0x20000000>;
>  			compatible = "arm,armv7-timer-mem";
>  			reg = <0x0 0x17c20000 0x0 0x1000>;
>  			clock-frequency = <19200000>;
> @@ -4878,49 +4878,49 @@ frame@17c21000 {
>  				frame-number = <0>;
>  				interrupts = <GIC_SPI 8 IRQ_TYPE_LEVEL_HIGH>,
>  					     <GIC_SPI 6 IRQ_TYPE_LEVEL_HIGH>;
> -				reg = <0x0 0x17c21000 0x0 0x1000>,
> -				      <0x0 0x17c22000 0x0 0x1000>;
> +				reg = <0x0 0x17c21000 0x1000>,
> +				      <0x0 0x17c22000 0x1000>;
>  			};
>  
>  			frame@17c23000 {
>  				frame-number = <1>;
>  				interrupts = <GIC_SPI 9 IRQ_TYPE_LEVEL_HIGH>;
> -				reg = <0x0 0x17c23000 0x0 0x1000>;
> +				reg = <0x0 0x17c23000 0x1000>;
>  				status = "disabled";
>  			};
>  
>  			frame@17c25000 {
>  				frame-number = <2>;
>  				interrupts = <GIC_SPI 10 IRQ_TYPE_LEVEL_HIGH>;
> -				reg = <0x0 0x17c25000 0x0 0x1000>;
> +				reg = <0x0 0x17c25000 0x1000>;
>  				status = "disabled";
>  			};
>  
>  			frame@17c27000 {
>  				frame-number = <3>;
>  				interrupts = <GIC_SPI 11 IRQ_TYPE_LEVEL_HIGH>;
> -				reg = <0x0 0x17c27000 0x0 0x1000>;
> +				reg = <0x0 0x17c27000 0x1000>;
>  				status = "disabled";
>  			};
>  
>  			frame@17c29000 {
>  				frame-number = <4>;
>  				interrupts = <GIC_SPI 12 IRQ_TYPE_LEVEL_HIGH>;
> -				reg = <0x0 0x17c29000 0x0 0x1000>;
> +				reg = <0x0 0x17c29000 0x1000>;
>  				status = "disabled";
>  			};
>  
>  			frame@17c2b000 {
>  				frame-number = <5>;
>  				interrupts = <GIC_SPI 13 IRQ_TYPE_LEVEL_HIGH>;
> -				reg = <0x0 0x17c2b000 0x0 0x1000>;
> +				reg = <0x0 0x17c2b000 0x1000>;
>  				status = "disabled";
>  			};
>  
>  			frame@17c2d000 {
>  				frame-number = <6>;
>  				interrupts = <GIC_SPI 14 IRQ_TYPE_LEVEL_HIGH>;
> -				reg = <0x0 0x17c2d000 0x0 0x1000>;
> +				reg = <0x0 0x17c2d000 0x1000>;
>  				status = "disabled";
>  			};
>  		};
> diff --git a/arch/arm64/boot/dts/qcom/sm8350.dtsi b/arch/arm64/boot/dts/qcom/sm8350.dtsi
> index 743cba9b683c..58e6bb4a1899 100644
> --- a/arch/arm64/boot/dts/qcom/sm8350.dtsi
> +++ b/arch/arm64/boot/dts/qcom/sm8350.dtsi
> @@ -1934,8 +1934,8 @@ intc: interrupt-controller@17a00000 {
>  		timer@17c20000 {
>  			compatible = "arm,armv7-timer-mem";
>  			#address-cells = <2>;
> -			#size-cells = <2>;
> -			ranges;
> +			#size-cells = <1>;
> +			ranges = <0 0 0 0 0x20000000>;
>  			reg = <0x0 0x17c20000 0x0 0x1000>;
>  			clock-frequency = <19200000>;
>  
> @@ -1943,49 +1943,49 @@ frame@17c21000 {
>  				frame-number = <0>;
>  				interrupts = <GIC_SPI 8 IRQ_TYPE_LEVEL_HIGH>,
>  					     <GIC_SPI 6 IRQ_TYPE_LEVEL_HIGH>;
> -				reg = <0x0 0x17c21000 0x0 0x1000>,
> -				      <0x0 0x17c22000 0x0 0x1000>;
> +				reg = <0x0 0x17c21000 0x1000>,
> +				      <0x0 0x17c22000 0x1000>;
>  			};
>  
>  			frame@17c23000 {
>  				frame-number = <1>;
>  				interrupts = <GIC_SPI 9 IRQ_TYPE_LEVEL_HIGH>;
> -				reg = <0x0 0x17c23000 0x0 0x1000>;
> +				reg = <0x0 0x17c23000 0x1000>;
>  				status = "disabled";
>  			};
>  
>  			frame@17c25000 {
>  				frame-number = <2>;
>  				interrupts = <GIC_SPI 10 IRQ_TYPE_LEVEL_HIGH>;
> -				reg = <0x0 0x17c25000 0x0 0x1000>;
> +				reg = <0x0 0x17c25000 0x1000>;
>  				status = "disabled";
>  			};
>  
>  			frame@17c27000 {
>  				frame-number = <3>;
>  				interrupts = <GIC_SPI 11 IRQ_TYPE_LEVEL_HIGH>;
> -				reg = <0x0 0x17c27000 0x0 0x1000>;
> +				reg = <0x0 0x17c27000 0x1000>;
>  				status = "disabled";
>  			};
>  
>  			frame@17c29000 {
>  				frame-number = <4>;
>  				interrupts = <GIC_SPI 12 IRQ_TYPE_LEVEL_HIGH>;
> -				reg = <0x0 0x17c29000 0x0 0x1000>;
> +				reg = <0x0 0x17c29000 0x1000>;
>  				status = "disabled";
>  			};
>  
>  			frame@17c2b000 {
>  				frame-number = <5>;
>  				interrupts = <GIC_SPI 13 IRQ_TYPE_LEVEL_HIGH>;
> -				reg = <0x0 0x17c2b000 0x0 0x1000>;
> +				reg = <0x0 0x17c2b000 0x1000>;
>  				status = "disabled";
>  			};
>  
>  			frame@17c2d000 {
>  				frame-number = <6>;
>  				interrupts = <GIC_SPI 14 IRQ_TYPE_LEVEL_HIGH>;
> -				reg = <0x0 0x17c2d000 0x0 0x1000>;
> +				reg = <0x0 0x17c2d000 0x1000>;
>  				status = "disabled";
>  			};
>  		};
> diff --git a/arch/arm64/boot/dts/qcom/sm8450.dtsi b/arch/arm64/boot/dts/qcom/sm8450.dtsi
> index 7d08fad76371..64ceff515337 100644
> --- a/arch/arm64/boot/dts/qcom/sm8450.dtsi
> +++ b/arch/arm64/boot/dts/qcom/sm8450.dtsi
> @@ -2858,8 +2858,8 @@ intc: interrupt-controller@17100000 {
>  		timer@17420000 {
>  			compatible = "arm,armv7-timer-mem";
>  			#address-cells = <2>;
> -			#size-cells = <2>;
> -			ranges;
> +			#size-cells = <1>;
> +			ranges = <0 0 0 0 0x20000000>;
>  			reg = <0x0 0x17420000 0x0 0x1000>;
>  			clock-frequency = <19200000>;
>  
> @@ -2867,49 +2867,49 @@ frame@17421000 {
>  				frame-number = <0>;
>  				interrupts = <GIC_SPI 8 IRQ_TYPE_LEVEL_HIGH>,
>  					     <GIC_SPI 6 IRQ_TYPE_LEVEL_HIGH>;
> -				reg = <0x0 0x17421000 0x0 0x1000>,
> -				      <0x0 0x17422000 0x0 0x1000>;
> +				reg = <0x0 0x17421000 0x1000>,
> +				      <0x0 0x17422000 0x1000>;
>  			};
>  
>  			frame@17423000 {
>  				frame-number = <1>;
>  				interrupts = <GIC_SPI 9 IRQ_TYPE_LEVEL_HIGH>;
> -				reg = <0x0 0x17423000 0x0 0x1000>;
> +				reg = <0x0 0x17423000 0x1000>;
>  				status = "disabled";
>  			};
>  
>  			frame@17425000 {
>  				frame-number = <2>;
>  				interrupts = <GIC_SPI 10 IRQ_TYPE_LEVEL_HIGH>;
> -				reg = <0x0 0x17425000 0x0 0x1000>;
> +				reg = <0x0 0x17425000 0x1000>;
>  				status = "disabled";
>  			};
>  
>  			frame@17427000 {
>  				frame-number = <3>;
>  				interrupts = <GIC_SPI 11 IRQ_TYPE_LEVEL_HIGH>;
> -				reg = <0x0 0x17427000 0x0 0x1000>;
> +				reg = <0x0 0x17427000 0x1000>;
>  				status = "disabled";
>  			};
>  
>  			frame@17429000 {
>  				frame-number = <4>;
>  				interrupts = <GIC_SPI 12 IRQ_TYPE_LEVEL_HIGH>;
> -				reg = <0x0 0x17429000 0x0 0x1000>;
> +				reg = <0x0 0x17429000 0x1000>;
>  				status = "disabled";
>  			};
>  
>  			frame@1742b000 {
>  				frame-number = <5>;
>  				interrupts = <GIC_SPI 13 IRQ_TYPE_LEVEL_HIGH>;
> -				reg = <0x0 0x1742b000 0x0 0x1000>;
> +				reg = <0x0 0x1742b000 0x1000>;
>  				status = "disabled";
>  			};
>  
>  			frame@1742d000 {
>  				frame-number = <6>;
>  				interrupts = <GIC_SPI 14 IRQ_TYPE_LEVEL_HIGH>;
> -				reg = <0x0 0x1742d000 0x0 0x1000>;
> +				reg = <0x0 0x1742d000 0x1000>;
>  				status = "disabled";
>  			};
>  		};
> -- 
> 2.35.1
>
David Heidelberg June 26, 2022, 9:51 a.m. UTC | #2
On 26/06/2022 05:12, Bjorn Andersson wrote:
> On Sun 15 May 17:17 CDT 2022, David Heidelberg wrote:
>
>> There's no reason the timer needs > 32-bits of size.
>> Since we using 32-bit size, we need to define ranges properly.
>>
>> Fixes warnings as:
>> ```
>> arch/arm64/boot/dts/qcom/sdm845-oneplus-fajita.dt.yaml: timer@17c90000: #size-cells:0:0: 1 was expected
>>          From schema: Documentation/devicetree/bindings/timer/arm,arch_timer_mmio.yaml
>> ```
>>
>> Signed-off-by: David Heidelberg <david@ixit.cz>
>>
>> ---
>> Replaces "dt-bindings: timer: add #size-cells 2 for ARM"
> I'm not able to find this patch, but I'm guessing that you tried to make
> it possible to allow #size-cells to be 2 in the binding?

Yes, here it's the original patch link: 
https://lkml.iu.edu/hypermail/linux/kernel/2112.1/09190.html


>
>> v2: fix accidental change of value 0x0 to 0
>>
>>   arch/arm64/boot/dts/qcom/ipq6018.dtsi | 20 ++++++++++----------
>>   arch/arm64/boot/dts/qcom/sc7180.dtsi  | 20 ++++++++++----------
>>   arch/arm64/boot/dts/qcom/sc7280.dtsi  | 20 ++++++++++----------
>>   arch/arm64/boot/dts/qcom/sdm845.dtsi  | 20 ++++++++++----------
>>   arch/arm64/boot/dts/qcom/sm6350.dtsi  | 20 ++++++++++----------
>>   arch/arm64/boot/dts/qcom/sm8150.dtsi  | 20 ++++++++++----------
>>   arch/arm64/boot/dts/qcom/sm8250.dtsi  | 20 ++++++++++----------
>>   arch/arm64/boot/dts/qcom/sm8350.dtsi  | 20 ++++++++++----------
>>   arch/arm64/boot/dts/qcom/sm8450.dtsi  | 20 ++++++++++----------
>>   9 files changed, 90 insertions(+), 90 deletions(-)
>>
>> diff --git a/arch/arm64/boot/dts/qcom/ipq6018.dtsi b/arch/arm64/boot/dts/qcom/ipq6018.dtsi
>> index c89499e366d3..8638231dbd2f 100644
>> --- a/arch/arm64/boot/dts/qcom/ipq6018.dtsi
>> +++ b/arch/arm64/boot/dts/qcom/ipq6018.dtsi
>> @@ -526,8 +526,8 @@ timer {
>>   
>>   		timer@b120000 {
>>   			#address-cells = <2>;
>> -			#size-cells = <2>;
>> -			ranges;
>> +			#size-cells = <1>;
> While perfectly valid, I don't like the asymmetry of #address-cells = 2
> and #size-cells = 1.
>
> If the binding maintainers don't approve of allowing #size-cells = 2,
> can you please make #address-cells = 1 as well?

Yes, for sure, I should noticed in reply to my original patch -

https://lkml.iu.edu/hypermail/linux/kernel/2112.1/09937.html

that both address and size should be 32-bit only.

Fixed patch coming soon! :)

David

>
> Regards,
> Bjorn
>
>> +			ranges = <0 0 0 0 0x10000000>;
>>   			compatible = "arm,armv7-timer-mem";
>>   			reg = <0x0 0x0b120000 0x0 0x1000>;
>>   
>> @@ -535,49 +535,49 @@ frame@b120000 {
>>   				frame-number = <0>;
>>   				interrupts = <GIC_SPI 8 IRQ_TYPE_LEVEL_HIGH>,
>>   					     <GIC_SPI 7 IRQ_TYPE_LEVEL_HIGH>;
>> -				reg = <0x0 0x0b121000 0x0 0x1000>,
>> -				      <0x0 0x0b122000 0x0 0x1000>;
>> +				reg = <0x0 0x0b121000 0x1000>,
>> +				      <0x0 0x0b122000 0x1000>;
>>   			};
>>   
>>   			frame@b123000 {
>>   				frame-number = <1>;
>>   				interrupts = <GIC_SPI 9 IRQ_TYPE_LEVEL_HIGH>;
>> -				reg = <0x0 0xb123000 0x0 0x1000>;
>> +				reg = <0x0 0x0b123000 0x1000>;
>>   				status = "disabled";
>>   			};
>>   
>>   			frame@b124000 {
>>   				frame-number = <2>;
>>   				interrupts = <GIC_SPI 10 IRQ_TYPE_LEVEL_HIGH>;
>> -				reg = <0x0 0x0b124000 0x0 0x1000>;
>> +				reg = <0x0 0x0b124000 0x1000>;
>>   				status = "disabled";
>>   			};
>>   
>>   			frame@b125000 {
>>   				frame-number = <3>;
>>   				interrupts = <GIC_SPI 11 IRQ_TYPE_LEVEL_HIGH>;
>> -				reg = <0x0 0x0b125000 0x0 0x1000>;
>> +				reg = <0x0 0x0b125000 0x1000>;
>>   				status = "disabled";
>>   			};
>>   
>>   			frame@b126000 {
>>   				frame-number = <4>;
>>   				interrupts = <GIC_SPI 12 IRQ_TYPE_LEVEL_HIGH>;
>> -				reg = <0x0 0x0b126000 0x0 0x1000>;
>> +				reg = <0x0 0x0b126000 0x1000>;
>>   				status = "disabled";
>>   			};
>>   
>>   			frame@b127000 {
>>   				frame-number = <5>;
>>   				interrupts = <GIC_SPI 13 IRQ_TYPE_LEVEL_HIGH>;
>> -				reg = <0x0 0x0b127000 0x0 0x1000>;
>> +				reg = <0x0 0x0b127000 0x1000>;
>>   				status = "disabled";
>>   			};
>>   
>>   			frame@b128000 {
>>   				frame-number = <6>;
>>   				interrupts = <GIC_SPI 14 IRQ_TYPE_LEVEL_HIGH>;
>> -				reg = <0x0 0x0b128000 0x0 0x1000>;
>> +				reg = <0x0 0x0b128000 0x1000>;
>>   				status = "disabled";
>>   			};
>>   		};
>> diff --git a/arch/arm64/boot/dts/qcom/sc7180.dtsi b/arch/arm64/boot/dts/qcom/sc7180.dtsi
>> index 5dcaac23a138..4b7be66e04df 100644
>> --- a/arch/arm64/boot/dts/qcom/sc7180.dtsi
>> +++ b/arch/arm64/boot/dts/qcom/sc7180.dtsi
>> @@ -3385,8 +3385,8 @@ watchdog@17c10000 {
>>   
>>   		timer@17c20000{
>>   			#address-cells = <2>;
>> -			#size-cells = <2>;
>> -			ranges;
>> +			#size-cells = <1>;
>> +			ranges = <0 0 0 0 0x20000000>;
>>   			compatible = "arm,armv7-timer-mem";
>>   			reg = <0 0x17c20000 0 0x1000>;
>>   
>> @@ -3394,49 +3394,49 @@ frame@17c21000 {
>>   				frame-number = <0>;
>>   				interrupts = <GIC_SPI 8 IRQ_TYPE_LEVEL_HIGH>,
>>   					     <GIC_SPI 6 IRQ_TYPE_LEVEL_HIGH>;
>> -				reg = <0 0x17c21000 0 0x1000>,
>> -				      <0 0x17c22000 0 0x1000>;
>> +				reg = <0 0x17c21000 0x1000>,
>> +				      <0 0x17c22000 0x1000>;
>>   			};
>>   
>>   			frame@17c23000 {
>>   				frame-number = <1>;
>>   				interrupts = <GIC_SPI 9 IRQ_TYPE_LEVEL_HIGH>;
>> -				reg = <0 0x17c23000 0 0x1000>;
>> +				reg = <0 0x17c23000 0x1000>;
>>   				status = "disabled";
>>   			};
>>   
>>   			frame@17c25000 {
>>   				frame-number = <2>;
>>   				interrupts = <GIC_SPI 10 IRQ_TYPE_LEVEL_HIGH>;
>> -				reg = <0 0x17c25000 0 0x1000>;
>> +				reg = <0 0x17c25000 0x1000>;
>>   				status = "disabled";
>>   			};
>>   
>>   			frame@17c27000 {
>>   				frame-number = <3>;
>>   				interrupts = <GIC_SPI 11 IRQ_TYPE_LEVEL_HIGH>;
>> -				reg = <0 0x17c27000 0 0x1000>;
>> +				reg = <0 0x17c27000 0x1000>;
>>   				status = "disabled";
>>   			};
>>   
>>   			frame@17c29000 {
>>   				frame-number = <4>;
>>   				interrupts = <GIC_SPI 12 IRQ_TYPE_LEVEL_HIGH>;
>> -				reg = <0 0x17c29000 0 0x1000>;
>> +				reg = <0 0x17c29000 0x1000>;
>>   				status = "disabled";
>>   			};
>>   
>>   			frame@17c2b000 {
>>   				frame-number = <5>;
>>   				interrupts = <GIC_SPI 13 IRQ_TYPE_LEVEL_HIGH>;
>> -				reg = <0 0x17c2b000 0 0x1000>;
>> +				reg = <0 0x17c2b000 0x1000>;
>>   				status = "disabled";
>>   			};
>>   
>>   			frame@17c2d000 {
>>   				frame-number = <6>;
>>   				interrupts = <GIC_SPI 14 IRQ_TYPE_LEVEL_HIGH>;
>> -				reg = <0 0x17c2d000 0 0x1000>;
>> +				reg = <0 0x17c2d000 0x1000>;
>>   				status = "disabled";
>>   			};
>>   		};
>> diff --git a/arch/arm64/boot/dts/qcom/sc7280.dtsi b/arch/arm64/boot/dts/qcom/sc7280.dtsi
>> index e66fc67de206..1b81cd1e84e9 100644
>> --- a/arch/arm64/boot/dts/qcom/sc7280.dtsi
>> +++ b/arch/arm64/boot/dts/qcom/sc7280.dtsi
>> @@ -4772,8 +4772,8 @@ watchdog@17c10000 {
>>   
>>   		timer@17c20000 {
>>   			#address-cells = <2>;
>> -			#size-cells = <2>;
>> -			ranges;
>> +			#size-cells = <1>;
>> +			ranges = <0 0 0 0 0x20000000>;
>>   			compatible = "arm,armv7-timer-mem";
>>   			reg = <0 0x17c20000 0 0x1000>;
>>   
>> @@ -4781,49 +4781,49 @@ frame@17c21000 {
>>   				frame-number = <0>;
>>   				interrupts = <GIC_SPI 8 IRQ_TYPE_LEVEL_HIGH>,
>>   					     <GIC_SPI 6 IRQ_TYPE_LEVEL_HIGH>;
>> -				reg = <0 0x17c21000 0 0x1000>,
>> -				      <0 0x17c22000 0 0x1000>;
>> +				reg = <0 0x17c21000 0x1000>,
>> +				      <0 0x17c22000 0x1000>;
>>   			};
>>   
>>   			frame@17c23000 {
>>   				frame-number = <1>;
>>   				interrupts = <GIC_SPI 9 IRQ_TYPE_LEVEL_HIGH>;
>> -				reg = <0 0x17c23000 0 0x1000>;
>> +				reg = <0 0x17c23000 0x1000>;
>>   				status = "disabled";
>>   			};
>>   
>>   			frame@17c25000 {
>>   				frame-number = <2>;
>>   				interrupts = <GIC_SPI 10 IRQ_TYPE_LEVEL_HIGH>;
>> -				reg = <0 0x17c25000 0 0x1000>;
>> +				reg = <0 0x17c25000 0x1000>;
>>   				status = "disabled";
>>   			};
>>   
>>   			frame@17c27000 {
>>   				frame-number = <3>;
>>   				interrupts = <GIC_SPI 11 IRQ_TYPE_LEVEL_HIGH>;
>> -				reg = <0 0x17c27000 0 0x1000>;
>> +				reg = <0 0x17c27000 0x1000>;
>>   				status = "disabled";
>>   			};
>>   
>>   			frame@17c29000 {
>>   				frame-number = <4>;
>>   				interrupts = <GIC_SPI 12 IRQ_TYPE_LEVEL_HIGH>;
>> -				reg = <0 0x17c29000 0 0x1000>;
>> +				reg = <0 0x17c29000 0x1000>;
>>   				status = "disabled";
>>   			};
>>   
>>   			frame@17c2b000 {
>>   				frame-number = <5>;
>>   				interrupts = <GIC_SPI 13 IRQ_TYPE_LEVEL_HIGH>;
>> -				reg = <0 0x17c2b000 0 0x1000>;
>> +				reg = <0 0x17c2b000 0x1000>;
>>   				status = "disabled";
>>   			};
>>   
>>   			frame@17c2d000 {
>>   				frame-number = <6>;
>>   				interrupts = <GIC_SPI 14 IRQ_TYPE_LEVEL_HIGH>;
>> -				reg = <0 0x17c2d000 0 0x1000>;
>> +				reg = <0 0x17c2d000 0x1000>;
>>   				status = "disabled";
>>   			};
>>   		};
>> diff --git a/arch/arm64/boot/dts/qcom/sdm845.dtsi b/arch/arm64/boot/dts/qcom/sdm845.dtsi
>> index 0692ae0e60a4..9a1705bfa425 100644
>> --- a/arch/arm64/boot/dts/qcom/sdm845.dtsi
>> +++ b/arch/arm64/boot/dts/qcom/sdm845.dtsi
>> @@ -4949,8 +4949,8 @@ slimbam: dma-controller@17184000 {
>>   
>>   		timer@17c90000 {
>>   			#address-cells = <2>;
>> -			#size-cells = <2>;
>> -			ranges;
>> +			#size-cells = <1>;
>> +			ranges = <0 0 0 0 0x20000000>;
>>   			compatible = "arm,armv7-timer-mem";
>>   			reg = <0 0x17c90000 0 0x1000>;
>>   
>> @@ -4958,49 +4958,49 @@ frame@17ca0000 {
>>   				frame-number = <0>;
>>   				interrupts = <GIC_SPI 7 IRQ_TYPE_LEVEL_HIGH>,
>>   					     <GIC_SPI 6 IRQ_TYPE_LEVEL_HIGH>;
>> -				reg = <0 0x17ca0000 0 0x1000>,
>> -				      <0 0x17cb0000 0 0x1000>;
>> +				reg = <0 0x17ca0000 0x1000>,
>> +				      <0 0x17cb0000 0x1000>;
>>   			};
>>   
>>   			frame@17cc0000 {
>>   				frame-number = <1>;
>>   				interrupts = <GIC_SPI 8 IRQ_TYPE_LEVEL_HIGH>;
>> -				reg = <0 0x17cc0000 0 0x1000>;
>> +				reg = <0 0x17cc0000 0x1000>;
>>   				status = "disabled";
>>   			};
>>   
>>   			frame@17cd0000 {
>>   				frame-number = <2>;
>>   				interrupts = <GIC_SPI 9 IRQ_TYPE_LEVEL_HIGH>;
>> -				reg = <0 0x17cd0000 0 0x1000>;
>> +				reg = <0 0x17cd0000 0x1000>;
>>   				status = "disabled";
>>   			};
>>   
>>   			frame@17ce0000 {
>>   				frame-number = <3>;
>>   				interrupts = <GIC_SPI 10 IRQ_TYPE_LEVEL_HIGH>;
>> -				reg = <0 0x17ce0000 0 0x1000>;
>> +				reg = <0 0x17ce0000 0x1000>;
>>   				status = "disabled";
>>   			};
>>   
>>   			frame@17cf0000 {
>>   				frame-number = <4>;
>>   				interrupts = <GIC_SPI 11 IRQ_TYPE_LEVEL_HIGH>;
>> -				reg = <0 0x17cf0000 0 0x1000>;
>> +				reg = <0 0x17cf0000 0x1000>;
>>   				status = "disabled";
>>   			};
>>   
>>   			frame@17d00000 {
>>   				frame-number = <5>;
>>   				interrupts = <GIC_SPI 12 IRQ_TYPE_LEVEL_HIGH>;
>> -				reg = <0 0x17d00000 0 0x1000>;
>> +				reg = <0 0x17d00000 0x1000>;
>>   				status = "disabled";
>>   			};
>>   
>>   			frame@17d10000 {
>>   				frame-number = <6>;
>>   				interrupts = <GIC_SPI 13 IRQ_TYPE_LEVEL_HIGH>;
>> -				reg = <0 0x17d10000 0 0x1000>;
>> +				reg = <0 0x17d10000 0x1000>;
>>   				status = "disabled";
>>   			};
>>   		};
>> diff --git a/arch/arm64/boot/dts/qcom/sm6350.dtsi b/arch/arm64/boot/dts/qcom/sm6350.dtsi
>> index d4f8f33f3f0c..fc473cb8c93b 100644
>> --- a/arch/arm64/boot/dts/qcom/sm6350.dtsi
>> +++ b/arch/arm64/boot/dts/qcom/sm6350.dtsi
>> @@ -1305,56 +1305,56 @@ timer@17c20000 {
>>   			reg = <0x0 0x17c20000 0x0 0x1000>;
>>   			clock-frequency = <19200000>;
>>   			#address-cells = <2>;
>> -			#size-cells = <2>;
>> -			ranges;
>> +			#size-cells = <1>;
>> +			ranges = <0 0 0 0 0x20000000>;
>>   
>>   			frame@17c21000 {
>>   				frame-number = <0>;
>>   				interrupts = <GIC_SPI 8 IRQ_TYPE_LEVEL_HIGH>,
>>   					     <GIC_SPI 6 IRQ_TYPE_LEVEL_HIGH>;
>> -				reg = <0x0 0x17c21000 0x0 0x1000>,
>> -				      <0x0 0x17c22000 0x0 0x1000>;
>> +				reg = <0x0 0x17c21000 0x1000>,
>> +				      <0x0 0x17c22000 0x1000>;
>>   			};
>>   
>>   			frame@17c23000 {
>>   				frame-number = <1>;
>>   				interrupts = <GIC_SPI 9 IRQ_TYPE_LEVEL_HIGH>;
>> -				reg = <0x0 0x17c23000 0x0 0x1000>;
>> +				reg = <0x0 0x17c23000 0x1000>;
>>   				status = "disabled";
>>   			};
>>   
>>   			frame@17c25000 {
>>   				frame-number = <2>;
>>   				interrupts = <GIC_SPI 10 IRQ_TYPE_LEVEL_HIGH>;
>> -				reg = <0x0 0x17c25000 0x0 0x1000>;
>> +				reg = <0x0 0x17c25000 0x1000>;
>>   				status = "disabled";
>>   			};
>>   
>>   			frame@17c27000 {
>>   				frame-number = <3>;
>>   				interrupts = <GIC_SPI 11 IRQ_TYPE_LEVEL_HIGH>;
>> -				reg = <0x0 0x17c27000 0x0 0x1000>;
>> +				reg = <0x0 0x17c27000 0x1000>;
>>   				status = "disabled";
>>   			};
>>   
>>   			frame@17c29000 {
>>   				frame-number = <4>;
>>   				interrupts = <GIC_SPI 12 IRQ_TYPE_LEVEL_HIGH>;
>> -				reg = <0x0 0x17c29000 0x0 0x1000>;
>> +				reg = <0x0 0x17c29000 0x1000>;
>>   				status = "disabled";
>>   			};
>>   
>>   			frame@17c2b000 {
>>   				frame-number = <5>;
>>   				interrupts = <GIC_SPI 13 IRQ_TYPE_LEVEL_HIGH>;
>> -				reg = <0x0 0x17c2b000 0x0 0x1000>;
>> +				reg = <0x0 0x17c2b000 0x1000>;
>>   				status = "disabled";
>>   			};
>>   
>>   			frame@17c2d000 {
>>   				frame-number = <6>;
>>   				interrupts = <GIC_SPI 14 IRQ_TYPE_LEVEL_HIGH>;
>> -				reg = <0x0 0x17c2d000 0x0 0x1000>;
>> +				reg = <0x0 0x17c2d000 0x1000>;
>>   				status = "disabled";
>>   			};
>>   		};
>> diff --git a/arch/arm64/boot/dts/qcom/sm8150.dtsi b/arch/arm64/boot/dts/qcom/sm8150.dtsi
>> index 8ea44c4b56b4..066cc00191b2 100644
>> --- a/arch/arm64/boot/dts/qcom/sm8150.dtsi
>> +++ b/arch/arm64/boot/dts/qcom/sm8150.dtsi
>> @@ -3945,8 +3945,8 @@ watchdog@17c10000 {
>>   
>>   		timer@17c20000 {
>>   			#address-cells = <2>;
>> -			#size-cells = <2>;
>> -			ranges;
>> +			#size-cells = <1>;
>> +			ranges = <0 0 0 0 0x20000000>;
>>   			compatible = "arm,armv7-timer-mem";
>>   			reg = <0x0 0x17c20000 0x0 0x1000>;
>>   			clock-frequency = <19200000>;
>> @@ -3955,49 +3955,49 @@ frame@17c21000{
>>   				frame-number = <0>;
>>   				interrupts = <GIC_SPI 8 IRQ_TYPE_LEVEL_HIGH>,
>>   					     <GIC_SPI 6 IRQ_TYPE_LEVEL_HIGH>;
>> -				reg = <0x0 0x17c21000 0x0 0x1000>,
>> -				      <0x0 0x17c22000 0x0 0x1000>;
>> +				reg = <0x0 0x17c21000 0x1000>,
>> +				      <0x0 0x17c22000 0x1000>;
>>   			};
>>   
>>   			frame@17c23000 {
>>   				frame-number = <1>;
>>   				interrupts = <GIC_SPI 9 IRQ_TYPE_LEVEL_HIGH>;
>> -				reg = <0x0 0x17c23000 0x0 0x1000>;
>> +				reg = <0x0 0x17c23000 0x1000>;
>>   				status = "disabled";
>>   			};
>>   
>>   			frame@17c25000 {
>>   				frame-number = <2>;
>>   				interrupts = <GIC_SPI 10 IRQ_TYPE_LEVEL_HIGH>;
>> -				reg = <0x0 0x17c25000 0x0 0x1000>;
>> +				reg = <0x0 0x17c25000 0x1000>;
>>   				status = "disabled";
>>   			};
>>   
>>   			frame@17c27000 {
>>   				frame-number = <3>;
>>   				interrupts = <GIC_SPI 11 IRQ_TYPE_LEVEL_HIGH>;
>> -				reg = <0x0 0x17c26000 0x0 0x1000>;
>> +				reg = <0x0 0x17c26000 0x1000>;
>>   				status = "disabled";
>>   			};
>>   
>>   			frame@17c29000 {
>>   				frame-number = <4>;
>>   				interrupts = <GIC_SPI 12 IRQ_TYPE_LEVEL_HIGH>;
>> -				reg = <0x0 0x17c29000 0x0 0x1000>;
>> +				reg = <0x0 0x17c29000 0x1000>;
>>   				status = "disabled";
>>   			};
>>   
>>   			frame@17c2b000 {
>>   				frame-number = <5>;
>>   				interrupts = <GIC_SPI 13 IRQ_TYPE_LEVEL_HIGH>;
>> -				reg = <0x0 0x17c2b000 0x0 0x1000>;
>> +				reg = <0x0 0x17c2b000 0x1000>;
>>   				status = "disabled";
>>   			};
>>   
>>   			frame@17c2d000 {
>>   				frame-number = <6>;
>>   				interrupts = <GIC_SPI 14 IRQ_TYPE_LEVEL_HIGH>;
>> -				reg = <0x0 0x17c2d000 0x0 0x1000>;
>> +				reg = <0x0 0x17c2d000 0x1000>;
>>   				status = "disabled";
>>   			};
>>   		};
>> diff --git a/arch/arm64/boot/dts/qcom/sm8250.dtsi b/arch/arm64/boot/dts/qcom/sm8250.dtsi
>> index cf0c97bd5ad3..ae7d0573e1dc 100644
>> --- a/arch/arm64/boot/dts/qcom/sm8250.dtsi
>> +++ b/arch/arm64/boot/dts/qcom/sm8250.dtsi
>> @@ -4868,8 +4868,8 @@ watchdog@17c10000 {
>>   
>>   		timer@17c20000 {
>>   			#address-cells = <2>;
>> -			#size-cells = <2>;
>> -			ranges;
>> +			#size-cells = <1>;
>> +			ranges = <0 0 0 0 0x20000000>;
>>   			compatible = "arm,armv7-timer-mem";
>>   			reg = <0x0 0x17c20000 0x0 0x1000>;
>>   			clock-frequency = <19200000>;
>> @@ -4878,49 +4878,49 @@ frame@17c21000 {
>>   				frame-number = <0>;
>>   				interrupts = <GIC_SPI 8 IRQ_TYPE_LEVEL_HIGH>,
>>   					     <GIC_SPI 6 IRQ_TYPE_LEVEL_HIGH>;
>> -				reg = <0x0 0x17c21000 0x0 0x1000>,
>> -				      <0x0 0x17c22000 0x0 0x1000>;
>> +				reg = <0x0 0x17c21000 0x1000>,
>> +				      <0x0 0x17c22000 0x1000>;
>>   			};
>>   
>>   			frame@17c23000 {
>>   				frame-number = <1>;
>>   				interrupts = <GIC_SPI 9 IRQ_TYPE_LEVEL_HIGH>;
>> -				reg = <0x0 0x17c23000 0x0 0x1000>;
>> +				reg = <0x0 0x17c23000 0x1000>;
>>   				status = "disabled";
>>   			};
>>   
>>   			frame@17c25000 {
>>   				frame-number = <2>;
>>   				interrupts = <GIC_SPI 10 IRQ_TYPE_LEVEL_HIGH>;
>> -				reg = <0x0 0x17c25000 0x0 0x1000>;
>> +				reg = <0x0 0x17c25000 0x1000>;
>>   				status = "disabled";
>>   			};
>>   
>>   			frame@17c27000 {
>>   				frame-number = <3>;
>>   				interrupts = <GIC_SPI 11 IRQ_TYPE_LEVEL_HIGH>;
>> -				reg = <0x0 0x17c27000 0x0 0x1000>;
>> +				reg = <0x0 0x17c27000 0x1000>;
>>   				status = "disabled";
>>   			};
>>   
>>   			frame@17c29000 {
>>   				frame-number = <4>;
>>   				interrupts = <GIC_SPI 12 IRQ_TYPE_LEVEL_HIGH>;
>> -				reg = <0x0 0x17c29000 0x0 0x1000>;
>> +				reg = <0x0 0x17c29000 0x1000>;
>>   				status = "disabled";
>>   			};
>>   
>>   			frame@17c2b000 {
>>   				frame-number = <5>;
>>   				interrupts = <GIC_SPI 13 IRQ_TYPE_LEVEL_HIGH>;
>> -				reg = <0x0 0x17c2b000 0x0 0x1000>;
>> +				reg = <0x0 0x17c2b000 0x1000>;
>>   				status = "disabled";
>>   			};
>>   
>>   			frame@17c2d000 {
>>   				frame-number = <6>;
>>   				interrupts = <GIC_SPI 14 IRQ_TYPE_LEVEL_HIGH>;
>> -				reg = <0x0 0x17c2d000 0x0 0x1000>;
>> +				reg = <0x0 0x17c2d000 0x1000>;
>>   				status = "disabled";
>>   			};
>>   		};
>> diff --git a/arch/arm64/boot/dts/qcom/sm8350.dtsi b/arch/arm64/boot/dts/qcom/sm8350.dtsi
>> index 743cba9b683c..58e6bb4a1899 100644
>> --- a/arch/arm64/boot/dts/qcom/sm8350.dtsi
>> +++ b/arch/arm64/boot/dts/qcom/sm8350.dtsi
>> @@ -1934,8 +1934,8 @@ intc: interrupt-controller@17a00000 {
>>   		timer@17c20000 {
>>   			compatible = "arm,armv7-timer-mem";
>>   			#address-cells = <2>;
>> -			#size-cells = <2>;
>> -			ranges;
>> +			#size-cells = <1>;
>> +			ranges = <0 0 0 0 0x20000000>;
>>   			reg = <0x0 0x17c20000 0x0 0x1000>;
>>   			clock-frequency = <19200000>;
>>   
>> @@ -1943,49 +1943,49 @@ frame@17c21000 {
>>   				frame-number = <0>;
>>   				interrupts = <GIC_SPI 8 IRQ_TYPE_LEVEL_HIGH>,
>>   					     <GIC_SPI 6 IRQ_TYPE_LEVEL_HIGH>;
>> -				reg = <0x0 0x17c21000 0x0 0x1000>,
>> -				      <0x0 0x17c22000 0x0 0x1000>;
>> +				reg = <0x0 0x17c21000 0x1000>,
>> +				      <0x0 0x17c22000 0x1000>;
>>   			};
>>   
>>   			frame@17c23000 {
>>   				frame-number = <1>;
>>   				interrupts = <GIC_SPI 9 IRQ_TYPE_LEVEL_HIGH>;
>> -				reg = <0x0 0x17c23000 0x0 0x1000>;
>> +				reg = <0x0 0x17c23000 0x1000>;
>>   				status = "disabled";
>>   			};
>>   
>>   			frame@17c25000 {
>>   				frame-number = <2>;
>>   				interrupts = <GIC_SPI 10 IRQ_TYPE_LEVEL_HIGH>;
>> -				reg = <0x0 0x17c25000 0x0 0x1000>;
>> +				reg = <0x0 0x17c25000 0x1000>;
>>   				status = "disabled";
>>   			};
>>   
>>   			frame@17c27000 {
>>   				frame-number = <3>;
>>   				interrupts = <GIC_SPI 11 IRQ_TYPE_LEVEL_HIGH>;
>> -				reg = <0x0 0x17c27000 0x0 0x1000>;
>> +				reg = <0x0 0x17c27000 0x1000>;
>>   				status = "disabled";
>>   			};
>>   
>>   			frame@17c29000 {
>>   				frame-number = <4>;
>>   				interrupts = <GIC_SPI 12 IRQ_TYPE_LEVEL_HIGH>;
>> -				reg = <0x0 0x17c29000 0x0 0x1000>;
>> +				reg = <0x0 0x17c29000 0x1000>;
>>   				status = "disabled";
>>   			};
>>   
>>   			frame@17c2b000 {
>>   				frame-number = <5>;
>>   				interrupts = <GIC_SPI 13 IRQ_TYPE_LEVEL_HIGH>;
>> -				reg = <0x0 0x17c2b000 0x0 0x1000>;
>> +				reg = <0x0 0x17c2b000 0x1000>;
>>   				status = "disabled";
>>   			};
>>   
>>   			frame@17c2d000 {
>>   				frame-number = <6>;
>>   				interrupts = <GIC_SPI 14 IRQ_TYPE_LEVEL_HIGH>;
>> -				reg = <0x0 0x17c2d000 0x0 0x1000>;
>> +				reg = <0x0 0x17c2d000 0x1000>;
>>   				status = "disabled";
>>   			};
>>   		};
>> diff --git a/arch/arm64/boot/dts/qcom/sm8450.dtsi b/arch/arm64/boot/dts/qcom/sm8450.dtsi
>> index 7d08fad76371..64ceff515337 100644
>> --- a/arch/arm64/boot/dts/qcom/sm8450.dtsi
>> +++ b/arch/arm64/boot/dts/qcom/sm8450.dtsi
>> @@ -2858,8 +2858,8 @@ intc: interrupt-controller@17100000 {
>>   		timer@17420000 {
>>   			compatible = "arm,armv7-timer-mem";
>>   			#address-cells = <2>;
>> -			#size-cells = <2>;
>> -			ranges;
>> +			#size-cells = <1>;
>> +			ranges = <0 0 0 0 0x20000000>;
>>   			reg = <0x0 0x17420000 0x0 0x1000>;
>>   			clock-frequency = <19200000>;
>>   
>> @@ -2867,49 +2867,49 @@ frame@17421000 {
>>   				frame-number = <0>;
>>   				interrupts = <GIC_SPI 8 IRQ_TYPE_LEVEL_HIGH>,
>>   					     <GIC_SPI 6 IRQ_TYPE_LEVEL_HIGH>;
>> -				reg = <0x0 0x17421000 0x0 0x1000>,
>> -				      <0x0 0x17422000 0x0 0x1000>;
>> +				reg = <0x0 0x17421000 0x1000>,
>> +				      <0x0 0x17422000 0x1000>;
>>   			};
>>   
>>   			frame@17423000 {
>>   				frame-number = <1>;
>>   				interrupts = <GIC_SPI 9 IRQ_TYPE_LEVEL_HIGH>;
>> -				reg = <0x0 0x17423000 0x0 0x1000>;
>> +				reg = <0x0 0x17423000 0x1000>;
>>   				status = "disabled";
>>   			};
>>   
>>   			frame@17425000 {
>>   				frame-number = <2>;
>>   				interrupts = <GIC_SPI 10 IRQ_TYPE_LEVEL_HIGH>;
>> -				reg = <0x0 0x17425000 0x0 0x1000>;
>> +				reg = <0x0 0x17425000 0x1000>;
>>   				status = "disabled";
>>   			};
>>   
>>   			frame@17427000 {
>>   				frame-number = <3>;
>>   				interrupts = <GIC_SPI 11 IRQ_TYPE_LEVEL_HIGH>;
>> -				reg = <0x0 0x17427000 0x0 0x1000>;
>> +				reg = <0x0 0x17427000 0x1000>;
>>   				status = "disabled";
>>   			};
>>   
>>   			frame@17429000 {
>>   				frame-number = <4>;
>>   				interrupts = <GIC_SPI 12 IRQ_TYPE_LEVEL_HIGH>;
>> -				reg = <0x0 0x17429000 0x0 0x1000>;
>> +				reg = <0x0 0x17429000 0x1000>;
>>   				status = "disabled";
>>   			};
>>   
>>   			frame@1742b000 {
>>   				frame-number = <5>;
>>   				interrupts = <GIC_SPI 13 IRQ_TYPE_LEVEL_HIGH>;
>> -				reg = <0x0 0x1742b000 0x0 0x1000>;
>> +				reg = <0x0 0x1742b000 0x1000>;
>>   				status = "disabled";
>>   			};
>>   
>>   			frame@1742d000 {
>>   				frame-number = <6>;
>>   				interrupts = <GIC_SPI 14 IRQ_TYPE_LEVEL_HIGH>;
>> -				reg = <0x0 0x1742d000 0x0 0x1000>;
>> +				reg = <0x0 0x1742d000 0x1000>;
>>   				status = "disabled";
>>   			};
>>   		};
>> -- 
>> 2.35.1
>>
diff mbox series

Patch

diff --git a/arch/arm64/boot/dts/qcom/ipq6018.dtsi b/arch/arm64/boot/dts/qcom/ipq6018.dtsi
index c89499e366d3..8638231dbd2f 100644
--- a/arch/arm64/boot/dts/qcom/ipq6018.dtsi
+++ b/arch/arm64/boot/dts/qcom/ipq6018.dtsi
@@ -526,8 +526,8 @@  timer {
 
 		timer@b120000 {
 			#address-cells = <2>;
-			#size-cells = <2>;
-			ranges;
+			#size-cells = <1>;
+			ranges = <0 0 0 0 0x10000000>;
 			compatible = "arm,armv7-timer-mem";
 			reg = <0x0 0x0b120000 0x0 0x1000>;
 
@@ -535,49 +535,49 @@  frame@b120000 {
 				frame-number = <0>;
 				interrupts = <GIC_SPI 8 IRQ_TYPE_LEVEL_HIGH>,
 					     <GIC_SPI 7 IRQ_TYPE_LEVEL_HIGH>;
-				reg = <0x0 0x0b121000 0x0 0x1000>,
-				      <0x0 0x0b122000 0x0 0x1000>;
+				reg = <0x0 0x0b121000 0x1000>,
+				      <0x0 0x0b122000 0x1000>;
 			};
 
 			frame@b123000 {
 				frame-number = <1>;
 				interrupts = <GIC_SPI 9 IRQ_TYPE_LEVEL_HIGH>;
-				reg = <0x0 0xb123000 0x0 0x1000>;
+				reg = <0x0 0x0b123000 0x1000>;
 				status = "disabled";
 			};
 
 			frame@b124000 {
 				frame-number = <2>;
 				interrupts = <GIC_SPI 10 IRQ_TYPE_LEVEL_HIGH>;
-				reg = <0x0 0x0b124000 0x0 0x1000>;
+				reg = <0x0 0x0b124000 0x1000>;
 				status = "disabled";
 			};
 
 			frame@b125000 {
 				frame-number = <3>;
 				interrupts = <GIC_SPI 11 IRQ_TYPE_LEVEL_HIGH>;
-				reg = <0x0 0x0b125000 0x0 0x1000>;
+				reg = <0x0 0x0b125000 0x1000>;
 				status = "disabled";
 			};
 
 			frame@b126000 {
 				frame-number = <4>;
 				interrupts = <GIC_SPI 12 IRQ_TYPE_LEVEL_HIGH>;
-				reg = <0x0 0x0b126000 0x0 0x1000>;
+				reg = <0x0 0x0b126000 0x1000>;
 				status = "disabled";
 			};
 
 			frame@b127000 {
 				frame-number = <5>;
 				interrupts = <GIC_SPI 13 IRQ_TYPE_LEVEL_HIGH>;
-				reg = <0x0 0x0b127000 0x0 0x1000>;
+				reg = <0x0 0x0b127000 0x1000>;
 				status = "disabled";
 			};
 
 			frame@b128000 {
 				frame-number = <6>;
 				interrupts = <GIC_SPI 14 IRQ_TYPE_LEVEL_HIGH>;
-				reg = <0x0 0x0b128000 0x0 0x1000>;
+				reg = <0x0 0x0b128000 0x1000>;
 				status = "disabled";
 			};
 		};
diff --git a/arch/arm64/boot/dts/qcom/sc7180.dtsi b/arch/arm64/boot/dts/qcom/sc7180.dtsi
index 5dcaac23a138..4b7be66e04df 100644
--- a/arch/arm64/boot/dts/qcom/sc7180.dtsi
+++ b/arch/arm64/boot/dts/qcom/sc7180.dtsi
@@ -3385,8 +3385,8 @@  watchdog@17c10000 {
 
 		timer@17c20000{
 			#address-cells = <2>;
-			#size-cells = <2>;
-			ranges;
+			#size-cells = <1>;
+			ranges = <0 0 0 0 0x20000000>;
 			compatible = "arm,armv7-timer-mem";
 			reg = <0 0x17c20000 0 0x1000>;
 
@@ -3394,49 +3394,49 @@  frame@17c21000 {
 				frame-number = <0>;
 				interrupts = <GIC_SPI 8 IRQ_TYPE_LEVEL_HIGH>,
 					     <GIC_SPI 6 IRQ_TYPE_LEVEL_HIGH>;
-				reg = <0 0x17c21000 0 0x1000>,
-				      <0 0x17c22000 0 0x1000>;
+				reg = <0 0x17c21000 0x1000>,
+				      <0 0x17c22000 0x1000>;
 			};
 
 			frame@17c23000 {
 				frame-number = <1>;
 				interrupts = <GIC_SPI 9 IRQ_TYPE_LEVEL_HIGH>;
-				reg = <0 0x17c23000 0 0x1000>;
+				reg = <0 0x17c23000 0x1000>;
 				status = "disabled";
 			};
 
 			frame@17c25000 {
 				frame-number = <2>;
 				interrupts = <GIC_SPI 10 IRQ_TYPE_LEVEL_HIGH>;
-				reg = <0 0x17c25000 0 0x1000>;
+				reg = <0 0x17c25000 0x1000>;
 				status = "disabled";
 			};
 
 			frame@17c27000 {
 				frame-number = <3>;
 				interrupts = <GIC_SPI 11 IRQ_TYPE_LEVEL_HIGH>;
-				reg = <0 0x17c27000 0 0x1000>;
+				reg = <0 0x17c27000 0x1000>;
 				status = "disabled";
 			};
 
 			frame@17c29000 {
 				frame-number = <4>;
 				interrupts = <GIC_SPI 12 IRQ_TYPE_LEVEL_HIGH>;
-				reg = <0 0x17c29000 0 0x1000>;
+				reg = <0 0x17c29000 0x1000>;
 				status = "disabled";
 			};
 
 			frame@17c2b000 {
 				frame-number = <5>;
 				interrupts = <GIC_SPI 13 IRQ_TYPE_LEVEL_HIGH>;
-				reg = <0 0x17c2b000 0 0x1000>;
+				reg = <0 0x17c2b000 0x1000>;
 				status = "disabled";
 			};
 
 			frame@17c2d000 {
 				frame-number = <6>;
 				interrupts = <GIC_SPI 14 IRQ_TYPE_LEVEL_HIGH>;
-				reg = <0 0x17c2d000 0 0x1000>;
+				reg = <0 0x17c2d000 0x1000>;
 				status = "disabled";
 			};
 		};
diff --git a/arch/arm64/boot/dts/qcom/sc7280.dtsi b/arch/arm64/boot/dts/qcom/sc7280.dtsi
index e66fc67de206..1b81cd1e84e9 100644
--- a/arch/arm64/boot/dts/qcom/sc7280.dtsi
+++ b/arch/arm64/boot/dts/qcom/sc7280.dtsi
@@ -4772,8 +4772,8 @@  watchdog@17c10000 {
 
 		timer@17c20000 {
 			#address-cells = <2>;
-			#size-cells = <2>;
-			ranges;
+			#size-cells = <1>;
+			ranges = <0 0 0 0 0x20000000>;
 			compatible = "arm,armv7-timer-mem";
 			reg = <0 0x17c20000 0 0x1000>;
 
@@ -4781,49 +4781,49 @@  frame@17c21000 {
 				frame-number = <0>;
 				interrupts = <GIC_SPI 8 IRQ_TYPE_LEVEL_HIGH>,
 					     <GIC_SPI 6 IRQ_TYPE_LEVEL_HIGH>;
-				reg = <0 0x17c21000 0 0x1000>,
-				      <0 0x17c22000 0 0x1000>;
+				reg = <0 0x17c21000 0x1000>,
+				      <0 0x17c22000 0x1000>;
 			};
 
 			frame@17c23000 {
 				frame-number = <1>;
 				interrupts = <GIC_SPI 9 IRQ_TYPE_LEVEL_HIGH>;
-				reg = <0 0x17c23000 0 0x1000>;
+				reg = <0 0x17c23000 0x1000>;
 				status = "disabled";
 			};
 
 			frame@17c25000 {
 				frame-number = <2>;
 				interrupts = <GIC_SPI 10 IRQ_TYPE_LEVEL_HIGH>;
-				reg = <0 0x17c25000 0 0x1000>;
+				reg = <0 0x17c25000 0x1000>;
 				status = "disabled";
 			};
 
 			frame@17c27000 {
 				frame-number = <3>;
 				interrupts = <GIC_SPI 11 IRQ_TYPE_LEVEL_HIGH>;
-				reg = <0 0x17c27000 0 0x1000>;
+				reg = <0 0x17c27000 0x1000>;
 				status = "disabled";
 			};
 
 			frame@17c29000 {
 				frame-number = <4>;
 				interrupts = <GIC_SPI 12 IRQ_TYPE_LEVEL_HIGH>;
-				reg = <0 0x17c29000 0 0x1000>;
+				reg = <0 0x17c29000 0x1000>;
 				status = "disabled";
 			};
 
 			frame@17c2b000 {
 				frame-number = <5>;
 				interrupts = <GIC_SPI 13 IRQ_TYPE_LEVEL_HIGH>;
-				reg = <0 0x17c2b000 0 0x1000>;
+				reg = <0 0x17c2b000 0x1000>;
 				status = "disabled";
 			};
 
 			frame@17c2d000 {
 				frame-number = <6>;
 				interrupts = <GIC_SPI 14 IRQ_TYPE_LEVEL_HIGH>;
-				reg = <0 0x17c2d000 0 0x1000>;
+				reg = <0 0x17c2d000 0x1000>;
 				status = "disabled";
 			};
 		};
diff --git a/arch/arm64/boot/dts/qcom/sdm845.dtsi b/arch/arm64/boot/dts/qcom/sdm845.dtsi
index 0692ae0e60a4..9a1705bfa425 100644
--- a/arch/arm64/boot/dts/qcom/sdm845.dtsi
+++ b/arch/arm64/boot/dts/qcom/sdm845.dtsi
@@ -4949,8 +4949,8 @@  slimbam: dma-controller@17184000 {
 
 		timer@17c90000 {
 			#address-cells = <2>;
-			#size-cells = <2>;
-			ranges;
+			#size-cells = <1>;
+			ranges = <0 0 0 0 0x20000000>;
 			compatible = "arm,armv7-timer-mem";
 			reg = <0 0x17c90000 0 0x1000>;
 
@@ -4958,49 +4958,49 @@  frame@17ca0000 {
 				frame-number = <0>;
 				interrupts = <GIC_SPI 7 IRQ_TYPE_LEVEL_HIGH>,
 					     <GIC_SPI 6 IRQ_TYPE_LEVEL_HIGH>;
-				reg = <0 0x17ca0000 0 0x1000>,
-				      <0 0x17cb0000 0 0x1000>;
+				reg = <0 0x17ca0000 0x1000>,
+				      <0 0x17cb0000 0x1000>;
 			};
 
 			frame@17cc0000 {
 				frame-number = <1>;
 				interrupts = <GIC_SPI 8 IRQ_TYPE_LEVEL_HIGH>;
-				reg = <0 0x17cc0000 0 0x1000>;
+				reg = <0 0x17cc0000 0x1000>;
 				status = "disabled";
 			};
 
 			frame@17cd0000 {
 				frame-number = <2>;
 				interrupts = <GIC_SPI 9 IRQ_TYPE_LEVEL_HIGH>;
-				reg = <0 0x17cd0000 0 0x1000>;
+				reg = <0 0x17cd0000 0x1000>;
 				status = "disabled";
 			};
 
 			frame@17ce0000 {
 				frame-number = <3>;
 				interrupts = <GIC_SPI 10 IRQ_TYPE_LEVEL_HIGH>;
-				reg = <0 0x17ce0000 0 0x1000>;
+				reg = <0 0x17ce0000 0x1000>;
 				status = "disabled";
 			};
 
 			frame@17cf0000 {
 				frame-number = <4>;
 				interrupts = <GIC_SPI 11 IRQ_TYPE_LEVEL_HIGH>;
-				reg = <0 0x17cf0000 0 0x1000>;
+				reg = <0 0x17cf0000 0x1000>;
 				status = "disabled";
 			};
 
 			frame@17d00000 {
 				frame-number = <5>;
 				interrupts = <GIC_SPI 12 IRQ_TYPE_LEVEL_HIGH>;
-				reg = <0 0x17d00000 0 0x1000>;
+				reg = <0 0x17d00000 0x1000>;
 				status = "disabled";
 			};
 
 			frame@17d10000 {
 				frame-number = <6>;
 				interrupts = <GIC_SPI 13 IRQ_TYPE_LEVEL_HIGH>;
-				reg = <0 0x17d10000 0 0x1000>;
+				reg = <0 0x17d10000 0x1000>;
 				status = "disabled";
 			};
 		};
diff --git a/arch/arm64/boot/dts/qcom/sm6350.dtsi b/arch/arm64/boot/dts/qcom/sm6350.dtsi
index d4f8f33f3f0c..fc473cb8c93b 100644
--- a/arch/arm64/boot/dts/qcom/sm6350.dtsi
+++ b/arch/arm64/boot/dts/qcom/sm6350.dtsi
@@ -1305,56 +1305,56 @@  timer@17c20000 {
 			reg = <0x0 0x17c20000 0x0 0x1000>;
 			clock-frequency = <19200000>;
 			#address-cells = <2>;
-			#size-cells = <2>;
-			ranges;
+			#size-cells = <1>;
+			ranges = <0 0 0 0 0x20000000>;
 
 			frame@17c21000 {
 				frame-number = <0>;
 				interrupts = <GIC_SPI 8 IRQ_TYPE_LEVEL_HIGH>,
 					     <GIC_SPI 6 IRQ_TYPE_LEVEL_HIGH>;
-				reg = <0x0 0x17c21000 0x0 0x1000>,
-				      <0x0 0x17c22000 0x0 0x1000>;
+				reg = <0x0 0x17c21000 0x1000>,
+				      <0x0 0x17c22000 0x1000>;
 			};
 
 			frame@17c23000 {
 				frame-number = <1>;
 				interrupts = <GIC_SPI 9 IRQ_TYPE_LEVEL_HIGH>;
-				reg = <0x0 0x17c23000 0x0 0x1000>;
+				reg = <0x0 0x17c23000 0x1000>;
 				status = "disabled";
 			};
 
 			frame@17c25000 {
 				frame-number = <2>;
 				interrupts = <GIC_SPI 10 IRQ_TYPE_LEVEL_HIGH>;
-				reg = <0x0 0x17c25000 0x0 0x1000>;
+				reg = <0x0 0x17c25000 0x1000>;
 				status = "disabled";
 			};
 
 			frame@17c27000 {
 				frame-number = <3>;
 				interrupts = <GIC_SPI 11 IRQ_TYPE_LEVEL_HIGH>;
-				reg = <0x0 0x17c27000 0x0 0x1000>;
+				reg = <0x0 0x17c27000 0x1000>;
 				status = "disabled";
 			};
 
 			frame@17c29000 {
 				frame-number = <4>;
 				interrupts = <GIC_SPI 12 IRQ_TYPE_LEVEL_HIGH>;
-				reg = <0x0 0x17c29000 0x0 0x1000>;
+				reg = <0x0 0x17c29000 0x1000>;
 				status = "disabled";
 			};
 
 			frame@17c2b000 {
 				frame-number = <5>;
 				interrupts = <GIC_SPI 13 IRQ_TYPE_LEVEL_HIGH>;
-				reg = <0x0 0x17c2b000 0x0 0x1000>;
+				reg = <0x0 0x17c2b000 0x1000>;
 				status = "disabled";
 			};
 
 			frame@17c2d000 {
 				frame-number = <6>;
 				interrupts = <GIC_SPI 14 IRQ_TYPE_LEVEL_HIGH>;
-				reg = <0x0 0x17c2d000 0x0 0x1000>;
+				reg = <0x0 0x17c2d000 0x1000>;
 				status = "disabled";
 			};
 		};
diff --git a/arch/arm64/boot/dts/qcom/sm8150.dtsi b/arch/arm64/boot/dts/qcom/sm8150.dtsi
index 8ea44c4b56b4..066cc00191b2 100644
--- a/arch/arm64/boot/dts/qcom/sm8150.dtsi
+++ b/arch/arm64/boot/dts/qcom/sm8150.dtsi
@@ -3945,8 +3945,8 @@  watchdog@17c10000 {
 
 		timer@17c20000 {
 			#address-cells = <2>;
-			#size-cells = <2>;
-			ranges;
+			#size-cells = <1>;
+			ranges = <0 0 0 0 0x20000000>;
 			compatible = "arm,armv7-timer-mem";
 			reg = <0x0 0x17c20000 0x0 0x1000>;
 			clock-frequency = <19200000>;
@@ -3955,49 +3955,49 @@  frame@17c21000{
 				frame-number = <0>;
 				interrupts = <GIC_SPI 8 IRQ_TYPE_LEVEL_HIGH>,
 					     <GIC_SPI 6 IRQ_TYPE_LEVEL_HIGH>;
-				reg = <0x0 0x17c21000 0x0 0x1000>,
-				      <0x0 0x17c22000 0x0 0x1000>;
+				reg = <0x0 0x17c21000 0x1000>,
+				      <0x0 0x17c22000 0x1000>;
 			};
 
 			frame@17c23000 {
 				frame-number = <1>;
 				interrupts = <GIC_SPI 9 IRQ_TYPE_LEVEL_HIGH>;
-				reg = <0x0 0x17c23000 0x0 0x1000>;
+				reg = <0x0 0x17c23000 0x1000>;
 				status = "disabled";
 			};
 
 			frame@17c25000 {
 				frame-number = <2>;
 				interrupts = <GIC_SPI 10 IRQ_TYPE_LEVEL_HIGH>;
-				reg = <0x0 0x17c25000 0x0 0x1000>;
+				reg = <0x0 0x17c25000 0x1000>;
 				status = "disabled";
 			};
 
 			frame@17c27000 {
 				frame-number = <3>;
 				interrupts = <GIC_SPI 11 IRQ_TYPE_LEVEL_HIGH>;
-				reg = <0x0 0x17c26000 0x0 0x1000>;
+				reg = <0x0 0x17c26000 0x1000>;
 				status = "disabled";
 			};
 
 			frame@17c29000 {
 				frame-number = <4>;
 				interrupts = <GIC_SPI 12 IRQ_TYPE_LEVEL_HIGH>;
-				reg = <0x0 0x17c29000 0x0 0x1000>;
+				reg = <0x0 0x17c29000 0x1000>;
 				status = "disabled";
 			};
 
 			frame@17c2b000 {
 				frame-number = <5>;
 				interrupts = <GIC_SPI 13 IRQ_TYPE_LEVEL_HIGH>;
-				reg = <0x0 0x17c2b000 0x0 0x1000>;
+				reg = <0x0 0x17c2b000 0x1000>;
 				status = "disabled";
 			};
 
 			frame@17c2d000 {
 				frame-number = <6>;
 				interrupts = <GIC_SPI 14 IRQ_TYPE_LEVEL_HIGH>;
-				reg = <0x0 0x17c2d000 0x0 0x1000>;
+				reg = <0x0 0x17c2d000 0x1000>;
 				status = "disabled";
 			};
 		};
diff --git a/arch/arm64/boot/dts/qcom/sm8250.dtsi b/arch/arm64/boot/dts/qcom/sm8250.dtsi
index cf0c97bd5ad3..ae7d0573e1dc 100644
--- a/arch/arm64/boot/dts/qcom/sm8250.dtsi
+++ b/arch/arm64/boot/dts/qcom/sm8250.dtsi
@@ -4868,8 +4868,8 @@  watchdog@17c10000 {
 
 		timer@17c20000 {
 			#address-cells = <2>;
-			#size-cells = <2>;
-			ranges;
+			#size-cells = <1>;
+			ranges = <0 0 0 0 0x20000000>;
 			compatible = "arm,armv7-timer-mem";
 			reg = <0x0 0x17c20000 0x0 0x1000>;
 			clock-frequency = <19200000>;
@@ -4878,49 +4878,49 @@  frame@17c21000 {
 				frame-number = <0>;
 				interrupts = <GIC_SPI 8 IRQ_TYPE_LEVEL_HIGH>,
 					     <GIC_SPI 6 IRQ_TYPE_LEVEL_HIGH>;
-				reg = <0x0 0x17c21000 0x0 0x1000>,
-				      <0x0 0x17c22000 0x0 0x1000>;
+				reg = <0x0 0x17c21000 0x1000>,
+				      <0x0 0x17c22000 0x1000>;
 			};
 
 			frame@17c23000 {
 				frame-number = <1>;
 				interrupts = <GIC_SPI 9 IRQ_TYPE_LEVEL_HIGH>;
-				reg = <0x0 0x17c23000 0x0 0x1000>;
+				reg = <0x0 0x17c23000 0x1000>;
 				status = "disabled";
 			};
 
 			frame@17c25000 {
 				frame-number = <2>;
 				interrupts = <GIC_SPI 10 IRQ_TYPE_LEVEL_HIGH>;
-				reg = <0x0 0x17c25000 0x0 0x1000>;
+				reg = <0x0 0x17c25000 0x1000>;
 				status = "disabled";
 			};
 
 			frame@17c27000 {
 				frame-number = <3>;
 				interrupts = <GIC_SPI 11 IRQ_TYPE_LEVEL_HIGH>;
-				reg = <0x0 0x17c27000 0x0 0x1000>;
+				reg = <0x0 0x17c27000 0x1000>;
 				status = "disabled";
 			};
 
 			frame@17c29000 {
 				frame-number = <4>;
 				interrupts = <GIC_SPI 12 IRQ_TYPE_LEVEL_HIGH>;
-				reg = <0x0 0x17c29000 0x0 0x1000>;
+				reg = <0x0 0x17c29000 0x1000>;
 				status = "disabled";
 			};
 
 			frame@17c2b000 {
 				frame-number = <5>;
 				interrupts = <GIC_SPI 13 IRQ_TYPE_LEVEL_HIGH>;
-				reg = <0x0 0x17c2b000 0x0 0x1000>;
+				reg = <0x0 0x17c2b000 0x1000>;
 				status = "disabled";
 			};
 
 			frame@17c2d000 {
 				frame-number = <6>;
 				interrupts = <GIC_SPI 14 IRQ_TYPE_LEVEL_HIGH>;
-				reg = <0x0 0x17c2d000 0x0 0x1000>;
+				reg = <0x0 0x17c2d000 0x1000>;
 				status = "disabled";
 			};
 		};
diff --git a/arch/arm64/boot/dts/qcom/sm8350.dtsi b/arch/arm64/boot/dts/qcom/sm8350.dtsi
index 743cba9b683c..58e6bb4a1899 100644
--- a/arch/arm64/boot/dts/qcom/sm8350.dtsi
+++ b/arch/arm64/boot/dts/qcom/sm8350.dtsi
@@ -1934,8 +1934,8 @@  intc: interrupt-controller@17a00000 {
 		timer@17c20000 {
 			compatible = "arm,armv7-timer-mem";
 			#address-cells = <2>;
-			#size-cells = <2>;
-			ranges;
+			#size-cells = <1>;
+			ranges = <0 0 0 0 0x20000000>;
 			reg = <0x0 0x17c20000 0x0 0x1000>;
 			clock-frequency = <19200000>;
 
@@ -1943,49 +1943,49 @@  frame@17c21000 {
 				frame-number = <0>;
 				interrupts = <GIC_SPI 8 IRQ_TYPE_LEVEL_HIGH>,
 					     <GIC_SPI 6 IRQ_TYPE_LEVEL_HIGH>;
-				reg = <0x0 0x17c21000 0x0 0x1000>,
-				      <0x0 0x17c22000 0x0 0x1000>;
+				reg = <0x0 0x17c21000 0x1000>,
+				      <0x0 0x17c22000 0x1000>;
 			};
 
 			frame@17c23000 {
 				frame-number = <1>;
 				interrupts = <GIC_SPI 9 IRQ_TYPE_LEVEL_HIGH>;
-				reg = <0x0 0x17c23000 0x0 0x1000>;
+				reg = <0x0 0x17c23000 0x1000>;
 				status = "disabled";
 			};
 
 			frame@17c25000 {
 				frame-number = <2>;
 				interrupts = <GIC_SPI 10 IRQ_TYPE_LEVEL_HIGH>;
-				reg = <0x0 0x17c25000 0x0 0x1000>;
+				reg = <0x0 0x17c25000 0x1000>;
 				status = "disabled";
 			};
 
 			frame@17c27000 {
 				frame-number = <3>;
 				interrupts = <GIC_SPI 11 IRQ_TYPE_LEVEL_HIGH>;
-				reg = <0x0 0x17c27000 0x0 0x1000>;
+				reg = <0x0 0x17c27000 0x1000>;
 				status = "disabled";
 			};
 
 			frame@17c29000 {
 				frame-number = <4>;
 				interrupts = <GIC_SPI 12 IRQ_TYPE_LEVEL_HIGH>;
-				reg = <0x0 0x17c29000 0x0 0x1000>;
+				reg = <0x0 0x17c29000 0x1000>;
 				status = "disabled";
 			};
 
 			frame@17c2b000 {
 				frame-number = <5>;
 				interrupts = <GIC_SPI 13 IRQ_TYPE_LEVEL_HIGH>;
-				reg = <0x0 0x17c2b000 0x0 0x1000>;
+				reg = <0x0 0x17c2b000 0x1000>;
 				status = "disabled";
 			};
 
 			frame@17c2d000 {
 				frame-number = <6>;
 				interrupts = <GIC_SPI 14 IRQ_TYPE_LEVEL_HIGH>;
-				reg = <0x0 0x17c2d000 0x0 0x1000>;
+				reg = <0x0 0x17c2d000 0x1000>;
 				status = "disabled";
 			};
 		};
diff --git a/arch/arm64/boot/dts/qcom/sm8450.dtsi b/arch/arm64/boot/dts/qcom/sm8450.dtsi
index 7d08fad76371..64ceff515337 100644
--- a/arch/arm64/boot/dts/qcom/sm8450.dtsi
+++ b/arch/arm64/boot/dts/qcom/sm8450.dtsi
@@ -2858,8 +2858,8 @@  intc: interrupt-controller@17100000 {
 		timer@17420000 {
 			compatible = "arm,armv7-timer-mem";
 			#address-cells = <2>;
-			#size-cells = <2>;
-			ranges;
+			#size-cells = <1>;
+			ranges = <0 0 0 0 0x20000000>;
 			reg = <0x0 0x17420000 0x0 0x1000>;
 			clock-frequency = <19200000>;
 
@@ -2867,49 +2867,49 @@  frame@17421000 {
 				frame-number = <0>;
 				interrupts = <GIC_SPI 8 IRQ_TYPE_LEVEL_HIGH>,
 					     <GIC_SPI 6 IRQ_TYPE_LEVEL_HIGH>;
-				reg = <0x0 0x17421000 0x0 0x1000>,
-				      <0x0 0x17422000 0x0 0x1000>;
+				reg = <0x0 0x17421000 0x1000>,
+				      <0x0 0x17422000 0x1000>;
 			};
 
 			frame@17423000 {
 				frame-number = <1>;
 				interrupts = <GIC_SPI 9 IRQ_TYPE_LEVEL_HIGH>;
-				reg = <0x0 0x17423000 0x0 0x1000>;
+				reg = <0x0 0x17423000 0x1000>;
 				status = "disabled";
 			};
 
 			frame@17425000 {
 				frame-number = <2>;
 				interrupts = <GIC_SPI 10 IRQ_TYPE_LEVEL_HIGH>;
-				reg = <0x0 0x17425000 0x0 0x1000>;
+				reg = <0x0 0x17425000 0x1000>;
 				status = "disabled";
 			};
 
 			frame@17427000 {
 				frame-number = <3>;
 				interrupts = <GIC_SPI 11 IRQ_TYPE_LEVEL_HIGH>;
-				reg = <0x0 0x17427000 0x0 0x1000>;
+				reg = <0x0 0x17427000 0x1000>;
 				status = "disabled";
 			};
 
 			frame@17429000 {
 				frame-number = <4>;
 				interrupts = <GIC_SPI 12 IRQ_TYPE_LEVEL_HIGH>;
-				reg = <0x0 0x17429000 0x0 0x1000>;
+				reg = <0x0 0x17429000 0x1000>;
 				status = "disabled";
 			};
 
 			frame@1742b000 {
 				frame-number = <5>;
 				interrupts = <GIC_SPI 13 IRQ_TYPE_LEVEL_HIGH>;
-				reg = <0x0 0x1742b000 0x0 0x1000>;
+				reg = <0x0 0x1742b000 0x1000>;
 				status = "disabled";
 			};
 
 			frame@1742d000 {
 				frame-number = <6>;
 				interrupts = <GIC_SPI 14 IRQ_TYPE_LEVEL_HIGH>;
-				reg = <0x0 0x1742d000 0x0 0x1000>;
+				reg = <0x0 0x1742d000 0x1000>;
 				status = "disabled";
 			};
 		};