* [PATCH] mmc: renesas_sdhi_sys_dmac: use DMA_SLAVE_BUSWIDTH_UNDEFINED
@ 2022-01-21 20:38 Sergey Shtylyov
2022-01-31 16:26 ` Ulf Hansson
0 siblings, 1 reply; 3+ messages in thread
From: Sergey Shtylyov @ 2022-01-21 20:38 UTC (permalink / raw)
To: Wolfram Sang, Ulf Hansson, linux-mmc
The 'dma_slave_config::{src|dst}_addr_width' fields have the *enum* type
which isn't isomorphic with the *bool* type, however is used as a boolean
expression. Use the *enum* dma_slave_buswidth's value corresponding to 0
instead.
Found by Linux Verification Center (linuxtesting.org) with the SVACE static
analysis tool.
Signed-off-by: Sergey Shtylyov <s.shtylyov@omp.ru>
---
This patch is against the 'next' branch of Ulf Hansson's 'mmc.git' repo.
drivers/mmc/host/renesas_sdhi_sys_dmac.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
Index: mmc/drivers/mmc/host/renesas_sdhi_sys_dmac.c
===================================================================
--- mmc.orig/drivers/mmc/host/renesas_sdhi_sys_dmac.c
+++ mmc/drivers/mmc/host/renesas_sdhi_sys_dmac.c
@@ -370,7 +370,7 @@ static void renesas_sdhi_sys_dmac_reques
cfg.dst_addr = res->start +
(CTL_SD_DATA_PORT << host->bus_shift);
cfg.dst_addr_width = priv->dma_priv.dma_buswidth;
- if (!cfg.dst_addr_width)
+ if (cfg.dst_addr_width == DMA_SLAVE_BUSWIDTH_UNDEFINED)
cfg.dst_addr_width = DMA_SLAVE_BUSWIDTH_2_BYTES;
cfg.src_addr = 0;
ret = dmaengine_slave_config(host->chan_tx, &cfg);
@@ -389,7 +389,7 @@ static void renesas_sdhi_sys_dmac_reques
cfg.direction = DMA_DEV_TO_MEM;
cfg.src_addr = cfg.dst_addr + host->pdata->dma_rx_offset;
cfg.src_addr_width = priv->dma_priv.dma_buswidth;
- if (!cfg.src_addr_width)
+ if (cfg.src_addr_width == DMA_SLAVE_BUSWIDTH_UNDEFINED)
cfg.src_addr_width = DMA_SLAVE_BUSWIDTH_2_BYTES;
cfg.dst_addr = 0;
ret = dmaengine_slave_config(host->chan_rx, &cfg);
^ permalink raw reply [flat|nested] 3+ messages in thread* Re: [PATCH] mmc: renesas_sdhi_sys_dmac: use DMA_SLAVE_BUSWIDTH_UNDEFINED
2022-01-21 20:38 [PATCH] mmc: renesas_sdhi_sys_dmac: use DMA_SLAVE_BUSWIDTH_UNDEFINED Sergey Shtylyov
@ 2022-01-31 16:26 ` Ulf Hansson
2022-02-02 4:13 ` Vinod Koul
0 siblings, 1 reply; 3+ messages in thread
From: Ulf Hansson @ 2022-01-31 16:26 UTC (permalink / raw)
To: Sergey Shtylyov, Vinod Koul; +Cc: Wolfram Sang, linux-mmc
+ Vinod
On Fri, 21 Jan 2022 at 21:38, Sergey Shtylyov <s.shtylyov@omp.ru> wrote:
>
> The 'dma_slave_config::{src|dst}_addr_width' fields have the *enum* type
> which isn't isomorphic with the *bool* type, however is used as a boolean
> expression. Use the *enum* dma_slave_buswidth's value corresponding to 0
> instead.
>
> Found by Linux Verification Center (linuxtesting.org) with the SVACE static
> analysis tool.
I have no strong opinion, but just wanted to make sure this is inline
with what Vinod wants.
There are not so many users checking DMA_SLAVE_BUSWIDTH_UNDEFINED,
perhaps we should simply drop it and instead leave the checks below as
is?
>
> Signed-off-by: Sergey Shtylyov <s.shtylyov@omp.ru>
Kind regards
Uffe
>
> ---
> This patch is against the 'next' branch of Ulf Hansson's 'mmc.git' repo.
>
> drivers/mmc/host/renesas_sdhi_sys_dmac.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> Index: mmc/drivers/mmc/host/renesas_sdhi_sys_dmac.c
> ===================================================================
> --- mmc.orig/drivers/mmc/host/renesas_sdhi_sys_dmac.c
> +++ mmc/drivers/mmc/host/renesas_sdhi_sys_dmac.c
> @@ -370,7 +370,7 @@ static void renesas_sdhi_sys_dmac_reques
> cfg.dst_addr = res->start +
> (CTL_SD_DATA_PORT << host->bus_shift);
> cfg.dst_addr_width = priv->dma_priv.dma_buswidth;
> - if (!cfg.dst_addr_width)
> + if (cfg.dst_addr_width == DMA_SLAVE_BUSWIDTH_UNDEFINED)
> cfg.dst_addr_width = DMA_SLAVE_BUSWIDTH_2_BYTES;
> cfg.src_addr = 0;
> ret = dmaengine_slave_config(host->chan_tx, &cfg);
> @@ -389,7 +389,7 @@ static void renesas_sdhi_sys_dmac_reques
> cfg.direction = DMA_DEV_TO_MEM;
> cfg.src_addr = cfg.dst_addr + host->pdata->dma_rx_offset;
> cfg.src_addr_width = priv->dma_priv.dma_buswidth;
> - if (!cfg.src_addr_width)
> + if (cfg.src_addr_width == DMA_SLAVE_BUSWIDTH_UNDEFINED)
> cfg.src_addr_width = DMA_SLAVE_BUSWIDTH_2_BYTES;
> cfg.dst_addr = 0;
> ret = dmaengine_slave_config(host->chan_rx, &cfg);
^ permalink raw reply [flat|nested] 3+ messages in thread* Re: [PATCH] mmc: renesas_sdhi_sys_dmac: use DMA_SLAVE_BUSWIDTH_UNDEFINED
2022-01-31 16:26 ` Ulf Hansson
@ 2022-02-02 4:13 ` Vinod Koul
0 siblings, 0 replies; 3+ messages in thread
From: Vinod Koul @ 2022-02-02 4:13 UTC (permalink / raw)
To: Ulf Hansson; +Cc: Sergey Shtylyov, Wolfram Sang, linux-mmc
On 31-01-22, 17:26, Ulf Hansson wrote:
> + Vinod
>
> On Fri, 21 Jan 2022 at 21:38, Sergey Shtylyov <s.shtylyov@omp.ru> wrote:
> >
> > The 'dma_slave_config::{src|dst}_addr_width' fields have the *enum* type
> > which isn't isomorphic with the *bool* type, however is used as a boolean
> > expression. Use the *enum* dma_slave_buswidth's value corresponding to 0
> > instead.
> >
> > Found by Linux Verification Center (linuxtesting.org) with the SVACE static
> > analysis tool.
>
> I have no strong opinion, but just wanted to make sure this is inline
> with what Vinod wants.
>
> There are not so many users checking DMA_SLAVE_BUSWIDTH_UNDEFINED,
> perhaps we should simply drop it and instead leave the checks below as
> is?
DMA_SLAVE_BUSWIDTH_UNDEFINED was defined as 0.
I think it might be overkill and should be fine to keep the check as
is..
>
> >
> > Signed-off-by: Sergey Shtylyov <s.shtylyov@omp.ru>
>
> Kind regards
> Uffe
>
> >
> > ---
> > This patch is against the 'next' branch of Ulf Hansson's 'mmc.git' repo.
> >
> > drivers/mmc/host/renesas_sdhi_sys_dmac.c | 4 ++--
> > 1 file changed, 2 insertions(+), 2 deletions(-)
> >
> > Index: mmc/drivers/mmc/host/renesas_sdhi_sys_dmac.c
> > ===================================================================
> > --- mmc.orig/drivers/mmc/host/renesas_sdhi_sys_dmac.c
> > +++ mmc/drivers/mmc/host/renesas_sdhi_sys_dmac.c
> > @@ -370,7 +370,7 @@ static void renesas_sdhi_sys_dmac_reques
> > cfg.dst_addr = res->start +
> > (CTL_SD_DATA_PORT << host->bus_shift);
> > cfg.dst_addr_width = priv->dma_priv.dma_buswidth;
> > - if (!cfg.dst_addr_width)
> > + if (cfg.dst_addr_width == DMA_SLAVE_BUSWIDTH_UNDEFINED)
> > cfg.dst_addr_width = DMA_SLAVE_BUSWIDTH_2_BYTES;
> > cfg.src_addr = 0;
> > ret = dmaengine_slave_config(host->chan_tx, &cfg);
> > @@ -389,7 +389,7 @@ static void renesas_sdhi_sys_dmac_reques
> > cfg.direction = DMA_DEV_TO_MEM;
> > cfg.src_addr = cfg.dst_addr + host->pdata->dma_rx_offset;
> > cfg.src_addr_width = priv->dma_priv.dma_buswidth;
> > - if (!cfg.src_addr_width)
> > + if (cfg.src_addr_width == DMA_SLAVE_BUSWIDTH_UNDEFINED)
> > cfg.src_addr_width = DMA_SLAVE_BUSWIDTH_2_BYTES;
> > cfg.dst_addr = 0;
> > ret = dmaengine_slave_config(host->chan_rx, &cfg);
--
~Vinod
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2022-02-02 4:20 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-01-21 20:38 [PATCH] mmc: renesas_sdhi_sys_dmac: use DMA_SLAVE_BUSWIDTH_UNDEFINED Sergey Shtylyov
2022-01-31 16:26 ` Ulf Hansson
2022-02-02 4:13 ` Vinod Koul
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.