Commit 6d4441be authored by Judith Mendez's avatar Judith Mendez Committed by Vignesh Raghavendra
Browse files

arm64: dts: ti: k3-am62p5-sk: Disable MMC1 internal pulls on data pins 

AM62P SK has external 10K pullups on MMC1 DAT1-DAT3 pins [0].
Disable internal pullups on DAT1-DAT3 so that each line has a
single pullup source:
- with both pullups enabled, the effective parallel resistance on
  DAT1-3 (~8.33K) drops below the 10K minimum pullup requirement
  for data lines (per SD Physical Layer Specification)
- removing internal pullups makes DAT1-3 match DAT0 10K
  external pullup so its consistent and within spec
- both internal and external pullups enabled equals unnecessary power
  consumption

[0] https://www.ti.com/lit/zip/SPRR487



Fixes: c00504ea ("arm64: dts: ti: k3-am62p5-sk: Updates for SK EVM")
Signed-off-by: default avatarJudith Mendez <jm@ti.com>
Reviewed-by: default avatarMoteen Shah <m-shah@ti.com>
Link: https://patch.msgid.link/20260223233731.2690472-2-jm@ti.com


Signed-off-by: default avatarVignesh Raghavendra <vigneshr@ti.com>
parent 8cb06da0

arch/arm64/boot/dts/ti/k3-am62p5-sk.dts

+3 −3
Original line number Diff line number Diff line
@@ -271,9 +271,9 @@ main_mmc1_pins_default: main-mmc1-default-pins { 
			AM62PX_IOPAD(0x023c, PIN_INPUT, 0) /* (H20) MMC1_CMD */

			AM62PX_IOPAD(0x0234, PIN_OUTPUT, 0) /* (J24) MMC1_CLK */

			AM62PX_IOPAD(0x0230, PIN_INPUT, 0) /* (H21) MMC1_DAT0 */

			AM62PX_IOPAD(0x022c, PIN_INPUT_PULLUP, 0) /* (H23) MMC1_DAT1 */

			AM62PX_IOPAD(0x0228, PIN_INPUT_PULLUP, 0) /* (H22) MMC1_DAT2 */

			AM62PX_IOPAD(0x0224, PIN_INPUT_PULLUP, 0) /* (H25) MMC1_DAT3 */

			AM62PX_IOPAD(0x022c, PIN_INPUT, 0) /* (H23) MMC1_DAT1 */

			AM62PX_IOPAD(0x0228, PIN_INPUT, 0) /* (H22) MMC1_DAT2 */

			AM62PX_IOPAD(0x0224, PIN_INPUT, 0) /* (H25) MMC1_DAT3 */

			AM62PX_IOPAD(0x0240, PIN_INPUT, 0) /* (D23) MMC1_SDCD */

		>;

		bootph-all;