Hi, On 21/03/19 2:41 PM, Thomas Petazzoni wrote: > Hello Kishon, > > On Thu, 21 Mar 2019 13:59:48 +0530 > Kishon Vijay Abraham I <kishon@xxxxxx> wrote: > >>> int ro = mmc_sd_get_ro(host); >>> >>> if (ro < 0) { >>> pr_warn("%s: host does not support reading read-only switch, assuming write-enable\n", >>> mmc_hostname(host)); >>> } else if (ro > 0) { >>> mmc_card_set_readonly(card); >>> } >>> >>> Do you see this warning message before my change ? >> >> yes. > > Ah, OK. Then it makes sense. > >>> (3) WP signal routed to a dedicated signal of the SDHCI controller. In >>> this case, the kernel needs to read the SDHCI_PRESENT_STATE >>> register of the SDHCI controller. >> >> This is not supported in OMAP platforms. > > Because the SDHCI controller WP signal is not implemented / routed to > an external pin ? MMC_SDWP is brought out but I think it has never worked reliably for us that we always relied on GPIO. It has been that way for multiple generation of SoCs. I haven't experimented much on this myself. Thanks Kishon