Message ID | 20201201102920.84051-1-marex@denx.de |
---|---|
State | Accepted |
Commit | 845c6720eca92be09127f7d306349ab31dbf372d |
Delegated to: | Patrick Delaunay |
Headers | show |
Series | [1/4] ARM: dts: stm32: Enable internal pull-ups for SDMMC1 on DHCOM SoM | expand |
Hi Marek, From: Marek Vasut <marex@denx.de> > Sent: mardi 1 décembre 2020 11:29 > > The default state of SD bus and clock line is logical HI. SD card IO is open-drain and pulls the bus lines LO. Always enable the SD bus pull ups to guarantee this behavior on DHCOM SoM. Note that on SoMs with SD bus voltage level shifter, the pull ups are built into the level shifter, however that has no negative impact. > > Signed-off-by: Marek Vasut <marex@denx.de> > Cc: Patrice Chotard <patrice.chotard@st.com> > Cc: Patrick Delaunay <patrick.delaunay@st.com> > --- > arch/arm/dts/stm32mp15xx-dhcom.dtsi | 14 ++++++++++++++ > 1 file changed, 14 insertions(+) > > diff --git a/arch/arm/dts/stm32mp15xx-dhcom.dtsi b/arch/arm/dts/stm32mp15xx-dhcom.dtsi > index f022d8395c..9049245c5b 100644 > --- a/arch/arm/dts/stm32mp15xx-dhcom.dtsi > +++ b/arch/arm/dts/stm32mp15xx-dhcom.dtsi > @@ -339,6 +339,20 @@ > status = "okay"; > }; > > +&sdmmc1_b4_pins_a { > + /* > + * SD bus pull-up resistors: > + * - optional on SoMs with SD voltage translator > + * - mandatory on SoMs without SD voltage translator > + */ > + pins1 { > + bias-pull-up; > + }; > + pins2 { > + bias-pull-up; > + }; > +}; > + > &sdmmc2 { > pinctrl-names = "default"; > pinctrl-0 = <&sdmmc2_b4_pins_a &sdmmc2_d47_pins_a>; > -- > 2.29.2 > Reviewed-by: Patrick Delaunay <patrick.delaunay@foss.st.com> For the serie: the target is next or it is a bugfix for master / v2021.01 ? Thanks Patrick
On 12/8/20 6:20 PM, Patrick DELAUNAY wrote: > Hi Marek, Hi, [...] > For the serie: the target is next or it is a bugfix for master / > v2021.01 ? This is for 2021.01 , it corrects a couple of random things here and there.
Hi Marek, On 12/8/20 6:26 PM, Marek Vasut wrote: > On 12/8/20 6:20 PM, Patrick DELAUNAY wrote: >> Hi Marek, > > Hi, > > [...] > >> For the serie: the target is next or it is a bugfix for master / >> v2021.01 ? > > This is for 2021.01 , it corrects a couple of random things here and > there. For the serie: Applied to u-boot-stm/master. Thanks Patrick
diff --git a/arch/arm/dts/stm32mp15xx-dhcom.dtsi b/arch/arm/dts/stm32mp15xx-dhcom.dtsi index f022d8395c..9049245c5b 100644 --- a/arch/arm/dts/stm32mp15xx-dhcom.dtsi +++ b/arch/arm/dts/stm32mp15xx-dhcom.dtsi @@ -339,6 +339,20 @@ status = "okay"; }; +&sdmmc1_b4_pins_a { + /* + * SD bus pull-up resistors: + * - optional on SoMs with SD voltage translator + * - mandatory on SoMs without SD voltage translator + */ + pins1 { + bias-pull-up; + }; + pins2 { + bias-pull-up; + }; +}; + &sdmmc2 { pinctrl-names = "default"; pinctrl-0 = <&sdmmc2_b4_pins_a &sdmmc2_d47_pins_a>;
The default state of SD bus and clock line is logical HI. SD card IO is open-drain and pulls the bus lines LO. Always enable the SD bus pull ups to guarantee this behavior on DHCOM SoM. Note that on SoMs with SD bus voltage level shifter, the pull ups are built into the level shifter, however that has no negative impact. Signed-off-by: Marek Vasut <marex@denx.de> Cc: Patrice Chotard <patrice.chotard@st.com> Cc: Patrick Delaunay <patrick.delaunay@st.com> --- arch/arm/dts/stm32mp15xx-dhcom.dtsi | 14 ++++++++++++++ 1 file changed, 14 insertions(+)