Re: [PATCH V2 1/2] arm64: dts: imx8mm: Correct OPP table according to latest datasheet

2019-07-18 Thread Shawn Guo 
On Sat, Jun 29, 2019 at 06:21:56PM +0800, anson.hu...@nxp.com wrote:
> From: Anson Huang 
> 
> According to latest datasheet (Rev.0.2, 04/2019) from below links,
> 1.8GHz is ONLY available for consumer part, so the market segment
> bits for 1.8GHz opp should ONLY available for consumer part accordingly.
> 
> https://www.nxp.com/docs/en/data-sheet/IMX8MMIEC.pdf
> https://www.nxp.com/docs/en/data-sheet/IMX8MMCEC.pdf
> 
> Fixes: f403a26c865b (arm64: dts: imx8mm: Add cpu speed grading and all OPPs)
> Signed-off-by: Anson Huang 

Applied both, thanks.

Re: [PATCH V2 1/2] arm64: dts: imx8mm: Correct OPP table according to latest datasheet

2019-07-01 Thread Leonard Crestez 
On 6/29/2019 1:31 PM, anson.hu...@nxp.com wrote:
> From: Anson Huang 
> 
> According to latest datasheet (Rev.0.2, 04/2019) from below links,
> 1.8GHz is ONLY available for consumer part, so the market segment
> bits for 1.8GHz opp should ONLY available for consumer part accordingly.
>  > Fixes: f403a26c865b (arm64: dts: imx8mm: Add cpu speed grading and 
all OPPs)
> Signed-off-by: Anson Huang 

For both:
Reviewed-by: Leonard Crestez 

The vendor tree goes through a lot of testing so switching to the exact 
speed grading interpretation from there does make sense.

[PATCH V2 1/2] arm64: dts: imx8mm: Correct OPP table according to latest datasheet

2019-06-29 Thread Anson . Huang 
From: Anson Huang 

According to latest datasheet (Rev.0.2, 04/2019) from below links,
1.8GHz is ONLY available for consumer part, so the market segment
bits for 1.8GHz opp should ONLY available for consumer part accordingly.

https://www.nxp.com/docs/en/data-sheet/IMX8MMIEC.pdf
https://www.nxp.com/docs/en/data-sheet/IMX8MMCEC.pdf

Fixes: f403a26c865b (arm64: dts: imx8mm: Add cpu speed grading and all OPPs)
Signed-off-by: Anson Huang 
---
Changes since V1:
- remove the comment to avoid any confusion.
---
 arch/arm64/boot/dts/freescale/imx8mm.dtsi | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/arch/arm64/boot/dts/freescale/imx8mm.dtsi 
b/arch/arm64/boot/dts/freescale/imx8mm.dtsi
index 70de15c..f0ac027 100644
--- a/arch/arm64/boot/dts/freescale/imx8mm.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx8mm.dtsi
@@ -134,8 +134,7 @@
opp-18 {
opp-hz = /bits/ 64 <18>;
opp-microvolt = <100>;
-   /* Consumer only but rely on speed grading */
-   opp-supported-hw = <0x8>, <0x7>;
+   opp-supported-hw = <0x8>, <0x3>;
clock-latency-ns = <15>;
};
};
-- 
2.7.4