* [PATCH] ata: ep93xx: Use proper enums for directions @ 2018-10-04 2:37 ` Nathan Chancellor 2018-10-04 11:48 ` Bartlomiej Zolnierkiewicz 2018-10-04 14:26 ` Jens Axboe 0 siblings, 2 replies; 3+ messages in thread From: Nathan Chancellor @ 2018-10-04 2:37 UTC (permalink / raw) To: Bartlomiej Zolnierkiewicz, Jens Axboe Cc: linux-ide, linux-kernel, Nick Desaulniers, Nathan Chancellor Clang warns when one enumerated type is implicitly converted to another. drivers/ata/pata_ep93xx.c:662:36: warning: implicit conversion from enumeration type 'enum dma_data_direction' to different enumeration type 'enum dma_transfer_direction' [-Wenum-conversion] drv_data->dma_rx_data.direction = DMA_FROM_DEVICE; ~ ^~~~~~~~~~~~~~~ drivers/ata/pata_ep93xx.c:670:36: warning: implicit conversion from enumeration type 'enum dma_data_direction' to different enumeration type 'enum dma_transfer_direction' [-Wenum-conversion] drv_data->dma_tx_data.direction = DMA_TO_DEVICE; ~ ^~~~~~~~~~~~~ drivers/ata/pata_ep93xx.c:681:19: warning: implicit conversion from enumeration type 'enum dma_data_direction' to different enumeration type 'enum dma_transfer_direction' [-Wenum-conversion] conf.direction = DMA_FROM_DEVICE; ~ ^~~~~~~~~~~~~~~ drivers/ata/pata_ep93xx.c:692:19: warning: implicit conversion from enumeration type 'enum dma_data_direction' to different enumeration type 'enum dma_transfer_direction' [-Wenum-conversion] conf.direction = DMA_TO_DEVICE; ~ ^~~~~~~~~~~~~ Use the equivalent valued enums from the expected type so that Clang no longer warns about a conversion. DMA_TO_DEVICE = DMA_MEM_TO_DEV = 1 DMA_FROM_DEVICE = DMA_DEV_TO_MEM = 2 Signed-off-by: Nathan Chancellor <natechancellor@gmail.com> --- drivers/ata/pata_ep93xx.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/ata/pata_ep93xx.c b/drivers/ata/pata_ep93xx.c index 0a550190955a..cc6d06c1b2c7 100644 --- a/drivers/ata/pata_ep93xx.c +++ b/drivers/ata/pata_ep93xx.c @@ -659,7 +659,7 @@ static void ep93xx_pata_dma_init(struct ep93xx_pata_data *drv_data) * start of new transfer. */ drv_data->dma_rx_data.port = EP93XX_DMA_IDE; - drv_data->dma_rx_data.direction = DMA_FROM_DEVICE; + drv_data->dma_rx_data.direction = DMA_DEV_TO_MEM; drv_data->dma_rx_data.name = "ep93xx-pata-rx"; drv_data->dma_rx_channel = dma_request_channel(mask, ep93xx_pata_dma_filter, &drv_data->dma_rx_data); @@ -667,7 +667,7 @@ static void ep93xx_pata_dma_init(struct ep93xx_pata_data *drv_data) return; drv_data->dma_tx_data.port = EP93XX_DMA_IDE; - drv_data->dma_tx_data.direction = DMA_TO_DEVICE; + drv_data->dma_tx_data.direction = DMA_MEM_TO_DEV; drv_data->dma_tx_data.name = "ep93xx-pata-tx"; drv_data->dma_tx_channel = dma_request_channel(mask, ep93xx_pata_dma_filter, &drv_data->dma_tx_data); @@ -678,7 +678,7 @@ static void ep93xx_pata_dma_init(struct ep93xx_pata_data *drv_data) /* Configure receive channel direction and source address */ memset(&conf, 0, sizeof(conf)); - conf.direction = DMA_FROM_DEVICE; + conf.direction = DMA_DEV_TO_MEM; conf.src_addr = drv_data->udma_in_phys; conf.src_addr_width = DMA_SLAVE_BUSWIDTH_4_BYTES; if (dmaengine_slave_config(drv_data->dma_rx_channel, &conf)) { @@ -689,7 +689,7 @@ static void ep93xx_pata_dma_init(struct ep93xx_pata_data *drv_data) /* Configure transmit channel direction and destination address */ memset(&conf, 0, sizeof(conf)); - conf.direction = DMA_TO_DEVICE; + conf.direction = DMA_MEM_TO_DEV; conf.dst_addr = drv_data->udma_out_phys; conf.dst_addr_width = DMA_SLAVE_BUSWIDTH_4_BYTES; if (dmaengine_slave_config(drv_data->dma_tx_channel, &conf)) { -- 2.19.0 ^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] ata: ep93xx: Use proper enums for directions 2018-10-04 2:37 ` [PATCH] ata: ep93xx: Use proper enums for directions Nathan Chancellor @ 2018-10-04 11:48 ` Bartlomiej Zolnierkiewicz 2018-10-04 14:26 ` Jens Axboe 1 sibling, 0 replies; 3+ messages in thread From: Bartlomiej Zolnierkiewicz @ 2018-10-04 11:48 UTC (permalink / raw) To: Nathan Chancellor; +Cc: Jens Axboe, linux-ide, linux-kernel, Nick Desaulniers On 10/04/2018 04:37 AM, Nathan Chancellor wrote: > Clang warns when one enumerated type is implicitly converted to another. > > drivers/ata/pata_ep93xx.c:662:36: warning: implicit conversion from > enumeration type 'enum dma_data_direction' to different enumeration type > 'enum dma_transfer_direction' [-Wenum-conversion] > drv_data->dma_rx_data.direction = DMA_FROM_DEVICE; > ~ ^~~~~~~~~~~~~~~ > drivers/ata/pata_ep93xx.c:670:36: warning: implicit conversion from > enumeration type 'enum dma_data_direction' to different enumeration type > 'enum dma_transfer_direction' [-Wenum-conversion] > drv_data->dma_tx_data.direction = DMA_TO_DEVICE; > ~ ^~~~~~~~~~~~~ > drivers/ata/pata_ep93xx.c:681:19: warning: implicit conversion from > enumeration type 'enum dma_data_direction' to different enumeration type > 'enum dma_transfer_direction' [-Wenum-conversion] > conf.direction = DMA_FROM_DEVICE; > ~ ^~~~~~~~~~~~~~~ > drivers/ata/pata_ep93xx.c:692:19: warning: implicit conversion from > enumeration type 'enum dma_data_direction' to different enumeration type > 'enum dma_transfer_direction' [-Wenum-conversion] > conf.direction = DMA_TO_DEVICE; > ~ ^~~~~~~~~~~~~ > > Use the equivalent valued enums from the expected type so that Clang no > longer warns about a conversion. > > DMA_TO_DEVICE = DMA_MEM_TO_DEV = 1 > DMA_FROM_DEVICE = DMA_DEV_TO_MEM = 2 > > Signed-off-by: Nathan Chancellor <natechancellor@gmail.com> Acked-by: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com> Best regards, -- Bartlomiej Zolnierkiewicz Samsung R&D Institute Poland Samsung Electronics ^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] ata: ep93xx: Use proper enums for directions 2018-10-04 2:37 ` [PATCH] ata: ep93xx: Use proper enums for directions Nathan Chancellor 2018-10-04 11:48 ` Bartlomiej Zolnierkiewicz @ 2018-10-04 14:26 ` Jens Axboe 1 sibling, 0 replies; 3+ messages in thread From: Jens Axboe @ 2018-10-04 14:26 UTC (permalink / raw) To: Nathan Chancellor, Bartlomiej Zolnierkiewicz Cc: linux-ide, linux-kernel, Nick Desaulniers On 10/3/18 8:37 PM, Nathan Chancellor wrote: > Clang warns when one enumerated type is implicitly converted to another. > > drivers/ata/pata_ep93xx.c:662:36: warning: implicit conversion from > enumeration type 'enum dma_data_direction' to different enumeration type > 'enum dma_transfer_direction' [-Wenum-conversion] > drv_data->dma_rx_data.direction = DMA_FROM_DEVICE; > ~ ^~~~~~~~~~~~~~~ > drivers/ata/pata_ep93xx.c:670:36: warning: implicit conversion from > enumeration type 'enum dma_data_direction' to different enumeration type > 'enum dma_transfer_direction' [-Wenum-conversion] > drv_data->dma_tx_data.direction = DMA_TO_DEVICE; > ~ ^~~~~~~~~~~~~ > drivers/ata/pata_ep93xx.c:681:19: warning: implicit conversion from > enumeration type 'enum dma_data_direction' to different enumeration type > 'enum dma_transfer_direction' [-Wenum-conversion] > conf.direction = DMA_FROM_DEVICE; > ~ ^~~~~~~~~~~~~~~ > drivers/ata/pata_ep93xx.c:692:19: warning: implicit conversion from > enumeration type 'enum dma_data_direction' to different enumeration type > 'enum dma_transfer_direction' [-Wenum-conversion] > conf.direction = DMA_TO_DEVICE; > ~ ^~~~~~~~~~~~~ > > Use the equivalent valued enums from the expected type so that Clang no > longer warns about a conversion. > > DMA_TO_DEVICE = DMA_MEM_TO_DEV = 1 > DMA_FROM_DEVICE = DMA_DEV_TO_MEM = 2 Applied, thanks. -- Jens Axboe ^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2018-10-04 14:26 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <CGME20181004023814epcas3p163f7ce4d927c86d837015cdbe58c2936@epcas3p1.samsung.com>
2018-10-04 2:37 ` [PATCH] ata: ep93xx: Use proper enums for directions Nathan Chancellor
2018-10-04 11:48 ` Bartlomiej Zolnierkiewicz
2018-10-04 14:26 ` Jens Axboe
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox