On 14-09-18, 09:11, Angelo Dureghello wrote:
> This patch fixes the compilation warning reported
> during x86_64 allmodconfig build.
How does it do so, I have no clue what to expect. Please describe the
change done here in the log...
Patch title also doesn't tell me anything about the fix.
>
> Reported-By: Stephen Rothwell <sfr@xxxxxxxxxxxxxxxx>
> Signed-off-by: Angelo Dureghello <angelo@xxxxxxxx>
> ---
> Changes for v2:
> - added Reported-By
> ---
> drivers/dma/mcf-edma.c | 3 ++-
> include/linux/platform_data/dma-mcf-edma.h | 2 +-
> 2 files changed, 3 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/dma/mcf-edma.c b/drivers/dma/mcf-edma.c
> index 4d30d5302649..e08e2409a102 100644
> --- a/drivers/dma/mcf-edma.c
> +++ b/drivers/dma/mcf-edma.c
> @@ -292,8 +292,9 @@ bool mcf_edma_filter_fn(struct dma_chan *chan, void *param)
> {
> if (chan->device->dev->driver == &mcf_edma_driver.driver) {
> struct fsl_edma_chan *mcf_chan = to_fsl_edma_chan(chan);
> + unsigned int req = *(unsigned int *)param;
>
> - return (mcf_chan->slave_id == (u32)param);
> + return (mcf_chan->slave_id == req);
> }
>
> return false;
> diff --git a/include/linux/platform_data/dma-mcf-edma.h b/include/linux/platform_data/dma-mcf-edma.h
> index d718ccfa3421..97cb79bda646 100644
> --- a/include/linux/platform_data/dma-mcf-edma.h
> +++ b/include/linux/platform_data/dma-mcf-edma.h
> @@ -21,7 +21,7 @@ struct dma_slave_map;
>
> bool mcf_edma_filter_fn(struct dma_chan *chan, void *param);
>
> -#define MCF_EDMA_FILTER_PARAM(ch) ((void *)ch)
> +#define MCF_EDMA_FILTER_PARAM(ch) ((int[]) { (ch) })
I dont think you answered me about this, why is this change required and
in the context of current patch, what does it fix?
--
~Vinod
