[PATCH] counter: Fix menuconfig "Counter support" submenu entries disappearance

[PATCH] counter: Fix menuconfig "Counter support" submenu entries disappearance

[William Breathitt Gray]

The current placement of the I8254 Kconfig entry results in the
disappearance of the "Counter support" submenu items in menuconfig. Move
the I8254 above the menuconfig COUNTER entry to restore the intended
submenu behavior.

Fixes: d428487471ba ("counter: i8254: Introduce the Intel 8254 interface library module")
Reported-by: Jarkko Nikula <jarkko.nikula@linux.intel.com>
Closes: https://lore.kernel.org/all/32ddaa7b-53a8-d61f-d526-b545bd561337@linux.intel.com/
Signed-off-by: William Breathitt Gray <william.gray@linaro.org>
---
 drivers/counter/Kconfig | 14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

diff --git a/drivers/counter/Kconfig b/drivers/counter/Kconfig
index a61a4b9b8ec6..86536c2cc531 100644
--- a/drivers/counter/Kconfig
+++ b/drivers/counter/Kconfig
@@ -3,13 +3,6 @@
 # Counter devices
 #
 
-menuconfig COUNTER
-	tristate "Counter support"
-	help
-	  This enables counter device support through the Generic Counter
-	  interface. You only need to enable this, if you also want to enable
-	  one or more of the counter device drivers below.
-
 config I8254
 	tristate
 	select COUNTER
@@ -25,6 +18,13 @@ config I8254
 
 	  If built as a module its name will be i8254.
 
+menuconfig COUNTER
+	tristate "Counter support"
+	help
+	  This enables counter device support through the Generic Counter
+	  interface. You only need to enable this, if you also want to enable
+	  one or more of the counter device drivers below.
+
 if COUNTER
 
 config 104_QUAD_8

base-commit: d428487471ba6640ee8bcdabaf830aec08b85400
-- 
2.40.1

Re: [PATCH] counter: Fix menuconfig "Counter support" submenu entries disappearance

[Randy Dunlap]

Hi,

On 6/20/23 10:01, William Breathitt Gray wrote:
> The current placement of the I8254 Kconfig entry results in the
> disappearance of the "Counter support" submenu items in menuconfig. Move
> the I8254 above the menuconfig COUNTER entry to restore the intended
> submenu behavior.
> 
> Fixes: d428487471ba ("counter: i8254: Introduce the Intel 8254 interface library module")
> Reported-by: Jarkko Nikula <jarkko.nikula@linux.intel.com>
> Closes: https://lore.kernel.org/all/32ddaa7b-53a8-d61f-d526-b545bd561337@linux.intel.com/
> Signed-off-by: William Breathitt Gray <william.gray@linaro.org>

Reviewed-by: Randy Dunlap <rdunlap@infradead.org>

but why is I8254 here at all? Users cannot enable it
and nothing selects it.  Is it a WIP?

Thanks.

> ---
>  drivers/counter/Kconfig | 14 +++++++-------
>  1 file changed, 7 insertions(+), 7 deletions(-)
> 
> diff --git a/drivers/counter/Kconfig b/drivers/counter/Kconfig
> index a61a4b9b8ec6..86536c2cc531 100644
> --- a/drivers/counter/Kconfig
> +++ b/drivers/counter/Kconfig
> @@ -3,13 +3,6 @@
>  # Counter devices
>  #
>  
> -menuconfig COUNTER
> -	tristate "Counter support"
> -	help
> -	  This enables counter device support through the Generic Counter
> -	  interface. You only need to enable this, if you also want to enable
> -	  one or more of the counter device drivers below.
> -
>  config I8254
>  	tristate
>  	select COUNTER
> @@ -25,6 +18,13 @@ config I8254
>  
>  	  If built as a module its name will be i8254.
>  
> +menuconfig COUNTER
> +	tristate "Counter support"
> +	help
> +	  This enables counter device support through the Generic Counter
> +	  interface. You only need to enable this, if you also want to enable
> +	  one or more of the counter device drivers below.
> +
>  if COUNTER
>  
>  config 104_QUAD_8
> 
> base-commit: d428487471ba6640ee8bcdabaf830aec08b85400

-- 
~Randy

Re: [PATCH] counter: Fix menuconfig "Counter support" submenu entries disappearance

[William Breathitt Gray]

[-- Attachment #1: Type: text/plain, Size: 1315 bytes --]

On Tue, Jun 20, 2023 at 02:00:37PM -0700, Randy Dunlap wrote:
> Hi,
> 
> On 6/20/23 10:01, William Breathitt Gray wrote:
> > The current placement of the I8254 Kconfig entry results in the
> > disappearance of the "Counter support" submenu items in menuconfig. Move
> > the I8254 above the menuconfig COUNTER entry to restore the intended
> > submenu behavior.
> > 
> > Fixes: d428487471ba ("counter: i8254: Introduce the Intel 8254 interface library module")
> > Reported-by: Jarkko Nikula <jarkko.nikula@linux.intel.com>
> > Closes: https://lore.kernel.org/all/32ddaa7b-53a8-d61f-d526-b545bd561337@linux.intel.com/
> > Signed-off-by: William Breathitt Gray <william.gray@linaro.org>
> 
> Reviewed-by: Randy Dunlap <rdunlap@infradead.org>
> 
> but why is I8254 here at all? Users cannot enable it
> and nothing selects it.  Is it a WIP?
> 
> Thanks.

There are pending patches for two drivers to use it: 104-dio-48e[^1] and
stx104[^2]. Those will be picked up in their respective subsystem trees
(by Bart and Jonathan I presume).

William Breathitt Gray

[^1]: https://lore.kernel.org/all/dc4d0d5ca6ea28eda18815df114ecb21226cb345.1681665189.git.william.gray@linaro.org/
[^2]: https://lore.kernel.org/all/45d35b6f6e8d51df788b2bc85c456bfd45476b1a.1681665189.git.william.gray@linaro.org/

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]

Re: [PATCH] counter: Fix menuconfig "Counter support" submenu entries disappearance

[Jarkko Nikula]

On 6/21/23 00:12, William Breathitt Gray wrote:
> On Tue, Jun 20, 2023 at 02:00:37PM -0700, Randy Dunlap wrote:
>> Hi,
>>
>> On 6/20/23 10:01, William Breathitt Gray wrote:
>>> The current placement of the I8254 Kconfig entry results in the
>>> disappearance of the "Counter support" submenu items in menuconfig. Move
>>> the I8254 above the menuconfig COUNTER entry to restore the intended
>>> submenu behavior.
>>>
>>> Fixes: d428487471ba ("counter: i8254: Introduce the Intel 8254 interface library module")
>>> Reported-by: Jarkko Nikula <jarkko.nikula@linux.intel.com>
>>> Closes: https://lore.kernel.org/all/32ddaa7b-53a8-d61f-d526-b545bd561337@linux.intel.com/
>>> Signed-off-by: William Breathitt Gray <william.gray@linaro.org>
>>
>> Reviewed-by: Randy Dunlap <rdunlap@infradead.org>
>>
Thanks, this fixes it.

Tested-by: Jarkko Nikula <jarkko.nikula@linux.intel.com>
Reviewed-by: Jarkko Nikula <jarkko.nikula@linux.intel.com>

Re: [PATCH] counter: Fix menuconfig "Counter support" submenu entries disappearance

[William Breathitt Gray]

[-- Attachment #1: Type: text/plain, Size: 685 bytes --]

On Tue, Jun 20, 2023 at 01:01:59PM -0400, William Breathitt Gray wrote:
> The current placement of the I8254 Kconfig entry results in the
> disappearance of the "Counter support" submenu items in menuconfig. Move
> the I8254 above the menuconfig COUNTER entry to restore the intended
> submenu behavior.
> 
> Fixes: d428487471ba ("counter: i8254: Introduce the Intel 8254 interface library module")
> Reported-by: Jarkko Nikula <jarkko.nikula@linux.intel.com>
> Closes: https://lore.kernel.org/all/32ddaa7b-53a8-d61f-d526-b545bd561337@linux.intel.com/
> Signed-off-by: William Breathitt Gray <william.gray@linaro.org>

Applied to counter-current.

William Breathitt Gray

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]

Re: [PATCH] counter: Fix menuconfig "Counter support" submenu entries disappearance

[Jonathan Cameron]

On Tue, 20 Jun 2023 17:12:13 -0400
William Breathitt Gray <william.gray@linaro.org> wrote:

> On Tue, Jun 20, 2023 at 02:00:37PM -0700, Randy Dunlap wrote:
> > Hi,
> > 
> > On 6/20/23 10:01, William Breathitt Gray wrote:  
> > > The current placement of the I8254 Kconfig entry results in the
> > > disappearance of the "Counter support" submenu items in menuconfig. Move
> > > the I8254 above the menuconfig COUNTER entry to restore the intended
> > > submenu behavior.
> > > 
> > > Fixes: d428487471ba ("counter: i8254: Introduce the Intel 8254 interface library module")
> > > Reported-by: Jarkko Nikula <jarkko.nikula@linux.intel.com>
> > > Closes: https://lore.kernel.org/all/32ddaa7b-53a8-d61f-d526-b545bd561337@linux.intel.com/
> > > Signed-off-by: William Breathitt Gray <william.gray@linaro.org>  
> > 
> > Reviewed-by: Randy Dunlap <rdunlap@infradead.org>
> > 
> > but why is I8254 here at all? Users cannot enable it
> > and nothing selects it.  Is it a WIP?
> > 
> > Thanks.  
> 
> There are pending patches for two drivers to use it: 104-dio-48e[^1] and
> stx104[^2]. Those will be picked up in their respective subsystem trees
> (by Bart and Jonathan I presume).
> 

I missed the IIO patch in that series. Could you resend please.

Jonathan

> William Breathitt Gray
> 
> [^1]: https://lore.kernel.org/all/dc4d0d5ca6ea28eda18815df114ecb21226cb345.1681665189.git.william.gray@linaro.org/
> [^2]: https://lore.kernel.org/all/45d35b6f6e8d51df788b2bc85c456bfd45476b1a.1681665189.git.william.gray@linaro.org/

Re: [PATCH] counter: Fix menuconfig "Counter support" submenu entries disappearance

[William Breathitt Gray]

[-- Attachment #1: Type: text/plain, Size: 1934 bytes --]

On Sun, Jun 25, 2023 at 12:11:08PM +0100, Jonathan Cameron wrote:
> On Tue, 20 Jun 2023 17:12:13 -0400
> William Breathitt Gray <william.gray@linaro.org> wrote:
> 
> > On Tue, Jun 20, 2023 at 02:00:37PM -0700, Randy Dunlap wrote:
> > > Hi,
> > > 
> > > On 6/20/23 10:01, William Breathitt Gray wrote:  
> > > > The current placement of the I8254 Kconfig entry results in the
> > > > disappearance of the "Counter support" submenu items in menuconfig. Move
> > > > the I8254 above the menuconfig COUNTER entry to restore the intended
> > > > submenu behavior.
> > > > 
> > > > Fixes: d428487471ba ("counter: i8254: Introduce the Intel 8254 interface library module")
> > > > Reported-by: Jarkko Nikula <jarkko.nikula@linux.intel.com>
> > > > Closes: https://lore.kernel.org/all/32ddaa7b-53a8-d61f-d526-b545bd561337@linux.intel.com/
> > > > Signed-off-by: William Breathitt Gray <william.gray@linaro.org>  
> > > 
> > > Reviewed-by: Randy Dunlap <rdunlap@infradead.org>
> > > 
> > > but why is I8254 here at all? Users cannot enable it
> > > and nothing selects it.  Is it a WIP?
> > > 
> > > Thanks.  
> > 
> > There are pending patches for two drivers to use it: 104-dio-48e[^1] and
> > stx104[^2]. Those will be picked up in their respective subsystem trees
> > (by Bart and Jonathan I presume).
> > 
> 
> I missed the IIO patch in that series. Could you resend please.
> 
> Jonathan
> 
> > William Breathitt Gray
> > 
> > [^1]: https://lore.kernel.org/all/dc4d0d5ca6ea28eda18815df114ecb21226cb345.1681665189.git.william.gray@linaro.org/
> > [^2]: https://lore.kernel.org/all/45d35b6f6e8d51df788b2bc85c456bfd45476b1a.1681665189.git.william.gray@linaro.org/

I8254 is expected to land in 6.5 so I'll wait and resend the IIO patch
rebased on 6.5-rc1 once it's released in a couple weeks. That'll make
things simple for you and avoid the need for an immutable branch.

William Breathitt Gray

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]