On 17:39-20230315, Sinthu Raja wrote: > From: Sinthu Raja <sinthu.raja@xxxxxx> > > Rev E2 of the AM68 SK baseboard has updated the GPIO IO expander pins > functionality. To match the Rev E2 schematics, update existing IO expander > GPIO line names and the corresponding node which uses the expansion(exp1) > node. > > Signed-off-by: Sinthu Raja <sinthu.raja@xxxxxx> > --- > > Schematics Ref: https://www.ti.com/lit/zip/sprr463 Looking at https://www.ti.com/tool/SK-AM68 looks like E1 never hit production, is that correct? Does this need a "Fixes" tag? > > arch/arm64/boot/dts/ti/k3-am68-sk-base-board.dts | 12 +++++------- > 1 file changed, 5 insertions(+), 7 deletions(-) > > diff --git a/arch/arm64/boot/dts/ti/k3-am68-sk-base-board.dts b/arch/arm64/boot/dts/ti/k3-am68-sk-base-board.dts > index 2091cd2431fb..27a43a8ecffd 100644 > --- a/arch/arm64/boot/dts/ti/k3-am68-sk-base-board.dts > +++ b/arch/arm64/boot/dts/ti/k3-am68-sk-base-board.dts > @@ -60,7 +60,7 @@ vdd_mmc1: regulator-sd { > regulator-boot-on; > enable-active-high; > vin-supply = <&vsys_3v3>; > - gpio = <&exp1 10 GPIO_ACTIVE_HIGH>; > + gpio = <&exp1 8 GPIO_ACTIVE_HIGH>; > }; > > vdd_sd_dv: regulator-tlv71033 { > @@ -264,12 +264,10 @@ exp1: gpio@21 { > reg = <0x21>; > gpio-controller; > #gpio-cells = <2>; > - gpio-line-names = "CSI_VIO_SEL", "CSI_SEL_FPC_EXPn", "HDMI_PDn", > - "HDMI_LS_OE", "DP0_3V3 _EN", "BOARDID_EEPROM_WP", > - "CAN_STB", " ", "GPIO_uSD_PWR_EN", "eDP_ENABLE", > - "IO_EXP_PCIe1_M.2_RTSz", "IO_EXP_MCU_RGMII_RSTz", > - "IO_EXP_CSI2_EXP_RSTz", " ", "CSI0_B_GPIO1", > - "CSI1_B_GPIO1"; > + gpio-line-names = " ", " ", " ", " ", " ", > + "BOARDID_EEPROM_WP", "CAN_STB", " ", > + "GPIO_uSD_PWR_EN", " ", "IO_EXP_PCIe1_M.2_RTSz", > + "IO_EXP_MCU_RGMII_RST#", " ", " ", " ", " "; > }; > }; > > -- > 2.36.1 > -- Regards, Nishanth Menon Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5 849D 1736 249D