dmaengine: altera: Use macros instead of structs to describe the registers
authorStefan Roese <sr@denx.de>
Sat, 26 Aug 2017 07:59:07 +0000 (09:59 +0200)
committerVinod Koul <vinod.koul@intel.com>
Tue, 29 Aug 2017 02:47:43 +0000 (08:17 +0530)
commit6084fc2ec478dbe5ab027c8dc233ead57b6032d8
treed2cd54ddc5af73577acb1eff3406a1f919e02710
parent491bea00a13600eaa146923b910b70e9a326e950
dmaengine: altera: Use macros instead of structs to describe the registers

This patch moves from a struct declaration for the DMA controller
registers to macros with offests to the base address. This is mainly
done to remove the sparse warnings, since the function parameter of
ioread32/iowrite32 is "void __iomem *" instead of a pointer to struct
members. With this patch applied, no sparse warning is seen anymore.

Please note that the struct for the descriptors is still kept in place,
as the code largely accesses the struct members as internal variables
before the complete struct is copied into the descriptor FIFO of the
DMA controller.

Additionally this patch also removes two warnings "variable xxx set but
not used" seen when compiling with "W=1". The registers need to be read
to flush the response FIFO, but nothing needs to be done with them. So
the code is correct here and the warning is a false one.

Signed-off-by: Stefan Roese <sr@denx.de>
Signed-off-by: Vinod Koul <vinod.koul@intel.com>
drivers/dma/altera-msgdma.c