diff options
author | Russell King <rmk+kernel@arm.linux.org.uk> | 2012-05-17 10:01:28 -0400 |
---|---|---|
committer | Russell King <rmk+kernel@arm.linux.org.uk> | 2012-07-01 09:15:38 -0400 |
commit | aeea1808fe752e917b966961bde3e9603f206dec (patch) | |
tree | a826ec3a86dd99203eb30dded5b2b82ac13db0bb /include | |
parent | 48afb3112e6373a292e54d675e986a5da14c0516 (diff) |
dmaengine: PL08x: clean up get_signal/put_signal
Try to avoid dereferencing the DMA engine's channel struct in these
platform helpers; instead, pass a pointer to the channel data into
get_signal(), and the returned signal number to put_signal().
Tested-by: Linus Walleij <linus.walleij@linaro.org>
Signed-off-by: Russell King <rmk+kernel@arm.linux.org.uk>
Diffstat (limited to 'include')
-rw-r--r-- | include/linux/amba/pl08x.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/include/linux/amba/pl08x.h b/include/linux/amba/pl08x.h index 0f5b34d668b6..88765a62c8f2 100644 --- a/include/linux/amba/pl08x.h +++ b/include/linux/amba/pl08x.h | |||
@@ -225,8 +225,8 @@ struct pl08x_platform_data { | |||
225 | const struct pl08x_channel_data *slave_channels; | 225 | const struct pl08x_channel_data *slave_channels; |
226 | unsigned int num_slave_channels; | 226 | unsigned int num_slave_channels; |
227 | struct pl08x_channel_data memcpy_channel; | 227 | struct pl08x_channel_data memcpy_channel; |
228 | int (*get_signal)(struct pl08x_dma_chan *); | 228 | int (*get_signal)(const struct pl08x_channel_data *); |
229 | void (*put_signal)(struct pl08x_dma_chan *); | 229 | void (*put_signal)(const struct pl08x_channel_data *, int); |
230 | u8 lli_buses; | 230 | u8 lli_buses; |
231 | u8 mem_buses; | 231 | u8 mem_buses; |
232 | }; | 232 | }; |