[PATCH 04/14] ARM: dts: dra76x: Create a common file with MMC/SD IOdelay data
From: tony@atomide.com (Tony Lindgren)
Date: 2017-12-15 16:32:14
Also in:
linux-devicetree, linux-mmc, linux-omap, lkml
* Kishon Vijay Abraham I [off-list ref] [171215 06:12]:
Hi Tony, On Thursday 14 December 2017 08:45 PM, Tony Lindgren wrote:quoted
* Kishon Vijay Abraham I [off-list ref] [171214 13:44]:quoted
+&dra7_pmx_core { + mmc1_pins_default: mmc1_pins_default { + pinctrl-single,pins = < + DRA7XX_CORE_IOPAD(0x3754, PIN_INPUT_PULLUP | MUX_MODE0) /* mmc1_clk.clk */ + DRA7XX_CORE_IOPAD(0x3758, PIN_INPUT_PULLUP | MUX_MODE0) /* mmc1_cmd.cmd */ + DRA7XX_CORE_IOPAD(0x375c, PIN_INPUT_PULLUP | MUX_MODE0) /* mmc1_dat0.dat0 */ + DRA7XX_CORE_IOPAD(0x3760, PIN_INPUT_PULLUP | MUX_MODE0) /* mmc1_dat1.dat1 */ + DRA7XX_CORE_IOPAD(0x3764, PIN_INPUT_PULLUP | MUX_MODE0) /* mmc1_dat2.dat2 */ + DRA7XX_CORE_IOPAD(0x3768, PIN_INPUT_PULLUP | MUX_MODE0) /* mmc1_dat3.dat3 */ + >; + }; + + mmc1_pins_sdr12: mmc1_pins_sdr12 { + pinctrl-single,pins = < + DRA7XX_CORE_IOPAD(0x3754, PIN_INPUT_PULLUP | MUX_MODE0) /* mmc1_clk.clk */ + DRA7XX_CORE_IOPAD(0x3758, PIN_INPUT_PULLUP | MUX_MODE0) /* mmc1_cmd.cmd */ + DRA7XX_CORE_IOPAD(0x375c, PIN_INPUT_PULLUP | MUX_MODE0) /* mmc1_dat0.dat0 */ + DRA7XX_CORE_IOPAD(0x3760, PIN_INPUT_PULLUP | MUX_MODE0) /* mmc1_dat1.dat1 */ + DRA7XX_CORE_IOPAD(0x3764, PIN_INPUT_PULLUP | MUX_MODE0) /* mmc1_dat2.dat2 */ + DRA7XX_CORE_IOPAD(0x3768, PIN_INPUT_PULLUP | MUX_MODE0) /* mmc1_dat3.dat3 */ + >; + };Can't you just do: pinctrl-0 = <&mmc1_pins_default>; pinctrl-1 = <&mmc1_pins_default>; pinctrl-2 = <&mmc1_pins_hs>; pinctrl-names = "default", "sdr12", "sdr25";just wanted to make sure every mode has it's own pinctrl group so that it's easy to review. Initially we were thinking something like mmc1_pins_default_sdr12_sdr25.
OK that naming works fine for me.
But if you'd prefer we just use mmc1_pins_default for all modes that uses default pinmux configuration, I can change it that way too.
No up to you with the naming thanks. Regards, Tony