[PATCH] Input: atkbd: Fix release quirk for Dell models

[PATCH] Input: atkbd: Fix release quirk for Dell models

[Pali Rohár]

This patch fixes commit 61579ba83934 ("Input: atkbd - expand Latitude's force
release quirk to other Dells"). Before that commit release quirks were called
for all Dell Latitude models. After that commit only for Portable Dell devices.
But lot of Latitude models are Laptop or Notebook DMI devices so quirks are not
called.

Release quirks are still needed also for new Dell Latitude models, so this patch
enables quirks for all Portable, Laptop, Notebook and Sub-Notebook Dell devices.

Signed-off-by: Pali Rohár <pali.rohar@gmail.com>
---
 drivers/input/keyboard/atkbd.c |   48 ++++++++++++++++++++++++++++++++++++++++
 1 file changed, 48 insertions(+)

diff --git a/drivers/input/keyboard/atkbd.c b/drivers/input/keyboard/atkbd.c
index 387c51f..3188493 100644
--- a/drivers/input/keyboard/atkbd.c
+++ b/drivers/input/keyboard/atkbd.c
@@ -1664,6 +1664,30 @@ static const struct dmi_system_id atkbd_dmi_quirk_table[] __initconst = {
 	},
 	{
 		.matches = {
+			DMI_MATCH(DMI_SYS_VENDOR, "Dell Inc."),
+			DMI_MATCH(DMI_CHASSIS_TYPE, "9"), /* Laptop */
+		},
+		.callback = atkbd_setup_forced_release,
+		.driver_data = atkbd_dell_laptop_forced_release_keys,
+	},
+	{
+		.matches = {
+			DMI_MATCH(DMI_SYS_VENDOR, "Dell Inc."),
+			DMI_MATCH(DMI_CHASSIS_TYPE, "10"), /* Notebook */
+		},
+		.callback = atkbd_setup_forced_release,
+		.driver_data = atkbd_dell_laptop_forced_release_keys,
+	},
+	{
+		.matches = {
+			DMI_MATCH(DMI_SYS_VENDOR, "Dell Inc."),
+			DMI_MATCH(DMI_CHASSIS_TYPE, "14"), /* Sub-Notebook */
+		},
+		.callback = atkbd_setup_forced_release,
+		.driver_data = atkbd_dell_laptop_forced_release_keys,
+	},
+	{
+		.matches = {
 			DMI_MATCH(DMI_SYS_VENDOR, "Dell Computer Corporation"),
 			DMI_MATCH(DMI_CHASSIS_TYPE, "8"), /* Portable */
 		},
@@ -1672,6 +1696,30 @@ static const struct dmi_system_id atkbd_dmi_quirk_table[] __initconst = {
 	},
 	{
 		.matches = {
+			DMI_MATCH(DMI_SYS_VENDOR, "Dell Computer Corporation"),
+			DMI_MATCH(DMI_CHASSIS_TYPE, "9"), /* Laptop */
+		},
+		.callback = atkbd_setup_forced_release,
+		.driver_data = atkbd_dell_laptop_forced_release_keys,
+	},
+	{
+		.matches = {
+			DMI_MATCH(DMI_SYS_VENDOR, "Dell Computer Corporation"),
+			DMI_MATCH(DMI_CHASSIS_TYPE, "10"), /* Notebook */
+		},
+		.callback = atkbd_setup_forced_release,
+		.driver_data = atkbd_dell_laptop_forced_release_keys,
+	},
+	{
+		.matches = {
+			DMI_MATCH(DMI_SYS_VENDOR, "Dell Computer Corporation"),
+			DMI_MATCH(DMI_CHASSIS_TYPE, "14"), /* Sub-Notebook */
+		},
+		.callback = atkbd_setup_forced_release,
+		.driver_data = atkbd_dell_laptop_forced_release_keys,
+	},
+	{
+		.matches = {
 			DMI_MATCH(DMI_SYS_VENDOR, "Hewlett-Packard"),
 			DMI_MATCH(DMI_PRODUCT_NAME, "HP 2133"),
 		},
-- 
1.7.9.5

--
To unsubscribe from this list: send the line "unsubscribe linux-input" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH] Input: atkbd: Fix release quirk for Dell models

[Pali Rohár]

[-- Attachment #1: Type: Text/Plain, Size: 887 bytes --]

On Sunday 29 March 2015 16:36:19 Pali Rohár wrote:
> This patch fixes commit 61579ba83934 ("Input: atkbd - expand
> Latitude's force release quirk to other Dells"). Before that
> commit release quirks were called for all Dell Latitude
> models. After that commit only for Portable Dell devices. But
> lot of Latitude models are Laptop or Notebook DMI devices so
> quirks are not called.
> 
> Release quirks are still needed also for new Dell Latitude
> models, so this patch enables quirks for all Portable,
> Laptop, Notebook and Sub-Notebook Dell devices.
> 
> Signed-off-by: Pali Rohár <pali.rohar@gmail.com>

Dmitry, Matthew: Can be this patch pushed to 4.0 and backported 
to stable releases? It fix regression caused by specified commit. 
I tested this patch on Latitude E6440 and now all keys are 
correctly released.

-- 
Pali Rohár
pali.rohar@gmail.com

[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 198 bytes --]

Re: [PATCH] Input: atkbd: Fix release quirk for Dell models

[Dmitry Torokhov]

Hi Pali,

On Sun, Mar 29, 2015 at 04:36:19PM +0200, Pali Rohár wrote:
> This patch fixes commit 61579ba83934 ("Input: atkbd - expand Latitude's force
> release quirk to other Dells"). Before that commit release quirks were called
> for all Dell Latitude models. After that commit only for Portable Dell devices.
> But lot of Latitude models are Laptop or Notebook DMI devices so quirks are not
> called.
> 
> Release quirks are still needed also for new Dell Latitude models, so this patch
> enables quirks for all Portable, Laptop, Notebook and Sub-Notebook Dell devices.

Does Dell use all these types for their laptops? What models do you know
that need this quirk?

Thanks.

> 
> Signed-off-by: Pali Rohár <pali.rohar@gmail.com>
> ---
>  drivers/input/keyboard/atkbd.c |   48 ++++++++++++++++++++++++++++++++++++++++
>  1 file changed, 48 insertions(+)
> 
> diff --git a/drivers/input/keyboard/atkbd.c b/drivers/input/keyboard/atkbd.c
> index 387c51f..3188493 100644
> --- a/drivers/input/keyboard/atkbd.c
> +++ b/drivers/input/keyboard/atkbd.c
> @@ -1664,6 +1664,30 @@ static const struct dmi_system_id atkbd_dmi_quirk_table[] __initconst = {
>  	},
>  	{
>  		.matches = {
> +			DMI_MATCH(DMI_SYS_VENDOR, "Dell Inc."),
> +			DMI_MATCH(DMI_CHASSIS_TYPE, "9"), /* Laptop */
> +		},
> +		.callback = atkbd_setup_forced_release,
> +		.driver_data = atkbd_dell_laptop_forced_release_keys,
> +	},
> +	{
> +		.matches = {
> +			DMI_MATCH(DMI_SYS_VENDOR, "Dell Inc."),
> +			DMI_MATCH(DMI_CHASSIS_TYPE, "10"), /* Notebook */
> +		},
> +		.callback = atkbd_setup_forced_release,
> +		.driver_data = atkbd_dell_laptop_forced_release_keys,
> +	},
> +	{
> +		.matches = {
> +			DMI_MATCH(DMI_SYS_VENDOR, "Dell Inc."),
> +			DMI_MATCH(DMI_CHASSIS_TYPE, "14"), /* Sub-Notebook */
> +		},
> +		.callback = atkbd_setup_forced_release,
> +		.driver_data = atkbd_dell_laptop_forced_release_keys,
> +	},
> +	{
> +		.matches = {
>  			DMI_MATCH(DMI_SYS_VENDOR, "Dell Computer Corporation"),
>  			DMI_MATCH(DMI_CHASSIS_TYPE, "8"), /* Portable */
>  		},
> @@ -1672,6 +1696,30 @@ static const struct dmi_system_id atkbd_dmi_quirk_table[] __initconst = {
>  	},
>  	{
>  		.matches = {
> +			DMI_MATCH(DMI_SYS_VENDOR, "Dell Computer Corporation"),
> +			DMI_MATCH(DMI_CHASSIS_TYPE, "9"), /* Laptop */
> +		},
> +		.callback = atkbd_setup_forced_release,
> +		.driver_data = atkbd_dell_laptop_forced_release_keys,
> +	},
> +	{
> +		.matches = {
> +			DMI_MATCH(DMI_SYS_VENDOR, "Dell Computer Corporation"),
> +			DMI_MATCH(DMI_CHASSIS_TYPE, "10"), /* Notebook */
> +		},
> +		.callback = atkbd_setup_forced_release,
> +		.driver_data = atkbd_dell_laptop_forced_release_keys,
> +	},
> +	{
> +		.matches = {
> +			DMI_MATCH(DMI_SYS_VENDOR, "Dell Computer Corporation"),
> +			DMI_MATCH(DMI_CHASSIS_TYPE, "14"), /* Sub-Notebook */
> +		},
> +		.callback = atkbd_setup_forced_release,
> +		.driver_data = atkbd_dell_laptop_forced_release_keys,
> +	},
> +	{
> +		.matches = {
>  			DMI_MATCH(DMI_SYS_VENDOR, "Hewlett-Packard"),
>  			DMI_MATCH(DMI_PRODUCT_NAME, "HP 2133"),
>  		},
> -- 
> 1.7.9.5
> 

-- 
Dmitry
--
To unsubscribe from this list: send the line "unsubscribe linux-input" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH] Input: atkbd: Fix release quirk for Dell models

[Pali Rohár]

[-- Attachment #1: Type: Text/Plain, Size: 1249 bytes --]

On Sunday 05 April 2015 23:48:33 Dmitry Torokhov wrote:
> Hi Pali,
> 
> On Sun, Mar 29, 2015 at 04:36:19PM +0200, Pali Rohár wrote:
> > This patch fixes commit 61579ba83934 ("Input: atkbd - expand
> > Latitude's force release quirk to other Dells"). Before
> > that commit release quirks were called for all Dell
> > Latitude models. After that commit only for Portable Dell
> > devices. But lot of Latitude models are Laptop or Notebook
> > DMI devices so quirks are not called.
> > 
> > Release quirks are still needed also for new Dell Latitude
> > models, so this patch enables quirks for all Portable,
> > Laptop, Notebook and Sub-Notebook Dell devices.
> 
> Does Dell use all these types for their laptops? What models
> do you know that need this quirk?
> 
> Thanks.
> 

I do not if Dell use all types, but months ago Matthew wrote to 
include also other numbers not only 9 (Laptop) and you agreed.

I do not know exact list of models which needs these quirks, but 
before that commit (61579ba83934) it was used for all Latitude 
models. At least I see that switches do not generate release 
events and on older Latitude machines some Fn keys do not 
generate them too.

-- 
Pali Rohár
pali.rohar@gmail.com

[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 198 bytes --]

Re: [PATCH] Input: atkbd: Fix release quirk for Dell models

[Dmitry Torokhov]

On Mon, Apr 06, 2015 at 12:00:32AM +0200, Pali Rohár wrote:
> On Sunday 05 April 2015 23:48:33 Dmitry Torokhov wrote:
> > Hi Pali,
> > 
> > On Sun, Mar 29, 2015 at 04:36:19PM +0200, Pali Rohár wrote:
> > > This patch fixes commit 61579ba83934 ("Input: atkbd - expand
> > > Latitude's force release quirk to other Dells"). Before
> > > that commit release quirks were called for all Dell
> > > Latitude models. After that commit only for Portable Dell
> > > devices. But lot of Latitude models are Laptop or Notebook
> > > DMI devices so quirks are not called.
> > > 
> > > Release quirks are still needed also for new Dell Latitude
> > > models, so this patch enables quirks for all Portable,
> > > Laptop, Notebook and Sub-Notebook Dell devices.
> > 
> > Does Dell use all these types for their laptops? What models
> > do you know that need this quirk?
> > 
> > Thanks.
> > 
> 
> I do not if Dell use all types, but months ago Matthew wrote to 
> include also other numbers not only 9 (Laptop) and you agreed.

Hmm, I tried looking back but I could not quite find the discussion.

> 
> I do not know exact list of models which needs these quirks, but 
> before that commit (61579ba83934) it was used for all Latitude 
> models. At least I see that switches do not generate release 
> events and on older Latitude machines some Fn keys do not 
> generate them too.

So the question is still: what models do need this quirk and what
chassis type they are used.

Regardless, I have not accepted quirks for force_release and atkbd
keymaps for many years now as the task to adjust both keymap and
force_release list has been offloaded to udev.

Thanks.

-- 
Dmitry
--
To unsubscribe from this list: send the line "unsubscribe linux-input" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH] Input: atkbd: Fix release quirk for Dell models

[Pali Rohár]

[-- Attachment #1: Type: Text/Plain, Size: 2319 bytes --]

On Monday 06 April 2015 01:53:43 Dmitry Torokhov wrote:
> On Mon, Apr 06, 2015 at 12:00:32AM +0200, Pali Rohár wrote:
> > On Sunday 05 April 2015 23:48:33 Dmitry Torokhov wrote:
> > > Hi Pali,
> > > 
> > > On Sun, Mar 29, 2015 at 04:36:19PM +0200, Pali Rohár wrote:
> > > > This patch fixes commit 61579ba83934 ("Input: atkbd -
> > > > expand Latitude's force release quirk to other Dells").
> > > > Before that commit release quirks were called for all
> > > > Dell Latitude models. After that commit only for
> > > > Portable Dell devices. But lot of Latitude models are
> > > > Laptop or Notebook DMI devices so quirks are not
> > > > called.
> > > > 
> > > > Release quirks are still needed also for new Dell
> > > > Latitude models, so this patch enables quirks for all
> > > > Portable, Laptop, Notebook and Sub-Notebook Dell
> > > > devices.
> > > 
> > > Does Dell use all these types for their laptops? What
> > > models do you know that need this quirk?
> > > 
> > > Thanks.
> > 
> > I do not if Dell use all types, but months ago Matthew wrote
> > to include also other numbers not only 9 (Laptop) and you
> > agreed.
> 
> Hmm, I tried looking back but I could not quite find the
> discussion.
> 

Search for Message-ID: <20141222175632.GB18556@dtor-ws>
(this should be unique identifier for emails)

> > I do not know exact list of models which needs these quirks,
> > but before that commit (61579ba83934) it was used for all
> > Latitude models. At least I see that switches do not
> > generate release events and on older Latitude machines some
> > Fn keys do not generate them too.
> 
> So the question is still: what models do need this quirk and
> what chassis type they are used.
> 

My model:

$ cat /sys/class/dmi/id/chassis_type 
9
$ cat /sys/class/dmi/id/sys_vendor
Dell Inc.
$ cat /sys/class/dmi/id/product_name
Latitude E6440

> Regardless, I have not accepted quirks for force_release and
> atkbd keymaps for many years now as the task to adjust both
> keymap and force_release list has been offloaded to udev.
> 
> Thanks.

Ok, and what to do with kernel regressions? As before commit 
61579ba83934 it worked, because quirk was called for all Latitude 
(independently on chassis_type).

-- 
Pali Rohár
pali.rohar@gmail.com

[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 198 bytes --]

Re: [PATCH] Input: atkbd: Fix release quirk for Dell models

[Dmitry Torokhov]

On Mon, Apr 06, 2015 at 10:06:10AM +0200, Pali Rohár wrote:
> On Monday 06 April 2015 01:53:43 Dmitry Torokhov wrote:
> > On Mon, Apr 06, 2015 at 12:00:32AM +0200, Pali Rohár wrote:
> > > On Sunday 05 April 2015 23:48:33 Dmitry Torokhov wrote:
> > > > Hi Pali,
> > > > 
> > > > On Sun, Mar 29, 2015 at 04:36:19PM +0200, Pali Rohár wrote:
> > > > > This patch fixes commit 61579ba83934 ("Input: atkbd -
> > > > > expand Latitude's force release quirk to other Dells").
> > > > > Before that commit release quirks were called for all
> > > > > Dell Latitude models. After that commit only for
> > > > > Portable Dell devices. But lot of Latitude models are
> > > > > Laptop or Notebook DMI devices so quirks are not
> > > > > called.
> > > > > 
> > > > > Release quirks are still needed also for new Dell
> > > > > Latitude models, so this patch enables quirks for all
> > > > > Portable, Laptop, Notebook and Sub-Notebook Dell
> > > > > devices.
> > > > 
> > > > Does Dell use all these types for their laptops? What
> > > > models do you know that need this quirk?
> > > > 
> > > > Thanks.
> > > 
> > > I do not if Dell use all types, but months ago Matthew wrote
> > > to include also other numbers not only 9 (Laptop) and you
> > > agreed.
> > 
> > Hmm, I tried looking back but I could not quite find the
> > discussion.
> > 
> 
> Search for Message-ID: <20141222175632.GB18556@dtor-ws>
> (this should be unique identifier for emails)

Ah, indeed. Sorry, at this time I completely forgot that we are
controlling this from userspace nowadays.

> 
> > > I do not know exact list of models which needs these quirks,
> > > but before that commit (61579ba83934) it was used for all
> > > Latitude models. At least I see that switches do not
> > > generate release events and on older Latitude machines some
> > > Fn keys do not generate them too.
> > 
> > So the question is still: what models do need this quirk and
> > what chassis type they are used.
> > 
> 
> My model:
> 
> $ cat /sys/class/dmi/id/chassis_type 
> 9
> $ cat /sys/class/dmi/id/sys_vendor
> Dell Inc.
> $ cat /sys/class/dmi/id/product_name
> Latitude E6440
> 
> > Regardless, I have not accepted quirks for force_release and
> > atkbd keymaps for many years now as the task to adjust both
> > keymap and force_release list has been offloaded to udev.
> > 
> > Thanks.
> 
> Ok, and what to do with kernel regressions? As before commit 
> 61579ba83934 it worked, because quirk was called for all Latitude 
> (independently on chassis_type).

No, this is not a kernel regression because it would mean that your
laptop worked properly with then current kernel and then stopped
working. Given that the change went in in 2008 and your laptop is a bit
newer than that you can't claim kernel regression here; as far as I can
see the change was reasonable for the lineage we had back then, or at
least noone notified us for about 6 years.

So please do adjust udev rules, there are quite a bit for tweaks for
Dell and other manufacturer's laptop keyboards there).

Thanks.

-- 
Dmitry
--
To unsubscribe from this list: send the line "unsubscribe linux-input" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH] Input: atkbd: Fix release quirk for Dell models

[Pali Rohár]

[-- Attachment #1: Type: Text/Plain, Size: 3600 bytes --]

On Monday 06 April 2015 18:20:56 Dmitry Torokhov wrote:
> On Mon, Apr 06, 2015 at 10:06:10AM +0200, Pali Rohár wrote:
> > On Monday 06 April 2015 01:53:43 Dmitry Torokhov wrote:
> > > On Mon, Apr 06, 2015 at 12:00:32AM +0200, Pali Rohár wrote:
> > > > On Sunday 05 April 2015 23:48:33 Dmitry Torokhov wrote:
> > > > > Hi Pali,
> > > > > 
> > > > > On Sun, Mar 29, 2015 at 04:36:19PM +0200, Pali Rohár
> > > > > wrote:
> > > > > > This patch fixes commit 61579ba83934 ("Input: atkbd
> > > > > > - expand Latitude's force release quirk to other
> > > > > > Dells"). Before that commit release quirks were
> > > > > > called for all Dell Latitude models. After that
> > > > > > commit only for Portable Dell devices. But lot of
> > > > > > Latitude models are Laptop or Notebook DMI devices
> > > > > > so quirks are not called.
> > > > > > 
> > > > > > Release quirks are still needed also for new Dell
> > > > > > Latitude models, so this patch enables quirks for
> > > > > > all Portable, Laptop, Notebook and Sub-Notebook
> > > > > > Dell devices.
> > > > > 
> > > > > Does Dell use all these types for their laptops? What
> > > > > models do you know that need this quirk?
> > > > > 
> > > > > Thanks.
> > > > 
> > > > I do not if Dell use all types, but months ago Matthew
> > > > wrote to include also other numbers not only 9 (Laptop)
> > > > and you agreed.
> > > 
> > > Hmm, I tried looking back but I could not quite find the
> > > discussion.
> > 
> > Search for Message-ID: <20141222175632.GB18556@dtor-ws>
> > (this should be unique identifier for emails)
> 
> Ah, indeed. Sorry, at this time I completely forgot that we
> are controlling this from userspace nowadays.
> 
> > > > I do not know exact list of models which needs these
> > > > quirks, but before that commit (61579ba83934) it was
> > > > used for all Latitude models. At least I see that
> > > > switches do not generate release events and on older
> > > > Latitude machines some Fn keys do not generate them
> > > > too.
> > > 
> > > So the question is still: what models do need this quirk
> > > and what chassis type they are used.
> > 
> > My model:
> > 
> > $ cat /sys/class/dmi/id/chassis_type
> > 9
> > $ cat /sys/class/dmi/id/sys_vendor
> > Dell Inc.
> > $ cat /sys/class/dmi/id/product_name
> > Latitude E6440
> > 
> > > Regardless, I have not accepted quirks for force_release
> > > and atkbd keymaps for many years now as the task to
> > > adjust both keymap and force_release list has been
> > > offloaded to udev.
> > > 
> > > Thanks.
> > 
> > Ok, and what to do with kernel regressions? As before commit
> > 61579ba83934 it worked, because quirk was called for all
> > Latitude (independently on chassis_type).
> 
> No, this is not a kernel regression because it would mean that
> your laptop worked properly with then current kernel and then
> stopped working. Given that the change went in in 2008 and
> your laptop is a bit newer than that you can't claim kernel
> regression here; as far as I can see the change was
> reasonable for the lineage we had back then, or at least
> noone notified us for about 6 years.
> 
> So please do adjust udev rules, there are quite a bit for
> tweaks for Dell and other manufacturer's laptop keyboards
> there).
> 
> Thanks.

Ok, thanks for information. Maybe you can add some comment to 
source code or documentation, that this part of code is now 
handled by userspace. So other people send patches to correct 
place/projects in future.

-- 
Pali Rohár
pali.rohar@gmail.com

[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 198 bytes --]

Re: [PATCH] Input: atkbd: Fix release quirk for Dell models

[Dmitry Torokhov]

On Wed, Apr 08, 2015 at 10:54:20PM +0200, Pali Rohár wrote:
> On Monday 06 April 2015 18:20:56 Dmitry Torokhov wrote:
> > On Mon, Apr 06, 2015 at 10:06:10AM +0200, Pali Rohár wrote:
> > > On Monday 06 April 2015 01:53:43 Dmitry Torokhov wrote:
> > > > On Mon, Apr 06, 2015 at 12:00:32AM +0200, Pali Rohár wrote:
> > > > > On Sunday 05 April 2015 23:48:33 Dmitry Torokhov wrote:
> > > > > > Hi Pali,
> > > > > > 
> > > > > > On Sun, Mar 29, 2015 at 04:36:19PM +0200, Pali Rohár
> > > > > > wrote:
> > > > > > > This patch fixes commit 61579ba83934 ("Input: atkbd
> > > > > > > - expand Latitude's force release quirk to other
> > > > > > > Dells"). Before that commit release quirks were
> > > > > > > called for all Dell Latitude models. After that
> > > > > > > commit only for Portable Dell devices. But lot of
> > > > > > > Latitude models are Laptop or Notebook DMI devices
> > > > > > > so quirks are not called.
> > > > > > > 
> > > > > > > Release quirks are still needed also for new Dell
> > > > > > > Latitude models, so this patch enables quirks for
> > > > > > > all Portable, Laptop, Notebook and Sub-Notebook
> > > > > > > Dell devices.
> > > > > > 
> > > > > > Does Dell use all these types for their laptops? What
> > > > > > models do you know that need this quirk?
> > > > > > 
> > > > > > Thanks.
> > > > > 
> > > > > I do not if Dell use all types, but months ago Matthew
> > > > > wrote to include also other numbers not only 9 (Laptop)
> > > > > and you agreed.
> > > > 
> > > > Hmm, I tried looking back but I could not quite find the
> > > > discussion.
> > > 
> > > Search for Message-ID: <20141222175632.GB18556@dtor-ws>
> > > (this should be unique identifier for emails)
> > 
> > Ah, indeed. Sorry, at this time I completely forgot that we
> > are controlling this from userspace nowadays.
> > 
> > > > > I do not know exact list of models which needs these
> > > > > quirks, but before that commit (61579ba83934) it was
> > > > > used for all Latitude models. At least I see that
> > > > > switches do not generate release events and on older
> > > > > Latitude machines some Fn keys do not generate them
> > > > > too.
> > > > 
> > > > So the question is still: what models do need this quirk
> > > > and what chassis type they are used.
> > > 
> > > My model:
> > > 
> > > $ cat /sys/class/dmi/id/chassis_type
> > > 9
> > > $ cat /sys/class/dmi/id/sys_vendor
> > > Dell Inc.
> > > $ cat /sys/class/dmi/id/product_name
> > > Latitude E6440
> > > 
> > > > Regardless, I have not accepted quirks for force_release
> > > > and atkbd keymaps for many years now as the task to
> > > > adjust both keymap and force_release list has been
> > > > offloaded to udev.
> > > > 
> > > > Thanks.
> > > 
> > > Ok, and what to do with kernel regressions? As before commit
> > > 61579ba83934 it worked, because quirk was called for all
> > > Latitude (independently on chassis_type).
> > 
> > No, this is not a kernel regression because it would mean that
> > your laptop worked properly with then current kernel and then
> > stopped working. Given that the change went in in 2008 and
> > your laptop is a bit newer than that you can't claim kernel
> > regression here; as far as I can see the change was
> > reasonable for the lineage we had back then, or at least
> > noone notified us for about 6 years.
> > 
> > So please do adjust udev rules, there are quite a bit for
> > tweaks for Dell and other manufacturer's laptop keyboards
> > there).
> > 
> > Thanks.
> 
> Ok, thanks for information. Maybe you can add some comment to 
> source code or documentation, that this part of code is now 
> handled by userspace. So other people send patches to correct 
> place/projects in future.

Fair enough. How about below?

Thanks.

-- 
Dmitry

Input: atkbd - document "no new force-release quirks" policy

From: Dmitry Torokhov <dmitry.torokhov@gmail.com>

To save people some time let's document that we do not want new quirks for
"force-release" keys in the kernel and that they should patch userspace
(udev) instead.

Suggested-by: Pali Rohár <pali.rohar@gmail.com>
Signed-off-by: Dmitry Torokhov <dmitry.torokhov@gmail.com>
---
 drivers/input/keyboard/atkbd.c |    6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/drivers/input/keyboard/atkbd.c b/drivers/input/keyboard/atkbd.c
index 387c51f..ec876b5 100644
--- a/drivers/input/keyboard/atkbd.c
+++ b/drivers/input/keyboard/atkbd.c
@@ -1653,6 +1653,12 @@ static int __init atkbd_deactivate_fixup(const struct dmi_system_id *id)
 	return 1;
 }
 
+/*
+ * NOTE: do not add any more "force release" quirks to this table.  The
+ * task of adjusting list of keys that should be "released" automatically
+ * by the driver is now delegated to userspace tools, such as udev, so
+ * submit such quirks there.
+ */
 static const struct dmi_system_id atkbd_dmi_quirk_table[] __initconst = {
 	{
 		.matches = {
--
To unsubscribe from this list: send the line "unsubscribe linux-input" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH] Input: atkbd: Fix release quirk for Dell models

[Pali Rohár]

[-- Attachment #1: Type: Text/Plain, Size: 1877 bytes --]

On Thursday 09 April 2015 18:57:19 Dmitry Torokhov wrote:
> On Wed, Apr 08, 2015 at 10:54:20PM +0200, Pali Rohár wrote:
> > On Monday 06 April 2015 18:20:56 Dmitry Torokhov wrote:
> > > On Mon, Apr 06, 2015 at 10:06:10AM +0200, Pali Rohár wrote:
> > > 
> > >
> > > Ah, indeed. Sorry, at this time I completely forgot that
> > > we are controlling this from userspace nowadays.
> > >
> > > 
> > >
> > > 
> > >
> > > No, this is not a kernel regression because it would mean
> > > that your laptop worked properly with then current kernel
> > > and then stopped working. Given that the change went in
> > > in 2008 and your laptop is a bit newer than that you
> > > can't claim kernel regression here; as far as I can see
> > > the change was reasonable for the lineage we had back
> > > then, or at least noone notified us for about 6 years.
> > >
> > > 
> > >
> > > So please do adjust udev rules, there are quite a bit for
> > > tweaks for Dell and other manufacturer's laptop keyboards
> > > there).
> > >
> > > 
> > >
> > > Thanks.
> >
> > 
> >
> > Ok, thanks for information. Maybe you can add some comment
> > to  source code or documentation, that this part of code is
> > now handled by userspace. So other people send patches to
> > correct place/projects in future.
> 
> Fair enough. How about below?
> 
> Thanks.
> 
> -- 
> Dmitry
> 
> Input: atkbd - document "no new force-release quirks" policy
> 
> From: Dmitry Torokhov <dmitry.torokhov@gmail.com>
> 
> To save people some time let's document that we do not want
> new quirks for "force-release" keys in the kernel and that
> they should patch userspace (udev) instead.
> 
> Suggested-by: Pali Rohár <pali.rohar@gmail.com>
> Signed-off-by: Dmitry Torokhov <dmitry.torokhov@gmail.com>
> ---

Looks good for me.

-- 
Pali Rohár
pali.rohar@gmail.com

[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 198 bytes --]