On Mon, Feb 28, 2011 at 07:32:05PM +0800, Richard Zhu wrote: > Fix the NO INT in the Multi-BLK IO in SD/MMC, and > Multi-BLK read in SDIO > > The CMDTYPE of the CMD register(offset 0xE) should be set to > "11" when the STOP CMD12 is issued on imx53 to abort one > open ended multi-blk IO. Otherwise one the TC INT wouldn't > be generated. > > In exact block transfer, the controller doesn't complete the > operations automatically as required at the end of the > transfer and remains on hold if the abort command is not sent. > As a result, the TC flag is not asserted and SW received timeout > exeception. set bit1 of Vendor Spec registor to fix it > > Signed-off-by: Richard Zhu <[email protected]> > Signed-off-by: Richard Zhao <[email protected]> > --- > drivers/mmc/host/sdhci-esdhc-imx.c | 77 ++++++++++++++++++++++++++++++++++- > drivers/mmc/host/sdhci-pltfm.h | 2 +- > 2 files changed, 75 insertions(+), 4 deletions(-) > > diff --git a/drivers/mmc/host/sdhci-esdhc-imx.c > b/drivers/mmc/host/sdhci-esdhc-imx.c > index 9b82910..32af7c4 100644 > --- a/drivers/mmc/host/sdhci-esdhc-imx.c > +++ b/drivers/mmc/host/sdhci-esdhc-imx.c > @@ -15,13 +15,41 @@ > #include <linux/delay.h> > #include <linux/err.h> > #include <linux/clk.h> > +#include <linux/slab.h> > #include <linux/mmc/host.h> > #include <linux/mmc/sdhci-pltfm.h> > +#include <linux/mmc/mmc.h> > +#include <linux/mmc/sdio.h> > #include <mach/hardware.h> > #include "sdhci.h" > #include "sdhci-pltfm.h" > #include "sdhci-esdhc.h" > > +/* Abort type definition in the command register */ > +#define SDHCI_CMD_ABORTCMD 0xC0
Won't that belong into sd.h (unless I misunderstood your last mail)?
> +/* VENDOR SPEC register */
> +#define SDHCI_VENDOR_SPEC 0xC0
> +
> +/*
> + * The CMDTYPE of the CMD register(offset 0xE) should be set to
Check spaces.
> + * "11" when the STOP CMD12 is issued on imx53 to abort one
> + * open ended multi-blk IO. Otherwise one the TC INT wouldn't
> + * be generated.
> + * In exact block transfer, the controller doesn't complete the
> + * operations automatically as required at the end of the
> + * transfer and remains on hold if the abort command is not sent.
> + * As a result, the TC flag is not asserted and SW received timeout
> + * exeception. Bit1 of Vendor Spec registor is used to fix it.
> + */
This is a better description, thanks. But what does the bit technically
do? Can't we keep it enabled all the time? Can't we use it to fix CMD12,
too?
> +#define IMX_MULTIBLK_NO_INT (1 << 0)
> +
> +struct pltfm_imx_data {
> + int flags;
> + u32 mod_val;
> +};
Hmm, to me, just using cpu_is_mx53() is more readable than introducing
another layer of flags/quirks.
> +
> +static struct sdhci_ops sdhci_esdhc_ops;
> +
Move them to the front. But I did this already, so no worries :) I will
ping Chris to merge my series, so we will have something better to
develop on.
> static inline void esdhc_clrset_le(struct sdhci_host *host, u32 mask, u32
> val, int reg)
> {
> void __iomem *base = host->ioaddr + (reg & ~0x3);
> @@ -38,20 +66,51 @@ static u16 esdhc_readw_le(struct sdhci_host *host, int
> reg)
> return readw(host->ioaddr + reg);
> }
>
> +static void esdhc_writel_le(struct sdhci_host *host, u32 val, int reg)
> +{
> + switch (reg) {
> + case SDHCI_INT_STATUS:
> + if (val & SDHCI_INT_DATA_END) {
> + u32 v;
> + v = readl(host->ioaddr + SDHCI_VENDOR_SPEC);
> + if (v & 0x2) {
> + v &= ~0x2;
Forgot to mention this before. Please use a define instead of hardocded
values.
I skipped the rest because we probably need to answer the questions
above first and to get a stable base with out conflicts.
Regards,
Wolfram
--
Pengutronix e.K. | Wolfram Sang |
Industrial Linux Solutions | http://www.pengutronix.de/ |
signature.asc
Description: Digital signature
