| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-ID: <YrfOn6T+cUn5mEe3@builder.lan>
Date: Sat, 25 Jun 2022 22:12:31 -0500
From: Bjorn Andersson <bjorn.andersson@...aro.org>
To: David Heidelberg <david@...t.cz>
Cc: Andy Gross <agross@...nel.org>, Rob Herring <robh+dt@...nel.org>,
Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>,
linux-arm-msm@...r.kernel.org, devicetree@...r.kernel.org,
linux-kernel@...r.kernel.org
Subject: Re: [PATCH v2] arm64: dts: qcom: timer should use only 32-bit size
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@...90000: #size-cells:0:0: 1 was expected
> From schema: Documentation/devicetree/bindings/timer/arm,arch_timer_mmio.yaml
> ```
>
> Signed-off-by: David Heidelberg <david@...t.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@...0000 {
> #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@...0000 {
> 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@...3000 {
> frame-number = <1>;
> interrupts = <GIC_SPI 9 IRQ_TYPE_LEVEL_HIGH>;
> - reg = <0x0 0xb123000 0x0 0x1000>;
> + reg = <0x0 0x0b123000 0x1000>;
> status = "disabled";
> };
>
> frame@...4000 {
> frame-number = <2>;
> interrupts = <GIC_SPI 10 IRQ_TYPE_LEVEL_HIGH>;
> - reg = <0x0 0x0b124000 0x0 0x1000>;
> + reg = <0x0 0x0b124000 0x1000>;
> status = "disabled";
> };
>
> frame@...5000 {
> frame-number = <3>;
> interrupts = <GIC_SPI 11 IRQ_TYPE_LEVEL_HIGH>;
> - reg = <0x0 0x0b125000 0x0 0x1000>;
> + reg = <0x0 0x0b125000 0x1000>;
> status = "disabled";
> };
>
> frame@...6000 {
> frame-number = <4>;
> interrupts = <GIC_SPI 12 IRQ_TYPE_LEVEL_HIGH>;
> - reg = <0x0 0x0b126000 0x0 0x1000>;
> + reg = <0x0 0x0b126000 0x1000>;
> status = "disabled";
> };
>
> frame@...7000 {
> frame-number = <5>;
> interrupts = <GIC_SPI 13 IRQ_TYPE_LEVEL_HIGH>;
> - reg = <0x0 0x0b127000 0x0 0x1000>;
> + reg = <0x0 0x0b127000 0x1000>;
> status = "disabled";
> };
>
> frame@...8000 {
> 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@...10000 {
>
> timer@...20000{
> #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@...21000 {
> 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@...23000 {
> frame-number = <1>;
> interrupts = <GIC_SPI 9 IRQ_TYPE_LEVEL_HIGH>;
> - reg = <0 0x17c23000 0 0x1000>;
> + reg = <0 0x17c23000 0x1000>;
> status = "disabled";
> };
>
> frame@...25000 {
> frame-number = <2>;
> interrupts = <GIC_SPI 10 IRQ_TYPE_LEVEL_HIGH>;
> - reg = <0 0x17c25000 0 0x1000>;
> + reg = <0 0x17c25000 0x1000>;
> status = "disabled";
> };
>
> frame@...27000 {
> frame-number = <3>;
> interrupts = <GIC_SPI 11 IRQ_TYPE_LEVEL_HIGH>;
> - reg = <0 0x17c27000 0 0x1000>;
> + reg = <0 0x17c27000 0x1000>;
> status = "disabled";
> };
>
> frame@...29000 {
> frame-number = <4>;
> interrupts = <GIC_SPI 12 IRQ_TYPE_LEVEL_HIGH>;
> - reg = <0 0x17c29000 0 0x1000>;
> + reg = <0 0x17c29000 0x1000>;
> status = "disabled";
> };
>
> frame@...2b000 {
> frame-number = <5>;
> interrupts = <GIC_SPI 13 IRQ_TYPE_LEVEL_HIGH>;
> - reg = <0 0x17c2b000 0 0x1000>;
> + reg = <0 0x17c2b000 0x1000>;
> status = "disabled";
> };
>
> frame@...2d000 {
> 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@...10000 {
>
> timer@...20000 {
> #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@...21000 {
> 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@...23000 {
> frame-number = <1>;
> interrupts = <GIC_SPI 9 IRQ_TYPE_LEVEL_HIGH>;
> - reg = <0 0x17c23000 0 0x1000>;
> + reg = <0 0x17c23000 0x1000>;
> status = "disabled";
> };
>
> frame@...25000 {
> frame-number = <2>;
> interrupts = <GIC_SPI 10 IRQ_TYPE_LEVEL_HIGH>;
> - reg = <0 0x17c25000 0 0x1000>;
> + reg = <0 0x17c25000 0x1000>;
> status = "disabled";
> };
>
> frame@...27000 {
> frame-number = <3>;
> interrupts = <GIC_SPI 11 IRQ_TYPE_LEVEL_HIGH>;
> - reg = <0 0x17c27000 0 0x1000>;
> + reg = <0 0x17c27000 0x1000>;
> status = "disabled";
> };
>
> frame@...29000 {
> frame-number = <4>;
> interrupts = <GIC_SPI 12 IRQ_TYPE_LEVEL_HIGH>;
> - reg = <0 0x17c29000 0 0x1000>;
> + reg = <0 0x17c29000 0x1000>;
> status = "disabled";
> };
>
> frame@...2b000 {
> frame-number = <5>;
> interrupts = <GIC_SPI 13 IRQ_TYPE_LEVEL_HIGH>;
> - reg = <0 0x17c2b000 0 0x1000>;
> + reg = <0 0x17c2b000 0x1000>;
> status = "disabled";
> };
>
> frame@...2d000 {
> 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@...84000 {
>
> timer@...90000 {
> #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@...a0000 {
> 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@...c0000 {
> frame-number = <1>;
> interrupts = <GIC_SPI 8 IRQ_TYPE_LEVEL_HIGH>;
> - reg = <0 0x17cc0000 0 0x1000>;
> + reg = <0 0x17cc0000 0x1000>;
> status = "disabled";
> };
>
> frame@...d0000 {
> frame-number = <2>;
> interrupts = <GIC_SPI 9 IRQ_TYPE_LEVEL_HIGH>;
> - reg = <0 0x17cd0000 0 0x1000>;
> + reg = <0 0x17cd0000 0x1000>;
> status = "disabled";
> };
>
> frame@...e0000 {
> frame-number = <3>;
> interrupts = <GIC_SPI 10 IRQ_TYPE_LEVEL_HIGH>;
> - reg = <0 0x17ce0000 0 0x1000>;
> + reg = <0 0x17ce0000 0x1000>;
> status = "disabled";
> };
>
> frame@...f0000 {
> frame-number = <4>;
> interrupts = <GIC_SPI 11 IRQ_TYPE_LEVEL_HIGH>;
> - reg = <0 0x17cf0000 0 0x1000>;
> + reg = <0 0x17cf0000 0x1000>;
> status = "disabled";
> };
>
> frame@...00000 {
> frame-number = <5>;
> interrupts = <GIC_SPI 12 IRQ_TYPE_LEVEL_HIGH>;
> - reg = <0 0x17d00000 0 0x1000>;
> + reg = <0 0x17d00000 0x1000>;
> status = "disabled";
> };
>
> frame@...10000 {
> 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@...20000 {
> 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@...21000 {
> 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@...23000 {
> frame-number = <1>;
> interrupts = <GIC_SPI 9 IRQ_TYPE_LEVEL_HIGH>;
> - reg = <0x0 0x17c23000 0x0 0x1000>;
> + reg = <0x0 0x17c23000 0x1000>;
> status = "disabled";
> };
>
> frame@...25000 {
> frame-number = <2>;
> interrupts = <GIC_SPI 10 IRQ_TYPE_LEVEL_HIGH>;
> - reg = <0x0 0x17c25000 0x0 0x1000>;
> + reg = <0x0 0x17c25000 0x1000>;
> status = "disabled";
> };
>
> frame@...27000 {
> frame-number = <3>;
> interrupts = <GIC_SPI 11 IRQ_TYPE_LEVEL_HIGH>;
> - reg = <0x0 0x17c27000 0x0 0x1000>;
> + reg = <0x0 0x17c27000 0x1000>;
> status = "disabled";
> };
>
> frame@...29000 {
> frame-number = <4>;
> interrupts = <GIC_SPI 12 IRQ_TYPE_LEVEL_HIGH>;
> - reg = <0x0 0x17c29000 0x0 0x1000>;
> + reg = <0x0 0x17c29000 0x1000>;
> status = "disabled";
> };
>
> frame@...2b000 {
> frame-number = <5>;
> interrupts = <GIC_SPI 13 IRQ_TYPE_LEVEL_HIGH>;
> - reg = <0x0 0x17c2b000 0x0 0x1000>;
> + reg = <0x0 0x17c2b000 0x1000>;
> status = "disabled";
> };
>
> frame@...2d000 {
> 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@...10000 {
>
> timer@...20000 {
> #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@...21000{
> 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@...23000 {
> frame-number = <1>;
> interrupts = <GIC_SPI 9 IRQ_TYPE_LEVEL_HIGH>;
> - reg = <0x0 0x17c23000 0x0 0x1000>;
> + reg = <0x0 0x17c23000 0x1000>;
> status = "disabled";
> };
>
> frame@...25000 {
> frame-number = <2>;
> interrupts = <GIC_SPI 10 IRQ_TYPE_LEVEL_HIGH>;
> - reg = <0x0 0x17c25000 0x0 0x1000>;
> + reg = <0x0 0x17c25000 0x1000>;
> status = "disabled";
> };
>
> frame@...27000 {
> frame-number = <3>;
> interrupts = <GIC_SPI 11 IRQ_TYPE_LEVEL_HIGH>;
> - reg = <0x0 0x17c26000 0x0 0x1000>;
> + reg = <0x0 0x17c26000 0x1000>;
> status = "disabled";
> };
>
> frame@...29000 {
> frame-number = <4>;
> interrupts = <GIC_SPI 12 IRQ_TYPE_LEVEL_HIGH>;
> - reg = <0x0 0x17c29000 0x0 0x1000>;
> + reg = <0x0 0x17c29000 0x1000>;
> status = "disabled";
> };
>
> frame@...2b000 {
> frame-number = <5>;
> interrupts = <GIC_SPI 13 IRQ_TYPE_LEVEL_HIGH>;
> - reg = <0x0 0x17c2b000 0x0 0x1000>;
> + reg = <0x0 0x17c2b000 0x1000>;
> status = "disabled";
> };
>
> frame@...2d000 {
> 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@...10000 {
>
> timer@...20000 {
> #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@...21000 {
> 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@...23000 {
> frame-number = <1>;
> interrupts = <GIC_SPI 9 IRQ_TYPE_LEVEL_HIGH>;
> - reg = <0x0 0x17c23000 0x0 0x1000>;
> + reg = <0x0 0x17c23000 0x1000>;
> status = "disabled";
> };
>
> frame@...25000 {
> frame-number = <2>;
> interrupts = <GIC_SPI 10 IRQ_TYPE_LEVEL_HIGH>;
> - reg = <0x0 0x17c25000 0x0 0x1000>;
> + reg = <0x0 0x17c25000 0x1000>;
> status = "disabled";
> };
>
> frame@...27000 {
> frame-number = <3>;
> interrupts = <GIC_SPI 11 IRQ_TYPE_LEVEL_HIGH>;
> - reg = <0x0 0x17c27000 0x0 0x1000>;
> + reg = <0x0 0x17c27000 0x1000>;
> status = "disabled";
> };
>
> frame@...29000 {
> frame-number = <4>;
> interrupts = <GIC_SPI 12 IRQ_TYPE_LEVEL_HIGH>;
> - reg = <0x0 0x17c29000 0x0 0x1000>;
> + reg = <0x0 0x17c29000 0x1000>;
> status = "disabled";
> };
>
> frame@...2b000 {
> frame-number = <5>;
> interrupts = <GIC_SPI 13 IRQ_TYPE_LEVEL_HIGH>;
> - reg = <0x0 0x17c2b000 0x0 0x1000>;
> + reg = <0x0 0x17c2b000 0x1000>;
> status = "disabled";
> };
>
> frame@...2d000 {
> 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@...00000 {
> timer@...20000 {
> 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@...21000 {
> 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@...23000 {
> frame-number = <1>;
> interrupts = <GIC_SPI 9 IRQ_TYPE_LEVEL_HIGH>;
> - reg = <0x0 0x17c23000 0x0 0x1000>;
> + reg = <0x0 0x17c23000 0x1000>;
> status = "disabled";
> };
>
> frame@...25000 {
> frame-number = <2>;
> interrupts = <GIC_SPI 10 IRQ_TYPE_LEVEL_HIGH>;
> - reg = <0x0 0x17c25000 0x0 0x1000>;
> + reg = <0x0 0x17c25000 0x1000>;
> status = "disabled";
> };
>
> frame@...27000 {
> frame-number = <3>;
> interrupts = <GIC_SPI 11 IRQ_TYPE_LEVEL_HIGH>;
> - reg = <0x0 0x17c27000 0x0 0x1000>;
> + reg = <0x0 0x17c27000 0x1000>;
> status = "disabled";
> };
>
> frame@...29000 {
> frame-number = <4>;
> interrupts = <GIC_SPI 12 IRQ_TYPE_LEVEL_HIGH>;
> - reg = <0x0 0x17c29000 0x0 0x1000>;
> + reg = <0x0 0x17c29000 0x1000>;
> status = "disabled";
> };
>
> frame@...2b000 {
> frame-number = <5>;
> interrupts = <GIC_SPI 13 IRQ_TYPE_LEVEL_HIGH>;
> - reg = <0x0 0x17c2b000 0x0 0x1000>;
> + reg = <0x0 0x17c2b000 0x1000>;
> status = "disabled";
> };
>
> frame@...2d000 {
> 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@...00000 {
> timer@...20000 {
> 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@...21000 {
> 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@...23000 {
> frame-number = <1>;
> interrupts = <GIC_SPI 9 IRQ_TYPE_LEVEL_HIGH>;
> - reg = <0x0 0x17423000 0x0 0x1000>;
> + reg = <0x0 0x17423000 0x1000>;
> status = "disabled";
> };
>
> frame@...25000 {
> frame-number = <2>;
> interrupts = <GIC_SPI 10 IRQ_TYPE_LEVEL_HIGH>;
> - reg = <0x0 0x17425000 0x0 0x1000>;
> + reg = <0x0 0x17425000 0x1000>;
> status = "disabled";
> };
>
> frame@...27000 {
> frame-number = <3>;
> interrupts = <GIC_SPI 11 IRQ_TYPE_LEVEL_HIGH>;
> - reg = <0x0 0x17427000 0x0 0x1000>;
> + reg = <0x0 0x17427000 0x1000>;
> status = "disabled";
> };
>
> frame@...29000 {
> frame-number = <4>;
> interrupts = <GIC_SPI 12 IRQ_TYPE_LEVEL_HIGH>;
> - reg = <0x0 0x17429000 0x0 0x1000>;
> + reg = <0x0 0x17429000 0x1000>;
> status = "disabled";
> };
>
> frame@...2b000 {
> frame-number = <5>;
> interrupts = <GIC_SPI 13 IRQ_TYPE_LEVEL_HIGH>;
> - reg = <0x0 0x1742b000 0x0 0x1000>;
> + reg = <0x0 0x1742b000 0x1000>;
> status = "disabled";
> };
>
> frame@...2d000 {
> 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
>
Powered by blists - more mailing lists