diff mbox series

[v2] arm64: dts: meson-sm1: fix typo in opp table

Message ID 20201130060320.GA30098@anyang-linuxfactory-or-kr
State Accepted
Commit b6a1c8a1eaa73b1e2ae251399308e9445d74cef7
Headers show
Series [v2] arm64: dts: meson-sm1: fix typo in opp table | expand

Commit Message

Dongjin Kim Nov. 30, 2020, 6:03 a.m. UTC
The freqency 1512000000 should be 1500000000.

Signed-off-by: Dongjin Kim <tobetter@gmail.com>

---

Change in v2:
  - wrong fix in the previous patch.
    https://patchwork.kernel.org/project/linux-amlogic/patch/20201130054221.GA25448@anyang-linuxfactory-or-kr/ 

---
 arch/arm64/boot/dts/amlogic/meson-sm1.dtsi | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

-- 
2.17.1

Comments

Neil Armstrong Nov. 30, 2020, 1:23 p.m. UTC | #1
On 30/11/2020 07:03, Dongjin Kim wrote:
> The freqency 1512000000 should be 1500000000.

> 


Fixes: 3d9e76483049 ("arm64: dts: meson-sm1-sei610: enable DVFS")

> Signed-off-by: Dongjin Kim <tobetter@gmail.com>

> ---

> 

> Change in v2:

>   - wrong fix in the previous patch.

>     https://patchwork.kernel.org/project/linux-amlogic/patch/20201130054221.GA25448@anyang-linuxfactory-or-kr/ 

> 

> ---

>  arch/arm64/boot/dts/amlogic/meson-sm1.dtsi | 2 +-

>  1 file changed, 1 insertion(+), 1 deletion(-)

> 

> diff --git a/arch/arm64/boot/dts/amlogic/meson-sm1.dtsi b/arch/arm64/boot/dts/amlogic/meson-sm1.dtsi

> index 71317f5aada1..c309517abae3 100644

> --- a/arch/arm64/boot/dts/amlogic/meson-sm1.dtsi

> +++ b/arch/arm64/boot/dts/amlogic/meson-sm1.dtsi

> @@ -130,7 +130,7 @@

>  			opp-microvolt = <790000>;

>  		};

>  

> -		opp-1512000000 {

> +		opp-1500000000 {

>  			opp-hz = /bits/ 64 <1500000000>;

>  			opp-microvolt = <800000>;

>  		};

> 


Reviewed-by: Neil Armstrong <narmstrong@baylibre.com>
Martin Blumenstingl Dec. 5, 2020, noon UTC | #2
On Mon, Nov 30, 2020 at 7:04 AM Dongjin Kim <tobetter@gmail.com> wrote:
>

> The freqency 1512000000 should be 1500000000.

there's a typo in "frequency". I hope that Kevin can fix this up while
applying the patch

>

> Signed-off-by: Dongjin Kim <tobetter@gmail.com>

apart from the typo above:
Reviewed-by: Martin Blumenstingl <martin.blumenstingl@googlemail.com>

(while reviewing I noticed that we generally use 0.3V less for each
OPP table step than the downstream kvim3l.dts - that's a discussion we
need to have elsewhere though)
diff mbox series

Patch

diff --git a/arch/arm64/boot/dts/amlogic/meson-sm1.dtsi b/arch/arm64/boot/dts/amlogic/meson-sm1.dtsi
index 71317f5aada1..c309517abae3 100644
--- a/arch/arm64/boot/dts/amlogic/meson-sm1.dtsi
+++ b/arch/arm64/boot/dts/amlogic/meson-sm1.dtsi
@@ -130,7 +130,7 @@ 
 			opp-microvolt = <790000>;
 		};
 
-		opp-1512000000 {
+		opp-1500000000 {
 			opp-hz = /bits/ 64 <1500000000>;
 			opp-microvolt = <800000>;
 		};