Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1710363 > unrolled thread
| Started by | Hoeze <git.ich@frhoelzlwimmer.de> |
|---|---|
| First post | 2017-08-12 22:20 +0200 |
| Last post | 2017-08-16 08:00 +0200 |
| Articles | 3 — 3 participants |
Back to article view | Back to linux.kernel
[PATCH v3] Fix missing PCI ID; Necessary for Lenovo Yoga 720-15 Touchpad. Hoeze <git.ich@frhoelzlwimmer.de> - 2017-08-12 22:20 +0200
Re: [PATCH v3] Fix missing PCI ID; Necessary for Lenovo Yoga 720-15 Touchpad. Daniel Axtens <dja@axtens.net> - 2017-08-16 02:50 +0200
Re: [PATCH v3] Fix missing PCI ID; Necessary for Lenovo Yoga 720-15 Touchpad. Andrew Donnellan <andrew.donnellan@au1.ibm.com> - 2017-08-16 08:00 +0200
| From | Hoeze <git.ich@frhoelzlwimmer.de> |
|---|---|
| Date | 2017-08-12 22:20 +0200 |
| Subject | [PATCH v3] Fix missing PCI ID; Necessary for Lenovo Yoga 720-15 Touchpad. |
| Message-ID | <udL35-5Bs-7@gated-at.bofh.it> |
From: Hoeze <Hoeze@users.noreply.github.com>
This patch adds a missing PCI ID to the Intel LPSS driver.
It fixes a bug causing the touchpad of the Lenovo Yoga 720-15 not to be recognized.
See also bug 1700657 on bugs.launchpad.net.
Many thanks to CoolStar, who found this solution!
Reported-by: CoolStar <coolstarorganization@gmail.com>
Tested-by: Mike Schwartz <mykesx@gmail.com>
Tested-by: Björn Dahlgren <bjodah@gmail.com>
Signed-off-by: Florian R. Hölzlwimmer <git.ich@frhoelzlwimmer.de>
---
drivers/mfd/intel-lpss-pci.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/mfd/intel-lpss-pci.c b/drivers/mfd/intel-lpss-pci.c
index ad388bb0..d1c46de8 100644
--- a/drivers/mfd/intel-lpss-pci.c
+++ b/drivers/mfd/intel-lpss-pci.c
@@ -221,6 +221,7 @@ static const struct pci_device_id intel_lpss_pci_ids[] = {
{ PCI_VDEVICE(INTEL, 0xa12a), (kernel_ulong_t)&spt_info },
{ PCI_VDEVICE(INTEL, 0xa160), (kernel_ulong_t)&spt_i2c_info },
{ PCI_VDEVICE(INTEL, 0xa161), (kernel_ulong_t)&spt_i2c_info },
+ { PCI_VDEVICE(INTEL, 0xa162), (kernel_ulong_t)&spt_i2c_info },
{ PCI_VDEVICE(INTEL, 0xa166), (kernel_ulong_t)&spt_uart_info },
/* KBL-H */
{ PCI_VDEVICE(INTEL, 0xa2a7), (kernel_ulong_t)&spt_uart_info },
--
2.14.1
[toc] | [next] | [standalone]
| From | Daniel Axtens <dja@axtens.net> |
|---|---|
| Date | 2017-08-16 02:50 +0200 |
| Message-ID | <ueUGZ-8tc-5@gated-at.bofh.it> |
| In reply to | #1710363 |
Hi all,
This has worked its way from a pull request on GitHub through to this
patch, which is a great credit to the determination and effort of
Florian.
I think it's an excellent first contribution.
Reviewed-by: Daniel Axtens <dja@axtens.net>
Regards,
Daniel
Hoeze <git.ich@frhoelzlwimmer.de> writes:
> From: Hoeze <Hoeze@users.noreply.github.com>
>
> This patch adds a missing PCI ID to the Intel LPSS driver.
> It fixes a bug causing the touchpad of the Lenovo Yoga 720-15 not to be recognized.
> See also bug 1700657 on bugs.launchpad.net.
>
> Many thanks to CoolStar, who found this solution!
>
> Reported-by: CoolStar <coolstarorganization@gmail.com>
> Tested-by: Mike Schwartz <mykesx@gmail.com>
> Tested-by: Björn Dahlgren <bjodah@gmail.com>
> Signed-off-by: Florian R. Hölzlwimmer <git.ich@frhoelzlwimmer.de>
> ---
> drivers/mfd/intel-lpss-pci.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/drivers/mfd/intel-lpss-pci.c b/drivers/mfd/intel-lpss-pci.c
> index ad388bb0..d1c46de8 100644
> --- a/drivers/mfd/intel-lpss-pci.c
> +++ b/drivers/mfd/intel-lpss-pci.c
> @@ -221,6 +221,7 @@ static const struct pci_device_id intel_lpss_pci_ids[] = {
> { PCI_VDEVICE(INTEL, 0xa12a), (kernel_ulong_t)&spt_info },
> { PCI_VDEVICE(INTEL, 0xa160), (kernel_ulong_t)&spt_i2c_info },
> { PCI_VDEVICE(INTEL, 0xa161), (kernel_ulong_t)&spt_i2c_info },
> + { PCI_VDEVICE(INTEL, 0xa162), (kernel_ulong_t)&spt_i2c_info },
> { PCI_VDEVICE(INTEL, 0xa166), (kernel_ulong_t)&spt_uart_info },
> /* KBL-H */
> { PCI_VDEVICE(INTEL, 0xa2a7), (kernel_ulong_t)&spt_uart_info },
> --
> 2.14.1
[toc] | [prev] | [next] | [standalone]
| From | Andrew Donnellan <andrew.donnellan@au1.ibm.com> |
|---|---|
| Date | 2017-08-16 08:00 +0200 |
| Subject | Re: [PATCH v3] Fix missing PCI ID; Necessary for Lenovo Yoga 720-15 Touchpad. |
| Message-ID | <ueZx0-39t-13@gated-at.bofh.it> |
| In reply to | #1710363 |
Hi Florian, As Daniel said, good work on figuring out how to get this far! There's a couple of minor issues with the formatting of this patch. Firstly, as Lee said in response to the last version, you should change the summary line of the commit to indicate the contents of the patch more accurately. Based on what I see in the git log, I'd suggest: "mfd: intel-lpss: Add missing PCI ID" The previous commits to this file also mention the type of chipset - I *think* this is Sunrise Point, but I don't work for Intel. :) Don't mention the Lenovo touchpad in the summary - in the main body of the commit message is better. On 13/08/17 06:16, Hoeze wrote: > From: Hoeze <Hoeze@users.noreply.github.com> The name and email here is what's going to appear in the git commit log as the author of the commit. Generally, it should be the same as both the details in your Signed-off-by line, and the email should be the same as the email address you're sending from. If the name/email address in the commit is the same as the name/email address you're sending from, git send-email will actually skip this line completely. You should probably try: $ git config --global user.name "Florian R. Hölzlwimmer" $ git config --global user.email "git.ich@frhoelzlwimmer.de" $ git commit --amend --reset-author This will set your details in your git configuration, and then re-create the last commit using the correct authorship details. If you then re-send it, it should skip this From: line and everything should be fine. Andrew -- Andrew Donnellan OzLabs, ADL Canberra andrew.donnellan@au1.ibm.com IBM Australia Limited
[toc] | [prev] | [standalone]
Back to top | Article view | linux.kernel
csiph-web