* [PATCH v1] HID: hid-elecom: extend to fix descriptor for HUGE trackball @ 2017-10-14 3:23 Alex Manoussakis 2017-10-16 20:20 ` Alex Manoussakis 0 siblings, 1 reply; 9+ messages in thread From: Alex Manoussakis @ 2017-10-14 3:23 UTC (permalink / raw) To: stable; +Cc: Alex Manoussakis, Jiri Kosina [-- Warning: decoded text below may be mangled, UTF-8 assumed --] [-- Attachment #1: Type: text/plain, Size: 4312 bytes --] commit a0933a456ff83a3b5ffa3a1903e0b8de4a56adf5 upstream In addition to DEFT, Elecom introduced a larger trackball called HUGE, in both wired (M-HT1URBK) and wireless (M-HT1DRBK) versions. It has the same buttons and behavior as the DEFT. This patch adds the two relevant USB IDs to enable operation of the three Fn buttons on the top of the device. Works fine in 4.13, in fact I am succefully using this patch on 4.4 as well (4.4 needs additional cherry pick 0bb7a37f8d15e5fb5d21776875f9fbc74e10753a) Cc: <stable@vger.kernel.org> # 4.12.x Signed-off-by: Alex Manoussakis <amanou@gnu.org> Signed-off-by: Jiri Kosina <jikos@kernel.org> diff --git a/drivers/hid/Kconfig b/drivers/hid/Kconfig index 0a3117c..374301f 100644 --- a/drivers/hid/Kconfig +++ b/drivers/hid/Kconfig @@ -281,6 +281,7 @@ config HID_ELECOM Support for ELECOM devices: - BM084 Bluetooth Mouse - DEFT Trackball (Wired and wireless) + - HUGE Trackball (Wired and wireless) config HID_ELO tristate "ELO USB 4000/4500 touchscreen" diff --git a/drivers/hid/hid-core.c b/drivers/hid/hid-core.c index 9bc9116..330ca98 100644 --- a/drivers/hid/hid-core.c +++ b/drivers/hid/hid-core.c @@ -2032,6 +2032,8 @@ static const struct hid_device_id hid_have_special_driver[] = { { HID_BLUETOOTH_DEVICE(USB_VENDOR_ID_ELECOM, USB_DEVICE_ID_ELECOM_BM084) }, { HID_USB_DEVICE(USB_VENDOR_ID_ELECOM, USB_DEVICE_ID_ELECOM_DEFT_WIRED) }, { HID_USB_DEVICE(USB_VENDOR_ID_ELECOM, USB_DEVICE_ID_ELECOM_DEFT_WIRELESS) }, + { HID_USB_DEVICE(USB_VENDOR_ID_ELECOM, USB_DEVICE_ID_ELECOM_HUGE_WIRED) }, + { HID_USB_DEVICE(USB_VENDOR_ID_ELECOM, USB_DEVICE_ID_ELECOM_HUGE_WIRELESS) }, #endif #if IS_ENABLED(CONFIG_HID_ELO) { HID_USB_DEVICE(USB_VENDOR_ID_ELO, 0x0009) }, diff --git a/drivers/hid/hid-elecom.c b/drivers/hid/hid-elecom.c index e2c7465..54aeea5 100644 --- a/drivers/hid/hid-elecom.c +++ b/drivers/hid/hid-elecom.c @@ -3,6 +3,7 @@ * Copyright (c) 2010 Richard Nauber <Richard.Nauber@gmail.com> * Copyright (c) 2016 Yuxuan Shui <yshuiv7@gmail.com> * Copyright (c) 2017 Diego Elio Pettenò <flameeyes@flameeyes.eu> + * Copyright (c) 2017 Alex Manoussakis <amanou@gnu.org> */ /* @@ -32,9 +33,11 @@ static __u8 *elecom_report_fixup(struct hid_device *hdev, __u8 *rdesc, break; case USB_DEVICE_ID_ELECOM_DEFT_WIRED: case USB_DEVICE_ID_ELECOM_DEFT_WIRELESS: - /* The DEFT trackball has eight buttons, but its descriptor only - * reports five, disabling the three Fn buttons on the top of - * the mouse. + case USB_DEVICE_ID_ELECOM_HUGE_WIRED: + case USB_DEVICE_ID_ELECOM_HUGE_WIRELESS: + /* The DEFT/HUGE trackball has eight buttons, but its descriptor + * only reports five, disabling the three Fn buttons on the top + * of the mouse. * * Apply the following diff to the descriptor: * @@ -62,7 +65,7 @@ static __u8 *elecom_report_fixup(struct hid_device *hdev, __u8 *rdesc, * End Collection, End Collection, */ if (*rsize == 213 && rdesc[13] == 5 && rdesc[21] == 5) { - hid_info(hdev, "Fixing up Elecom DEFT Fn buttons\n"); + hid_info(hdev, "Fixing up Elecom DEFT/HUGE Fn buttons\n"); rdesc[13] = 8; /* Button/Variable Report Count */ rdesc[21] = 8; /* Button/Variable Usage Maximum */ rdesc[29] = 0; /* Button/Constant Report Count */ @@ -76,6 +79,8 @@ static const struct hid_device_id elecom_devices[] = { { HID_BLUETOOTH_DEVICE(USB_VENDOR_ID_ELECOM, USB_DEVICE_ID_ELECOM_BM084) }, { HID_USB_DEVICE(USB_VENDOR_ID_ELECOM, USB_DEVICE_ID_ELECOM_DEFT_WIRED) }, { HID_USB_DEVICE(USB_VENDOR_ID_ELECOM, USB_DEVICE_ID_ELECOM_DEFT_WIRELESS) }, + { HID_USB_DEVICE(USB_VENDOR_ID_ELECOM, USB_DEVICE_ID_ELECOM_HUGE_WIRED) }, + { HID_USB_DEVICE(USB_VENDOR_ID_ELECOM, USB_DEVICE_ID_ELECOM_HUGE_WIRELESS) }, { } }; MODULE_DEVICE_TABLE(hid, elecom_devices); diff --git a/drivers/hid/hid-ids.h b/drivers/hid/hid-ids.h index b397a14..e9da185 100644 --- a/drivers/hid/hid-ids.h +++ b/drivers/hid/hid-ids.h @@ -368,6 +368,8 @@ #define USB_DEVICE_ID_ELECOM_BM084 0x0061 #define USB_DEVICE_ID_ELECOM_DEFT_WIRED 0x00fe #define USB_DEVICE_ID_ELECOM_DEFT_WIRELESS 0x00ff +#define USB_DEVICE_ID_ELECOM_HUGE_WIRED 0x010c +#define USB_DEVICE_ID_ELECOM_HUGE_WIRELESS 0x010d #define USB_VENDOR_ID_DREAM_CHEEKY 0x1d34 #define USB_DEVICE_ID_DREAM_CHEEKY_WN 0x0004 ^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH v1] HID: hid-elecom: extend to fix descriptor for HUGE trackball 2017-10-14 3:23 [PATCH v1] HID: hid-elecom: extend to fix descriptor for HUGE trackball Alex Manoussakis @ 2017-10-16 20:20 ` Alex Manoussakis 2017-10-17 6:58 ` Greg Kroah-Hartman 2017-10-19 9:35 ` Greg Kroah-Hartman 0 siblings, 2 replies; 9+ messages in thread From: Alex Manoussakis @ 2017-10-16 20:20 UTC (permalink / raw) To: stable; +Cc: Jiri Kosina, Greg Kroah-Hartman, Alex Manoussakis commit a0933a456ff83a3b5ffa3a1903e0b8de4a56adf5 upstream Hello, while you evaluate my request to commit 2 USB IDs to 4.13, I would like you to consider putting it in 4.4 and 4.9 also. I tested tested both of these (patch, build, boot, and usage of the device). It would be great for users of those longterm kernels to have their Elecom HUGE trackball work properly! The original patch does not apply cleanly to 4.4 and 4.9 however; a trivial adjustment is needed (simply in the unmodified positioning lines of one file). Below is the adjusted patch that will apply without conflicts to 4.4 and 4.9. As mentioned earlier, for kernels prior to 4.12 you would need to cherry pick 68f9e3d71650c0f4c48d039c83e3baa4e278deed first. Thanks! Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org> Cc: <stable@stable@vger.kernel.org> # 4.12.x Cc: <stable@stable@vger.kernel.org> # 4.9.x: 0bb7a37: HID: elecom: ... DEFT trackballs Cc: <stable@stable@vger.kernel.org> # 4.4.x: 0bb7a37: HID: elecom: ... DEFT trackballs diff --git a/drivers/hid/Kconfig b/drivers/hid/Kconfig index c376394a..d6e01a1 100644 --- a/drivers/hid/Kconfig +++ b/drivers/hid/Kconfig @@ -264,6 +264,7 @@ config HID_ELECOM Support for ELECOM devices: - BM084 Bluetooth Mouse - DEFT Trackball (Wired and wireless) + - HUGE Trackball (Wired and wireless) config HID_ELO tristate "ELO USB 4000/4500 touchscreen" diff --git a/drivers/hid/hid-core.c b/drivers/hid/hid-core.c index 456fcf0..fbd5e66 100644 --- a/drivers/hid/hid-core.c +++ b/drivers/hid/hid-core.c @@ -1887,6 +1887,8 @@ static const struct hid_device_id hid_have_special_driver[] = { { HID_BLUETOOTH_DEVICE(USB_VENDOR_ID_ELECOM, USB_DEVICE_ID_ELECOM_BM084) }, { HID_USB_DEVICE(USB_VENDOR_ID_ELECOM, USB_DEVICE_ID_ELECOM_DEFT_WIRED) }, { HID_USB_DEVICE(USB_VENDOR_ID_ELECOM, USB_DEVICE_ID_ELECOM_DEFT_WIRELESS) }, + { HID_USB_DEVICE(USB_VENDOR_ID_ELECOM, USB_DEVICE_ID_ELECOM_HUGE_WIRED) }, + { HID_USB_DEVICE(USB_VENDOR_ID_ELECOM, USB_DEVICE_ID_ELECOM_HUGE_WIRELESS) }, { HID_USB_DEVICE(USB_VENDOR_ID_ELO, 0x0009) }, { HID_USB_DEVICE(USB_VENDOR_ID_ELO, 0x0030) }, { HID_USB_DEVICE(USB_VENDOR_ID_EMS, USB_DEVICE_ID_EMS_TRIO_LINKER_PLUS_II) }, diff --git a/drivers/hid/hid-elecom.c b/drivers/hid/hid-elecom.c index e2c7465..54aeea5 100644 --- a/drivers/hid/hid-elecom.c +++ b/drivers/hid/hid-elecom.c @@ -3,6 +3,7 @@ * Copyright (c) 2010 Richard Nauber <Richard.Nauber@gmail.com> * Copyright (c) 2016 Yuxuan Shui <yshuiv7@gmail.com> * Copyright (c) 2017 Diego Elio Petten� <flameeyes@flameeyes.eu> + * Copyright (c) 2017 Alex Manoussakis <amanou@gnu.org> */ /* @@ -32,9 +33,11 @@ static __u8 *elecom_report_fixup(struct hid_device *hdev, __u8 *rdesc, break; case USB_DEVICE_ID_ELECOM_DEFT_WIRED: case USB_DEVICE_ID_ELECOM_DEFT_WIRELESS: - /* The DEFT trackball has eight buttons, but its descriptor only - * reports five, disabling the three Fn buttons on the top of - * the mouse. + case USB_DEVICE_ID_ELECOM_HUGE_WIRED: + case USB_DEVICE_ID_ELECOM_HUGE_WIRELESS: + /* The DEFT/HUGE trackball has eight buttons, but its descriptor + * only reports five, disabling the three Fn buttons on the top + * of the mouse. * * Apply the following diff to the descriptor: * @@ -62,7 +65,7 @@ static __u8 *elecom_report_fixup(struct hid_device *hdev, __u8 *rdesc, * End Collection, End Collection, */ if (*rsize == 213 && rdesc[13] == 5 && rdesc[21] == 5) { - hid_info(hdev, "Fixing up Elecom DEFT Fn buttons\n"); + hid_info(hdev, "Fixing up Elecom DEFT/HUGE Fn buttons\n"); rdesc[13] = 8; /* Button/Variable Report Count */ rdesc[21] = 8; /* Button/Variable Usage Maximum */ rdesc[29] = 0; /* Button/Constant Report Count */ @@ -76,6 +79,8 @@ static const struct hid_device_id elecom_devices[] = { { HID_BLUETOOTH_DEVICE(USB_VENDOR_ID_ELECOM, USB_DEVICE_ID_ELECOM_BM084) }, { HID_USB_DEVICE(USB_VENDOR_ID_ELECOM, USB_DEVICE_ID_ELECOM_DEFT_WIRED) }, { HID_USB_DEVICE(USB_VENDOR_ID_ELECOM, USB_DEVICE_ID_ELECOM_DEFT_WIRELESS) }, + { HID_USB_DEVICE(USB_VENDOR_ID_ELECOM, USB_DEVICE_ID_ELECOM_HUGE_WIRED) }, + { HID_USB_DEVICE(USB_VENDOR_ID_ELECOM, USB_DEVICE_ID_ELECOM_HUGE_WIRELESS) }, { } }; MODULE_DEVICE_TABLE(hid, elecom_devices); diff --git a/drivers/hid/hid-ids.h b/drivers/hid/hid-ids.h index dbf7077..06ce846 100644 --- a/drivers/hid/hid-ids.h +++ b/drivers/hid/hid-ids.h @@ -355,6 +355,8 @@ #define USB_DEVICE_ID_ELECOM_BM084 0x0061 #define USB_DEVICE_ID_ELECOM_DEFT_WIRED 0x00fe #define USB_DEVICE_ID_ELECOM_DEFT_WIRELESS 0x00ff +#define USB_DEVICE_ID_ELECOM_HUGE_WIRED 0x010c +#define USB_DEVICE_ID_ELECOM_HUGE_WIRELESS 0x010d #define USB_VENDOR_ID_DREAM_CHEEKY 0x1d34 #define USB_DEVICE_ID_DREAM_CHEEKY_WN 0x0004 ^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH v1] HID: hid-elecom: extend to fix descriptor for HUGE trackball 2017-10-16 20:20 ` Alex Manoussakis @ 2017-10-17 6:58 ` Greg Kroah-Hartman 2017-10-17 8:25 ` Alex Manoussakis 2017-10-19 9:35 ` Greg Kroah-Hartman 1 sibling, 1 reply; 9+ messages in thread From: Greg Kroah-Hartman @ 2017-10-17 6:58 UTC (permalink / raw) To: Alex Manoussakis; +Cc: stable, Jiri Kosina On Mon, Oct 16, 2017 at 04:20:35PM -0400, Alex Manoussakis wrote: > --- a/drivers/hid/hid-elecom.c > +++ b/drivers/hid/hid-elecom.c > @@ -3,6 +3,7 @@ > * Copyright (c) 2010 Richard Nauber <Richard.Nauber@gmail.com> > * Copyright (c) 2016 Yuxuan Shui <yshuiv7@gmail.com> > * Copyright (c) 2017 Diego Elio Petten� <flameeyes@flameeyes.eu> > + * Copyright (c) 2017 Alex Manoussakis <amanou@gnu.org> Note, I'm not the maintainer, but you should be aware that adding a line like this, with only the changes as expressed in this patch, is very dubious according to every lawyer that I know. Please be careful about stuff like this, and only add such a line if you really know what you are doing[1] If you have done other changes to this file, my appologies... thanks, greg k-h [1] There is the real fact that these types of lines mean absolutely nothing, and that lawyers love to cargo-cult worse than programmers do, but that being said, please always know the norms for what this means and represents before using. ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH v1] HID: hid-elecom: extend to fix descriptor for HUGE trackball 2017-10-17 6:58 ` Greg Kroah-Hartman @ 2017-10-17 8:25 ` Alex Manoussakis 2017-10-17 8:35 ` Greg Kroah-Hartman 0 siblings, 1 reply; 9+ messages in thread From: Alex Manoussakis @ 2017-10-17 8:25 UTC (permalink / raw) To: Greg Kroah-Hartman; +Cc: stable, Jiri Kosina, Diego Elio Petteo > > * Copyright (c) 2010 Richard Nauber <Richard.Nauber@gmail.com> > > * Copyright (c) 2016 Yuxuan Shui <yshuiv7@gmail.com> > > * Copyright (c) 2017 Diego Elio Petten� <flameeyes@flameeyes.eu> > > + * Copyright (c) 2017 Alex Manoussakis <amanou@gnu.org> > > Note, I'm not the maintainer, but you should be aware that adding a line > like this, with only the changes as expressed in this patch, is very > dubious according to every lawyer that I know. Please be careful about > stuff like this, and only add such a line if you really know what you > are doing[1] > > If you have done other changes to this file, my appologies... Hi Greg, thanks for the note. The changes I have done is add new USB IDs and change the text in a couple of places (both comments and messages) to include the new model "HUGE" where it only said "DEFT". We could remove the copyright line I added if it's a problem. I didn't give it much thought. I copied the linux-input maintainer Jiri, and Diego who added the code for the DEFT trackballs, in my mainline submission and both were ok (Diego wrote me "The patch looks good to me", and Jiri submitted it) and now it's in mainline. Copying Diego in this thread now too. My goal is simply to make the trackball I purchased work in my favorite OS, and benefit other users who have found or will find themselves with a semi- working device otherwise. And push as far back as 4.4 since some very popular long-term-support distros use this kernel, I use it too, and it works. Can you commit the patch upstream ? Below is the patch that will apply clean to 4.4 and 4.9, without the extra copyright line if you prefer to use that. Note 4.4 and 4.9 need to cherry pick 0bb7a37f8d15e5fb5d21776875f9fbc74e10753a as a prerequisite which adds support for the DEFT trackballs. Should I submit a mainline patch that removes the Copyright line I added ? Let me know what I should do and I'll be happy to do it. Thanks! Alex Cc: Diego Elio Petten� <flameeyes@flameeyes.eu> diff --git a/drivers/hid/Kconfig b/drivers/hid/Kconfig index c376394a..d6e01a1 100644 --- a/drivers/hid/Kconfig +++ b/drivers/hid/Kconfig @@ -264,6 +264,7 @@ config HID_ELECOM Support for ELECOM devices: - BM084 Bluetooth Mouse - DEFT Trackball (Wired and wireless) + - HUGE Trackball (Wired and wireless) config HID_ELO tristate "ELO USB 4000/4500 touchscreen" diff --git a/drivers/hid/hid-core.c b/drivers/hid/hid-core.c index 456fcf0..fbd5e66 100644 --- a/drivers/hid/hid-core.c +++ b/drivers/hid/hid-core.c @@ -1887,6 +1887,8 @@ static const struct hid_device_id hid_have_special_driver[] = { { HID_BLUETOOTH_DEVICE(USB_VENDOR_ID_ELECOM, USB_DEVICE_ID_ELECOM_BM084) }, { HID_USB_DEVICE(USB_VENDOR_ID_ELECOM, USB_DEVICE_ID_ELECOM_DEFT_WIRED) }, { HID_USB_DEVICE(USB_VENDOR_ID_ELECOM, USB_DEVICE_ID_ELECOM_DEFT_WIRELESS) }, + { HID_USB_DEVICE(USB_VENDOR_ID_ELECOM, USB_DEVICE_ID_ELECOM_HUGE_WIRED) }, + { HID_USB_DEVICE(USB_VENDOR_ID_ELECOM, USB_DEVICE_ID_ELECOM_HUGE_WIRELESS) }, { HID_USB_DEVICE(USB_VENDOR_ID_ELO, 0x0009) }, { HID_USB_DEVICE(USB_VENDOR_ID_ELO, 0x0030) }, { HID_USB_DEVICE(USB_VENDOR_ID_EMS, USB_DEVICE_ID_EMS_TRIO_LINKER_PLUS_II) }, diff --git a/drivers/hid/hid-elecom.c b/drivers/hid/hid-elecom.c index e2c7465..54aeea5 100644 --- a/drivers/hid/hid-elecom.c +++ b/drivers/hid/hid-elecom.c @@ -32,9 +33,11 @@ static __u8 *elecom_report_fixup(struct hid_device *hdev, __u8 *rdesc, break; case USB_DEVICE_ID_ELECOM_DEFT_WIRED: case USB_DEVICE_ID_ELECOM_DEFT_WIRELESS: - /* The DEFT trackball has eight buttons, but its descriptor only - * reports five, disabling the three Fn buttons on the top of - * the mouse. + case USB_DEVICE_ID_ELECOM_HUGE_WIRED: + case USB_DEVICE_ID_ELECOM_HUGE_WIRELESS: + /* The DEFT/HUGE trackball has eight buttons, but its descriptor + * only reports five, disabling the three Fn buttons on the top + * of the mouse. * * Apply the following diff to the descriptor: * @@ -62,7 +65,7 @@ static __u8 *elecom_report_fixup(struct hid_device *hdev, __u8 *rdesc, * End Collection, End Collection, */ if (*rsize == 213 && rdesc[13] == 5 && rdesc[21] == 5) { - hid_info(hdev, "Fixing up Elecom DEFT Fn buttons\n"); + hid_info(hdev, "Fixing up Elecom DEFT/HUGE Fn buttons\n"); rdesc[13] = 8; /* Button/Variable Report Count */ rdesc[21] = 8; /* Button/Variable Usage Maximum */ rdesc[29] = 0; /* Button/Constant Report Count */ @@ -76,6 +79,8 @@ static const struct hid_device_id elecom_devices[] = { { HID_BLUETOOTH_DEVICE(USB_VENDOR_ID_ELECOM, USB_DEVICE_ID_ELECOM_BM084) }, { HID_USB_DEVICE(USB_VENDOR_ID_ELECOM, USB_DEVICE_ID_ELECOM_DEFT_WIRED) }, { HID_USB_DEVICE(USB_VENDOR_ID_ELECOM, USB_DEVICE_ID_ELECOM_DEFT_WIRELESS) }, + { HID_USB_DEVICE(USB_VENDOR_ID_ELECOM, USB_DEVICE_ID_ELECOM_HUGE_WIRED) }, + { HID_USB_DEVICE(USB_VENDOR_ID_ELECOM, USB_DEVICE_ID_ELECOM_HUGE_WIRELESS) }, { } }; MODULE_DEVICE_TABLE(hid, elecom_devices); diff --git a/drivers/hid/hid-ids.h b/drivers/hid/hid-ids.h index dbf7077..06ce846 100644 --- a/drivers/hid/hid-ids.h +++ b/drivers/hid/hid-ids.h @@ -355,6 +355,8 @@ #define USB_DEVICE_ID_ELECOM_BM084 0x0061 #define USB_DEVICE_ID_ELECOM_DEFT_WIRED 0x00fe #define USB_DEVICE_ID_ELECOM_DEFT_WIRELESS 0x00ff +#define USB_DEVICE_ID_ELECOM_HUGE_WIRED 0x010c +#define USB_DEVICE_ID_ELECOM_HUGE_WIRELESS 0x010d #define USB_VENDOR_ID_DREAM_CHEEKY 0x1d34 #define USB_DEVICE_ID_DREAM_CHEEKY_WN 0x0004 ^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH v1] HID: hid-elecom: extend to fix descriptor for HUGE trackball 2017-10-17 8:25 ` Alex Manoussakis @ 2017-10-17 8:35 ` Greg Kroah-Hartman 0 siblings, 0 replies; 9+ messages in thread From: Greg Kroah-Hartman @ 2017-10-17 8:35 UTC (permalink / raw) To: Alex Manoussakis; +Cc: stable, Jiri Kosina, Diego Elio Petteo On Tue, Oct 17, 2017 at 04:25:59AM -0400, Alex Manoussakis wrote: > > > * Copyright (c) 2010 Richard Nauber <Richard.Nauber@gmail.com> > > > * Copyright (c) 2016 Yuxuan Shui <yshuiv7@gmail.com> > > > * Copyright (c) 2017 Diego Elio Petten� <flameeyes@flameeyes.eu> > > > + * Copyright (c) 2017 Alex Manoussakis <amanou@gnu.org> > > > > Note, I'm not the maintainer, but you should be aware that adding a line > > like this, with only the changes as expressed in this patch, is very > > dubious according to every lawyer that I know. Please be careful about > > stuff like this, and only add such a line if you really know what you > > are doing[1] > > > > If you have done other changes to this file, my appologies... > > Hi Greg, thanks for the note. The changes I have done is add new USB IDs and > change the text in a couple of places (both comments and messages) to include > the new model "HUGE" where it only said "DEFT". > > We could remove the copyright line I added if it's a problem. I didn't give it > much thought. I copied the linux-input maintainer Jiri, and Diego who added the > code for the DEFT trackballs, in my mainline submission and both were ok > (Diego wrote me "The patch looks good to me", and Jiri submitted it) and now > it's in mainline. Copying Diego in this thread now too. > > My goal is simply to make the trackball I purchased work in my favorite OS, > and benefit other users who have found or will find themselves with a semi- > working device otherwise. And push as far back as 4.4 since some very popular > long-term-support distros use this kernel, I use it too, and it works. > > Can you commit the patch upstream ? Below is the patch that will apply clean > to 4.4 and 4.9, without the extra copyright line if you prefer to use that. > Note 4.4 and 4.9 need to cherry pick 0bb7a37f8d15e5fb5d21776875f9fbc74e10753a > as a prerequisite which adds support for the DEFT trackballs. > > Should I submit a mainline patch that removes the Copyright line I added ? > Let me know what I should do and I'll be happy to do it. Ok, if this is already merged, that's fine, just wanted to point out the obvious copyright statement issues :) I'll add it to my queue of patches to apply to as stable tree after these next kernels get released, thanks. greg k-h ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH v1] HID: hid-elecom: extend to fix descriptor for HUGE trackball 2017-10-16 20:20 ` Alex Manoussakis 2017-10-17 6:58 ` Greg Kroah-Hartman @ 2017-10-19 9:35 ` Greg Kroah-Hartman 2017-10-19 11:07 ` Jiri Kosina 2017-10-19 13:44 ` Alex Manoussakis 1 sibling, 2 replies; 9+ messages in thread From: Greg Kroah-Hartman @ 2017-10-19 9:35 UTC (permalink / raw) To: Alex Manoussakis; +Cc: stable, Jiri Kosina On Mon, Oct 16, 2017 at 04:20:35PM -0400, Alex Manoussakis wrote: > commit a0933a456ff83a3b5ffa3a1903e0b8de4a56adf5 upstream > > Hello, while you evaluate my request to commit 2 USB IDs to 4.13, > I would like you to consider putting it in 4.4 and 4.9 also. > I tested tested both of these (patch, build, boot, and usage of the device). > It would be great for users of those longterm kernels to have their Elecom HUGE > trackball work properly! > > The original patch does not apply cleanly to 4.4 and 4.9 however; a trivial > adjustment is needed (simply in the unmodified positioning lines of one file). > Below is the adjusted patch that will apply without conflicts to 4.4 and 4.9. > As mentioned earlier, for kernels prior to 4.12 you > would need to cherry pick 68f9e3d71650c0f4c48d039c83e3baa4e278deed first. That commit id is not a patch :( > > Thanks! > > Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org> > Cc: <stable@stable@vger.kernel.org> # 4.12.x > Cc: <stable@stable@vger.kernel.org> # 4.9.x: 0bb7a37: HID: elecom: ... DEFT trackballs > Cc: <stable@stable@vger.kernel.org> # 4.4.x: 0bb7a37: HID: elecom: ... DEFT trackballs Commit 0bb7a37 is a "new feature", and as such, shouldn't be backported to older kernels, right? I need a very strong agrument from the HID maintainer as to why this should be accepted to the older kernels... thanks, greg k-h ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH v1] HID: hid-elecom: extend to fix descriptor for HUGE trackball 2017-10-19 9:35 ` Greg Kroah-Hartman @ 2017-10-19 11:07 ` Jiri Kosina 2017-10-19 12:53 ` Greg Kroah-Hartman 2017-10-19 13:44 ` Alex Manoussakis 1 sibling, 1 reply; 9+ messages in thread From: Jiri Kosina @ 2017-10-19 11:07 UTC (permalink / raw) To: Greg Kroah-Hartman; +Cc: Alex Manoussakis, stable On Thu, 19 Oct 2017, Greg Kroah-Hartman wrote: > > Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org> > > Cc: <stable@stable@vger.kernel.org> # 4.12.x > > Cc: <stable@stable@vger.kernel.org> # 4.9.x: 0bb7a37: HID: elecom: ... DEFT trackballs > > Cc: <stable@stable@vger.kernel.org> # 4.4.x: 0bb7a37: HID: elecom: ... DEFT trackballs > > Commit 0bb7a37 is a "new feature", and as such, shouldn't be backported > to older kernels, right? > > I need a very strong agrument from the HID maintainer as to why this > should be accepted to the older kernels... Hi Greg, it basically is a new device ID addition (with a small glue code applied only to that device ID), and as such a left the stable markers there. But I don't have strong preference either way -- if you feel that this is beyond the 'new device ID' policy, I wouldn't object strongly. Thanks, -- Jiri Kosina SUSE Labs ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH v1] HID: hid-elecom: extend to fix descriptor for HUGE trackball 2017-10-19 11:07 ` Jiri Kosina @ 2017-10-19 12:53 ` Greg Kroah-Hartman 0 siblings, 0 replies; 9+ messages in thread From: Greg Kroah-Hartman @ 2017-10-19 12:53 UTC (permalink / raw) To: Jiri Kosina; +Cc: Alex Manoussakis, stable On Thu, Oct 19, 2017 at 01:07:25PM +0200, Jiri Kosina wrote: > On Thu, 19 Oct 2017, Greg Kroah-Hartman wrote: > > > > Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org> > > > Cc: <stable@stable@vger.kernel.org> # 4.12.x > > > Cc: <stable@stable@vger.kernel.org> # 4.9.x: 0bb7a37: HID: elecom: ... DEFT trackballs > > > Cc: <stable@stable@vger.kernel.org> # 4.4.x: 0bb7a37: HID: elecom: ... DEFT trackballs > > > > Commit 0bb7a37 is a "new feature", and as such, shouldn't be backported > > to older kernels, right? > > > > I need a very strong agrument from the HID maintainer as to why this > > should be accepted to the older kernels... > > Hi Greg, > > it basically is a new device ID addition (with a small glue code applied > only to that device ID), and as such a left the stable markers there. But > I don't have strong preference either way -- if you feel that this is > beyond the 'new device ID' policy, I wouldn't object strongly. I think it's a bit more than just the normal "add a new id or quirk" logic here. For new devices, it's always best to just update to a newer kernel, there's no reason anyone with these devices should be using 4.9 or older :) thanks, greg k-h ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH v1] HID: hid-elecom: extend to fix descriptor for HUGE trackball 2017-10-19 9:35 ` Greg Kroah-Hartman 2017-10-19 11:07 ` Jiri Kosina @ 2017-10-19 13:44 ` Alex Manoussakis 1 sibling, 0 replies; 9+ messages in thread From: Alex Manoussakis @ 2017-10-19 13:44 UTC (permalink / raw) To: Greg Kroah-Hartman; +Cc: stable, Jiri Kosina > > As mentioned earlier, for kernels prior to 4.12 you > > would need to cherry pick 68f9e3d71650c0f4c48d039c83e3baa4e278deed first. > > That commit id is not a patch :( Sorry, I meant 0bb7a37f8d15e5fb5d21776875f9fbc74e10753a. Yes, that cherry pick is a bit more involved than just new device IDs. Saw your other email regarding your reservations for the older kernels, it's ok, thanks for accepting it for 4.13 ! Alex ^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2017-10-19 13:45 UTC | newest] Thread overview: 9+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2017-10-14 3:23 [PATCH v1] HID: hid-elecom: extend to fix descriptor for HUGE trackball Alex Manoussakis 2017-10-16 20:20 ` Alex Manoussakis 2017-10-17 6:58 ` Greg Kroah-Hartman 2017-10-17 8:25 ` Alex Manoussakis 2017-10-17 8:35 ` Greg Kroah-Hartman 2017-10-19 9:35 ` Greg Kroah-Hartman 2017-10-19 11:07 ` Jiri Kosina 2017-10-19 12:53 ` Greg Kroah-Hartman 2017-10-19 13:44 ` Alex Manoussakis
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).