Re: [PATCH 2/4] ARM: shmobile: dma: Use defines instead of hardcoded numbers

From: Simon Horman
Date: Tue Jul 15 2014 - 05:02:22 EST


On Fri, Jun 20, 2014 at 02:37:39PM +0200, Geert Uytterhoeven wrote:
> Signed-off-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx>
> ---
> arch/arm/mach-shmobile/dma-register.h | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/arch/arm/mach-shmobile/dma-register.h b/arch/arm/mach-shmobile/dma-register.h
> index 97c40bd9b94f..52a2f66e600f 100644
> --- a/arch/arm/mach-shmobile/dma-register.h
> +++ b/arch/arm/mach-shmobile/dma-register.h
> @@ -52,8 +52,8 @@ static const unsigned int dma_ts_shift[] = {
> ((((i) & TS_LOW_BIT) << TS_LOW_SHIFT) |\
> (((i) & TS_HI_BIT) << TS_HI_SHIFT))
>
> -#define CHCR_TX(xmit_sz) (DM_FIX | SM_INC | 0x800 | TS_INDEX2VAL((xmit_sz)))
> -#define CHCR_RX(xmit_sz) (DM_INC | SM_FIX | 0x800 | TS_INDEX2VAL((xmit_sz)))
> +#define CHCR_TX(xmit_sz) (DM_FIX | SM_INC | RS_ERS | TS_INDEX2VAL((xmit_sz)))
> +#define CHCR_RX(xmit_sz) (DM_INC | SM_FIX | RS_ERS | TS_INDEX2VAL((xmit_sz)))

I am dropping this patch from the shdma-for-v3.17 branch (which I am
preparing for Vinod) as it depends on a patch in another branch of my tree
(which I am preparing for arm-SoC).

Could you please repost this patch once the other patches
in this tree have been accepted by Vinod.

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/