| Message ID | 20200729162910.13196-2-laurent.pinchart@ideasonboard.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | Fix Kconfig dependency issue with DMAENGINES selection | expand |
Hi Laurent. On Wed, Jul 29, 2020 at 07:29:08PM +0300, Laurent Pinchart wrote: > Enabling a whole subsystem from a single driver 'select' is frowned > upon and won't be accepted in new drivers, that need to use 'depends on' > instead. Existing selection of DMAENGINES will then cause circular > dependencies. Replace them with a dependency. > > Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> > Acked-by: Randy Dunlap <rdunlap@infradead.org> > --- > drivers/rapidio/Kconfig | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/rapidio/Kconfig b/drivers/rapidio/Kconfig > index e4c422d806be..b9f8514909bf 100644 > --- a/drivers/rapidio/Kconfig > +++ b/drivers/rapidio/Kconfig > @@ -37,7 +37,7 @@ config RAPIDIO_ENABLE_RX_TX_PORTS > config RAPIDIO_DMA_ENGINE > bool "DMA Engine support for RapidIO" > depends on RAPIDIO > - select DMADEVICES > + depends on DMADEVICES $subject and changlog talks about DMAENGINES but the patch touches DMADEVICES?? Sam > select DMA_ENGINE > help > Say Y here if you want to use DMA Engine frameork for RapidIO data > -- > Regards, > > Laurent Pinchart > > _______________________________________________ > dri-devel mailing list > dri-devel@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/dri-devel
Hi Sam, On Wed, Jul 29, 2020 at 06:43:26PM +0200, Sam Ravnborg wrote: > On Wed, Jul 29, 2020 at 07:29:08PM +0300, Laurent Pinchart wrote: > > Enabling a whole subsystem from a single driver 'select' is frowned > > upon and won't be accepted in new drivers, that need to use 'depends on' > > instead. Existing selection of DMAENGINES will then cause circular > > dependencies. Replace them with a dependency. > > > > Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> > > Acked-by: Randy Dunlap <rdunlap@infradead.org> > > --- > > drivers/rapidio/Kconfig | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/drivers/rapidio/Kconfig b/drivers/rapidio/Kconfig > > index e4c422d806be..b9f8514909bf 100644 > > --- a/drivers/rapidio/Kconfig > > +++ b/drivers/rapidio/Kconfig > > @@ -37,7 +37,7 @@ config RAPIDIO_ENABLE_RX_TX_PORTS > > config RAPIDIO_DMA_ENGINE > > bool "DMA Engine support for RapidIO" > > depends on RAPIDIO > > - select DMADEVICES > > + depends on DMADEVICES > > $subject and changlog talks about DMAENGINES but the patch touches DMADEVICES?? Clearly a typo in the subject line, it should be DMADEVICES. I'll wait a little bit to see if there are more comments, and will send a v2. > > select DMA_ENGINE > > help > > Say Y here if you want to use DMA Engine frameork for RapidIO data
diff --git a/drivers/rapidio/Kconfig b/drivers/rapidio/Kconfig index e4c422d806be..b9f8514909bf 100644 --- a/drivers/rapidio/Kconfig +++ b/drivers/rapidio/Kconfig @@ -37,7 +37,7 @@ config RAPIDIO_ENABLE_RX_TX_PORTS config RAPIDIO_DMA_ENGINE bool "DMA Engine support for RapidIO" depends on RAPIDIO - select DMADEVICES + depends on DMADEVICES select DMA_ENGINE help Say Y here if you want to use DMA Engine frameork for RapidIO data