Hi Morimoto-san,
On Fri, Feb 20, 2015 at 11:22 AM, Kuninori Morimoto kuninori.morimoto.gx@renesas.com wrote:
--- a/include/linux/of_dma.h +++ b/include/linux/of_dma.h @@ -37,8 +37,8 @@ extern int of_dma_controller_register(struct device_node *np, (struct of_phandle_args *, struct of_dma *), void *data); extern void of_dma_controller_free(struct device_node *np); -extern struct dma_chan *of_dma_request_slave_channel(struct device_node *np,
const char *name);+struct dma_chan *of_dma_request_slave_channel(struct device_node *np,
const char *name);
Why have you dropped the "extern" keyword? Checkpach? All prototypes in the neighborhood have "extern" keywords, so I think it's better to keep it for consistency.
extern struct dma_chan *of_dma_simple_xlate(struct of_phandle_args *dma_spec, struct of_dma *ofdma); extern struct dma_chan *of_dma_xlate_by_chan_id(struct of_phandle_args *dma_spec,