Re: [PATCH] dma: mmp_tdma: add multiple burst size support for 910-squ

From: zhangfei gao
Date: Thu Oct 10 2013 - 09:54:34 EST


On Wed, Oct 9, 2013 at 1:40 PM, Qiao Zhou <zhouqiao@xxxxxxxxxxx> wrote:
> add multiple burst size support for 910-squ.
>
> Signed-off-by: Qiao Zhou <zhouqiao@xxxxxxxxxxx>
> ---
> drivers/dma/mmp_tdma.c | 25 ++++++++++++++++++++++++-
> 1 files changed, 24 insertions(+), 1 deletions(-)
>
> diff --git a/drivers/dma/mmp_tdma.c b/drivers/dma/mmp_tdma.c
> index 38cb517..d84354b 100644
> --- a/drivers/dma/mmp_tdma.c
> +++ b/drivers/dma/mmp_tdma.c
> @@ -228,8 +228,31 @@ static int mmp_tdma_config_chan(struct mmp_tdma_chan *tdmac)
> return -EINVAL;
> }
> } else if (tdmac->type == PXA910_SQU) {
> - tdcr |= TDCR_BURSTSZ_SQU_32B;
> tdcr |= TDCR_SSPMOD;
> +
> + switch (tdmac->burst_sz) {
> + case 1:
> + tdcr |= TDCR_BURSTSZ_SQU_1B;
> + break;
> + case 2:
> + tdcr |= TDCR_BURSTSZ_SQU_2B;
> + break;
> + case 4:
> + tdcr |= TDCR_BURSTSZ_SQU_4B;
> + break;
> + case 8:
> + tdcr |= TDCR_BURSTSZ_SQU_8B;
> + break;
> + case 16:
> + tdcr |= TDCR_BURSTSZ_SQU_16B;
> + break;
> + case 32:
> + tdcr |= TDCR_BURSTSZ_SQU_32B;
> + break;
> + default:
> + dev_err(tdmac->dev, "mmp_tdma: unknown burst size.\n");
> + return -EINVAL;
> + }

Sorry, do I make mistake,
Not find definition TDCR_BURSTSZ_SQU_16B ~
Only find
#define TDCR_BURSTSZ_SQU_32B (0x7 << 6)

Thanks
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/