On Mon, 16 Jan 2023 at 20:44, Doug Brown <doug@xxxxxxxxxxxxx> wrote: > > This is a revival of an earlier patch series from 2013 to add support > for the PXA168 SDHC controller, with an additional SDIO IRQ errata fix. > It also cleans up the clock naming to be consistent with the existing DT > schema shared with the pxav3 driver (in a backwards-compatible way). > > Here is the original patch series this is based on: > https://lore.kernel.org/linux-mmc/1363544206-3671-1-git-send-email-tanmay.upadhyay@xxxxxxxxxxxxxx/ > > Note that I left out the platform_specific_completion and clock gating > changes from the original patches. They both seemed controversial, and > don't seem necessary based on my testing. I've been running this code on > a PXA168 for months without any issues. > > Changes in v5: > - Fix missing assignment to ret in core clock patch found by test robot > > Changes in v4: > - Rebase on latest mmc/next to fix conflict with DT binding > > Changes in v3: > - Use OF match data rather than of_match_device and of_device_is_compatible > - Simplify some instances of pdev->dev that could have just been "dev" > - Handle EPROBE_DEFER when getting the clock > - Use devm_clk_get_optional_enabled for the core clock (it's simpler) > - Clear sdio_mrq before calling mmc_request_done > - Small tweaks to devicetree binding requested by Krzysztof > > Changes in v2: > - Fix mistakes in devicetree binding > - Use cleaner code for pxav1_readw suggested by Adrian > - Switch to request_done() and irq() for SDIO workaround CMD0 handling > > Doug Brown (8): > mmc: sdhci-pxav2: add initial support for PXA168 V1 controller > mmc: sdhci-pxav2: enable CONFIG_MMC_SDHCI_IO_ACCESSORS > mmc: sdhci-pxav2: add register workaround for PXA168 silicon bug > mmc: sdhci-pxav2: change clock name to match DT bindings > mmc: sdhci-pxav2: add optional core clock > mmc: sdhci-pxav2: add SDIO card IRQ workaround for PXA168 V1 > controller > mmc: sdhci-pxav2: add optional pinctrl for SDIO IRQ workaround > dt-bindings: mmc: sdhci-pxa: add pxav1 > > .../devicetree/bindings/mmc/sdhci-pxa.yaml | 19 ++- > drivers/mmc/host/Kconfig | 1 + > drivers/mmc/host/sdhci-pxav2.c | 154 ++++++++++++++++-- > 3 files changed, 160 insertions(+), 14 deletions(-) > Applied for next, thanks! Kind regards Uffe