On Thu, 10 Oct 2013, Will Deacon wrote: > On Wed, Oct 09, 2013 at 09:48:15PM +0100, Stefano Stabellini wrote: > > Signed-off-by: Stefano Stabellini <stefano.stabell...@eu.citrix.com> > > Acked-by: Catalin Marinas <catalin.mari...@arm.com> > > CC: will.dea...@arm.com > > --- > > arch/arm64/include/asm/dma-mapping.h | 1 + > > 1 files changed, 1 insertions(+), 0 deletions(-) > > > > diff --git a/arch/arm64/include/asm/dma-mapping.h > > b/arch/arm64/include/asm/dma-mapping.h > > index 8d18100..c2cb8a0 100644 > > --- a/arch/arm64/include/asm/dma-mapping.h > > +++ b/arch/arm64/include/asm/dma-mapping.h > > @@ -25,6 +25,7 @@ > > > > #define ARCH_HAS_DMA_GET_REQUIRED_MASK > > > > +#define DMA_ERROR_CODE (~0) > > Hmm, is this guaranteed to be large enough for a dma_addr_t? Other 64-bit > architectures using a non-zero error code cast to dma_addr_t before the > negation.
Good catch! I'll fix it. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/