On Tue, Oct 05, 2021 at 03:14:40PM +0200, Frieder Schrempf wrote:

On 05.10.21 08:56, Shawn Guo wrote:

...

On Thu, Sep 30, 2021 at 05:56:26PM +0200, Frieder Schrempf wrote:

...

From: Frieder Schrempf frieder.schrempf@kontron.de

According to the datasheet VDD_SNVS should be 800 mV, so let's make sure that the voltage won't be different.

Could you share the datasheet?

Sure, it's here: https://www.nxp.com/docs/en/data-sheet/IMX8MMIEC.pdf. See table 10 for the operating voltages.

Thanks for the link!

But datasheet specifies a voltage range. 800 mV is just the typical one. The only problem I see is that regulator-min-microvolt should be lowered to 800 mV. regulator-max-microvolt looks correct to me.

Shawn

...

...

Fixes: 21c4f45b335f ("arm64: dts: Add the Kontron i.MX8M Mini SoMs and baseboards") Cc: stable@vger.kernel.org Signed-off-by: Frieder Schrempf frieder.schrempf@kontron.de

---

arch/arm64/boot/dts/freescale/imx8mm-kontron-n801x-som.dtsi | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/arm64/boot/dts/freescale/imx8mm-kontron-n801x-som.dtsi b/arch/arm64/boot/dts/freescale/imx8mm-kontron-n801x-som.dtsi index b12fb7ce6686..213014f59b46 100644 --- a/arch/arm64/boot/dts/freescale/imx8mm-kontron-n801x-som.dtsi +++ b/arch/arm64/boot/dts/freescale/imx8mm-kontron-n801x-som.dtsi @@ -152,8 +152,8 @@ reg_nvcc_snvs: LDO1 { reg_vdd_snvs: LDO2 { regulator-name = "ldo2";

• 		regulator-min-microvolt = <850000>;
  

• 		regulator-max-microvolt = <900000>;
  

• 		regulator-min-microvolt = <800000>;
  

• 		regulator-max-microvolt = <800000>;
  	regulator-boot-on;
  	regulator-always-on;
  };
  

-- 2.33.0