- Linux-stable-mirror - lists.linaro.org

stable-rc/linux-5.4.y boot: 85 boots: 0 failed, 77 passed with 7 offline, 1 untried/unknown (v5.4.16-111-g1bde1c513887)

by kernelci.org bot

stable-rc/linux-5.4.y boot: 85 boots: 0 failed, 77 passed with 7 offline, 1 untried/unknown (v5.4.16-111-g1bde1c513887) Full Boot Summary: https://kernelci.org/boot/all/job/stable-rc/branch/linux-5.4.y/kernel/v5.4.… Full Build Summary: https://kernelci.org/build/stable-rc/branch/linux-5.4.y/kernel/v5.4.16-111-… Tree: stable-rc Branch: linux-5.4.y Git Describe: v5.4.16-111-g1bde1c513887 Git Commit: 1bde1c513887dd7bac46be466f1cd06fe3a64292 Git URL: https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.git Tested: 56 unique boards, 20 SoC families, 16 builds out of 162 Boot Regressions Detected: arm: davinci_all_defconfig: gcc-8: dm365evm,legacy: lab-baylibre-seattle: new failure (last pass: v5.4.16) multi_v7_defconfig: gcc-8: mt7629-rfb: lab-baylibre-seattle: new failure (last pass: v5.4.16) arm64: defconfig: gcc-8: meson-axg-s400: lab-baylibre-seattle: new failure (last pass: v5.4.16) Offline Platforms: arm: multi_v7_defconfig: gcc-8 mt7629-rfb: 1 offline lab qcom-apq8064-cm-qs600: 1 offline lab sun5i-r8-chip: 1 offline lab sunxi_defconfig: gcc-8 sun5i-r8-chip: 1 offline lab qcom_defconfig: gcc-8 qcom-apq8064-cm-qs600: 1 offline lab davinci_all_defconfig: gcc-8 dm365evm,legacy: 1 offline lab arm64: defconfig: gcc-8 meson-axg-s400: 1 offline lab --- For more info write to <info(a)kernelci.org>

5 years, 9 months

1
0
0 0

stable-rc/linux-4.9.y boot: 48 boots: 1 failed, 42 passed with 5 offline (v4.9.212-26-gc8f671aa81fa)

by kernelci.org bot

stable-rc/linux-4.9.y boot: 48 boots: 1 failed, 42 passed with 5 offline (v4.9.212-26-gc8f671aa81fa) Full Boot Summary: https://kernelci.org/boot/all/job/stable-rc/branch/linux-4.9.y/kernel/v4.9.… Full Build Summary: https://kernelci.org/build/stable-rc/branch/linux-4.9.y/kernel/v4.9.212-26-… Tree: stable-rc Branch: linux-4.9.y Git Describe: v4.9.212-26-gc8f671aa81fa Git Commit: c8f671aa81faf69b2573521f6b97adde12b72925 Git URL: https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.git Tested: 29 unique boards, 15 SoC families, 14 builds out of 156 Boot Regressions Detected: arm: davinci_all_defconfig: gcc-8: dm365evm,legacy: lab-baylibre-seattle: new failure (last pass: v4.9.212) Boot Failure Detected: arm: multi_v7_defconfig: gcc-8: sun4i-a10-cubieboard: 1 failed lab Offline Platforms: arm: multi_v7_defconfig: gcc-8 qcom-apq8064-cm-qs600: 1 offline lab sun5i-r8-chip: 1 offline lab sunxi_defconfig: gcc-8 sun5i-r8-chip: 1 offline lab qcom_defconfig: gcc-8 qcom-apq8064-cm-qs600: 1 offline lab davinci_all_defconfig: gcc-8 dm365evm,legacy: 1 offline lab --- For more info write to <info(a)kernelci.org>

5 years, 9 months

1
0
0 0

stable-rc/linux-4.14.y boot: 67 boots: 1 failed, 61 passed with 5 offline (v4.14.169-33-ga3d2ee7a48e7)

by kernelci.org bot

stable-rc/linux-4.14.y boot: 67 boots: 1 failed, 61 passed with 5 offline (v4.14.169-33-ga3d2ee7a48e7) Full Boot Summary: https://kernelci.org/boot/all/job/stable-rc/branch/linux-4.14.y/kernel/v4.1… Full Build Summary: https://kernelci.org/build/stable-rc/branch/linux-4.14.y/kernel/v4.14.169-3… Tree: stable-rc Branch: linux-4.14.y Git Describe: v4.14.169-33-ga3d2ee7a48e7 Git Commit: a3d2ee7a48e78477fc9ad73102510602923091ae Git URL: https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.git Tested: 40 unique boards, 15 SoC families, 12 builds out of 159 Boot Regressions Detected: arm: davinci_all_defconfig: gcc-8: dm365evm,legacy: lab-baylibre-seattle: new failure (last pass: v4.14.169) Boot Failure Detected: arm: multi_v7_defconfig: gcc-8: sun4i-a10-cubieboard: 1 failed lab Offline Platforms: arm: multi_v7_defconfig: gcc-8 qcom-apq8064-cm-qs600: 1 offline lab sun5i-r8-chip: 1 offline lab sunxi_defconfig: gcc-8 sun5i-r8-chip: 1 offline lab qcom_defconfig: gcc-8 qcom-apq8064-cm-qs600: 1 offline lab davinci_all_defconfig: gcc-8 dm365evm,legacy: 1 offline lab --- For more info write to <info(a)kernelci.org>

5 years, 9 months

1
0
0 0

Re: [PATCH] HID: ite: Only bind to keyboard USB interface on Acer SW5-012 keyboard dock

by Benjamin Tissoires

On Fri, Jan 31, 2020 at 6:17 PM Z R <zdenda.rampas(a)gmail.com> wrote: > > Hi Benjamin, > in last log touchpad.log (omg, should be keyboard.log), I pressed fn-f3 multiple times. I got one two liner: > > # ReportID: 3 / Wireless Radio Button: 0 | # > E: 000007.606583 2 03 00 great, thanks. > > every time i release f3. Does not matter what is happening with fn-key. (could be released already or still pushed down). This log was collected with last patch from Hans applied. Actually, it doesn't really matter if the patch from Hans was applied or not. I am looking at the raw event from the device before kernel processing :) May I ask you one more thing: can you run `libinput record` (as root) on the touchpad? I need to know how many fingers the touchpad is capable to detect, as right now, my tests are failing because of that. > > Sorry for the mess I caused :-) I still don't get how you guys manage to have your emails so well polished and readable. No worries. Having a responsive user is much more valuable than someone who takes ages to reply even in a clear and well polished message :) Cheers, Benjamin > > Bye > Zdeněk > > pá 31. 1. 2020 v 18:00 odesílatel Benjamin Tissoires <benjamin.tissoires(a)redhat.com> napsal: >> >> On Fri, Jan 31, 2020 at 5:09 PM Z R <zdenda.rampas(a)gmail.com> wrote: >> > >> > I believe I pressed wifi button on both replays for keyboard. >> >> Yep, I can see that. Just to double check, on the last log, you >> pressed the wifi button twice? >> >> Anyway, thanks for all the logs, I should have enough to implement the >> regression tests. >> >> Cheers, >> Benjamin >> >> > >> > With latest patch from Hans on top of v5.5.0 touchpads "two finger scrolling" is working again. Attaching current hid-record for keyboard with Wifi button pressed. Events in log appeared after f3 button was "released". >> > >> > Thanks >> > >> > Zdeněk >> > >> > pá 31. 1. 2020 v 16:45 odesílatel Hans de Goede <hdegoede(a)redhat.com> napsal: >> >> >> >> Hi, >> >> >> >> On 1/31/20 4:38 PM, Z R wrote: >> >> > Hi Benjamin, >> >> > hid-record for keyboard and touchpad. With Commit 8f18eca9ebc5 reverted and from unmodified kernel. >> >> > >> >> > I hope it is what you asked for :-) >> >> > >> >> > Currently waiting for reworked patch from Hans. >> >> >> >> I just send you the reworked patch. >> >> >> >> Does the recordning include pressing of the wlan on/off key (Fn + F3 I believe) ? >> >> That is the whole reason why the special hid-ite driver is necessary. >> >> >> >> Benjamin about the wlan on/off key. AFAICR on a press + release of the key a >> >> single hid input report for the generic-desktop Wireless Radio Controls group >> >> is send. This input-report only has the one button with usage code HID_GD_RFKILL_BTN >> >> in there and it is always 0. It is as if the input-report is only send on release >> >> and not on press. So the hid-ite code emulates a press + release whenever the >> >> input-report is send. >> >> >> >> IOW the receiving of the input report is (ab)used as indication of the button >> >> having been pressed. >> >> >> >> Regards, >> >> >> >> Hans >> >> >> >> >> >> > >> >> > Bye for now >> >> > Zdeněk >> >> > >> >> > pá 31. 1. 2020 v 15:12 odesílatel Benjamin Tissoires <benjamin.tissoires(a)redhat.com <mailto:benjamin.tissoires@redhat.com>> napsal: >> >> > >> >> > On Fri, Jan 31, 2020 at 3:04 PM Hans de Goede <hdegoede(a)redhat.com <mailto:hdegoede@redhat.com>> wrote: >> >> > > >> >> > > Hi, >> >> > > >> >> > > On 1/31/20 2:54 PM, Benjamin Tissoires wrote: >> >> > > > On Fri, Jan 31, 2020 at 2:41 PM Hans de Goede <hdegoede(a)redhat.com <mailto:hdegoede@redhat.com>> wrote: >> >> > > >> >> >> > > >> Hi, >> >> > > >> >> >> > > >> On 1/31/20 2:10 PM, Benjamin Tissoires wrote: >> >> > > >>> Hi Hans, >> >> > > >>> >> >> > > >>> On Fri, Jan 31, 2020 at 1:46 PM Hans de Goede <hdegoede(a)redhat.com <mailto:hdegoede@redhat.com>> wrote: >> >> > > >>>> >> >> > > >>>> Commit 8f18eca9ebc5 ("HID: ite: Add USB id match for Acer SW5-012 keyboard >> >> > > >>>> dock") added the USB id for the Acer SW5-012's keyboard dock to the >> >> > > >>>> hid-ite driver to fix the rfkill driver not working. >> >> > > >>>> >> >> > > >>>> Most keyboard docks with an ITE 8595 keyboard/touchpad controller have the >> >> > > >>>> "Wireless Radio Control" bits which need the special hid-ite driver on the >> >> > > >>>> second USB interface (the mouse interface) and their touchpad only supports >> >> > > >>>> mouse emulation, so using generic hid-input handling for anything but >> >> > > >>>> the "Wireless Radio Control" bits is fine. On these devices we simply bind >> >> > > >>>> to all USB interfaces. >> >> > > >>>> >> >> > > >>>> But unlike other ITE8595 using keyboard docks, the Acer Aspire Switch 10 >> >> > > >>>> (SW5-012)'s touchpad not only does mouse emulation it also supports >> >> > > >>>> HID-multitouch and all the keys including the "Wireless Radio Control" >> >> > > >>>> bits have been moved to the first USB interface (the keyboard intf). >> >> > > >>>> >> >> > > >>>> So we need hid-ite to handle the first (keyboard) USB interface and have >> >> > > >>>> it NOT bind to the second (mouse) USB interface so that that can be >> >> > > >>>> handled by hid-multitouch.c and we get proper multi-touch support. >> >> > > >>>> >> >> > > >>>> This commit adds a match callback to hid-ite which makes it only >> >> > > >>>> match the first USB interface when running on the Acer SW5-012, >> >> > > >>>> fixing the regression to mouse-emulation mode introduced by adding the >> >> > > >>>> keyboard dock USB id. >> >> > > >>>> >> >> > > >>>> Note the match function only does the special only bind to the first >> >> > > >>>> USB interface on the Acer SW5-012, on other devices the hid-ite driver >> >> > > >>>> actually must bind to the second interface as that is where the >> >> > > >>>> "Wireless Radio Control" bits are. >> >> > > >>> >> >> > > >>> This is not a full review, but a couple of things that popped out >> >> > > >>> while scrolling through the patch. >> >> > > >>> >> >> > > >>>> >> >> > > >>>> Cc: stable(a)vger.kernel.org <mailto:stable@vger.kernel.org> >> >> > > >>>> Fixes: 8f18eca9ebc5 ("HID: ite: Add USB id match for Acer SW5-012 keyboard dock") >> >> > > >>>> Reported-by: Zdeněk Rampas <zdenda.rampas(a)gmail.com <mailto:zdenda.rampas@gmail.com>> >> >> > > >>>> Signed-off-by: Hans de Goede <hdegoede(a)redhat.com <mailto:hdegoede@redhat.com>> >> >> > > >>>> --- >> >> > > >>>> drivers/hid/hid-ite.c | 34 ++++++++++++++++++++++++++++++++++ >> >> > > >>>> 1 file changed, 34 insertions(+) >> >> > > >>>> >> >> > > >>>> diff --git a/drivers/hid/hid-ite.c b/drivers/hid/hid-ite.c >> >> > > >>>> index c436e12feb23..69a4ddfd033d 100644 >> >> > > >>>> --- a/drivers/hid/hid-ite.c >> >> > > >>>> +++ b/drivers/hid/hid-ite.c >> >> > > >>>> @@ -8,9 +8,12 @@ >> >> > > >>>> #include <linux/input.h> >> >> > > >>>> #include <linux/hid.h> >> >> > > >>>> #include <linux/module.h> >> >> > > >>>> +#include <linux/usb.h> >> >> > > >>>> >> >> > > >>>> #include "hid-ids.h" >> >> > > >>>> >> >> > > >>>> +#define ITE8595_KBD_USB_INTF 0 >> >> > > >>>> + >> >> > > >>>> static int ite_event(struct hid_device *hdev, struct hid_field *field, >> >> > > >>>> struct hid_usage *usage, __s32 value) >> >> > > >>>> { >> >> > > >>>> @@ -37,6 +40,36 @@ static int ite_event(struct hid_device *hdev, struct hid_field *field, >> >> > > >>>> return 0; >> >> > > >>>> } >> >> > > >>>> >> >> > > >>>> +static bool ite_match(struct hid_device *hdev, bool ignore_special_driver) >> >> > > >>>> +{ >> >> > > >>>> + struct usb_interface *intf; >> >> > > >>>> + >> >> > > >>>> + if (ignore_special_driver) >> >> > > >>>> + return false; >> >> > > >>>> + >> >> > > >>>> + /* >> >> > > >>>> + * Most keyboard docks with an ITE 8595 keyboard/touchpad controller >> >> > > >>>> + * have the "Wireless Radio Control" bits which need this special >> >> > > >>>> + * driver on the second USB interface (the mouse interface). On >> >> > > >>>> + * these devices we simply bind to all USB interfaces. >> >> > > >>>> + * >> >> > > >>>> + * The Acer Aspire Switch 10 (SW5-012) is special, its touchpad >> >> > > >>>> + * not only does mouse emulation it also supports HID-multitouch >> >> > > >>>> + * and all the keys including the "Wireless Radio Control" bits >> >> > > >>>> + * have been moved to the first USB interface (the keyboard intf). >> >> > > >>>> + * >> >> > > >>>> + * We want the hid-multitouch driver to bind to the touchpad, so on >> >> > > >>>> + * the Acer SW5-012 we should only bind to the keyboard USB intf. >> >> > > >>>> + */ >> >> > > >>>> + if (hdev->bus != BUS_USB || hdev->vendor != USB_VENDOR_ID_SYNAPTICS || >> >> > > >>>> + hdev->product != USB_DEVICE_ID_SYNAPTICS_ACER_SWITCH5_012) >> >> > > >>> >> >> > > >>> Isn't there an existing matching function we can use here, instead of >> >> > > >>> checking each individual field? >> >> > > >> >> >> > > >> There is hid_match_one_id() but that is not exported (can be fixed) and it >> >> > > >> requires a struct hid_device_id, which either requires declaring an extra >> >> > > >> standalone struct hid_device_id for the SW5-012 kbd-dock, or hardcoding an >> >> > > >> index into the existing hid_device_id array for the driver (with the hardcoding >> >> > > >> being error prone, so not a good idea). >> >> > > >> >> >> > > >> Given the problems with using hid_match_one_id() I decided to just go with >> >> > > >> the above. >> >> > > > >> >> > > > right. An other solution would be to have a local macro/function that >> >> > > > does that. Because as soon as you start adding a quirk, an other comes >> >> > > > right after. >> >> > > > >> >> > > >> >> >> > > >> But see below. >> >> > > >> >> >> > > >>> >> >> > > >>>> + return true; >> >> > > >>>> + >> >> > > >>>> + intf = to_usb_interface(hdev->dev.parent); >> >> > > >>> >> >> > > >>> And this is oops-prone. You need: >> >> > > >>> - ensure hid_is_using_ll_driver(hdev, &usb_hid_driver) returns true. >> >> > > >>> - add a dependency on USBHID in the KConfig now that you are checking >> >> > > >>> on the USB transport layer. >> >> > > >>> >> >> > > >>> That being said, I would love instead: >> >> > > >>> - to have a non USB version of this match, where you decide which >> >> > > >>> component needs to be handled based on the report descriptor >> >> > > >> >> >> > > >> Actually your idea to use the desciptors is not bad, but since what >> >> > > >> we really want is to not bind to the interface which is marked for the >> >> > > >> hid-multitouch driver I just realized we can just check that. >> >> > > >> >> >> > > >> So how about: >> >> > > >> >> >> > > >> static bool ite_match(struct hid_device *hdev, bool ignore_special_driver) >> >> > > >> { >> >> > > >> if (ignore_special_driver) >> >> > > >> return false; >> >> > > >> >> >> > > >> /* >> >> > > >> * Some keyboard docks with an ITE 8595 keyboard/touchpad controller >> >> > > >> * support the HID multitouch protocol for the touchpad, in that >> >> > > >> * case the "Wireless Radio Control" bits which we care about are >> >> > > >> * on the other interface; and we should not bind to the multitouch >> >> > > >> * capable interface as that breaks multitouch support. >> >> > > >> */ >> >> > > >> return hdev->group != HID_GROUP_MULTITOUCH_WIN_8; >> >> > > >> } >> >> > > > >> >> > > > Yep, I like that very much :) >> >> > > >> >> > > Actually if we want to check the group and there are only 2 interfaces we do >> >> > > not need to use the match callback at all, w e can simply match on the >> >> > > group of the interface which we do want: >> >> > > >> >> > > diff --git a/drivers/hid/hid-ite.c b/drivers/hid/hid-ite.c >> >> > > index db0f35be5a8b..21bd48f16033 100644 >> >> > > --- a/drivers/hid/hid-ite.c >> >> > > +++ b/drivers/hid/hid-ite.c >> >> > > @@ -56,8 +56,9 @@ static const struct hid_device_id ite_devices[] = { >> >> > > { HID_USB_DEVICE(USB_VENDOR_ID_ITE, USB_DEVICE_ID_ITE8595) }, >> >> > > { HID_USB_DEVICE(USB_VENDOR_ID_258A, USB_DEVICE_ID_258A_6A88) }, >> >> > > /* ITE8595 USB kbd ctlr, with Synaptics touchpad connected to it. */ >> >> > > - { HID_USB_DEVICE(USB_VENDOR_ID_SYNAPTICS, >> >> > > - USB_DEVICE_ID_SYNAPTICS_ACER_SWITCH5_012) }, >> >> > > + { HID_DEVICE(BUS_USB, HID_GROUP_GENERIC, >> >> > > + USB_VENDOR_ID_SYNAPTICS, >> >> > > + USB_DEVICE_ID_SYNAPTICS_ACER_SWITCH5_012) }, >> >> > > { } >> >> > > }; >> >> > > MODULE_DEVICE_TABLE(hid, ite_devices); >> >> > > >> >> > > Much cleaner >> >> > >> >> > yep >> >> > >> >> > > (and now I don't need to write a test, which is always >> >> > > a good motivation to come up with a cleaner solution :) >> >> > >> >> > Hehe, too bad, you already picked up my curiosity on this one, and I >> >> > really would like to see the report descriptors and some events of the >> >> > keys that are fixed by hid-ite.c. >> >> > <with a low voice>This will be a hard requirement to accept this patch </joke>. >> >> > >> >> > More seriously, Zdeněk, can you run hid-recorder from >> >> > https://gitlab.freedesktop.org/libevdev/hid-tools/ and provide me the >> >> > report descriptor for all of your ITE HID devices? I'll add the >> >> > matching tests in hid-tools and be sure we do not regress in the >> >> > future. >> >> > >> >> > > >> >> > > Let me turn this into a proper patch and then I will send that to >> >> > > Zdeněk (off-list) for him to test (note don't worry if you do >> >> > > not have time to test this weekend, then I'll do it on Monday). >> >> > > >> >> > > Regards, >> >> > > >> >> > > Hans >> >> > > >> >> > > p.s. >> >> > > >> >> > > 1. My train is approaching Brussels (Fosdem) so my email response >> >> > > time will soon become irregular. >> >> > >> >> > How dare you? :) >> >> > >> >> > > >> >> > > 2. Benjamin will you be at Fosdem too ? >> >> > > >> >> > >> >> > Unfortunately no. Already got my quota of meeting people for this year >> >> > between Kernel Recipes in September, XDC in October and LCA last week. >> >> > So I need to keep in a quiet environment for a little bit :) >> >> > >> >> > Cheers, >> >> > Benjamin >> >> > >> >> >>

5 years, 9 months

1
0
0 0

Re: [PATCH] HID: ite: Only bind to keyboard USB interface on Acer SW5-012 keyboard dock

by Benjamin Tissoires

On Fri, Jan 31, 2020 at 5:09 PM Z R <zdenda.rampas(a)gmail.com> wrote: > > I believe I pressed wifi button on both replays for keyboard. Yep, I can see that. Just to double check, on the last log, you pressed the wifi button twice? Anyway, thanks for all the logs, I should have enough to implement the regression tests. Cheers, Benjamin > > With latest patch from Hans on top of v5.5.0 touchpads "two finger scrolling" is working again. Attaching current hid-record for keyboard with Wifi button pressed. Events in log appeared after f3 button was "released". > > Thanks > > Zdeněk > > pá 31. 1. 2020 v 16:45 odesílatel Hans de Goede <hdegoede(a)redhat.com> napsal: >> >> Hi, >> >> On 1/31/20 4:38 PM, Z R wrote: >> > Hi Benjamin, >> > hid-record for keyboard and touchpad. With Commit 8f18eca9ebc5 reverted and from unmodified kernel. >> > >> > I hope it is what you asked for :-) >> > >> > Currently waiting for reworked patch from Hans. >> >> I just send you the reworked patch. >> >> Does the recordning include pressing of the wlan on/off key (Fn + F3 I believe) ? >> That is the whole reason why the special hid-ite driver is necessary. >> >> Benjamin about the wlan on/off key. AFAICR on a press + release of the key a >> single hid input report for the generic-desktop Wireless Radio Controls group >> is send. This input-report only has the one button with usage code HID_GD_RFKILL_BTN >> in there and it is always 0. It is as if the input-report is only send on release >> and not on press. So the hid-ite code emulates a press + release whenever the >> input-report is send. >> >> IOW the receiving of the input report is (ab)used as indication of the button >> having been pressed. >> >> Regards, >> >> Hans >> >> >> > >> > Bye for now >> > Zdeněk >> > >> > pá 31. 1. 2020 v 15:12 odesílatel Benjamin Tissoires <benjamin.tissoires(a)redhat.com <mailto:benjamin.tissoires@redhat.com>> napsal: >> > >> > On Fri, Jan 31, 2020 at 3:04 PM Hans de Goede <hdegoede(a)redhat.com <mailto:hdegoede@redhat.com>> wrote: >> > > >> > > Hi, >> > > >> > > On 1/31/20 2:54 PM, Benjamin Tissoires wrote: >> > > > On Fri, Jan 31, 2020 at 2:41 PM Hans de Goede <hdegoede(a)redhat.com <mailto:hdegoede@redhat.com>> wrote: >> > > >> >> > > >> Hi, >> > > >> >> > > >> On 1/31/20 2:10 PM, Benjamin Tissoires wrote: >> > > >>> Hi Hans, >> > > >>> >> > > >>> On Fri, Jan 31, 2020 at 1:46 PM Hans de Goede <hdegoede(a)redhat.com <mailto:hdegoede@redhat.com>> wrote: >> > > >>>> >> > > >>>> Commit 8f18eca9ebc5 ("HID: ite: Add USB id match for Acer SW5-012 keyboard >> > > >>>> dock") added the USB id for the Acer SW5-012's keyboard dock to the >> > > >>>> hid-ite driver to fix the rfkill driver not working. >> > > >>>> >> > > >>>> Most keyboard docks with an ITE 8595 keyboard/touchpad controller have the >> > > >>>> "Wireless Radio Control" bits which need the special hid-ite driver on the >> > > >>>> second USB interface (the mouse interface) and their touchpad only supports >> > > >>>> mouse emulation, so using generic hid-input handling for anything but >> > > >>>> the "Wireless Radio Control" bits is fine. On these devices we simply bind >> > > >>>> to all USB interfaces. >> > > >>>> >> > > >>>> But unlike other ITE8595 using keyboard docks, the Acer Aspire Switch 10 >> > > >>>> (SW5-012)'s touchpad not only does mouse emulation it also supports >> > > >>>> HID-multitouch and all the keys including the "Wireless Radio Control" >> > > >>>> bits have been moved to the first USB interface (the keyboard intf). >> > > >>>> >> > > >>>> So we need hid-ite to handle the first (keyboard) USB interface and have >> > > >>>> it NOT bind to the second (mouse) USB interface so that that can be >> > > >>>> handled by hid-multitouch.c and we get proper multi-touch support. >> > > >>>> >> > > >>>> This commit adds a match callback to hid-ite which makes it only >> > > >>>> match the first USB interface when running on the Acer SW5-012, >> > > >>>> fixing the regression to mouse-emulation mode introduced by adding the >> > > >>>> keyboard dock USB id. >> > > >>>> >> > > >>>> Note the match function only does the special only bind to the first >> > > >>>> USB interface on the Acer SW5-012, on other devices the hid-ite driver >> > > >>>> actually must bind to the second interface as that is where the >> > > >>>> "Wireless Radio Control" bits are. >> > > >>> >> > > >>> This is not a full review, but a couple of things that popped out >> > > >>> while scrolling through the patch. >> > > >>> >> > > >>>> >> > > >>>> Cc: stable(a)vger.kernel.org <mailto:stable@vger.kernel.org> >> > > >>>> Fixes: 8f18eca9ebc5 ("HID: ite: Add USB id match for Acer SW5-012 keyboard dock") >> > > >>>> Reported-by: Zdeněk Rampas <zdenda.rampas(a)gmail.com <mailto:zdenda.rampas@gmail.com>> >> > > >>>> Signed-off-by: Hans de Goede <hdegoede(a)redhat.com <mailto:hdegoede@redhat.com>> >> > > >>>> --- >> > > >>>> drivers/hid/hid-ite.c | 34 ++++++++++++++++++++++++++++++++++ >> > > >>>> 1 file changed, 34 insertions(+) >> > > >>>> >> > > >>>> diff --git a/drivers/hid/hid-ite.c b/drivers/hid/hid-ite.c >> > > >>>> index c436e12feb23..69a4ddfd033d 100644 >> > > >>>> --- a/drivers/hid/hid-ite.c >> > > >>>> +++ b/drivers/hid/hid-ite.c >> > > >>>> @@ -8,9 +8,12 @@ >> > > >>>> #include <linux/input.h> >> > > >>>> #include <linux/hid.h> >> > > >>>> #include <linux/module.h> >> > > >>>> +#include <linux/usb.h> >> > > >>>> >> > > >>>> #include "hid-ids.h" >> > > >>>> >> > > >>>> +#define ITE8595_KBD_USB_INTF 0 >> > > >>>> + >> > > >>>> static int ite_event(struct hid_device *hdev, struct hid_field *field, >> > > >>>> struct hid_usage *usage, __s32 value) >> > > >>>> { >> > > >>>> @@ -37,6 +40,36 @@ static int ite_event(struct hid_device *hdev, struct hid_field *field, >> > > >>>> return 0; >> > > >>>> } >> > > >>>> >> > > >>>> +static bool ite_match(struct hid_device *hdev, bool ignore_special_driver) >> > > >>>> +{ >> > > >>>> + struct usb_interface *intf; >> > > >>>> + >> > > >>>> + if (ignore_special_driver) >> > > >>>> + return false; >> > > >>>> + >> > > >>>> + /* >> > > >>>> + * Most keyboard docks with an ITE 8595 keyboard/touchpad controller >> > > >>>> + * have the "Wireless Radio Control" bits which need this special >> > > >>>> + * driver on the second USB interface (the mouse interface). On >> > > >>>> + * these devices we simply bind to all USB interfaces. >> > > >>>> + * >> > > >>>> + * The Acer Aspire Switch 10 (SW5-012) is special, its touchpad >> > > >>>> + * not only does mouse emulation it also supports HID-multitouch >> > > >>>> + * and all the keys including the "Wireless Radio Control" bits >> > > >>>> + * have been moved to the first USB interface (the keyboard intf). >> > > >>>> + * >> > > >>>> + * We want the hid-multitouch driver to bind to the touchpad, so on >> > > >>>> + * the Acer SW5-012 we should only bind to the keyboard USB intf. >> > > >>>> + */ >> > > >>>> + if (hdev->bus != BUS_USB || hdev->vendor != USB_VENDOR_ID_SYNAPTICS || >> > > >>>> + hdev->product != USB_DEVICE_ID_SYNAPTICS_ACER_SWITCH5_012) >> > > >>> >> > > >>> Isn't there an existing matching function we can use here, instead of >> > > >>> checking each individual field? >> > > >> >> > > >> There is hid_match_one_id() but that is not exported (can be fixed) and it >> > > >> requires a struct hid_device_id, which either requires declaring an extra >> > > >> standalone struct hid_device_id for the SW5-012 kbd-dock, or hardcoding an >> > > >> index into the existing hid_device_id array for the driver (with the hardcoding >> > > >> being error prone, so not a good idea). >> > > >> >> > > >> Given the problems with using hid_match_one_id() I decided to just go with >> > > >> the above. >> > > > >> > > > right. An other solution would be to have a local macro/function that >> > > > does that. Because as soon as you start adding a quirk, an other comes >> > > > right after. >> > > > >> > > >> >> > > >> But see below. >> > > >> >> > > >>> >> > > >>>> + return true; >> > > >>>> + >> > > >>>> + intf = to_usb_interface(hdev->dev.parent); >> > > >>> >> > > >>> And this is oops-prone. You need: >> > > >>> - ensure hid_is_using_ll_driver(hdev, &usb_hid_driver) returns true. >> > > >>> - add a dependency on USBHID in the KConfig now that you are checking >> > > >>> on the USB transport layer. >> > > >>> >> > > >>> That being said, I would love instead: >> > > >>> - to have a non USB version of this match, where you decide which >> > > >>> component needs to be handled based on the report descriptor >> > > >> >> > > >> Actually your idea to use the desciptors is not bad, but since what >> > > >> we really want is to not bind to the interface which is marked for the >> > > >> hid-multitouch driver I just realized we can just check that. >> > > >> >> > > >> So how about: >> > > >> >> > > >> static bool ite_match(struct hid_device *hdev, bool ignore_special_driver) >> > > >> { >> > > >> if (ignore_special_driver) >> > > >> return false; >> > > >> >> > > >> /* >> > > >> * Some keyboard docks with an ITE 8595 keyboard/touchpad controller >> > > >> * support the HID multitouch protocol for the touchpad, in that >> > > >> * case the "Wireless Radio Control" bits which we care about are >> > > >> * on the other interface; and we should not bind to the multitouch >> > > >> * capable interface as that breaks multitouch support. >> > > >> */ >> > > >> return hdev->group != HID_GROUP_MULTITOUCH_WIN_8; >> > > >> } >> > > > >> > > > Yep, I like that very much :) >> > > >> > > Actually if we want to check the group and there are only 2 interfaces we do >> > > not need to use the match callback at all, w e can simply match on the >> > > group of the interface which we do want: >> > > >> > > diff --git a/drivers/hid/hid-ite.c b/drivers/hid/hid-ite.c >> > > index db0f35be5a8b..21bd48f16033 100644 >> > > --- a/drivers/hid/hid-ite.c >> > > +++ b/drivers/hid/hid-ite.c >> > > @@ -56,8 +56,9 @@ static const struct hid_device_id ite_devices[] = { >> > > { HID_USB_DEVICE(USB_VENDOR_ID_ITE, USB_DEVICE_ID_ITE8595) }, >> > > { HID_USB_DEVICE(USB_VENDOR_ID_258A, USB_DEVICE_ID_258A_6A88) }, >> > > /* ITE8595 USB kbd ctlr, with Synaptics touchpad connected to it. */ >> > > - { HID_USB_DEVICE(USB_VENDOR_ID_SYNAPTICS, >> > > - USB_DEVICE_ID_SYNAPTICS_ACER_SWITCH5_012) }, >> > > + { HID_DEVICE(BUS_USB, HID_GROUP_GENERIC, >> > > + USB_VENDOR_ID_SYNAPTICS, >> > > + USB_DEVICE_ID_SYNAPTICS_ACER_SWITCH5_012) }, >> > > { } >> > > }; >> > > MODULE_DEVICE_TABLE(hid, ite_devices); >> > > >> > > Much cleaner >> > >> > yep >> > >> > > (and now I don't need to write a test, which is always >> > > a good motivation to come up with a cleaner solution :) >> > >> > Hehe, too bad, you already picked up my curiosity on this one, and I >> > really would like to see the report descriptors and some events of the >> > keys that are fixed by hid-ite.c. >> > <with a low voice>This will be a hard requirement to accept this patch </joke>. >> > >> > More seriously, Zdeněk, can you run hid-recorder from >> > https://gitlab.freedesktop.org/libevdev/hid-tools/ and provide me the >> > report descriptor for all of your ITE HID devices? I'll add the >> > matching tests in hid-tools and be sure we do not regress in the >> > future. >> > >> > > >> > > Let me turn this into a proper patch and then I will send that to >> > > Zdeněk (off-list) for him to test (note don't worry if you do >> > > not have time to test this weekend, then I'll do it on Monday). >> > > >> > > Regards, >> > > >> > > Hans >> > > >> > > p.s. >> > > >> > > 1. My train is approaching Brussels (Fosdem) so my email response >> > > time will soon become irregular. >> > >> > How dare you? :) >> > >> > > >> > > 2. Benjamin will you be at Fosdem too ? >> > > >> > >> > Unfortunately no. Already got my quota of meeting people for this year >> > between Kernel Recipes in September, XDC in October and LCA last week. >> > So I need to keep in a quiet environment for a little bit :) >> > >> > Cheers, >> > Benjamin >> > >>

5 years, 9 months

1
0
0 0

stable/linux-4.19.y build: 203 builds: 0 failed, 203 passed, 10 warnings (v4.19.100)

by kernelci.org bot

stable/linux-4.19.y build: 203 builds: 0 failed, 203 passed, 10 warnings (v4.19.100) Full Build Summary: https://kernelci.org/build/stable/branch/linux-4.19.y/kernel/v4.19.100/ Tree: stable Branch: linux-4.19.y Git Describe: v4.19.100 Git Commit: 7cdefde351b6911ec5ef39322980296c091f6c52 Git URL: https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable.git Built: 7 unique architectures Warnings Detected: arc: arm64: arm: multi_v7_defconfig (gcc-8): 2 warnings sunxi_defconfig (gcc-8): 2 warnings i386: mips: lemote2f_defconfig (gcc-8): 1 warning loongson3_defconfig (gcc-8): 2 warnings malta_qemu_32r6_defconfig (gcc-8): 1 warning nlm_xlp_defconfig (gcc-8): 1 warning riscv: x86_64: tinyconfig (gcc-8): 1 warning Warnings summary: 6 net/core/rtnetlink.c:3190:1: warning: the frame size of 1312 bytes is larger than 1024 bytes [-Wframe-larger-than=] 4 drivers/gpu/drm/sun4i/sun4i_hdmi_tmds_clk.c:159:5: warning: ‘is_double’ may be used uninitialized in this function [-Wmaybe-uninitialized] 4 arch/arm/boot/dts/sun8i-h3-beelink-x2.dtb: Warning (clocks_property): /wifi_pwrseq: Missing property '#clock-cells' in node /soc/rtc@1f00000 or bad phandle (referred from clocks[0]) 2 arch/mips/configs/loongson3_defconfig:55:warning: symbol value 'm' invalid for HOTPLUG_PCI_SHPC 2 .config:1010:warning: override: UNWINDER_GUESS changes choice state 1 {standard input}:131: Warning: macro instruction expanded into multiple instructions ================================================================================ Detailed per-defconfig build reports: -------------------------------------------------------------------------------- 32r2el_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- acs5k_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- acs5k_tiny_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- allnoconfig (arc, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- allnoconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- allnoconfig (arm64, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- allnoconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- allnoconfig (x86_64, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- allnoconfig (riscv, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- allnoconfig (i386, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- am200epdkit_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- ar7_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- aspeed_g4_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- aspeed_g5_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- assabet_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- at91_dt_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- ath25_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- ath79_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- axm55xx_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- axs103_defconfig (arc, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- axs103_smp_defconfig (arc, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- badge4_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- bcm2835_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- bcm47xx_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- bcm63xx_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- bigsur_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- bmips_be_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- bmips_stb_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- capcella_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- cavium_octeon_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- cerfcube_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- ci20_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- clps711x_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- cm_x2xx_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- cm_x300_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- cns3420vb_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- cobalt_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- colibri_pxa270_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- colibri_pxa300_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- collie_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- corgi_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- davinci_all_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- db1xxx_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- decstation_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- defconfig (arm64, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- defconfig (riscv, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- dove_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- e55_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- ebsa110_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- efm32_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- em_x270_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- ep93xx_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- eseries_pxa_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- exynos_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- ezx_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- footbridge_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- fuloong2e_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- gcw0_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- gemini_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- gpr_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- h3600_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- h5000_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- hackkit_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- haps_hs_defconfig (arc, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- haps_hs_smp_defconfig (arc, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- hisi_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- hsdk_defconfig (arc, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- i386_defconfig (i386, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- imote2_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- imx_v4_v5_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- imx_v6_v7_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- integrator_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- iop13xx_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- iop32x_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- iop33x_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- ip22_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- ip27_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- ip28_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- ip32_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- ixp4xx_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- jazz_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- jmr3927_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- jornada720_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- keystone_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- ks8695_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- lart_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- lemote2f_defconfig (mips, gcc-8) — PASS, 0 errors, 1 warning, 0 section mismatches Warnings: net/core/rtnetlink.c:3190:1: warning: the frame size of 1312 bytes is larger than 1024 bytes [-Wframe-larger-than=] -------------------------------------------------------------------------------- loongson1b_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- loongson1c_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- loongson3_defconfig (mips, gcc-8) — PASS, 0 errors, 2 warnings, 0 section mismatches Warnings: arch/mips/configs/loongson3_defconfig:55:warning: symbol value 'm' invalid for HOTPLUG_PCI_SHPC net/core/rtnetlink.c:3190:1: warning: the frame size of 1312 bytes is larger than 1024 bytes [-Wframe-larger-than=] -------------------------------------------------------------------------------- lpc18xx_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- lpc32xx_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- lpd270_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- lubbock_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- magician_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- mainstone_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- malta_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- malta_kvm_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- malta_kvm_guest_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- malta_qemu_32r6_defconfig (mips, gcc-8) — PASS, 0 errors, 1 warning, 0 section mismatches Warnings: {standard input}:131: Warning: macro instruction expanded into multiple instructions -------------------------------------------------------------------------------- maltaaprp_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- maltasmvp_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- maltasmvp_eva_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- maltaup_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- maltaup_xpa_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- markeins_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- mini2440_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- mips_paravirt_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- mmp2_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- moxart_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- mpc30x_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- mps2_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- msp71xx_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- mtx1_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- multi_v4t_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- multi_v5_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- multi_v7_defconfig (arm, gcc-8) — PASS, 0 errors, 2 warnings, 0 section mismatches Warnings: drivers/gpu/drm/sun4i/sun4i_hdmi_tmds_clk.c:159:5: warning: ‘is_double’ may be used uninitialized in this function [-Wmaybe-uninitialized] arch/arm/boot/dts/sun8i-h3-beelink-x2.dtb: Warning (clocks_property): /wifi_pwrseq: Missing property '#clock-cells' in node /soc/rtc@1f00000 or bad phandle (referred from clocks[0]) -------------------------------------------------------------------------------- mv78xx0_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- mvebu_v5_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- mvebu_v7_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- mxs_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- neponset_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- netwinder_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- netx_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- nhk8815_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- nlm_xlp_defconfig (mips, gcc-8) — PASS, 0 errors, 1 warning, 0 section mismatches Warnings: net/core/rtnetlink.c:3190:1: warning: the frame size of 1312 bytes is larger than 1024 bytes [-Wframe-larger-than=] -------------------------------------------------------------------------------- nlm_xlr_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- nsim_hs_defconfig (arc, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- nsim_hs_smp_defconfig (arc, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- nsimosci_hs_defconfig (arc, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- nsimosci_hs_smp_defconfig (arc, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- nuc910_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- nuc950_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- nuc960_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- omap2plus_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- omega2p_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- orion5x_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- oxnas_v6_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- palmz72_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- pcm027_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- pic32mzda_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- pistachio_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- pleb_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- pnx8335_stb225_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- prima2_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- pxa168_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- pxa255-idp_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- pxa3xx_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- pxa910_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- pxa_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- qcom_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- qi_lb60_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- raumfeld_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- rb532_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- rbtx49xx_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- realview_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- rm200_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- rpc_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- rt305x_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- s3c2410_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- s3c6400_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- s5pv210_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- sama5_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- sb1250_swarm_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- shannon_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- shmobile_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- simpad_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- socfpga_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- spear13xx_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- spear3xx_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- spear6xx_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- spitz_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- stm32_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- sunxi_defconfig (arm, gcc-8) — PASS, 0 errors, 2 warnings, 0 section mismatches Warnings: drivers/gpu/drm/sun4i/sun4i_hdmi_tmds_clk.c:159:5: warning: ‘is_double’ may be used uninitialized in this function [-Wmaybe-uninitialized] arch/arm/boot/dts/sun8i-h3-beelink-x2.dtb: Warning (clocks_property): /wifi_pwrseq: Missing property '#clock-cells' in node /soc/rtc@1f00000 or bad phandle (referred from clocks[0]) -------------------------------------------------------------------------------- tango4_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- tb0219_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- tb0226_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- tb0287_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- tct_hammer_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- tegra_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- tinyconfig (riscv, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- tinyconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- tinyconfig (x86_64, gcc-8) — PASS, 0 errors, 1 warning, 0 section mismatches Warnings: .config:1010:warning: override: UNWINDER_GUESS changes choice state -------------------------------------------------------------------------------- tinyconfig (arm64, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- tinyconfig (i386, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- tinyconfig (arc, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- trizeps4_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- u300_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- u8500_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- vdk_hs38_defconfig (arc, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- vdk_hs38_smp_defconfig (arc, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- versatile_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- vexpress_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- vf610m4_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- viper_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- vocore2_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- vt8500_v6_v7_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- workpad_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- x86_64_defconfig (x86_64, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- xcep_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- xway_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- zeus_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- zx_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches --- For more info write to <info(a)kernelci.org>

5 years, 9 months

1
0
0 0

Re: [PATCH] HID: ite: Only bind to keyboard USB interface on Acer SW5-012 keyboard dock

by Hans de Goede

Hi, On 1/31/20 4:38 PM, Z R wrote: > Hi Benjamin, > hid-record for keyboard and touchpad. With Commit 8f18eca9ebc5 reverted and from unmodified kernel. > > I hope it is what you asked for :-) > > Currently waiting for reworked patch from Hans. I just send you the reworked patch. Does the recordning include pressing of the wlan on/off key (Fn + F3 I believe) ? That is the whole reason why the special hid-ite driver is necessary. Benjamin about the wlan on/off key. AFAICR on a press + release of the key a single hid input report for the generic-desktop Wireless Radio Controls group is send. This input-report only has the one button with usage code HID_GD_RFKILL_BTN in there and it is always 0. It is as if the input-report is only send on release and not on press. So the hid-ite code emulates a press + release whenever the input-report is send. IOW the receiving of the input report is (ab)used as indication of the button having been pressed. Regards, Hans > > Bye for now > Zdeněk > > pá 31. 1. 2020 v 15:12 odesílatel Benjamin Tissoires <benjamin.tissoires(a)redhat.com <mailto:benjamin.tissoires@redhat.com>> napsal: > > On Fri, Jan 31, 2020 at 3:04 PM Hans de Goede <hdegoede(a)redhat.com <mailto:hdegoede@redhat.com>> wrote: > > > > Hi, > > > > On 1/31/20 2:54 PM, Benjamin Tissoires wrote: > > > On Fri, Jan 31, 2020 at 2:41 PM Hans de Goede <hdegoede(a)redhat.com <mailto:hdegoede@redhat.com>> wrote: > > >> > > >> Hi, > > >> > > >> On 1/31/20 2:10 PM, Benjamin Tissoires wrote: > > >>> Hi Hans, > > >>> > > >>> On Fri, Jan 31, 2020 at 1:46 PM Hans de Goede <hdegoede(a)redhat.com <mailto:hdegoede@redhat.com>> wrote: > > >>>> > > >>>> Commit 8f18eca9ebc5 ("HID: ite: Add USB id match for Acer SW5-012 keyboard > > >>>> dock") added the USB id for the Acer SW5-012's keyboard dock to the > > >>>> hid-ite driver to fix the rfkill driver not working. > > >>>> > > >>>> Most keyboard docks with an ITE 8595 keyboard/touchpad controller have the > > >>>> "Wireless Radio Control" bits which need the special hid-ite driver on the > > >>>> second USB interface (the mouse interface) and their touchpad only supports > > >>>> mouse emulation, so using generic hid-input handling for anything but > > >>>> the "Wireless Radio Control" bits is fine. On these devices we simply bind > > >>>> to all USB interfaces. > > >>>> > > >>>> But unlike other ITE8595 using keyboard docks, the Acer Aspire Switch 10 > > >>>> (SW5-012)'s touchpad not only does mouse emulation it also supports > > >>>> HID-multitouch and all the keys including the "Wireless Radio Control" > > >>>> bits have been moved to the first USB interface (the keyboard intf). > > >>>> > > >>>> So we need hid-ite to handle the first (keyboard) USB interface and have > > >>>> it NOT bind to the second (mouse) USB interface so that that can be > > >>>> handled by hid-multitouch.c and we get proper multi-touch support. > > >>>> > > >>>> This commit adds a match callback to hid-ite which makes it only > > >>>> match the first USB interface when running on the Acer SW5-012, > > >>>> fixing the regression to mouse-emulation mode introduced by adding the > > >>>> keyboard dock USB id. > > >>>> > > >>>> Note the match function only does the special only bind to the first > > >>>> USB interface on the Acer SW5-012, on other devices the hid-ite driver > > >>>> actually must bind to the second interface as that is where the > > >>>> "Wireless Radio Control" bits are. > > >>> > > >>> This is not a full review, but a couple of things that popped out > > >>> while scrolling through the patch. > > >>> > > >>>> > > >>>> Cc: stable(a)vger.kernel.org <mailto:stable@vger.kernel.org> > > >>>> Fixes: 8f18eca9ebc5 ("HID: ite: Add USB id match for Acer SW5-012 keyboard dock") > > >>>> Reported-by: Zdeněk Rampas <zdenda.rampas(a)gmail.com <mailto:zdenda.rampas@gmail.com>> > > >>>> Signed-off-by: Hans de Goede <hdegoede(a)redhat.com <mailto:hdegoede@redhat.com>> > > >>>> --- > > >>>> drivers/hid/hid-ite.c | 34 ++++++++++++++++++++++++++++++++++ > > >>>> 1 file changed, 34 insertions(+) > > >>>> > > >>>> diff --git a/drivers/hid/hid-ite.c b/drivers/hid/hid-ite.c > > >>>> index c436e12feb23..69a4ddfd033d 100644 > > >>>> --- a/drivers/hid/hid-ite.c > > >>>> +++ b/drivers/hid/hid-ite.c > > >>>> @@ -8,9 +8,12 @@ > > >>>> #include <linux/input.h> > > >>>> #include <linux/hid.h> > > >>>> #include <linux/module.h> > > >>>> +#include <linux/usb.h> > > >>>> > > >>>> #include "hid-ids.h" > > >>>> > > >>>> +#define ITE8595_KBD_USB_INTF 0 > > >>>> + > > >>>> static int ite_event(struct hid_device *hdev, struct hid_field *field, > > >>>> struct hid_usage *usage, __s32 value) > > >>>> { > > >>>> @@ -37,6 +40,36 @@ static int ite_event(struct hid_device *hdev, struct hid_field *field, > > >>>> return 0; > > >>>> } > > >>>> > > >>>> +static bool ite_match(struct hid_device *hdev, bool ignore_special_driver) > > >>>> +{ > > >>>> + struct usb_interface *intf; > > >>>> + > > >>>> + if (ignore_special_driver) > > >>>> + return false; > > >>>> + > > >>>> + /* > > >>>> + * Most keyboard docks with an ITE 8595 keyboard/touchpad controller > > >>>> + * have the "Wireless Radio Control" bits which need this special > > >>>> + * driver on the second USB interface (the mouse interface). On > > >>>> + * these devices we simply bind to all USB interfaces. > > >>>> + * > > >>>> + * The Acer Aspire Switch 10 (SW5-012) is special, its touchpad > > >>>> + * not only does mouse emulation it also supports HID-multitouch > > >>>> + * and all the keys including the "Wireless Radio Control" bits > > >>>> + * have been moved to the first USB interface (the keyboard intf). > > >>>> + * > > >>>> + * We want the hid-multitouch driver to bind to the touchpad, so on > > >>>> + * the Acer SW5-012 we should only bind to the keyboard USB intf. > > >>>> + */ > > >>>> + if (hdev->bus != BUS_USB || hdev->vendor != USB_VENDOR_ID_SYNAPTICS || > > >>>> + hdev->product != USB_DEVICE_ID_SYNAPTICS_ACER_SWITCH5_012) > > >>> > > >>> Isn't there an existing matching function we can use here, instead of > > >>> checking each individual field? > > >> > > >> There is hid_match_one_id() but that is not exported (can be fixed) and it > > >> requires a struct hid_device_id, which either requires declaring an extra > > >> standalone struct hid_device_id for the SW5-012 kbd-dock, or hardcoding an > > >> index into the existing hid_device_id array for the driver (with the hardcoding > > >> being error prone, so not a good idea). > > >> > > >> Given the problems with using hid_match_one_id() I decided to just go with > > >> the above. > > > > > > right. An other solution would be to have a local macro/function that > > > does that. Because as soon as you start adding a quirk, an other comes > > > right after. > > > > > >> > > >> But see below. > > >> > > >>> > > >>>> + return true; > > >>>> + > > >>>> + intf = to_usb_interface(hdev->dev.parent); > > >>> > > >>> And this is oops-prone. You need: > > >>> - ensure hid_is_using_ll_driver(hdev, &usb_hid_driver) returns true. > > >>> - add a dependency on USBHID in the KConfig now that you are checking > > >>> on the USB transport layer. > > >>> > > >>> That being said, I would love instead: > > >>> - to have a non USB version of this match, where you decide which > > >>> component needs to be handled based on the report descriptor > > >> > > >> Actually your idea to use the desciptors is not bad, but since what > > >> we really want is to not bind to the interface which is marked for the > > >> hid-multitouch driver I just realized we can just check that. > > >> > > >> So how about: > > >> > > >> static bool ite_match(struct hid_device *hdev, bool ignore_special_driver) > > >> { > > >> if (ignore_special_driver) > > >> return false; > > >> > > >> /* > > >> * Some keyboard docks with an ITE 8595 keyboard/touchpad controller > > >> * support the HID multitouch protocol for the touchpad, in that > > >> * case the "Wireless Radio Control" bits which we care about are > > >> * on the other interface; and we should not bind to the multitouch > > >> * capable interface as that breaks multitouch support. > > >> */ > > >> return hdev->group != HID_GROUP_MULTITOUCH_WIN_8; > > >> } > > > > > > Yep, I like that very much :) > > > > Actually if we want to check the group and there are only 2 interfaces we do > > not need to use the match callback at all, w e can simply match on the > > group of the interface which we do want: > > > > diff --git a/drivers/hid/hid-ite.c b/drivers/hid/hid-ite.c > > index db0f35be5a8b..21bd48f16033 100644 > > --- a/drivers/hid/hid-ite.c > > +++ b/drivers/hid/hid-ite.c > > @@ -56,8 +56,9 @@ static const struct hid_device_id ite_devices[] = { > > { HID_USB_DEVICE(USB_VENDOR_ID_ITE, USB_DEVICE_ID_ITE8595) }, > > { HID_USB_DEVICE(USB_VENDOR_ID_258A, USB_DEVICE_ID_258A_6A88) }, > > /* ITE8595 USB kbd ctlr, with Synaptics touchpad connected to it. */ > > - { HID_USB_DEVICE(USB_VENDOR_ID_SYNAPTICS, > > - USB_DEVICE_ID_SYNAPTICS_ACER_SWITCH5_012) }, > > + { HID_DEVICE(BUS_USB, HID_GROUP_GENERIC, > > + USB_VENDOR_ID_SYNAPTICS, > > + USB_DEVICE_ID_SYNAPTICS_ACER_SWITCH5_012) }, > > { } > > }; > > MODULE_DEVICE_TABLE(hid, ite_devices); > > > > Much cleaner > > yep > > > (and now I don't need to write a test, which is always > > a good motivation to come up with a cleaner solution :) > > Hehe, too bad, you already picked up my curiosity on this one, and I > really would like to see the report descriptors and some events of the > keys that are fixed by hid-ite.c. > <with a low voice>This will be a hard requirement to accept this patch </joke>. > > More seriously, Zdeněk, can you run hid-recorder from > https://gitlab.freedesktop.org/libevdev/hid-tools/ and provide me the > report descriptor for all of your ITE HID devices? I'll add the > matching tests in hid-tools and be sure we do not regress in the > future. > > > > > Let me turn this into a proper patch and then I will send that to > > Zdeněk (off-list) for him to test (note don't worry if you do > > not have time to test this weekend, then I'll do it on Monday). > > > > Regards, > > > > Hans > > > > p.s. > > > > 1. My train is approaching Brussels (Fosdem) so my email response > > time will soon become irregular. > > How dare you? :) > > > > > 2. Benjamin will you be at Fosdem too ? > > > > Unfortunately no. Already got my quota of meeting people for this year > between Kernel Recipes in September, XDC in October and LCA last week. > So I need to keep in a quiet environment for a little bit :) > > Cheers, > Benjamin >

5 years, 9 months

1
0
0 0

stable-rc/linux-4.4.y boot: 53 boots: 1 failed, 47 passed with 5 offline (v4.4.212-23-g986b37ed163d)

by kernelci.org bot

stable-rc/linux-4.4.y boot: 53 boots: 1 failed, 47 passed with 5 offline (v4.4.212-23-g986b37ed163d) Full Boot Summary: https://kernelci.org/boot/all/job/stable-rc/branch/linux-4.4.y/kernel/v4.4.… Full Build Summary: https://kernelci.org/build/stable-rc/branch/linux-4.4.y/kernel/v4.4.212-23-… Tree: stable-rc Branch: linux-4.4.y Git Describe: v4.4.212-23-g986b37ed163d Git Commit: 986b37ed163d3c83afeaf1b1886e07178e590d26 Git URL: https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.git Tested: 34 unique boards, 13 SoC families, 12 builds out of 134 Boot Regressions Detected: arm: davinci_all_defconfig: gcc-8: dm365evm,legacy: lab-baylibre-seattle: new failure (last pass: v4.4.211-184-g475d90ca735c) Boot Failure Detected: arm: multi_v7_defconfig: gcc-8: sun4i-a10-cubieboard: 1 failed lab Offline Platforms: arm: multi_v7_defconfig: gcc-8 qcom-apq8064-cm-qs600: 1 offline lab sun5i-r8-chip: 1 offline lab sunxi_defconfig: gcc-8 sun5i-r8-chip: 1 offline lab qcom_defconfig: gcc-8 qcom-apq8064-cm-qs600: 1 offline lab davinci_all_defconfig: gcc-8 dm365evm,legacy: 1 offline lab --- For more info write to <info(a)kernelci.org>

5 years, 9 months

1
0
0 0

[PATCH v3] media: v4l2-core: fix a use-after-free bug of sd->devnode

by Dafna Hirschfeld

sd->devnode is released after calling v4l2_subdev_release. Therefore it should be set to NULL so that the subdev won't hold a pointer to a released object. This fixes a reference after free bug in function v4l2_device_unregister_subdev Cc: stable(a)vger.kernel.org Fixes: 0e43734d4c46e ("media: v4l2-subdev: add release() internal op") Signed-off-by: Dafna Hirschfeld <dafna.hirschfeld(a)collabora.com> Reviewed-by: Ezequiel Garcia <ezequiel(a)collabora.com> --- changes since v2: - since this is a regresion fix, I added Fixes and Cc to stable tags, - change the commit title and log to be more clear. drivers/media/v4l2-core/v4l2-device.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/media/v4l2-core/v4l2-device.c b/drivers/media/v4l2-core/v4l2-device.c index 63d6b147b21e..2b3595671d62 100644 --- a/drivers/media/v4l2-core/v4l2-device.c +++ b/drivers/media/v4l2-core/v4l2-device.c @@ -177,6 +177,7 @@ static void v4l2_subdev_release(struct v4l2_subdev *sd) { struct module *owner = !sd->owner_v4l2_dev ? sd->owner : NULL; + sd->devnode = NULL; if (sd->internal_ops && sd->internal_ops->release) sd->internal_ops->release(sd); module_put(owner); -- 2.20.1

5 years, 9 months

3
5
0 0

stable/linux-5.4.y build: 175 builds: 1 failed, 174 passed, 1 error, 99 warnings (v5.4.16)

by kernelci.org bot

stable/linux-5.4.y build: 175 builds: 1 failed, 174 passed, 1 error, 99 warnings (v5.4.16) Full Build Summary: https://kernelci.org/build/stable/branch/linux-5.4.y/kernel/v5.4.16/ Tree: stable Branch: linux-5.4.y Git Describe: v5.4.16 Git Commit: 60b6aa2b71efa7e0bd5393ce292ace4a0cf2e71b Git URL: https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable.git Built: 7 unique architectures Build Failure Detected: mips: ip27_defconfig: (gcc-8) FAIL Errors and Warnings Detected: arc: allnoconfig (gcc-8): 1 warning axs103_defconfig (gcc-8): 2 warnings axs103_smp_defconfig (gcc-8): 2 warnings haps_hs_defconfig (gcc-8): 2 warnings haps_hs_smp_defconfig (gcc-8): 2 warnings hsdk_defconfig (gcc-8): 2 warnings nsim_hs_defconfig (gcc-8): 2 warnings nsim_hs_smp_defconfig (gcc-8): 2 warnings nsimosci_hs_defconfig (gcc-8): 2 warnings nsimosci_hs_smp_defconfig (gcc-8): 2 warnings tinyconfig (gcc-8): 1 warning vdk_hs38_defconfig (gcc-8): 1 warning vdk_hs38_smp_defconfig (gcc-8): 1 warning arm64: defconfig (gcc-8): 12 warnings arm: assabet_defconfig (gcc-8): 1 warning at91_dt_defconfig (gcc-8): 1 warning axm55xx_defconfig (gcc-8): 1 warning cm_x2xx_defconfig (gcc-8): 1 warning cm_x300_defconfig (gcc-8): 1 warning cns3420vb_defconfig (gcc-8): 1 warning colibri_pxa270_defconfig (gcc-8): 1 warning colibri_pxa300_defconfig (gcc-8): 1 warning davinci_all_defconfig (gcc-8): 1 warning dove_defconfig (gcc-8): 1 warning em_x270_defconfig (gcc-8): 1 warning ep93xx_defconfig (gcc-8): 1 warning eseries_pxa_defconfig (gcc-8): 1 warning exynos_defconfig (gcc-8): 1 warning ezx_defconfig (gcc-8): 1 warning h3600_defconfig (gcc-8): 1 warning h5000_defconfig (gcc-8): 1 warning imote2_defconfig (gcc-8): 1 warning imx_v4_v5_defconfig (gcc-8): 1 warning imx_v6_v7_defconfig (gcc-8): 1 warning integrator_defconfig (gcc-8): 1 warning ixp4xx_defconfig (gcc-8): 1 warning lpc32xx_defconfig (gcc-8): 1 warning milbeaut_m10v_defconfig (gcc-8): 1 warning mini2440_defconfig (gcc-8): 1 warning mmp2_defconfig (gcc-8): 1 warning multi_v5_defconfig (gcc-8): 1 warning multi_v7_defconfig (gcc-8): 1 warning mv78xx0_defconfig (gcc-8): 1 warning mvebu_v5_defconfig (gcc-8): 1 warning mvebu_v7_defconfig (gcc-8): 1 warning neponset_defconfig (gcc-8): 1 warning nhk8815_defconfig (gcc-8): 1 warning omap1_defconfig (gcc-8): 1 warning omap2plus_defconfig (gcc-8): 1 warning oxnas_v6_defconfig (gcc-8): 1 warning palmz72_defconfig (gcc-8): 1 warning pcm027_defconfig (gcc-8): 1 warning prima2_defconfig (gcc-8): 1 warning pxa168_defconfig (gcc-8): 1 warning pxa3xx_defconfig (gcc-8): 1 warning pxa910_defconfig (gcc-8): 1 warning qcom_defconfig (gcc-8): 1 warning realview_defconfig (gcc-8): 1 warning s3c6400_defconfig (gcc-8): 1 warning sama5_defconfig (gcc-8): 1 warning shannon_defconfig (gcc-8): 1 warning spear13xx_defconfig (gcc-8): 1 warning sunxi_defconfig (gcc-8): 1 warning tango4_defconfig (gcc-8): 1 warning tegra_defconfig (gcc-8): 1 warning trizeps4_defconfig (gcc-8): 1 warning u300_defconfig (gcc-8): 1 warning versatile_defconfig (gcc-8): 1 warning vexpress_defconfig (gcc-8): 1 warning viper_defconfig (gcc-8): 1 warning xcep_defconfig (gcc-8): 1 warning zeus_defconfig (gcc-8): 1 warning i386: mips: ip27_defconfig (gcc-8): 1 error riscv: rv32_defconfig (gcc-8): 6 warnings x86_64: tinyconfig (gcc-8): 1 warning Errors summary: 1 arch/mips/pci/pci-xtalk-bridge.c:287:9: error: ‘struct bridge_irq_chip_data’ has no member named ‘nnasid’; did you mean ‘nasid’? Warnings summary: 63 WARNING: "return_address" [vmlinux] is a static EXPORT_SYMBOL_GPL 24 <stdin>:1511:2: warning: #warning syscall clone3 not implemented [-Wcpp] 4 drivers/video/fbdev/sa1100fb.c:975:21: warning: ‘sa1100fb_min_dma_period’ defined but not used [-Wunused-function] 4 arch/arm64/boot/dts/exynos/exynos5433.dtsi:254.3-29: Warning (reg_format): /gpu@14ac0000:reg: property has invalid length (8 bytes) (#address-cells == 2, #size-cells == 2) 2 arch/arm64/boot/dts/exynos/exynos7.dtsi:83.3-29: Warning (reg_format): /gpu@14ac0000:reg: property has invalid length (8 bytes) (#address-cells == 2, #size-cells == 2) 2 arch/arm64/boot/dts/exynos/exynos7-espresso.dtb: Warning (spi_bus_reg): Failed prerequisite 'reg_format' 2 arch/arm64/boot/dts/exynos/exynos7-espresso.dtb: Warning (pci_device_bus_num): Failed prerequisite 'reg_format' 2 arch/arm64/boot/dts/exynos/exynos7-espresso.dtb: Warning (i2c_bus_reg): Failed prerequisite 'reg_format' 2 arch/arm64/boot/dts/exynos/exynos5433-tm2e.dtb: Warning (spi_bus_reg): Failed prerequisite 'reg_format' 2 arch/arm64/boot/dts/exynos/exynos5433-tm2e.dtb: Warning (pci_device_bus_num): Failed prerequisite 'reg_format' 2 arch/arm64/boot/dts/exynos/exynos5433-tm2e.dtb: Warning (i2c_bus_reg): Failed prerequisite 'reg_format' 2 arch/arm64/boot/dts/exynos/exynos5433-tm2.dtb: Warning (spi_bus_reg): Failed prerequisite 'reg_format' 2 arch/arm64/boot/dts/exynos/exynos5433-tm2.dtb: Warning (pci_device_bus_num): Failed prerequisite 'reg_format' 2 arch/arm64/boot/dts/exynos/exynos5433-tm2.dtb: Warning (i2c_bus_reg): Failed prerequisite 'reg_format' 2 <stdin>:830:2: warning: #warning syscall fstat64 not implemented [-Wcpp] 2 <stdin>:1127:2: warning: #warning syscall fstatat64 not implemented [-Wcpp] 1 .config:1158:warning: override: UNWINDER_GUESS changes choice state Section mismatches summary: 4 WARNING: vmlinux.o(.text.unlikely+0x8c4): Section mismatch in reference from the function free_memmap() to the function .meminit.text:memblock_free() 2 WARNING: vmlinux.o(.text.unlikely+0x84c): Section mismatch in reference from the function free_memmap() to the function .meminit.text:memblock_free() 2 WARNING: vmlinux.o(.text.unlikely+0x840): Section mismatch in reference from the function free_memmap() to the function .meminit.text:memblock_free() 2 WARNING: vmlinux.o(.text.unlikely+0x710): Section mismatch in reference from the function free_memmap() to the function .meminit.text:memblock_free() 1 WARNING: vmlinux.o(.text.unlikely+0x8e4): Section mismatch in reference from the function free_memmap() to the function .meminit.text:memblock_free() 1 WARNING: vmlinux.o(.text.unlikely+0x7c8): Section mismatch in reference from the function free_memmap() to the function .meminit.text:memblock_free() 1 WARNING: vmlinux.o(.text.unlikely+0x3598): Section mismatch in reference from the function pmax_setup_memory_region() to the function .init.text:add_memory_region() 1 WARNING: vmlinux.o(.text.unlikely+0x321c): Section mismatch in reference from the function pmax_setup_memory_region() to the function .init.text:add_memory_region() ================================================================================ Detailed per-defconfig build reports: -------------------------------------------------------------------------------- 32r2el_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- allnoconfig (riscv, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- allnoconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- allnoconfig (arc, gcc-8) — PASS, 0 errors, 1 warning, 0 section mismatches Warnings: <stdin>:1511:2: warning: #warning syscall clone3 not implemented [-Wcpp] -------------------------------------------------------------------------------- allnoconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- allnoconfig (arm64, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- allnoconfig (x86_64, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- ar7_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- aspeed_g4_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- aspeed_g5_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- assabet_defconfig (arm, gcc-8) — PASS, 0 errors, 1 warning, 0 section mismatches Warnings: drivers/video/fbdev/sa1100fb.c:975:21: warning: ‘sa1100fb_min_dma_period’ defined but not used [-Wunused-function] Section mismatches: WARNING: vmlinux.o(.text.unlikely+0x8c4): Section mismatch in reference from the function free_memmap() to the function .meminit.text:memblock_free() -------------------------------------------------------------------------------- at91_dt_defconfig (arm, gcc-8) — PASS, 0 errors, 1 warning, 0 section mismatches Warnings: WARNING: "return_address" [vmlinux] is a static EXPORT_SYMBOL_GPL -------------------------------------------------------------------------------- ath25_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- ath79_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- axm55xx_defconfig (arm, gcc-8) — PASS, 0 errors, 1 warning, 0 section mismatches Warnings: WARNING: "return_address" [vmlinux] is a static EXPORT_SYMBOL_GPL -------------------------------------------------------------------------------- axs103_defconfig (arc, gcc-8) — PASS, 0 errors, 2 warnings, 0 section mismatches Warnings: <stdin>:1511:2: warning: #warning syscall clone3 not implemented [-Wcpp] <stdin>:1511:2: warning: #warning syscall clone3 not implemented [-Wcpp] -------------------------------------------------------------------------------- axs103_smp_defconfig (arc, gcc-8) — PASS, 0 errors, 2 warnings, 0 section mismatches Warnings: <stdin>:1511:2: warning: #warning syscall clone3 not implemented [-Wcpp] <stdin>:1511:2: warning: #warning syscall clone3 not implemented [-Wcpp] -------------------------------------------------------------------------------- bcm2835_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- bcm63xx_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- bigsur_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- bmips_be_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- bmips_stb_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- capcella_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- cavium_octeon_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- cerfcube_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches Section mismatches: WARNING: vmlinux.o(.text.unlikely+0x84c): Section mismatch in reference from the function free_memmap() to the function .meminit.text:memblock_free() -------------------------------------------------------------------------------- ci20_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- clps711x_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- cm_x2xx_defconfig (arm, gcc-8) — PASS, 0 errors, 1 warning, 0 section mismatches Warnings: WARNING: "return_address" [vmlinux] is a static EXPORT_SYMBOL_GPL -------------------------------------------------------------------------------- cm_x300_defconfig (arm, gcc-8) — PASS, 0 errors, 1 warning, 0 section mismatches Warnings: WARNING: "return_address" [vmlinux] is a static EXPORT_SYMBOL_GPL -------------------------------------------------------------------------------- cns3420vb_defconfig (arm, gcc-8) — PASS, 0 errors, 1 warning, 0 section mismatches Warnings: WARNING: "return_address" [vmlinux] is a static EXPORT_SYMBOL_GPL -------------------------------------------------------------------------------- cobalt_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- colibri_pxa270_defconfig (arm, gcc-8) — PASS, 0 errors, 1 warning, 0 section mismatches Warnings: WARNING: "return_address" [vmlinux] is a static EXPORT_SYMBOL_GPL -------------------------------------------------------------------------------- colibri_pxa300_defconfig (arm, gcc-8) — PASS, 0 errors, 1 warning, 0 section mismatches Warnings: WARNING: "return_address" [vmlinux] is a static EXPORT_SYMBOL_GPL -------------------------------------------------------------------------------- corgi_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- davinci_all_defconfig (arm, gcc-8) — PASS, 0 errors, 1 warning, 0 section mismatches Warnings: WARNING: "return_address" [vmlinux] is a static EXPORT_SYMBOL_GPL -------------------------------------------------------------------------------- db1xxx_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- decstation_64_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- decstation_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches Section mismatches: WARNING: vmlinux.o(.text.unlikely+0x3598): Section mismatch in reference from the function pmax_setup_memory_region() to the function .init.text:add_memory_region() -------------------------------------------------------------------------------- decstation_r4k_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches Section mismatches: WARNING: vmlinux.o(.text.unlikely+0x321c): Section mismatch in reference from the function pmax_setup_memory_region() to the function .init.text:add_memory_region() -------------------------------------------------------------------------------- defconfig (arm64, gcc-8) — PASS, 0 errors, 12 warnings, 0 section mismatches Warnings: arch/arm64/boot/dts/exynos/exynos5433.dtsi:254.3-29: Warning (reg_format): /gpu@14ac0000:reg: property has invalid length (8 bytes) (#address-cells == 2, #size-cells == 2) arch/arm64/boot/dts/exynos/exynos5433-tm2.dtb: Warning (pci_device_bus_num): Failed prerequisite 'reg_format' arch/arm64/boot/dts/exynos/exynos5433-tm2.dtb: Warning (i2c_bus_reg): Failed prerequisite 'reg_format' arch/arm64/boot/dts/exynos/exynos5433-tm2.dtb: Warning (spi_bus_reg): Failed prerequisite 'reg_format' arch/arm64/boot/dts/exynos/exynos5433.dtsi:254.3-29: Warning (reg_format): /gpu@14ac0000:reg: property has invalid length (8 bytes) (#address-cells == 2, #size-cells == 2) arch/arm64/boot/dts/exynos/exynos5433-tm2e.dtb: Warning (pci_device_bus_num): Failed prerequisite 'reg_format' arch/arm64/boot/dts/exynos/exynos5433-tm2e.dtb: Warning (i2c_bus_reg): Failed prerequisite 'reg_format' arch/arm64/boot/dts/exynos/exynos5433-tm2e.dtb: Warning (spi_bus_reg): Failed prerequisite 'reg_format' arch/arm64/boot/dts/exynos/exynos7.dtsi:83.3-29: Warning (reg_format): /gpu@14ac0000:reg: property has invalid length (8 bytes) (#address-cells == 2, #size-cells == 2) arch/arm64/boot/dts/exynos/exynos7-espresso.dtb: Warning (pci_device_bus_num): Failed prerequisite 'reg_format' arch/arm64/boot/dts/exynos/exynos7-espresso.dtb: Warning (i2c_bus_reg): Failed prerequisite 'reg_format' arch/arm64/boot/dts/exynos/exynos7-espresso.dtb: Warning (spi_bus_reg): Failed prerequisite 'reg_format' -------------------------------------------------------------------------------- dove_defconfig (arm, gcc-8) — PASS, 0 errors, 1 warning, 0 section mismatches Warnings: WARNING: "return_address" [vmlinux] is a static EXPORT_SYMBOL_GPL -------------------------------------------------------------------------------- e55_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- ebsa110_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- efm32_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- em_x270_defconfig (arm, gcc-8) — PASS, 0 errors, 1 warning, 0 section mismatches Warnings: WARNING: "return_address" [vmlinux] is a static EXPORT_SYMBOL_GPL -------------------------------------------------------------------------------- ep93xx_defconfig (arm, gcc-8) — PASS, 0 errors, 1 warning, 0 section mismatches Warnings: WARNING: "return_address" [vmlinux] is a static EXPORT_SYMBOL_GPL Section mismatches: WARNING: vmlinux.o(.text.unlikely+0x710): Section mismatch in reference from the function free_memmap() to the function .meminit.text:memblock_free() -------------------------------------------------------------------------------- eseries_pxa_defconfig (arm, gcc-8) — PASS, 0 errors, 1 warning, 0 section mismatches Warnings: WARNING: "return_address" [vmlinux] is a static EXPORT_SYMBOL_GPL -------------------------------------------------------------------------------- exynos_defconfig (arm, gcc-8) — PASS, 0 errors, 1 warning, 0 section mismatches Warnings: WARNING: "return_address" [vmlinux] is a static EXPORT_SYMBOL_GPL -------------------------------------------------------------------------------- ezx_defconfig (arm, gcc-8) — PASS, 0 errors, 1 warning, 0 section mismatches Warnings: WARNING: "return_address" [vmlinux] is a static EXPORT_SYMBOL_GPL -------------------------------------------------------------------------------- footbridge_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- fuloong2e_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- gcw0_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- gemini_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- gpr_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- h3600_defconfig (arm, gcc-8) — PASS, 0 errors, 1 warning, 0 section mismatches Warnings: drivers/video/fbdev/sa1100fb.c:975:21: warning: ‘sa1100fb_min_dma_period’ defined but not used [-Wunused-function] Section mismatches: WARNING: vmlinux.o(.text.unlikely+0x840): Section mismatch in reference from the function free_memmap() to the function .meminit.text:memblock_free() -------------------------------------------------------------------------------- h5000_defconfig (arm, gcc-8) — PASS, 0 errors, 1 warning, 0 section mismatches Warnings: WARNING: "return_address" [vmlinux] is a static EXPORT_SYMBOL_GPL -------------------------------------------------------------------------------- haps_hs_defconfig (arc, gcc-8) — PASS, 0 errors, 2 warnings, 0 section mismatches Warnings: <stdin>:1511:2: warning: #warning syscall clone3 not implemented [-Wcpp] <stdin>:1511:2: warning: #warning syscall clone3 not implemented [-Wcpp] -------------------------------------------------------------------------------- haps_hs_smp_defconfig (arc, gcc-8) — PASS, 0 errors, 2 warnings, 0 section mismatches Warnings: <stdin>:1511:2: warning: #warning syscall clone3 not implemented [-Wcpp] <stdin>:1511:2: warning: #warning syscall clone3 not implemented [-Wcpp] -------------------------------------------------------------------------------- hisi_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- hsdk_defconfig (arc, gcc-8) — PASS, 0 errors, 2 warnings, 0 section mismatches Warnings: <stdin>:1511:2: warning: #warning syscall clone3 not implemented [-Wcpp] <stdin>:1511:2: warning: #warning syscall clone3 not implemented [-Wcpp] -------------------------------------------------------------------------------- i386_defconfig (i386, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- imote2_defconfig (arm, gcc-8) — PASS, 0 errors, 1 warning, 0 section mismatches Warnings: WARNING: "return_address" [vmlinux] is a static EXPORT_SYMBOL_GPL -------------------------------------------------------------------------------- imx_v4_v5_defconfig (arm, gcc-8) — PASS, 0 errors, 1 warning, 0 section mismatches Warnings: WARNING: "return_address" [vmlinux] is a static EXPORT_SYMBOL_GPL -------------------------------------------------------------------------------- imx_v6_v7_defconfig (arm, gcc-8) — PASS, 0 errors, 1 warning, 0 section mismatches Warnings: WARNING: "return_address" [vmlinux] is a static EXPORT_SYMBOL_GPL -------------------------------------------------------------------------------- integrator_defconfig (arm, gcc-8) — PASS, 0 errors, 1 warning, 0 section mismatches Warnings: WARNING: "return_address" [vmlinux] is a static EXPORT_SYMBOL_GPL -------------------------------------------------------------------------------- iop32x_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- ip22_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- ip27_defconfig (mips, gcc-8) — FAIL, 1 error, 0 warnings, 0 section mismatches Errors: arch/mips/pci/pci-xtalk-bridge.c:287:9: error: ‘struct bridge_irq_chip_data’ has no member named ‘nnasid’; did you mean ‘nasid’? -------------------------------------------------------------------------------- ip28_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- ip32_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- ixp4xx_defconfig (arm, gcc-8) — PASS, 0 errors, 1 warning, 0 section mismatches Warnings: WARNING: "return_address" [vmlinux] is a static EXPORT_SYMBOL_GPL -------------------------------------------------------------------------------- jazz_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- jmr3927_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- jornada720_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches Section mismatches: WARNING: vmlinux.o(.text.unlikely+0x7c8): Section mismatch in reference from the function free_memmap() to the function .meminit.text:memblock_free() -------------------------------------------------------------------------------- lart_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches Section mismatches: WARNING: vmlinux.o(.text.unlikely+0x8c4): Section mismatch in reference from the function free_memmap() to the function .meminit.text:memblock_free() -------------------------------------------------------------------------------- lasat_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- lemote2f_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- loongson1b_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- loongson1c_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- loongson3_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- lpc18xx_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- lpc32xx_defconfig (arm, gcc-8) — PASS, 0 errors, 1 warning, 0 section mismatches Warnings: WARNING: "return_address" [vmlinux] is a static EXPORT_SYMBOL_GPL -------------------------------------------------------------------------------- lpd270_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- lubbock_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- malta_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- malta_kvm_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- malta_kvm_guest_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- malta_qemu_32r6_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- maltaup_xpa_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- markeins_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- milbeaut_m10v_defconfig (arm, gcc-8) — PASS, 0 errors, 1 warning, 0 section mismatches Warnings: WARNING: "return_address" [vmlinux] is a static EXPORT_SYMBOL_GPL -------------------------------------------------------------------------------- mini2440_defconfig (arm, gcc-8) — PASS, 0 errors, 1 warning, 0 section mismatches Warnings: WARNING: "return_address" [vmlinux] is a static EXPORT_SYMBOL_GPL -------------------------------------------------------------------------------- mips_paravirt_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- mmp2_defconfig (arm, gcc-8) — PASS, 0 errors, 1 warning, 0 section mismatches Warnings: WARNING: "return_address" [vmlinux] is a static EXPORT_SYMBOL_GPL -------------------------------------------------------------------------------- mpc30x_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- mps2_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- msp71xx_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- mtx1_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- multi_v4t_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- multi_v5_defconfig (arm, gcc-8) — PASS, 0 errors, 1 warning, 0 section mismatches Warnings: WARNING: "return_address" [vmlinux] is a static EXPORT_SYMBOL_GPL -------------------------------------------------------------------------------- multi_v7_defconfig (arm, gcc-8) — PASS, 0 errors, 1 warning, 0 section mismatches Warnings: WARNING: "return_address" [vmlinux] is a static EXPORT_SYMBOL_GPL -------------------------------------------------------------------------------- mv78xx0_defconfig (arm, gcc-8) — PASS, 0 errors, 1 warning, 0 section mismatches Warnings: WARNING: "return_address" [vmlinux] is a static EXPORT_SYMBOL_GPL -------------------------------------------------------------------------------- mvebu_v5_defconfig (arm, gcc-8) — PASS, 0 errors, 1 warning, 0 section mismatches Warnings: WARNING: "return_address" [vmlinux] is a static EXPORT_SYMBOL_GPL -------------------------------------------------------------------------------- mvebu_v7_defconfig (arm, gcc-8) — PASS, 0 errors, 1 warning, 0 section mismatches Warnings: WARNING: "return_address" [vmlinux] is a static EXPORT_SYMBOL_GPL -------------------------------------------------------------------------------- neponset_defconfig (arm, gcc-8) — PASS, 0 errors, 1 warning, 0 section mismatches Warnings: drivers/video/fbdev/sa1100fb.c:975:21: warning: ‘sa1100fb_min_dma_period’ defined but not used [-Wunused-function] Section mismatches: WARNING: vmlinux.o(.text.unlikely+0x8c4): Section mismatch in reference from the function free_memmap() to the function .meminit.text:memblock_free() -------------------------------------------------------------------------------- netwinder_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- nhk8815_defconfig (arm, gcc-8) — PASS, 0 errors, 1 warning, 0 section mismatches Warnings: WARNING: "return_address" [vmlinux] is a static EXPORT_SYMBOL_GPL -------------------------------------------------------------------------------- nlm_xlp_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- nlm_xlr_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- nsim_hs_defconfig (arc, gcc-8) — PASS, 0 errors, 2 warnings, 0 section mismatches Warnings: <stdin>:1511:2: warning: #warning syscall clone3 not implemented [-Wcpp] <stdin>:1511:2: warning: #warning syscall clone3 not implemented [-Wcpp] -------------------------------------------------------------------------------- nsim_hs_smp_defconfig (arc, gcc-8) — PASS, 0 errors, 2 warnings, 0 section mismatches Warnings: <stdin>:1511:2: warning: #warning syscall clone3 not implemented [-Wcpp] <stdin>:1511:2: warning: #warning syscall clone3 not implemented [-Wcpp] -------------------------------------------------------------------------------- nsimosci_hs_defconfig (arc, gcc-8) — PASS, 0 errors, 2 warnings, 0 section mismatches Warnings: <stdin>:1511:2: warning: #warning syscall clone3 not implemented [-Wcpp] <stdin>:1511:2: warning: #warning syscall clone3 not implemented [-Wcpp] -------------------------------------------------------------------------------- nsimosci_hs_smp_defconfig (arc, gcc-8) — PASS, 0 errors, 2 warnings, 0 section mismatches Warnings: <stdin>:1511:2: warning: #warning syscall clone3 not implemented [-Wcpp] <stdin>:1511:2: warning: #warning syscall clone3 not implemented [-Wcpp] -------------------------------------------------------------------------------- omap1_defconfig (arm, gcc-8) — PASS, 0 errors, 1 warning, 0 section mismatches Warnings: WARNING: "return_address" [vmlinux] is a static EXPORT_SYMBOL_GPL -------------------------------------------------------------------------------- omap2plus_defconfig (arm, gcc-8) — PASS, 0 errors, 1 warning, 0 section mismatches Warnings: WARNING: "return_address" [vmlinux] is a static EXPORT_SYMBOL_GPL -------------------------------------------------------------------------------- omega2p_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- oxnas_v6_defconfig (arm, gcc-8) — PASS, 0 errors, 1 warning, 0 section mismatches Warnings: WARNING: "return_address" [vmlinux] is a static EXPORT_SYMBOL_GPL -------------------------------------------------------------------------------- palmz72_defconfig (arm, gcc-8) — PASS, 0 errors, 1 warning, 0 section mismatches Warnings: WARNING: "return_address" [vmlinux] is a static EXPORT_SYMBOL_GPL -------------------------------------------------------------------------------- pcm027_defconfig (arm, gcc-8) — PASS, 0 errors, 1 warning, 0 section mismatches Warnings: WARNING: "return_address" [vmlinux] is a static EXPORT_SYMBOL_GPL -------------------------------------------------------------------------------- pistachio_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- pleb_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches Section mismatches: WARNING: vmlinux.o(.text.unlikely+0x840): Section mismatch in reference from the function free_memmap() to the function .meminit.text:memblock_free() -------------------------------------------------------------------------------- pnx8335_stb225_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- prima2_defconfig (arm, gcc-8) — PASS, 0 errors, 1 warning, 0 section mismatches Warnings: WARNING: "return_address" [vmlinux] is a static EXPORT_SYMBOL_GPL -------------------------------------------------------------------------------- pxa168_defconfig (arm, gcc-8) — PASS, 0 errors, 1 warning, 0 section mismatches Warnings: WARNING: "return_address" [vmlinux] is a static EXPORT_SYMBOL_GPL -------------------------------------------------------------------------------- pxa255-idp_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- pxa3xx_defconfig (arm, gcc-8) — PASS, 0 errors, 1 warning, 0 section mismatches Warnings: WARNING: "return_address" [vmlinux] is a static EXPORT_SYMBOL_GPL -------------------------------------------------------------------------------- pxa910_defconfig (arm, gcc-8) — PASS, 0 errors, 1 warning, 0 section mismatches Warnings: WARNING: "return_address" [vmlinux] is a static EXPORT_SYMBOL_GPL -------------------------------------------------------------------------------- pxa_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- qcom_defconfig (arm, gcc-8) — PASS, 0 errors, 1 warning, 0 section mismatches Warnings: WARNING: "return_address" [vmlinux] is a static EXPORT_SYMBOL_GPL -------------------------------------------------------------------------------- qi_lb60_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- rbtx49xx_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- realview_defconfig (arm, gcc-8) — PASS, 0 errors, 1 warning, 0 section mismatches Warnings: WARNING: "return_address" [vmlinux] is a static EXPORT_SYMBOL_GPL -------------------------------------------------------------------------------- rpc_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches Section mismatches: WARNING: vmlinux.o(.text.unlikely+0x8e4): Section mismatch in reference from the function free_memmap() to the function .meminit.text:memblock_free() -------------------------------------------------------------------------------- rt305x_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- rv32_defconfig (riscv, gcc-8) — PASS, 0 errors, 6 warnings, 0 section mismatches Warnings: <stdin>:830:2: warning: #warning syscall fstat64 not implemented [-Wcpp] <stdin>:1127:2: warning: #warning syscall fstatat64 not implemented [-Wcpp] <stdin>:1511:2: warning: #warning syscall clone3 not implemented [-Wcpp] <stdin>:830:2: warning: #warning syscall fstat64 not implemented [-Wcpp] <stdin>:1127:2: warning: #warning syscall fstatat64 not implemented [-Wcpp] <stdin>:1511:2: warning: #warning syscall clone3 not implemented [-Wcpp] -------------------------------------------------------------------------------- s3c2410_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- s3c6400_defconfig (arm, gcc-8) — PASS, 0 errors, 1 warning, 0 section mismatches Warnings: WARNING: "return_address" [vmlinux] is a static EXPORT_SYMBOL_GPL -------------------------------------------------------------------------------- sama5_defconfig (arm, gcc-8) — PASS, 0 errors, 1 warning, 0 section mismatches Warnings: WARNING: "return_address" [vmlinux] is a static EXPORT_SYMBOL_GPL -------------------------------------------------------------------------------- sb1250_swarm_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- shannon_defconfig (arm, gcc-8) — PASS, 0 errors, 1 warning, 0 section mismatches Warnings: drivers/video/fbdev/sa1100fb.c:975:21: warning: ‘sa1100fb_min_dma_period’ defined but not used [-Wunused-function] Section mismatches: WARNING: vmlinux.o(.text.unlikely+0x8c4): Section mismatch in reference from the function free_memmap() to the function .meminit.text:memblock_free() -------------------------------------------------------------------------------- shmobile_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- simpad_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches Section mismatches: WARNING: vmlinux.o(.text.unlikely+0x84c): Section mismatch in reference from the function free_memmap() to the function .meminit.text:memblock_free() -------------------------------------------------------------------------------- socfpga_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- spear13xx_defconfig (arm, gcc-8) — PASS, 0 errors, 1 warning, 0 section mismatches Warnings: WARNING: "return_address" [vmlinux] is a static EXPORT_SYMBOL_GPL -------------------------------------------------------------------------------- spear3xx_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- spear6xx_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- stm32_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- sunxi_defconfig (arm, gcc-8) — PASS, 0 errors, 1 warning, 0 section mismatches Warnings: WARNING: "return_address" [vmlinux] is a static EXPORT_SYMBOL_GPL -------------------------------------------------------------------------------- tango4_defconfig (arm, gcc-8) — PASS, 0 errors, 1 warning, 0 section mismatches Warnings: WARNING: "return_address" [vmlinux] is a static EXPORT_SYMBOL_GPL -------------------------------------------------------------------------------- tb0219_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- tb0287_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- tct_hammer_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- tegra_defconfig (arm, gcc-8) — PASS, 0 errors, 1 warning, 0 section mismatches Warnings: WARNING: "return_address" [vmlinux] is a static EXPORT_SYMBOL_GPL -------------------------------------------------------------------------------- tinyconfig (i386, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- tinyconfig (riscv, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- tinyconfig (x86_64, gcc-8) — PASS, 0 errors, 1 warning, 0 section mismatches Warnings: .config:1158:warning: override: UNWINDER_GUESS changes choice state -------------------------------------------------------------------------------- tinyconfig (arc, gcc-8) — PASS, 0 errors, 1 warning, 0 section mismatches Warnings: <stdin>:1511:2: warning: #warning syscall clone3 not implemented [-Wcpp] -------------------------------------------------------------------------------- tinyconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- tinyconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- tinyconfig (arm64, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- trizeps4_defconfig (arm, gcc-8) — PASS, 0 errors, 1 warning, 0 section mismatches Warnings: WARNING: "return_address" [vmlinux] is a static EXPORT_SYMBOL_GPL -------------------------------------------------------------------------------- u300_defconfig (arm, gcc-8) — PASS, 0 errors, 1 warning, 0 section mismatches Warnings: WARNING: "return_address" [vmlinux] is a static EXPORT_SYMBOL_GPL -------------------------------------------------------------------------------- vdk_hs38_defconfig (arc, gcc-8) — PASS, 0 errors, 1 warning, 0 section mismatches Warnings: <stdin>:1511:2: warning: #warning syscall clone3 not implemented [-Wcpp] -------------------------------------------------------------------------------- vdk_hs38_smp_defconfig (arc, gcc-8) — PASS, 0 errors, 1 warning, 0 section mismatches Warnings: <stdin>:1511:2: warning: #warning syscall clone3 not implemented [-Wcpp] -------------------------------------------------------------------------------- versatile_defconfig (arm, gcc-8) — PASS, 0 errors, 1 warning, 0 section mismatches Warnings: WARNING: "return_address" [vmlinux] is a static EXPORT_SYMBOL_GPL -------------------------------------------------------------------------------- vexpress_defconfig (arm, gcc-8) — PASS, 0 errors, 1 warning, 0 section mismatches Warnings: WARNING: "return_address" [vmlinux] is a static EXPORT_SYMBOL_GPL -------------------------------------------------------------------------------- vf610m4_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- viper_defconfig (arm, gcc-8) — PASS, 0 errors, 1 warning, 0 section mismatches Warnings: WARNING: "return_address" [vmlinux] is a static EXPORT_SYMBOL_GPL -------------------------------------------------------------------------------- vocore2_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- vt8500_v6_v7_defconfig (arm, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- workpad_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- x86_64_defconfig (x86_64, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- xcep_defconfig (arm, gcc-8) — PASS, 0 errors, 1 warning, 0 section mismatches Warnings: WARNING: "return_address" [vmlinux] is a static EXPORT_SYMBOL_GPL -------------------------------------------------------------------------------- xway_defconfig (mips, gcc-8) — PASS, 0 errors, 0 warnings, 0 section mismatches -------------------------------------------------------------------------------- zeus_defconfig (arm, gcc-8) — PASS, 0 errors, 1 warning, 0 section mismatches Warnings: WARNING: "return_address" [vmlinux] is a static EXPORT_SYMBOL_GPL --- For more info write to <info(a)kernelci.org>

5 years, 9 months

1
0
0 0

2025

2024

2023

2022

2021

2020

2019

2018

2017

Linux-stable-mirror