On 22 May 2014 14:05, Chris Ball <chris@xxxxxxxxxx> wrote: > Hi Ulf, > > On Thu, May 22 2014, Ulf Hansson wrote: >> Here are a third and final collection of patches for 3.16. >> >> I have as you and I discussed (off list), collected the patches from >> Russell's sdhci patchset. I needed to resolve some conflicts while >> applying them, but it was manageable. I also omitted the patch which >> was reported to caused problem with eMMC, (mmc: sdhci: hack up driver >> to make it more compliant with UHS-1). > > Pulled, but: Great! > > drivers/mmc/host/sdhci.c: In function ‘sdhci_set_uhs_signaling’: > drivers/mmc/host/sdhci.c:1432:12: error: ‘ios’ undeclared (first use > in this function) > else if ((ios->timing == MMC_TIMING_UHS_DDR50) || > ^ Seems like I manage to break it while I resolved the conflict for the below commit: "mmc: sdhci: convert sdhci_set_uhs_signaling() into a library function" The following lines... + else if ((ios->timing == MMC_TIMING_UHS_DDR50) || + (ios->timing == MMC_TIMING_MMC_DDR52)) + ctrl_2 |= SDHCI_CTRL_UHS_DDR50; Should be replaced by; + else if ((timing == MMC_TIMING_UHS_DDR50) || + (timing == MMC_TIMING_MMC_DDR52)) + ctrl_2 |= SDHCI_CTRL_UHS_DDR50; Can you update the code and amend the commit? Kind regards Uffe -- To unsubscribe from this list: send the line "unsubscribe linux-mmc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html