* [PATCH 1/2] ARM: dts: imx7d: Correct speed grading fuse settings
@ 2019-09-12 2:56 Anson Huang
2019-09-12 2:56 ` [PATCH 2/2] ARM: dts: imx7d: Add opp-suspend property Anson Huang
` (2 more replies)
0 siblings, 3 replies; 6+ messages in thread
From: Anson Huang @ 2019-09-12 2:56 UTC (permalink / raw)
To: robh+dt, mark.rutland, shawnguo, s.hauer, kernel, festevam,
devicetree, linux-arm-kernel, linux-kernel
Cc: Linux-imx
The 800MHz opp speed grading fuse mask should be 0xd instead
of 0xf according to fuse map definition:
SPEED_GRADING[1:0] MHz
00 800
01 500
10 1000
11 1200
Signed-off-by: Anson Huang <Anson.Huang@nxp.com>
---
arch/arm/boot/dts/imx7d.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/boot/dts/imx7d.dtsi b/arch/arm/boot/dts/imx7d.dtsi
index 9c8dd32..0083272 100644
--- a/arch/arm/boot/dts/imx7d.dtsi
+++ b/arch/arm/boot/dts/imx7d.dtsi
@@ -43,7 +43,7 @@
opp-hz = /bits/ 64 <792000000>;
opp-microvolt = <1000000>;
clock-latency-ns = <150000>;
- opp-supported-hw = <0xf>, <0xf>;
+ opp-supported-hw = <0xd>, <0xf>;
};
opp-996000000 {
--
2.7.4
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [PATCH 2/2] ARM: dts: imx7d: Add opp-suspend property
2019-09-12 2:56 [PATCH 1/2] ARM: dts: imx7d: Correct speed grading fuse settings Anson Huang
@ 2019-09-12 2:56 ` Anson Huang
2019-09-12 12:46 ` Leonard Crestez
2019-09-12 12:45 ` [PATCH 1/2] ARM: dts: imx7d: Correct speed grading fuse settings Leonard Crestez
2019-10-06 8:09 ` Shawn Guo
2 siblings, 1 reply; 6+ messages in thread
From: Anson Huang @ 2019-09-12 2:56 UTC (permalink / raw)
To: robh+dt, mark.rutland, shawnguo, s.hauer, kernel, festevam,
devicetree, linux-arm-kernel, linux-kernel
Cc: Linux-imx
Add "opp-suspend" property for i.MX7D to make sure system
suspend with max available opp.
Signed-off-by: Anson Huang <Anson.Huang@nxp.com>
---
arch/arm/boot/dts/imx7d.dtsi | 3 +++
1 file changed, 3 insertions(+)
diff --git a/arch/arm/boot/dts/imx7d.dtsi b/arch/arm/boot/dts/imx7d.dtsi
index 0083272..2792767 100644
--- a/arch/arm/boot/dts/imx7d.dtsi
+++ b/arch/arm/boot/dts/imx7d.dtsi
@@ -44,6 +44,7 @@
opp-microvolt = <1000000>;
clock-latency-ns = <150000>;
opp-supported-hw = <0xd>, <0xf>;
+ opp-suspend;
};
opp-996000000 {
@@ -51,6 +52,7 @@
opp-microvolt = <1100000>;
clock-latency-ns = <150000>;
opp-supported-hw = <0xc>, <0xf>;
+ opp-suspend;
};
opp-1200000000 {
@@ -58,6 +60,7 @@
opp-microvolt = <1225000>;
clock-latency-ns = <150000>;
opp-supported-hw = <0x8>, <0xf>;
+ opp-suspend;
};
};
--
2.7.4
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH 1/2] ARM: dts: imx7d: Correct speed grading fuse settings
2019-09-12 2:56 [PATCH 1/2] ARM: dts: imx7d: Correct speed grading fuse settings Anson Huang
2019-09-12 2:56 ` [PATCH 2/2] ARM: dts: imx7d: Add opp-suspend property Anson Huang
@ 2019-09-12 12:45 ` Leonard Crestez
2019-09-16 2:55 ` Anson Huang
2019-10-06 8:09 ` Shawn Guo
2 siblings, 1 reply; 6+ messages in thread
From: Leonard Crestez @ 2019-09-12 12:45 UTC (permalink / raw)
To: Anson Huang, shawnguo
Cc: robh+dt, mark.rutland, s.hauer, kernel, festevam, devicetree,
linux-arm-kernel, linux-kernel, dl-linux-imx
On 2019-09-12 5:57 AM, Anson Huang wrote:
> The 800MHz opp speed grading fuse mask should be 0xd instead
> of 0xf according to fuse map definition:
>
> SPEED_GRADING[1:0] MHz
> 00 800
> 01 500
> 10 1000
> 11 1200
>
> Signed-off-by: Anson Huang <Anson.Huang@nxp.com>
Reviewed-by: Leonard Crestez <leonard.crestez@nxp.com>
Are you going to add the 500mhz OPP as well?
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 2/2] ARM: dts: imx7d: Add opp-suspend property
2019-09-12 2:56 ` [PATCH 2/2] ARM: dts: imx7d: Add opp-suspend property Anson Huang
@ 2019-09-12 12:46 ` Leonard Crestez
0 siblings, 0 replies; 6+ messages in thread
From: Leonard Crestez @ 2019-09-12 12:46 UTC (permalink / raw)
To: Anson Huang, shawnguo
Cc: robh+dt, mark.rutland, s.hauer, kernel, festevam, devicetree,
linux-arm-kernel, linux-kernel, dl-linux-imx
On 2019-09-12 5:57 AM, Anson Huang wrote:
> Add "opp-suspend" property for i.MX7D to make sure system
> suspend with max available opp.
>
> Signed-off-by: Anson Huang <Anson.Huang@nxp.com>
Reviewed-by: Leonard Crestez <leonard.crestez@nxp.com>
^ permalink raw reply [flat|nested] 6+ messages in thread
* RE: [PATCH 1/2] ARM: dts: imx7d: Correct speed grading fuse settings
2019-09-12 12:45 ` [PATCH 1/2] ARM: dts: imx7d: Correct speed grading fuse settings Leonard Crestez
@ 2019-09-16 2:55 ` Anson Huang
0 siblings, 0 replies; 6+ messages in thread
From: Anson Huang @ 2019-09-16 2:55 UTC (permalink / raw)
To: Leonard Crestez, shawnguo
Cc: robh+dt, mark.rutland, s.hauer, kernel, festevam, devicetree,
linux-arm-kernel, linux-kernel, dl-linux-imx
Hi, Leonard
> On 2019-09-12 5:57 AM, Anson Huang wrote:
> > The 800MHz opp speed grading fuse mask should be 0xd instead of 0xf
> > according to fuse map definition:
> >
> > SPEED_GRADING[1:0] MHz
> > 00 800
> > 01 500
> > 10 1000
> > 11 1200
> >
> > Signed-off-by: Anson Huang <Anson.Huang@nxp.com>
>
> Reviewed-by: Leonard Crestez <leonard.crestez@nxp.com>
>
> Are you going to add the 500mhz OPP as well?
So far no, as datasheet does NOT list 500MHz as OPP, this patch is just to make sure
the speed grading mask settings matches the fusemap file, NOT sure if 500MHz will be supported
officially.
Anson
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 1/2] ARM: dts: imx7d: Correct speed grading fuse settings
2019-09-12 2:56 [PATCH 1/2] ARM: dts: imx7d: Correct speed grading fuse settings Anson Huang
2019-09-12 2:56 ` [PATCH 2/2] ARM: dts: imx7d: Add opp-suspend property Anson Huang
2019-09-12 12:45 ` [PATCH 1/2] ARM: dts: imx7d: Correct speed grading fuse settings Leonard Crestez
@ 2019-10-06 8:09 ` Shawn Guo
2 siblings, 0 replies; 6+ messages in thread
From: Shawn Guo @ 2019-10-06 8:09 UTC (permalink / raw)
To: Anson Huang
Cc: robh+dt, mark.rutland, s.hauer, kernel, festevam, devicetree,
linux-arm-kernel, linux-kernel, Linux-imx
On Thu, Sep 12, 2019 at 10:56:31AM +0800, Anson Huang wrote:
> The 800MHz opp speed grading fuse mask should be 0xd instead
> of 0xf according to fuse map definition:
>
> SPEED_GRADING[1:0] MHz
> 00 800
> 01 500
> 10 1000
> 11 1200
>
> Signed-off-by: Anson Huang <Anson.Huang@nxp.com>
Applied both, thanks.
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2019-10-06 8:10 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-09-12 2:56 [PATCH 1/2] ARM: dts: imx7d: Correct speed grading fuse settings Anson Huang
2019-09-12 2:56 ` [PATCH 2/2] ARM: dts: imx7d: Add opp-suspend property Anson Huang
2019-09-12 12:46 ` Leonard Crestez
2019-09-12 12:45 ` [PATCH 1/2] ARM: dts: imx7d: Correct speed grading fuse settings Leonard Crestez
2019-09-16 2:55 ` Anson Huang
2019-10-06 8:09 ` Shawn Guo
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).