2020-09-04 14:02:01

by Ian Abbott

Subject: {PATCH 0/2] HID: wiimote: Minor change to spinlock usage

[I have posted these patches previously, but that was over a year ago.
--IA]

A couple of minor changes to the wiimote core module. They do not
change functionality of the driver:

1) HID: wiimote: make handlers[] const
2) HID: wiimote: narrow spinlock range in wiimote_hid_event()

drivers/hid/hid-wiimote-core.c | 10 ++++------
1 file changed, 4 insertions(+), 6 deletions(-)

2020-09-07 07:38:39

by David Rheinsberg

Subject: Re: {PATCH 0/2] HID: wiimote: Minor change to spinlock usage

Hi

On Fri, 4 Sep 2020 at 15:21, Ian Abbott <[email protected]> wrote:
>
> [I have posted these patches previously, but that was over a year ago.
> --IA]
>
> A couple of minor changes to the wiimote core module. They do not
> change functionality of the driver:
>
> 1) HID: wiimote: make handlers[] const
> 2) HID: wiimote: narrow spinlock range in wiimote_hid_event()
>
> drivers/hid/hid-wiimote-core.c | 10 ++++------
> 1 file changed, 4 insertions(+), 6 deletions(-)

Looks still good ;)

Reviewed-by: David Rheinsberg <[email protected]>

Thanks!
David

2020-09-07 14:08:48

by Jiri Kosina

Subject: Re: {PATCH 0/2] HID: wiimote: Minor change to spinlock usage

On Fri, 4 Sep 2020, Ian Abbott wrote:

> [I have posted these patches previously, but that was over a year ago.
> --IA]
>
> A couple of minor changes to the wiimote core module. They do not
> change functionality of the driver:
>
> 1) HID: wiimote: make handlers[] const
> 2) HID: wiimote: narrow spinlock range in wiimote_hid_event()
>
> drivers/hid/hid-wiimote-core.c | 10 ++++------
> 1 file changed, 4 insertions(+), 6 deletions(-)

Applied to for-5.10/wiimote. Thanks,

--
Jiri Kosina
SUSE Labs