Re: CVS commit: src/sys/dev/acpi/wmi

[bch <brad.harder%gmail.com@localhost>]

...
/usr/src/sys/dev/acpi/wmi/wmi_dell.c: In function 'wmi_dell_action':
/usr/src/sys/dev/acpi/wmi/wmi_dell.c:234:16: error: comparison between
signed and unsigned integer expressions [-Werror=sign-compare]
  for (i = 0; i < __arraycount(wmi_dell_actions); i++) {
                ^
cc1: all warnings being treated as errors




On 12/3/17, Manuel Bouyer <bouyer%netbsd.org@localhost> wrote:
> Module Name:	src
> Committed By:	bouyer
> Date:		Sun Dec  3 17:40:48 UTC 2017
>
> Modified Files:
> 	src/sys/dev/acpi/wmi: wmi_dell.c
>
> Log Message:
> Fix dell WMI mappings:
> - query the descriptor to get the interface version, needed to workaround
>   a bug in the BIOS/ACPI
> - properly decode the event buffer in type/subtype, and handle multiple
> events
>   per handler call
> - record some known type/subtype in a table, with associated actions.
>
> Informations mostly from linux. Tested on a Dell 5480 laptop.
>
>
> To generate a diff of this commit:
> cvs rdiff -u -r1.9 -r1.10 src/sys/dev/acpi/wmi/wmi_dell.c
>
> Please note that diffs are not public domain; they are subject to the
> copyright notices on the relevant files.
>
>