[U-Boot] [PATCH 1/2] mmc: at91: add multi block read/write support.
Andreas Bießmann
andreas.devel at googlemail.com
Thu Sep 13 13:11:58 CEST 2012
Dear Josh Wu,
sorry for beeing late. Bu2013.01 release.t this submission was way after
merge window close so it will anyhow end up in 2013.01 release.
On 07.09.2012 12:39, Josh Wu wrote:
> Since the at91sam9263, the mmc hardware support multi blocks read/write. So this driver enable it.
> This patch also change delay time to max value, which can avoid the timeout error in rare case.
>
> Signed-off-by: Josh Wu <josh.wu at atmel.com>
> ---
> drivers/mmc/gen_atmel_mci.c | 16 +++++++++++++---
> include/atmel_mci.h | 5 ++++-
> 2 files changed, 17 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/mmc/gen_atmel_mci.c b/drivers/mmc/gen_atmel_mci.c
> index 4968c5e..ec075f7 100644
> --- a/drivers/mmc/gen_atmel_mci.c
> +++ b/drivers/mmc/gen_atmel_mci.c
> @@ -24,7 +24,6 @@
> * Foundation, Inc., 59 Temple Place, Suite 330, Boston,
> * MA 02111-1307 USA
> */
> -
please leave this line untouched.
> #include <common.h>
> #include <mmc.h>
> #include <part.h>
> @@ -87,6 +86,11 @@ static void mci_set_mode(struct mmc *mmc, u32 hz, u32 blklen)
> | MMCI_BF(BLKLEN, blklen)
> | MMCI_BIT(RDPROOF)
> | MMCI_BIT(WRPROOF)), &mci->mr);
> + /*
> + * On some new platforms BLKLEN in mci->mr is ignored.
> + * Should use the BLKLEN in the block register.
> + */
> + writel(blklen << MMCI_BLKLEN_OFFSET, &mci->blkr);
please use the MMCI_BF macro here
> initialized = 1;
> }
>
> @@ -183,6 +187,12 @@ mci_send_cmd(struct mmc *mmc, struct mmc_cmd *cmd, struct mmc_data *data)
> /* Figure out the transfer arguments */
> cmdr = mci_encode_cmd(cmd, data, &error_flags);
>
> + /* For multi blocks read/write, set the block register */
> + if ((cmd->cmdidx == MMC_CMD_READ_MULTIPLE_BLOCK)
> + || (cmd->cmdidx == MMC_CMD_WRITE_MULTIPLE_BLOCK))
> + writel(data->blocks | mmc->read_bl_len << MMCI_BLKLEN_OFFSET,
> + &mci->blkr);
please use the MMCI_BF macros here.
> +
> /* Send the command */
> writel(cmd->cmdarg, &mci->argr);
> writel(cmdr, &mci->cmdr);
> @@ -310,8 +320,8 @@ static int mci_init(struct mmc *mmc)
> writel(MMCI_BIT(MCIEN), &mci->cr); /* enable mci */
> writel(MMCI_BF(SCDSEL, MCI_BUS), &mci->sdcr); /* select port */
>
> - /* Initial Time-outs */
> - writel(0x5f, &mci->dtor);
> + /* This delay can be optimized, but stick with max value */
> + writel(0x7f, &mci->dtor);
can we split this in a separate patch?
> /* Disable Interrupts */
> writel(~0UL, &mci->idr);
>
> diff --git a/include/atmel_mci.h b/include/atmel_mci.h
> index 3dd5d67..85468d4 100644
> --- a/include/atmel_mci.h
> +++ b/include/atmel_mci.h
> @@ -38,7 +38,7 @@ typedef struct atmel_mci {
> u32 sdcr; /* 0x0c */
> u32 argr; /* 0x10 */
> u32 cmdr; /* 0x14 */
> - u32 _18; /* 0x18 */
> + u32 blkr; /* 0x18 */
> u32 _1c; /* 0x1c */
> u32 rspr; /* 0x20 */
> u32 rspr1; /* 0x24 */
> @@ -118,6 +118,9 @@ typedef struct atmel_mci {
> #define MMCI_TRTYP_OFFSET 19
> #define MMCI_TRTYP_SIZE 2
>
> +/* Bitfields in BLKR */
> +#define MMCI_BLKLEN_OFFSET 16
NAK, this is defined some lines up (under MR), please add an reminder
here that it is defined under MR. I wonder if this will produce 'defined
previously' warnings.
> +
> /* Bitfields in RSPRx */
> #define MMCI_RSP_OFFSET 0
> #define MMCI_RSP_SIZE 32
>
Best regards
Andreas Bießmann
More information about the U-Boot
mailing list