diff options
author | Lars-Peter Clausen <lars@metafoo.de> | 2013-04-22 10:33:33 +0200 |
---|---|---|
committer | Vinod Koul <vinod.koul@intel.com> | 2013-06-12 07:49:19 +0530 |
commit | 8552bb4f16800d5ebc176a2cf5f2aa55b22731ea (patch) | |
tree | 307a72f913ce3e887c0ae3d5a0962039ff81db93 /include/linux/of_dma.h | |
parent | ff0e0f4f568e8d7593e0035c0c58067bcaf4ab07 (diff) | |
download | lwn-8552bb4f16800d5ebc176a2cf5f2aa55b22731ea.tar.gz lwn-8552bb4f16800d5ebc176a2cf5f2aa55b22731ea.zip |
dma: of: Remove check on always true condition
Both of_dma_nbcells field of the of_dma_controller and the args_count field of
the dma_spec are initialized by parsing the #dma-cells attribute of their device
tree node. So if the device tree nodes of a DMA controller and the dma_spec
match this means that of_dma_nbcells and args_count will also match. So the
second test in the of_dma_find_controller loop is redundant because given the
first test yields true the second test will also yield true. So we can safely
remove the test whether of_dma_nbcells matches args_count. Since this was the
last user of the of_dma_nbcells field we can remove it altogether.
Signed-off-by: Lars-Peter Clausen <lars@metafoo.de>
Acked-by: Arnd Bergmann <arnd@arndb.de>
Signed-off-by: Vinod Koul <vinod.koul@intel.com>
Diffstat (limited to 'include/linux/of_dma.h')
-rw-r--r-- | include/linux/of_dma.h | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/include/linux/of_dma.h b/include/linux/of_dma.h index 364dda734877..ae36298ba076 100644 --- a/include/linux/of_dma.h +++ b/include/linux/of_dma.h @@ -21,7 +21,6 @@ struct device_node; struct of_dma { struct list_head of_dma_controllers; struct device_node *of_node; - int of_dma_nbcells; struct dma_chan *(*of_dma_xlate) (struct of_phandle_args *, struct of_dma *); void *of_dma_data; |