• Javier Martinez Canillas's avatar
    ARM: dts: Mux XMMCnDATA[0] pad correctly for Exynos5420 boards · af6ad88a
    Javier Martinez Canillas authored
    The Exynos5420 SoC has 3 sets of 8 pads used as data lines for the 3
    MMC/SD/SDIO slots. These needs to be muxed as SD_n_DATA instead of a
    GPIO or external interrupt to allow the MMC controller to communicate
    with the attached cards or SDIO devices.
    
    Which data lines needs to be muxed as SD_n_DATA depends on the bus
    width used for data transfer:
    
    * bus-width = <1> needs SD_n_DATA[0]
    * bus-width = <4> needs SD_n_DATA[0-3]
    * bus-width = <8> needs SD_n_DATA[0-7]
    
    The Exynos5250-pinctrl.dtsi file that defines the groups of pins has
    SD_n_DATA[0] muxed for both sdn_bus1 and sdn_bus4 so just one of them
    needs to be included in the device node's pinctrl property.
    
    But Exynos5420-pinctrl has a different definition and only includes
    SD_n_DATA[1-3] for sdn_bus4. So for a bus-width = <4>, both sdn_bus1
    and sdn_bus4 have to be in the dev pinctrl to mux all the needed pads.
    
    It seems all Exynos5420 boards had just cargo cult the pinctrl lines
    assuming that sdn_bus4 also included SD_n_DATA[0] and it only works
    because the bootloader muxes the pads correctly. But that is not the
    case for the devices not used by the bootloader such as WiFi modules.
    
    Add sdn_bus1 too in the nodes pinctrl to not rely on the bootloader.
    Suggested-by: default avatarDoug Anderson <dianders@chromium.org>
    Signed-off-by: default avatarJavier Martinez Canillas <javier.martinez@collabora.co.uk>
    Signed-off-by: default avatarKukjin Kim <kgene@kernel.org>
    af6ad88a
exynos5420-arndale-octa.dts 9 KB