x86: edison: Don't take SD card detect pin into consideration
authorAndy Shevchenko <andriy.shevchenko@linux.intel.com>
Wed, 27 Oct 2021 11:28:10 +0000 (14:28 +0300)
committerBin Meng <bmeng.cn@gmail.com>
Mon, 1 Nov 2021 01:50:55 +0000 (09:50 +0800)
There are two PCB designs in the wild which use the opposite
signaling for SD card detection. This makes U-Boot working
in one case and failing in the other. Quirk this out by
disconnecting SD card detection pin from the PCB by switching
it to mode 3. In the disconnected state the read value is always
the same and inverted to what we are expecting in the code.

BugLink: https://github.com/edison-fw/meta-intel-edison/issues/136
Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Reviewed-by: Bin Meng <bmeng.cn@gmail.com>
arch/x86/dts/edison.dts

index 2c8cf6c..b3658b8 100644 (file)
        sdcard: mmc@ff3fa000 {
                compatible = "intel,sdhci-tangier";
                reg = <0xff3fa000 0x1000>;
+               /*
+                * In the disconnected state of the SD Card Detection pin
+                * the read value is always the same and inverted to what
+                * we are expecting in the code.
+                */
+               cd-inverted;
        };
 
        pmu: power@ff00b000 {
                reg = <0xff0c0000 0x8000>;
 
                /*
+                * Disconnect SD card detection pin, so it won't affect
+                * the reality on two different PCB designs where it's
+                * using the opposite signaling: Edison/Arduino uses
+                * Active Low, while SparkFun went with Active High.
+                */
+               sd_cd@0 {
+                       pad-offset = <37>;
+                       mode-func = <3>;
+               };
+
+               /*
                 * Initial configuration came from the firmware.
                 * Which quite likely has been used in the phones, where I2C #8,
                 * that is not part of Atom peripheral, is in use.