On Tue, May 04, 2010 at 11:43:22AM +0200, Maurus Cuelenaere wrote:
The 'From:' line seems to have been lost?
> The s3c6410 sdhci controller does not support the 'End' attribute and
> NOP attribute in the same 8-Byte ADMA descriptor. This patch adds a
> new quirk to identify sdhci host contollers with such behaviour.
> In addition to this, for controllers using the new quirk, the last
> entry in the ADMA descritor table is marked with the 'End' attribute
> (instead of using a NOP descriptor with 'End' attribute).
>
> Signed-off-by: Thomas Abraham <thomas.ab@xxxxxxxxxxx>
> Acked-by: Ben Dooks <ben-linux@xxxxxxxxx>
> Signed-off-by: Maurus Cuelenaere <mcuelenaere@xxxxxxxxx>
> ---
> drivers/mmc/host/sdhci.c | 20 +++++++++++++++-----
> drivers/mmc/host/sdhci.h | 2 ++
> 2 files changed, 17 insertions(+), 5 deletions(-)
>
> diff --git a/drivers/mmc/host/sdhci.c b/drivers/mmc/host/sdhci.c
> index 9d4fdfa..db5d566 100644
> --- a/drivers/mmc/host/sdhci.c
> +++ b/drivers/mmc/host/sdhci.c
> @@ -496,12 +496,22 @@ static int sdhci_adma_table_pre(struct sdhci_host *host,
> WARN_ON((desc - host->adma_desc) > (128 * 2 + 1) * 4);
> }
>
> - /*
> - * Add a terminating entry.
> - */
> + if (host->quirks & SDHCI_QUIRK_NO_ENDATTR_IN_NOPDESC) {
> + /*
> + * Mark the last descriptor as the terminating descriptor
> + */
> + if (desc != host->adma_desc) {
> + desc -= 8;
> + desc[0] |= 0x2; /* end */
> + }
> + } else {
> + /*
> + * Add a terminating entry.
> + */
>
> - /* nop, end, valid */
> - sdhci_set_adma_desc(desc, 0, 0, 0x3);
> + /* nop, end, valid */
> + sdhci_set_adma_desc(desc, 0, 0, 0x3);
> + }
>
> /*
> * Resync align buffer as we might have changed it.
> diff --git a/drivers/mmc/host/sdhci.h b/drivers/mmc/host/sdhci.h
> index 842f46f..7ba0006 100644
> --- a/drivers/mmc/host/sdhci.h
> +++ b/drivers/mmc/host/sdhci.h
> @@ -236,6 +236,8 @@ struct sdhci_host {
> #define SDHCI_QUIRK_DELAY_AFTER_POWER (1<<23)
> /* Controller uses SDCLK instead of TMCLK for data timeouts */
> #define SDHCI_QUIRK_DATA_TIMEOUT_USES_SDCLK (1<<24)
> +/* Controller cannot support End Attribute in NOP ADMA descriptor */
> +#define SDHCI_QUIRK_NO_ENDATTR_IN_NOPDESC (1<<25)
>
> int irq; /* Device IRQ */
> void __iomem * ioaddr; /* Mapped address */
> --
> 1.7.1
>
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at http://vger.kernel.org/majordomo-info.html
--
--
Ben
Q: What's a light-year?
A: One-third less calories than a regular year.
--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
