diff options
| author | Nathan Chancellor <natechancellor@gmail.com> | 2018-10-25 20:05:25 +0200 |
|---|---|---|
| committer | Vinod Koul <vkoul@kernel.org> | 2018-11-24 14:41:20 +0100 |
| commit | aeaebcc17cdf37065d2693865eeb1ff1c7dc5bf3 (patch) | |
| tree | d08a8dc8518a5ff4634218bda3ba9ba6bd9a6bc0 /drivers/dma/xilinx | |
| parent | dmaengine: xilinx_dma: Fix 64-bit simple CDMA transfer (diff) | |
| download | linux-aeaebcc17cdf37065d2693865eeb1ff1c7dc5bf3.tar.xz linux-aeaebcc17cdf37065d2693865eeb1ff1c7dc5bf3.zip | |
dmaengine: xilinx_dma: Remove __aligned attribute on zynqmp_dma_desc_ll
Clang warns:
drivers/dma/xilinx/zynqmp_dma.c:166:4: warning: attribute 'aligned' is
ignored, place it after "struct" to apply attribute to type declaration
[-Wignored-attributes]
}; __aligned(64)
^
./include/linux/compiler_types.h:200:38: note: expanded from macro
'__aligned'
^
1 warning generated.
As Nick pointed out in the previous version of this patch, the author
likely intended for this struct to be 8-byte (64-bit) aligned, not
64-byte, which is the default. Remove the hanging __aligned attribute.
Fixes: b0cc417c1637 ("dmaengine: Add Xilinx zynqmp dma engine driver support")
Reported-by: Nick Desaulniers <ndesaulniers@google.com>
Suggested-by: Nick Desaulniers <ndesaulniers@google.com>
Signed-off-by: Nathan Chancellor <natechancellor@gmail.com>
Reviewed-by: Nick Desaulniers <ndesaulniers@google.com>
Signed-off-by: Vinod Koul <vkoul@kernel.org>
Diffstat (limited to 'drivers/dma/xilinx')
| -rw-r--r-- | drivers/dma/xilinx/zynqmp_dma.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/dma/xilinx/zynqmp_dma.c b/drivers/dma/xilinx/zynqmp_dma.c index c74a88b65039..73de6a6179fc 100644 --- a/drivers/dma/xilinx/zynqmp_dma.c +++ b/drivers/dma/xilinx/zynqmp_dma.c @@ -163,7 +163,7 @@ struct zynqmp_dma_desc_ll { u32 ctrl; u64 nxtdscraddr; u64 rsvd; -}; __aligned(64) +}; /** * struct zynqmp_dma_desc_sw - Per Transaction structure |