Plippo / asus-wmi-screenpad

Variation of the asus-wmi kernel module with screenpad brightness support
Other
159 stars 20 forks source link

Step 4: Build Error on Kernel 5.6 #6

Closed matsuki820 closed 4 years ago

matsuki820 commented 4 years ago

Hey, I'm getting this error when building on kernel 5.6.

make: Entering directory '/usr/src/kernels/5.6.16-300.fc32.x86_64'
  AR      /var/lib/dkms/asus-wmi/1.0/build/built-in.a
  CC [M]  /var/lib/dkms/asus-wmi/1.0/build/asus-wmi.o
  CC [M]  /var/lib/dkms/asus-wmi/1.0/build/asus-nb-wmi.o
/var/lib/dkms/asus-wmi/1.0/build/asus-wmi.c: In function ‘asus_hwmon_temp1’:
/var/lib/dkms/asus-wmi/1.0/build/asus-wmi.c:1597:10: error: implicit declaration of function ‘DECI_KELVIN_TO_CELSIUS’ [-Werror=implicit-function-declaration]
 1597 |  value = DECI_KELVIN_TO_CELSIUS((value & 0xFFFF)) * 1000;
      |          ^~~~~~~~~~~~~~~~~~~~~~
cc1: some warnings being treated as errors
make[1]: *** [scripts/Makefile.build:267: /var/lib/dkms/asus-wmi/1.0/build/asus-wmi.o] Error 1
make: *** [Makefile:1695: /var/lib/dkms/asus-wmi/1.0/build] Error 2
make: Leaving directory '/usr/src/kernels/5.6.16-300.fc32.x86_64'

Any idea what might be causing this?

Plippo commented 4 years ago

Yes the driver is kernel specific, the version in the repository is for kernel 5.4. But I now added a script that downloads and patches the correct version for the currently install kernel. See the new step 3 in the instructions. I hope it works for you!

bundyo commented 4 years ago

Seems to build now, thanks.

matsuki820 commented 4 years ago

@Plippo that worked, thanks!