| Message ID | 20210930155633.2745201-4-frieder@fris.de (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | Devicetree fixes and additions for Kontron SL/BL i.MX8MM boards | expand |
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? 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 >
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. >> >> 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 >>
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 > >>
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; };