All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 0/3] iio: add titles to some Kconfig symbols
@ 2020-09-24 11:17 Alexandru Ardelean
  2020-09-24 11:17 ` [PATCH 1/3] iio: dma-buffer: Kconfig: Provide titles for IIO DMA " Alexandru Ardelean
                   ` (3 more replies)
  0 siblings, 4 replies; 5+ messages in thread
From: Alexandru Ardelean @ 2020-09-24 11:17 UTC (permalink / raw)
  To: linux-iio, linux-kernel; +Cc: jic23, lars, Alexandru Ardelean

For some embedded systems, a workflow involving external kernel modules
that implement IIO devices is more practical than working with in-tree
sources. However, there are number of features in the IIO KConfig that can
only be switched on by enabling a particular sensor.

This came as a request a few years back:
   https://github.com/analogdevicesinc/linux/issues/140

This patch implements that request.

After a quick run-through the iio Kconfig files, 4 seem to be useful to add
titles to.
i.e. IIO_TRIGGERED_EVENT, IIO_TRIGGERED_BUFFER, IIO_BUFFER_DMAENGINE &
IIO_BUFFER_DMA.

It's possible that for some out-of-tree drivers this could be useful.

Alexandru Ardelean (3):
  iio: dma-buffer: Kconfig: Provide titles for IIO DMA Kconfig symbols
  iio: Kconfig: Provide title for IIO_TRIGGERED_EVENT symbol
  iio: buffer: Kconfig: add title for IIO_TRIGGERED_BUFFER symbol

 drivers/iio/Kconfig        |  2 +-
 drivers/iio/buffer/Kconfig | 10 +++++-----
 2 files changed, 6 insertions(+), 6 deletions(-)

-- 
2.25.1


^ permalink raw reply	[flat|nested] 5+ messages in thread

* [PATCH 1/3] iio: dma-buffer: Kconfig: Provide titles for IIO DMA Kconfig symbols
  2020-09-24 11:17 [PATCH 0/3] iio: add titles to some Kconfig symbols Alexandru Ardelean
@ 2020-09-24 11:17 ` Alexandru Ardelean
  2020-09-24 11:17 ` [PATCH 2/3] iio: Kconfig: Provide title for IIO_TRIGGERED_EVENT symbol Alexandru Ardelean
                   ` (2 subsequent siblings)
  3 siblings, 0 replies; 5+ messages in thread
From: Alexandru Ardelean @ 2020-09-24 11:17 UTC (permalink / raw)
  To: linux-iio, linux-kernel; +Cc: jic23, lars, Alexandru Ardelean

For some embedded systems, a workflow involving external kernel modules
that implement IIO devices is more practical than working with in-tree
sources.

Kconfig symbols without any titles do not show up in menuconfig, and as
such are more difficult to configure granularly, as they need to be
selected by potentially unused/un-needed drivers.

This change adds titles to the IIO DMA Kconfig symbols to address this.

This change also updates DMAengine -> DMAEngine, which is the
correct/nitpick-y name of the framework.

Signed-off-by: Alexandru Ardelean <alexandru.ardelean@analog.com>
---
 drivers/iio/buffer/Kconfig | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/drivers/iio/buffer/Kconfig b/drivers/iio/buffer/Kconfig
index 63f265c8b466..747d21f84188 100644
--- a/drivers/iio/buffer/Kconfig
+++ b/drivers/iio/buffer/Kconfig
@@ -11,7 +11,7 @@ config IIO_BUFFER_CB
 	  usage.  That is, those where the data is pushed to the consumer.
 
 config IIO_BUFFER_DMA
-	tristate
+	tristate "Industrial I/O DMA buffer infrastructure"
 	help
 	  Provides the generic IIO DMA buffer infrastructure that can be used by
 	  drivers for devices with DMA support to implement the IIO buffer.
@@ -20,13 +20,13 @@ config IIO_BUFFER_DMA
 	  infrastructure.
 
 config IIO_BUFFER_DMAENGINE
-	tristate
+	tristate "Industrial I/O DMA buffer integration with DMAEngine"
 	select IIO_BUFFER_DMA
 	help
 	  Provides a bonding of the generic IIO DMA buffer infrastructure with the
-	  DMAengine framework. This can be used by converter drivers with a DMA port
+	  DMAEngine framework. This can be used by converter drivers with a DMA port
 	  connected to an external DMA controller which is supported by the
-	  DMAengine framework.
+	  DMAEngine framework.
 
 	  Should be selected by drivers that want to use this functionality.
 
-- 
2.25.1


^ permalink raw reply related	[flat|nested] 5+ messages in thread

* [PATCH 2/3] iio: Kconfig: Provide title for IIO_TRIGGERED_EVENT symbol
  2020-09-24 11:17 [PATCH 0/3] iio: add titles to some Kconfig symbols Alexandru Ardelean
  2020-09-24 11:17 ` [PATCH 1/3] iio: dma-buffer: Kconfig: Provide titles for IIO DMA " Alexandru Ardelean
@ 2020-09-24 11:17 ` Alexandru Ardelean
  2020-09-24 11:17 ` [PATCH 3/3] iio: buffer: Kconfig: add title for IIO_TRIGGERED_BUFFER symbol Alexandru Ardelean
  2020-09-26 15:59 ` [PATCH 0/3] iio: add titles to some Kconfig symbols Jonathan Cameron
  3 siblings, 0 replies; 5+ messages in thread
From: Alexandru Ardelean @ 2020-09-24 11:17 UTC (permalink / raw)
  To: linux-iio, linux-kernel; +Cc: jic23, lars, Alexandru Ardelean

For some embedded systems, a workflow involving external kernel modules
that implement IIO devices is more practical than working with in-tree
sources.

Kconfig symbols without any titles do not show up in menuconfig, and as
such are more difficult to configure granularly, as they need to be
selected by potentially unused/un-needed drivers.

Albeit, the IIO_TRIGGERED_EVENT is used by a single mainline driver, this
could allow for some out-of-tree drivers to use this kmod.

This change adds a title to the IIO_TRIGGERED_EVENT Kconfig symbol.

Signed-off-by: Alexandru Ardelean <alexandru.ardelean@analog.com>
---
 drivers/iio/Kconfig | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/iio/Kconfig b/drivers/iio/Kconfig
index d5c073a8aa3e..267553386c71 100644
--- a/drivers/iio/Kconfig
+++ b/drivers/iio/Kconfig
@@ -63,7 +63,7 @@ config IIO_SW_TRIGGER
 	  using the API provided.
 
 config IIO_TRIGGERED_EVENT
-	tristate
+	tristate "Enable triggered events support"
 	select IIO_TRIGGER
 	help
 	  Provides helper functions for setting up triggered events.
-- 
2.25.1


^ permalink raw reply related	[flat|nested] 5+ messages in thread

* [PATCH 3/3] iio: buffer: Kconfig: add title for IIO_TRIGGERED_BUFFER symbol
  2020-09-24 11:17 [PATCH 0/3] iio: add titles to some Kconfig symbols Alexandru Ardelean
  2020-09-24 11:17 ` [PATCH 1/3] iio: dma-buffer: Kconfig: Provide titles for IIO DMA " Alexandru Ardelean
  2020-09-24 11:17 ` [PATCH 2/3] iio: Kconfig: Provide title for IIO_TRIGGERED_EVENT symbol Alexandru Ardelean
@ 2020-09-24 11:17 ` Alexandru Ardelean
  2020-09-26 15:59 ` [PATCH 0/3] iio: add titles to some Kconfig symbols Jonathan Cameron
  3 siblings, 0 replies; 5+ messages in thread
From: Alexandru Ardelean @ 2020-09-24 11:17 UTC (permalink / raw)
  To: linux-iio, linux-kernel; +Cc: jic23, lars, Alexandru Ardelean

For some embedded systems, a workflow involving external kernel modules
that implement IIO devices is more practical than working with in-tree
sources.

Kconfig symbols without any titles do not show up in menuconfig, and as
such are more difficult to configure granularly, as they need to be
selected by potentially unused/un-needed drivers.

This change adds a title to the IIO_TRIGGERED_BUFFER Kconfig symbol.

Signed-off-by: Alexandru Ardelean <alexandru.ardelean@analog.com>
---
 drivers/iio/buffer/Kconfig | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/iio/buffer/Kconfig b/drivers/iio/buffer/Kconfig
index 747d21f84188..047b931591a9 100644
--- a/drivers/iio/buffer/Kconfig
+++ b/drivers/iio/buffer/Kconfig
@@ -48,7 +48,7 @@ config IIO_KFIFO_BUF
 	  often to read from the buffer.
 
 config IIO_TRIGGERED_BUFFER
-	tristate
+	tristate "Industrial I/O triggered buffer support"
 	select IIO_TRIGGER
 	select IIO_KFIFO_BUF
 	help
-- 
2.25.1


^ permalink raw reply related	[flat|nested] 5+ messages in thread

* Re: [PATCH 0/3] iio: add titles to some Kconfig symbols
  2020-09-24 11:17 [PATCH 0/3] iio: add titles to some Kconfig symbols Alexandru Ardelean
                   ` (2 preceding siblings ...)
  2020-09-24 11:17 ` [PATCH 3/3] iio: buffer: Kconfig: add title for IIO_TRIGGERED_BUFFER symbol Alexandru Ardelean
@ 2020-09-26 15:59 ` Jonathan Cameron
  3 siblings, 0 replies; 5+ messages in thread
From: Jonathan Cameron @ 2020-09-26 15:59 UTC (permalink / raw)
  To: Alexandru Ardelean; +Cc: linux-iio, linux-kernel, lars

On Thu, 24 Sep 2020 14:17:55 +0300
Alexandru Ardelean <alexandru.ardelean@analog.com> wrote:

> For some embedded systems, a workflow involving external kernel modules
> that implement IIO devices is more practical than working with in-tree
> sources. However, there are number of features in the IIO KConfig that can
> only be switched on by enabling a particular sensor.
> 
> This came as a request a few years back:
>    https://github.com/analogdevicesinc/linux/issues/140
> 
> This patch implements that request.
> 
> After a quick run-through the iio Kconfig files, 4 seem to be useful to add
> titles to.
> i.e. IIO_TRIGGERED_EVENT, IIO_TRIGGERED_BUFFER, IIO_BUFFER_DMAENGINE &
> IIO_BUFFER_DMA.
> 
> It's possible that for some out-of-tree drivers this could be useful.
I'm not particular keen on changes to support out of tree stuff, but
these are so trivial there is effectively 0 cost to doing it

Hence, series applied to the togreg branch of iio.git and pushed out
as testing for the autobuilders to find out if I'm wrong :)

Thanks,

Jonathan

> 
> Alexandru Ardelean (3):
>   iio: dma-buffer: Kconfig: Provide titles for IIO DMA Kconfig symbols
>   iio: Kconfig: Provide title for IIO_TRIGGERED_EVENT symbol
>   iio: buffer: Kconfig: add title for IIO_TRIGGERED_BUFFER symbol
> 
>  drivers/iio/Kconfig        |  2 +-
>  drivers/iio/buffer/Kconfig | 10 +++++-----
>  2 files changed, 6 insertions(+), 6 deletions(-)
> 


^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2020-09-26 15:59 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-09-24 11:17 [PATCH 0/3] iio: add titles to some Kconfig symbols Alexandru Ardelean
2020-09-24 11:17 ` [PATCH 1/3] iio: dma-buffer: Kconfig: Provide titles for IIO DMA " Alexandru Ardelean
2020-09-24 11:17 ` [PATCH 2/3] iio: Kconfig: Provide title for IIO_TRIGGERED_EVENT symbol Alexandru Ardelean
2020-09-24 11:17 ` [PATCH 3/3] iio: buffer: Kconfig: add title for IIO_TRIGGERED_BUFFER symbol Alexandru Ardelean
2020-09-26 15:59 ` [PATCH 0/3] iio: add titles to some Kconfig symbols Jonathan Cameron

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.