diff mbox series

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

Message ID 20201130060320.GA30098@anyang-linuxfactory-or-kr (mailing list archive)
State New, archived
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(-)

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>;
 		};