diff options
author | Julia Lawall <julia.lawall@lip6.fr> | 2015-11-13 06:46:00 -0500 |
---|---|---|
committer | Vinod Koul <vinod.koul@intel.com> | 2015-11-15 22:57:32 -0500 |
commit | 2bb129ebb23d2dfec3cd9c22dc7defd681cfcd58 (patch) | |
tree | f664dcc6cf2a09864a6ff40aedbbe1c0c9a97d71 | |
parent | d3cd63f91b84c60e0b30ee8a45b8f291dac7bbff (diff) |
dmaengine: ioatdma: constify dca_ops structures
The dca_ops structure is never modified, so declare it as const.
Done with the help of Coccinelle.
Signed-off-by: Julia Lawall <Julia.Lawall@lip6.fr>
Acked-by: Dan Williams <dan.j.williams@intel.com>
Signed-off-by: Vinod Koul <vinod.koul@intel.com>
-rw-r--r-- | drivers/dca/dca-core.c | 3 | ||||
-rw-r--r-- | drivers/dma/ioat/dca.c | 2 | ||||
-rw-r--r-- | include/linux/dca.h | 5 |
3 files changed, 6 insertions, 4 deletions
diff --git a/drivers/dca/dca-core.c b/drivers/dca/dca-core.c index 819dfda88236..7afbb28d6a0f 100644 --- a/drivers/dca/dca-core.c +++ b/drivers/dca/dca-core.c | |||
@@ -321,7 +321,8 @@ EXPORT_SYMBOL_GPL(dca_get_tag); | |||
321 | * @ops - pointer to struct of dca operation function pointers | 321 | * @ops - pointer to struct of dca operation function pointers |
322 | * @priv_size - size of extra mem to be added for provider's needs | 322 | * @priv_size - size of extra mem to be added for provider's needs |
323 | */ | 323 | */ |
324 | struct dca_provider *alloc_dca_provider(struct dca_ops *ops, int priv_size) | 324 | struct dca_provider *alloc_dca_provider(const struct dca_ops *ops, |
325 | int priv_size) | ||
325 | { | 326 | { |
326 | struct dca_provider *dca; | 327 | struct dca_provider *dca; |
327 | int alloc_size; | 328 | int alloc_size; |
diff --git a/drivers/dma/ioat/dca.c b/drivers/dma/ioat/dca.c index 2cb7c308d5c7..0b9b6b07db9e 100644 --- a/drivers/dma/ioat/dca.c +++ b/drivers/dma/ioat/dca.c | |||
@@ -224,7 +224,7 @@ static u8 ioat_dca_get_tag(struct dca_provider *dca, | |||
224 | return tag; | 224 | return tag; |
225 | } | 225 | } |
226 | 226 | ||
227 | static struct dca_ops ioat_dca_ops = { | 227 | static const struct dca_ops ioat_dca_ops = { |
228 | .add_requester = ioat_dca_add_requester, | 228 | .add_requester = ioat_dca_add_requester, |
229 | .remove_requester = ioat_dca_remove_requester, | 229 | .remove_requester = ioat_dca_remove_requester, |
230 | .get_tag = ioat_dca_get_tag, | 230 | .get_tag = ioat_dca_get_tag, |
diff --git a/include/linux/dca.h b/include/linux/dca.h index d27a7a05718d..ad956c2e07a8 100644 --- a/include/linux/dca.h +++ b/include/linux/dca.h | |||
@@ -34,7 +34,7 @@ void dca_unregister_notify(struct notifier_block *nb); | |||
34 | 34 | ||
35 | struct dca_provider { | 35 | struct dca_provider { |
36 | struct list_head node; | 36 | struct list_head node; |
37 | struct dca_ops *ops; | 37 | const struct dca_ops *ops; |
38 | struct device *cd; | 38 | struct device *cd; |
39 | int id; | 39 | int id; |
40 | }; | 40 | }; |
@@ -53,7 +53,8 @@ struct dca_ops { | |||
53 | int (*dev_managed) (struct dca_provider *, struct device *); | 53 | int (*dev_managed) (struct dca_provider *, struct device *); |
54 | }; | 54 | }; |
55 | 55 | ||
56 | struct dca_provider *alloc_dca_provider(struct dca_ops *ops, int priv_size); | 56 | struct dca_provider *alloc_dca_provider(const struct dca_ops *ops, |
57 | int priv_size); | ||
57 | void free_dca_provider(struct dca_provider *dca); | 58 | void free_dca_provider(struct dca_provider *dca); |
58 | int register_dca_provider(struct dca_provider *dca, struct device *dev); | 59 | int register_dca_provider(struct dca_provider *dca, struct device *dev); |
59 | void unregister_dca_provider(struct dca_provider *dca, struct device *dev); | 60 | void unregister_dca_provider(struct dca_provider *dca, struct device *dev); |