| Message ID | CAPDyKFo44fucVebB3f_qby1iWqjyq5P4XPAyD6og5ZWpqgO4=A@mail.gmail.com |
|---|---|
| State | New |
| Headers | show |
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: 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) || ^ drivers/mmc/host/sdhci.c:1432:12: note: each undeclared identifier is reported only once for each function it appears in make[3]: *** [drivers/mmc/host/sdhci.o] Error 1 - Chris.
On 22 May 2014 14:05, Chris Ball <chris@printf.net> 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@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Hi, On Thu, May 22 2014, Ulf Hansson wrote: > "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; Done and pushed out, thanks. - Chris.
Hi Chris, 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). Unless you have some additional review comments, please pull them into your mmc-next branch to get some more testing in linux-next. Kind regards Uffe The following changes since commit 197160d52e859575ff0ad1afd674ab2a825b9f83: mmc: sdhci: remove mdelay in eMMC tuning (2014-05-13 21:21:10 -0400) are available in the git repository at: git://git.linaro.org/people/ulf.hansson/mmc.git mmc-next_for_chris_3.16_take3 for you to fetch changes up to c47a823d0ee8097c08047bbc13728c9429a99e91: MAINTAINERS: mmc: Add path to git tree (2014-05-22 12:41:16 +0200) ---------------------------------------------------------------- Aisheng Dong (1): mmc: sdhci-esdhc-imx: fix mmc ddr mode regression issue Al Cooper (1): mmc: sdhci: SD tuning is broken for some controllers Andreas Fenkart (6): mmc: omap_hsmmc: Enable SDIO interrupt mmc: omap_hsmmc: enable wakeup event for sdio OMAP4 mmc: omap_hsmmc: Extend debugfs by SDIO IRQ handling, runtime state mmc: omap_hsmmc: abort runtime suspend if pending sdio irq detected mmc: omap_hsmmc: switch default/idle pinctrl states in runtime hooks mmc: omap_hsmmc: Pin remux workaround to support SDIO interrupt on AM335x Axel Lin (2): mmc: omap: Use DIV_ROUND_UP instead of open coded mmc: wmt-sdmmc: Use GFP_KERNEL instead of hard-coded value Balaji T K (7): mmc: omap_hsmmc: use devm_clk_get mmc: omap_hsmmc: use devm_request_irq mmc: omap_hsmmc: use devm_request_threaded_irq mmc: omap_hsmmc: use devm_ioremap_resource mmc: omap_hsmmc: fix cmd23 multiblock read/write mmc: omap_hsmmc: split omap-dma header file mmc: omap_hsmmc: use IS_ERR macro for error checking David Lanzendörfer (1): ARM: sunxi: Add driver for SD/MMC hosts found on Allwinner sunxi SoCs Doug Anderson (1): mmc: dw_mmc: Don't print data errors Peter Guo (1): mmc: sdhci-pci-o2micro: Add SeaBird SeaEagle SD3 support Russell King (31): mmc: sdio_irq: rework sdio irq handling mmc: sdhci: clean up interrupt handling mmc: sdhci: clean up sdio interrupt enable handling mmc: sdhci: convert to new SDIO IRQ handling mmc: sdhci: push card_tasklet into threaded irq handler mmc: sdhci: allow sdio interrupts while sdhci runtime suspended mmc: sdhci: more efficient interrupt enable register handling mmc: sdhci: plug hole in disabling card detection interrupts mmc: sdhci: convert generic bus width setup to library function mmc: sdhci: convert reset into a library function mmc: sdhci: move FSL ESDHC reset handling quirk into esdhc code mmc: sdhci: avoid sync'ing the SG if there's no misalignment mmc: sdhci: convert ADMA descriptors to a coherent allocation mmc: sdhci: clean up sdhci_update_clock()/sdhci_set_clock() mmc: sdhci: move setting host->clock into sdhci_do_set_ios() mmc: sdhci: move setting mmc->actual_clock into set_clock handlers mmc: sdhci: convert sdhci_set_clock() into a library function mmc: sdhci-esdhc-imx: avoid DMA to kernel stack mmc: sdhci-esdhc-imx: comment runtime_pm_get_sync() in esdhc_prepare_tuning() mmc: sdhci-esdhc-imx: fix lockdep splat upon tuning mmc: sdhci: set_uhs_signaling() need not return a value mmc: sdhci: convert sdhci_set_uhs_signaling() into a library function mmc: sdhci: cache timing information locally mmc: sdhci: clean up sdhci_execute_tuning() decision mmc: sdhci-esdhc-imx: remove emulation of uhs_mode mmc: sdhci-of-esdhc: remove platform_suspend/platform_resume callbacks mmc: sdhci: remove platform_suspend/platform_resume callbacks mmc: sdhci-tegra: get rid of special PRESENT_STATE register handling mmc: sdhci: move regulator handling into sdhci_set_power() mmc: sdhci: move remaining power handling into sdhci_set_power() mmc: sdhci: track whether preset mode is currently enabled in hardware Sebastian Hesselbarth (1): mmc: dove: fix missing MACH_DOVE dependency Ulf Hansson (1): MAINTAINERS: mmc: Add path to git tree .../devicetree/bindings/mmc/sunxi-mmc.txt | 43 + .../devicetree/bindings/mmc/ti-omap-hsmmc.txt | 54 + MAINTAINERS | 1 + drivers/mmc/core/sdio_irq.c | 41 +- drivers/mmc/host/Kconfig | 9 +- drivers/mmc/host/Makefile | 1 + drivers/mmc/host/dw_mmc.c | 2 +- drivers/mmc/host/omap.c | 10 +- drivers/mmc/host/omap_hsmmc.c | 351 +++++-- drivers/mmc/host/sdhci-acpi.c | 8 + drivers/mmc/host/sdhci-bcm-kona.c | 4 + drivers/mmc/host/sdhci-bcm2835.c | 4 + drivers/mmc/host/sdhci-cns3xxx.c | 13 +- drivers/mmc/host/sdhci-dove.c | 4 + drivers/mmc/host/sdhci-esdhc-imx.c | 83 +- drivers/mmc/host/sdhci-esdhc.h | 4 +- drivers/mmc/host/sdhci-of-arasan.c | 4 + drivers/mmc/host/sdhci-of-esdhc.c | 70 +- drivers/mmc/host/sdhci-of-hlwd.c | 4 + drivers/mmc/host/sdhci-pci-o2micro.c | 78 +- drivers/mmc/host/sdhci-pci-o2micro.h | 3 + drivers/mmc/host/sdhci-pci.c | 9 +- drivers/mmc/host/sdhci-pltfm.c | 4 + drivers/mmc/host/sdhci-pxav2.c | 14 +- drivers/mmc/host/sdhci-pxav3.c | 13 +- drivers/mmc/host/sdhci-s3c.c | 36 +- drivers/mmc/host/sdhci-sirf.c | 4 + drivers/mmc/host/sdhci-spear.c | 5 +- drivers/mmc/host/sdhci-tegra.c | 27 +- drivers/mmc/host/sdhci.c | 744 +++++++------- drivers/mmc/host/sdhci.h | 20 +- drivers/mmc/host/sunxi-mmc.c | 1049 ++++++++++++++++++++ drivers/mmc/host/wmt-sdmmc.c | 2 +- include/linux/mmc/host.h | 3 + include/linux/mmc/sdhci.h | 15 +- include/linux/omap-dma.h | 19 +- include/linux/omap-dmaengine.h | 21 + include/linux/platform_data/mmc-omap.h | 1 + 38 files changed, 2154 insertions(+), 623 deletions(-) create mode 100644 Documentation/devicetree/bindings/mmc/sunxi-mmc.txt create mode 100644 drivers/mmc/host/sunxi-mmc.c create mode 100644 include/linux/omap-dmaengine.h -- To unsubscribe from this list: send the line "unsubscribe linux-mmc" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html