* [PATCH] input: allow SERIO=m even without EXPERT=y
@ 2013-09-02 18:47 Tom Gundersen
2013-09-02 21:01 ` richard -rw- weinberger
2013-09-03 15:02 ` Matthew Garrett
0 siblings, 2 replies; 12+ messages in thread
From: Tom Gundersen @ 2013-09-02 18:47 UTC (permalink / raw)
To: linux-input; +Cc: linux-kernel, Tom Gundersen, Dmitry Torokhov
There is plenty of consumer hardware (e.g., mac books) that does not use
AT keyboards or PS/2 mice. It therefore makes sense for distro kernels
to build the related drivers as modules to avoid loading them on hardware
that does not need them. As such, these options should no longer be protected
by EXPERT.
Moreover, building these drivers as modules gets rid of the following ugly
error during boot:
[ 2.337745] i8042: PNP: No PS/2 controller found. Probing ports directly.
[ 3.439537] i8042: No controller found
Signed-off-by: Tom Gundersen <teg@jklm.no>
Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>
---
drivers/input/keyboard/Kconfig | 4 ++--
drivers/input/serio/Kconfig | 6 +++---
2 files changed, 5 insertions(+), 5 deletions(-)
diff --git a/drivers/input/keyboard/Kconfig b/drivers/input/keyboard/Kconfig
index 269d4c3..2d31cec 100644
--- a/drivers/input/keyboard/Kconfig
+++ b/drivers/input/keyboard/Kconfig
@@ -2,7 +2,7 @@
# Input core configuration
#
menuconfig INPUT_KEYBOARD
- bool "Keyboards" if EXPERT || !X86
+ bool "Keyboards"
default y
help
Say Y here, and a list of supported keyboards will be displayed.
@@ -67,7 +67,7 @@ config KEYBOARD_ATARI
module will be called atakbd.
config KEYBOARD_ATKBD
- tristate "AT keyboard" if EXPERT || !X86
+ tristate "AT keyboard"
default y
select SERIO
select SERIO_LIBPS2
diff --git a/drivers/input/serio/Kconfig b/drivers/input/serio/Kconfig
index 1e691a3..39c869d 100644
--- a/drivers/input/serio/Kconfig
+++ b/drivers/input/serio/Kconfig
@@ -2,7 +2,7 @@
# Input core configuration
#
config SERIO
- tristate "Serial I/O support" if EXPERT || !X86
+ tristate "Serial I/O support"
default y
help
Say Yes here if you have any input device that uses serial I/O to
@@ -19,7 +19,7 @@ config SERIO
if SERIO
config SERIO_I8042
- tristate "i8042 PC Keyboard controller" if EXPERT || !X86
+ tristate "i8042 PC Keyboard controller"
default y
depends on !PARISC && (!ARM || ARCH_SHARK || FOOTBRIDGE_HOST) && \
(!SUPERH || SH_CAYMAN) && !M68K && !BLACKFIN && !S390 && \
@@ -170,7 +170,7 @@ config SERIO_MACEPS2
module will be called maceps2.
config SERIO_LIBPS2
- tristate "PS/2 driver library" if EXPERT
+ tristate "PS/2 driver library"
depends on SERIO_I8042 || SERIO_I8042=n
help
Say Y here if you are using a driver for device connected
--
1.8.4
^ permalink raw reply related [flat|nested] 12+ messages in thread
* Re: [PATCH] input: allow SERIO=m even without EXPERT=y
2013-09-02 18:47 [PATCH] input: allow SERIO=m even without EXPERT=y Tom Gundersen
@ 2013-09-02 21:01 ` richard -rw- weinberger
2013-09-02 21:30 ` Geert Uytterhoeven
2013-09-03 15:02 ` Matthew Garrett
1 sibling, 1 reply; 12+ messages in thread
From: richard -rw- weinberger @ 2013-09-02 21:01 UTC (permalink / raw)
To: Tom Gundersen; +Cc: linux-input@vger.kernel.org, LKML, Dmitry Torokhov
On Mon, Sep 2, 2013 at 8:47 PM, Tom Gundersen <teg@jklm.no> wrote:
> There is plenty of consumer hardware (e.g., mac books) that does not use
> AT keyboards or PS/2 mice. It therefore makes sense for distro kernels
> to build the related drivers as modules to avoid loading them on hardware
> that does not need them. As such, these options should no longer be protected
> by EXPERT.
>
> Moreover, building these drivers as modules gets rid of the following ugly
> error during boot:
>
> [ 2.337745] i8042: PNP: No PS/2 controller found. Probing ports directly.
> [ 3.439537] i8042: No controller found
>
> Signed-off-by: Tom Gundersen <teg@jklm.no>
> Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>
> ---
> drivers/input/keyboard/Kconfig | 4 ++--
> drivers/input/serio/Kconfig | 6 +++---
> 2 files changed, 5 insertions(+), 5 deletions(-)
>
> diff --git a/drivers/input/keyboard/Kconfig b/drivers/input/keyboard/Kconfig
> index 269d4c3..2d31cec 100644
> --- a/drivers/input/keyboard/Kconfig
> +++ b/drivers/input/keyboard/Kconfig
> @@ -2,7 +2,7 @@
> # Input core configuration
> #
> menuconfig INPUT_KEYBOARD
> - bool "Keyboards" if EXPERT || !X86
> + bool "Keyboards"
Maybe this a stupid question, but why are you dropping the "!X86" too?
> default y
> help
> Say Y here, and a list of supported keyboards will be displayed.
> @@ -67,7 +67,7 @@ config KEYBOARD_ATARI
> module will be called atakbd.
>
> config KEYBOARD_ATKBD
> - tristate "AT keyboard" if EXPERT || !X86
> + tristate "AT keyboard"
> default y
> select SERIO
> select SERIO_LIBPS2
> diff --git a/drivers/input/serio/Kconfig b/drivers/input/serio/Kconfig
> index 1e691a3..39c869d 100644
> --- a/drivers/input/serio/Kconfig
> +++ b/drivers/input/serio/Kconfig
> @@ -2,7 +2,7 @@
> # Input core configuration
> #
> config SERIO
> - tristate "Serial I/O support" if EXPERT || !X86
> + tristate "Serial I/O support"
> default y
> help
> Say Yes here if you have any input device that uses serial I/O to
> @@ -19,7 +19,7 @@ config SERIO
> if SERIO
>
> config SERIO_I8042
> - tristate "i8042 PC Keyboard controller" if EXPERT || !X86
> + tristate "i8042 PC Keyboard controller"
> default y
> depends on !PARISC && (!ARM || ARCH_SHARK || FOOTBRIDGE_HOST) && \
> (!SUPERH || SH_CAYMAN) && !M68K && !BLACKFIN && !S390 && \
> @@ -170,7 +170,7 @@ config SERIO_MACEPS2
> module will be called maceps2.
>
> config SERIO_LIBPS2
> - tristate "PS/2 driver library" if EXPERT
> + tristate "PS/2 driver library"
> depends on SERIO_I8042 || SERIO_I8042=n
> help
> Say Y here if you are using a driver for device connected
> --
> 1.8.4
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at http://www.tux.org/lkml/
--
Thanks,
//richard
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH] input: allow SERIO=m even without EXPERT=y
2013-09-02 21:01 ` richard -rw- weinberger
@ 2013-09-02 21:30 ` Geert Uytterhoeven
2013-09-02 21:46 ` Tom Gundersen
0 siblings, 1 reply; 12+ messages in thread
From: Geert Uytterhoeven @ 2013-09-02 21:30 UTC (permalink / raw)
To: richard -rw- weinberger
Cc: Tom Gundersen, linux-input@vger.kernel.org, LKML, Dmitry Torokhov
On Mon, Sep 2, 2013 at 11:01 PM, richard -rw- weinberger
<richard.weinberger@gmail.com> wrote:
> On Mon, Sep 2, 2013 at 8:47 PM, Tom Gundersen <teg@jklm.no> wrote:
>> There is plenty of consumer hardware (e.g., mac books) that does not use
>> AT keyboards or PS/2 mice. It therefore makes sense for distro kernels
>> to build the related drivers as modules to avoid loading them on hardware
>> that does not need them. As such, these options should no longer be protected
>> by EXPERT.
>>
>> Moreover, building these drivers as modules gets rid of the following ugly
>> error during boot:
>>
>> [ 2.337745] i8042: PNP: No PS/2 controller found. Probing ports directly.
>> [ 3.439537] i8042: No controller found
>>
>> Signed-off-by: Tom Gundersen <teg@jklm.no>
>> Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>
>> ---
>> drivers/input/keyboard/Kconfig | 4 ++--
>> drivers/input/serio/Kconfig | 6 +++---
>> 2 files changed, 5 insertions(+), 5 deletions(-)
>>
>> diff --git a/drivers/input/keyboard/Kconfig b/drivers/input/keyboard/Kconfig
>> index 269d4c3..2d31cec 100644
>> --- a/drivers/input/keyboard/Kconfig
>> +++ b/drivers/input/keyboard/Kconfig
>> @@ -2,7 +2,7 @@
>> # Input core configuration
>> #
>> menuconfig INPUT_KEYBOARD
>> - bool "Keyboards" if EXPERT || !X86
>> + bool "Keyboards"
>
> Maybe this a stupid question, but why are you dropping the "!X86" too?
Because the "if EXPERT" only mattered for X86.
Apply De Morgan and it read like "do not ask the question if !EXPERT && X86".
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
^ permalink raw reply [flat|nested] 12+ messages in thread* Re: [PATCH] input: allow SERIO=m even without EXPERT=y
2013-09-02 21:30 ` Geert Uytterhoeven
@ 2013-09-02 21:46 ` Tom Gundersen
0 siblings, 0 replies; 12+ messages in thread
From: Tom Gundersen @ 2013-09-02 21:46 UTC (permalink / raw)
To: Geert Uytterhoeven
Cc: richard -rw- weinberger, linux-input@vger.kernel.org, LKML,
Dmitry Torokhov
On Mon, Sep 2, 2013 at 11:30 PM, Geert Uytterhoeven
<geert@linux-m68k.org> wrote:
> On Mon, Sep 2, 2013 at 11:01 PM, richard -rw- weinberger
> <richard.weinberger@gmail.com> wrote:
>> On Mon, Sep 2, 2013 at 8:47 PM, Tom Gundersen <teg@jklm.no> wrote:
>>> There is plenty of consumer hardware (e.g., mac books) that does not use
>>> AT keyboards or PS/2 mice. It therefore makes sense for distro kernels
>>> to build the related drivers as modules to avoid loading them on hardware
>>> that does not need them. As such, these options should no longer be protected
>>> by EXPERT.
>>>
>>> Moreover, building these drivers as modules gets rid of the following ugly
>>> error during boot:
>>>
>>> [ 2.337745] i8042: PNP: No PS/2 controller found. Probing ports directly.
>>> [ 3.439537] i8042: No controller found
>>>
>>> Signed-off-by: Tom Gundersen <teg@jklm.no>
>>> Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>
>>> ---
>>> drivers/input/keyboard/Kconfig | 4 ++--
>>> drivers/input/serio/Kconfig | 6 +++---
>>> 2 files changed, 5 insertions(+), 5 deletions(-)
>>>
>>> diff --git a/drivers/input/keyboard/Kconfig b/drivers/input/keyboard/Kconfig
>>> index 269d4c3..2d31cec 100644
>>> --- a/drivers/input/keyboard/Kconfig
>>> +++ b/drivers/input/keyboard/Kconfig
>>> @@ -2,7 +2,7 @@
>>> # Input core configuration
>>> #
>>> menuconfig INPUT_KEYBOARD
>>> - bool "Keyboards" if EXPERT || !X86
>>> + bool "Keyboards"
>>
>> Maybe this a stupid question, but why are you dropping the "!X86" too?
>
> Because the "if EXPERT" only mattered for X86.
>
> Apply De Morgan and it read like "do not ask the question if !EXPERT && X86".
Precisely.
-t
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH] input: allow SERIO=m even without EXPERT=y
2013-09-02 18:47 [PATCH] input: allow SERIO=m even without EXPERT=y Tom Gundersen
2013-09-02 21:01 ` richard -rw- weinberger
@ 2013-09-03 15:02 ` Matthew Garrett
2013-09-03 16:25 ` Tom Gundersen
1 sibling, 1 reply; 12+ messages in thread
From: Matthew Garrett @ 2013-09-03 15:02 UTC (permalink / raw)
To: Tom Gundersen; +Cc: linux-input, linux-kernel, Dmitry Torokhov
On Mon, Sep 02, 2013 at 08:47:10PM +0200, Tom Gundersen wrote:
> There is plenty of consumer hardware (e.g., mac books) that does not use
> AT keyboards or PS/2 mice. It therefore makes sense for distro kernels
> to build the related drivers as modules to avoid loading them on hardware
> that does not need them. As such, these options should no longer be protected
> by EXPERT.
There are systems (although, with luck, only *very* old ones) where the
modules won't get autoloaded. You should probably mention that in the
help text.
--
Matthew Garrett | mjg59@srcf.ucam.org
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH] input: allow SERIO=m even without EXPERT=y
2013-09-03 15:02 ` Matthew Garrett
@ 2013-09-03 16:25 ` Tom Gundersen
2013-09-03 19:24 ` Dmitry Torokhov
0 siblings, 1 reply; 12+ messages in thread
From: Tom Gundersen @ 2013-09-03 16:25 UTC (permalink / raw)
To: Matthew Garrett; +Cc: linux-input@vger.kernel.org, LKML, Dmitry Torokhov
On Tue, Sep 3, 2013 at 5:02 PM, Matthew Garrett <mjg59@srcf.ucam.org> wrote:
> On Mon, Sep 02, 2013 at 08:47:10PM +0200, Tom Gundersen wrote:
>> There is plenty of consumer hardware (e.g., mac books) that does not use
>> AT keyboards or PS/2 mice. It therefore makes sense for distro kernels
>> to build the related drivers as modules to avoid loading them on hardware
>> that does not need them. As such, these options should no longer be protected
>> by EXPERT.
>
> There are systems (although, with luck, only *very* old ones) where the
> modules won't get autoloaded. You should probably mention that in the
> help text.
Good point, if this turns out to be a problem on current hardware
distros probably want to load the modules automatically at boot by
default.
Does this problem only apply to the i8042 module or also atkbd and libps2?
Cheers,
Tom
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH] input: allow SERIO=m even without EXPERT=y
2013-09-03 16:25 ` Tom Gundersen
@ 2013-09-03 19:24 ` Dmitry Torokhov
2013-09-03 19:35 ` Matthew Garrett
0 siblings, 1 reply; 12+ messages in thread
From: Dmitry Torokhov @ 2013-09-03 19:24 UTC (permalink / raw)
To: Tom Gundersen; +Cc: Matthew Garrett, linux-input@vger.kernel.org, LKML
On Tue, Sep 03, 2013 at 06:25:28PM +0200, Tom Gundersen wrote:
> On Tue, Sep 3, 2013 at 5:02 PM, Matthew Garrett <mjg59@srcf.ucam.org> wrote:
> > On Mon, Sep 02, 2013 at 08:47:10PM +0200, Tom Gundersen wrote:
> >> There is plenty of consumer hardware (e.g., mac books) that does not use
> >> AT keyboards or PS/2 mice. It therefore makes sense for distro kernels
> >> to build the related drivers as modules to avoid loading them on hardware
> >> that does not need them. As such, these options should no longer be protected
> >> by EXPERT.
> >
> > There are systems (although, with luck, only *very* old ones) where the
> > modules won't get autoloaded. You should probably mention that in the
> > help text.
>
> Good point, if this turns out to be a problem on current hardware
> distros probably want to load the modules automatically at boot by
> default.
Presumably such systems would carry over their .config which would have
these as built-in.
>
> Does this problem only apply to the i8042 module or also atkbd and libps2?
Atkbd should be able to load automatically (as long as modutils support
SERIO bus, which any recentish - 5 years? - modutils should do).
libps2 should load automatically to resolve dependencies of
atkbd/psmouse.
i8042 is old-style platform driver, so nothing will load it if it is
compiled as a module. This one worries me most.
mousedev should load automatically if something tries to open it's
device nodes.
Thanks.
--
Dmitry
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH] input: allow SERIO=m even without EXPERT=y
2013-09-03 19:24 ` Dmitry Torokhov
@ 2013-09-03 19:35 ` Matthew Garrett
2013-09-04 9:27 ` [PATCH] input: i8042 - add PNP modaliases Tom Gundersen
0 siblings, 1 reply; 12+ messages in thread
From: Matthew Garrett @ 2013-09-03 19:35 UTC (permalink / raw)
To: Dmitry Torokhov; +Cc: Tom Gundersen, linux-input@vger.kernel.org, LKML
On Tue, Sep 03, 2013 at 12:24:21PM -0700, Dmitry Torokhov wrote:
> Presumably such systems would carry over their .config which would have
> these as built-in.
Yeah, but distributions should still be given guidance on their default
configs.
> i8042 is old-style platform driver, so nothing will load it if it is
> compiled as a module. This one worries me most.
We could add the PNP aliases for it.
--
Matthew Garrett | mjg59@srcf.ucam.org
^ permalink raw reply [flat|nested] 12+ messages in thread
* [PATCH] input: i8042 - add PNP modaliases
2013-09-03 19:35 ` Matthew Garrett
@ 2013-09-04 9:27 ` Tom Gundersen
2013-10-04 12:26 ` Tom Gundersen
2013-10-31 7:44 ` Dmitry Torokhov
0 siblings, 2 replies; 12+ messages in thread
From: Tom Gundersen @ 2013-09-04 9:27 UTC (permalink / raw)
To: linux-input; +Cc: linux-kernel, Tom Gundersen, Matthew Garrett, Dmitry Torokhov
This allows the module to be autoloaded in the common case.
In order to work on non-PnP systems the module should be compiled in or loaded
unconditionally at boot (c.f. modules-load.d(5)), as before.
Cc: Matthew Garrett <mjg59@srcf.ucam.org>
Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>
Signed-off-by: Tom Gundersen <teg@jklm.no>
---
This appears to work for me (though I don't have the real hardware to test), I get the following aliases:
alias: acpi*:CPQA0D7:*
alias: pnp:dCPQA0D7*
alias: acpi*:PNP0345:*
alias: pnp:dPNP0345*
alias: acpi*:PNP0344:*
alias: pnp:dPNP0344*
alias: acpi*:PNP0343:*
alias: pnp:dPNP0343*
alias: acpi*:PNP0320:*
alias: pnp:dPNP0320*
alias: acpi*:PNP030B:*
alias: pnp:dPNP030b*
alias: acpi*:PNP030A:*
alias: pnp:dPNP030a*
alias: acpi*:PNP0309:*
alias: pnp:dPNP0309*
alias: acpi*:PNP0306:*
alias: pnp:dPNP0306*
alias: acpi*:PNP0305:*
alias: pnp:dPNP0305*
alias: acpi*:PNP0304:*
alias: pnp:dPNP0304*
alias: acpi*:PNP0303:*
alias: pnp:dPNP0303*
alias: acpi*:PNP0302:*
alias: pnp:dPNP0302*
alias: acpi*:PNP0301:*
alias: pnp:dPNP0301*
alias: acpi*:PNP0300:*
alias: pnp:dPNP0300*
drivers/input/serio/i8042-x86ia64io.h | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/input/serio/i8042-x86ia64io.h b/drivers/input/serio/i8042-x86ia64io.h
index 5f306f7..0ec9abb 100644
--- a/drivers/input/serio/i8042-x86ia64io.h
+++ b/drivers/input/serio/i8042-x86ia64io.h
@@ -765,6 +765,7 @@ static struct pnp_device_id pnp_kbd_devids[] = {
{ .id = "CPQA0D7", .driver_data = 0 },
{ .id = "", },
};
+MODULE_DEVICE_TABLE(pnp, pnp_kbd_devids);
static struct pnp_driver i8042_pnp_kbd_driver = {
.name = "i8042 kbd",
@@ -786,6 +787,7 @@ static struct pnp_device_id pnp_aux_devids[] = {
{ .id = "SYN0801", .driver_data = 0 },
{ .id = "", },
};
+MODULE_DEVICE_TABLE(pnp, pnp_aux_devids);
static struct pnp_driver i8042_pnp_aux_driver = {
.name = "i8042 aux",
--
1.8.4
^ permalink raw reply related [flat|nested] 12+ messages in thread* Re: [PATCH] input: i8042 - add PNP modaliases
2013-09-04 9:27 ` [PATCH] input: i8042 - add PNP modaliases Tom Gundersen
@ 2013-10-04 12:26 ` Tom Gundersen
2013-10-30 14:30 ` Tom Gundersen
2013-10-31 7:44 ` Dmitry Torokhov
1 sibling, 1 reply; 12+ messages in thread
From: Tom Gundersen @ 2013-10-04 12:26 UTC (permalink / raw)
To: linux-input@vger.kernel.org
Cc: LKML, Tom Gundersen, Matthew Garrett, Dmitry Torokhov
On Wed, Sep 4, 2013 at 11:27 AM, Tom Gundersen <teg@jklm.no> wrote:
> This allows the module to be autoloaded in the common case.
>
> In order to work on non-PnP systems the module should be compiled in or loaded
> unconditionally at boot (c.f. modules-load.d(5)), as before.
>
> Cc: Matthew Garrett <mjg59@srcf.ucam.org>
> Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>
> Signed-off-by: Tom Gundersen <teg@jklm.no>
> --
Hi Dmitry,
Any comments on this? Any chance of having this (and the two patches
dropping EXPERT=y requirements) included for 3.13 (or even 3.12 if it
is not too late for this kind of stuff)? Let me know if I should
resend the three patches.
Cheers,
Tom
> This appears to work for me (though I don't have the real hardware to test), I get the following aliases:
> alias: acpi*:CPQA0D7:*
> alias: pnp:dCPQA0D7*
> alias: acpi*:PNP0345:*
> alias: pnp:dPNP0345*
> alias: acpi*:PNP0344:*
> alias: pnp:dPNP0344*
> alias: acpi*:PNP0343:*
> alias: pnp:dPNP0343*
> alias: acpi*:PNP0320:*
> alias: pnp:dPNP0320*
> alias: acpi*:PNP030B:*
> alias: pnp:dPNP030b*
> alias: acpi*:PNP030A:*
> alias: pnp:dPNP030a*
> alias: acpi*:PNP0309:*
> alias: pnp:dPNP0309*
> alias: acpi*:PNP0306:*
> alias: pnp:dPNP0306*
> alias: acpi*:PNP0305:*
> alias: pnp:dPNP0305*
> alias: acpi*:PNP0304:*
> alias: pnp:dPNP0304*
> alias: acpi*:PNP0303:*
> alias: pnp:dPNP0303*
> alias: acpi*:PNP0302:*
> alias: pnp:dPNP0302*
> alias: acpi*:PNP0301:*
> alias: pnp:dPNP0301*
> alias: acpi*:PNP0300:*
> alias: pnp:dPNP0300*
>
> drivers/input/serio/i8042-x86ia64io.h | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/drivers/input/serio/i8042-x86ia64io.h b/drivers/input/serio/i8042-x86ia64io.h
> index 5f306f7..0ec9abb 100644
> --- a/drivers/input/serio/i8042-x86ia64io.h
> +++ b/drivers/input/serio/i8042-x86ia64io.h
> @@ -765,6 +765,7 @@ static struct pnp_device_id pnp_kbd_devids[] = {
> { .id = "CPQA0D7", .driver_data = 0 },
> { .id = "", },
> };
> +MODULE_DEVICE_TABLE(pnp, pnp_kbd_devids);
>
> static struct pnp_driver i8042_pnp_kbd_driver = {
> .name = "i8042 kbd",
> @@ -786,6 +787,7 @@ static struct pnp_device_id pnp_aux_devids[] = {
> { .id = "SYN0801", .driver_data = 0 },
> { .id = "", },
> };
> +MODULE_DEVICE_TABLE(pnp, pnp_aux_devids);
>
> static struct pnp_driver i8042_pnp_aux_driver = {
> .name = "i8042 aux",
> --
> 1.8.4
>
^ permalink raw reply [flat|nested] 12+ messages in thread* Re: [PATCH] input: i8042 - add PNP modaliases
2013-10-04 12:26 ` Tom Gundersen
@ 2013-10-30 14:30 ` Tom Gundersen
0 siblings, 0 replies; 12+ messages in thread
From: Tom Gundersen @ 2013-10-30 14:30 UTC (permalink / raw)
To: linux-input@vger.kernel.org
Cc: LKML, Tom Gundersen, Matthew Garrett, Dmitry Torokhov
On Fri, Oct 4, 2013 at 2:26 PM, Tom Gundersen <teg@jklm.no> wrote:
> On Wed, Sep 4, 2013 at 11:27 AM, Tom Gundersen <teg@jklm.no> wrote:
>> This allows the module to be autoloaded in the common case.
>>
>> In order to work on non-PnP systems the module should be compiled in or loaded
>> unconditionally at boot (c.f. modules-load.d(5)), as before.
>>
>> Cc: Matthew Garrett <mjg59@srcf.ucam.org>
>> Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>
>> Signed-off-by: Tom Gundersen <teg@jklm.no>
>> --
>
>
> Hi Dmitry,
>
> Any comments on this? Any chance of having this (and the two patches
> dropping EXPERT=y requirements) included for 3.13 (or even 3.12 if it
> is not too late for this kind of stuff)? Let me know if I should
> resend the three patches.
Ping? Any chance of seeing this in 3.13?
Cheers,
Tom
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH] input: i8042 - add PNP modaliases
2013-09-04 9:27 ` [PATCH] input: i8042 - add PNP modaliases Tom Gundersen
2013-10-04 12:26 ` Tom Gundersen
@ 2013-10-31 7:44 ` Dmitry Torokhov
1 sibling, 0 replies; 12+ messages in thread
From: Dmitry Torokhov @ 2013-10-31 7:44 UTC (permalink / raw)
To: Tom Gundersen; +Cc: linux-input, linux-kernel, Matthew Garrett
On Wed, Sep 04, 2013 at 11:27:36AM +0200, Tom Gundersen wrote:
> This allows the module to be autoloaded in the common case.
>
> In order to work on non-PnP systems the module should be compiled in or loaded
> unconditionally at boot (c.f. modules-load.d(5)), as before.
>
> Cc: Matthew Garrett <mjg59@srcf.ucam.org>
> Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>
> Signed-off-by: Tom Gundersen <teg@jklm.no>
Applied, thank you.
--
Dmitry
^ permalink raw reply [flat|nested] 12+ messages in thread
end of thread, other threads:[~2013-10-31 7:44 UTC | newest]
Thread overview: 12+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-09-02 18:47 [PATCH] input: allow SERIO=m even without EXPERT=y Tom Gundersen
2013-09-02 21:01 ` richard -rw- weinberger
2013-09-02 21:30 ` Geert Uytterhoeven
2013-09-02 21:46 ` Tom Gundersen
2013-09-03 15:02 ` Matthew Garrett
2013-09-03 16:25 ` Tom Gundersen
2013-09-03 19:24 ` Dmitry Torokhov
2013-09-03 19:35 ` Matthew Garrett
2013-09-04 9:27 ` [PATCH] input: i8042 - add PNP modaliases Tom Gundersen
2013-10-04 12:26 ` Tom Gundersen
2013-10-30 14:30 ` Tom Gundersen
2013-10-31 7:44 ` Dmitry Torokhov
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).