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 <[email protected]>
Closes: https://lore.kernel.org/all/[email protected]/
Signed-off-by: William Breathitt Gray <[email protected]>
---
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
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 <[email protected]>
> Closes: https://lore.kernel.org/all/[email protected]/
> Signed-off-by: William Breathitt Gray <[email protected]>
Reviewed-by: Randy Dunlap <[email protected]>
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
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 <[email protected]>
> > Closes: https://lore.kernel.org/all/[email protected]/
> > Signed-off-by: William Breathitt Gray <[email protected]>
>
> Reviewed-by: Randy Dunlap <[email protected]>
>
> 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/
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 <[email protected]>
>>> Closes: https://lore.kernel.org/all/[email protected]/
>>> Signed-off-by: William Breathitt Gray <[email protected]>
>>
>> Reviewed-by: Randy Dunlap <[email protected]>
>>
Thanks, this fixes it.
Tested-by: Jarkko Nikula <[email protected]>
Reviewed-by: Jarkko Nikula <[email protected]>
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 <[email protected]>
> Closes: https://lore.kernel.org/all/[email protected]/
> Signed-off-by: William Breathitt Gray <[email protected]>
Applied to counter-current.
William Breathitt Gray
On Tue, 20 Jun 2023 17:12:13 -0400
William Breathitt Gray <[email protected]> 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 <[email protected]>
> > > Closes: https://lore.kernel.org/all/[email protected]/
> > > Signed-off-by: William Breathitt Gray <[email protected]>
> >
> > Reviewed-by: Randy Dunlap <[email protected]>
> >
> > 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/
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 <[email protected]> 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 <[email protected]>
> > > > Closes: https://lore.kernel.org/all/[email protected]/
> > > > Signed-off-by: William Breathitt Gray <[email protected]>
> > >
> > > Reviewed-by: Randy Dunlap <[email protected]>
> > >
> > > 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