On Thu, Oct 22, 2020 at 2:35 PM Niclas Zeising <zeis...@freebsd.org> wrote: > > On 2020-10-22 15:22, Kyle Evans wrote: > > On Sat, Oct 17, 2020 at 11:40 AM Warner Losh <i...@bsdimp.com> wrote: > >> > >> > >> > >> On Sat, Oct 17, 2020, 10:11 AM Alexander V. Chernikov <melif...@ipfw.ru> > >> wrote: > >>> > >>> 17.10.2020, 14:07, "Hans Petter Selasky" <h...@selasky.org>: > >>> > >>> On 2020-10-17 14:34, Alexander V. Chernikov wrote: > >>> > >>> 17.10.2020, 12:32, "Hans Petter Selasky" <h...@selasky.org>: > >>> > >>> On 2020-10-17 13:27, Alexander V. Chernikov wrote: > >>> > >>> 02.10.2020, 19:26, "Emmanuel Vadot" <m...@freebsd.org > >>> <mailto:m...@freebsd.org>>: > >>> > >>> Author: manu > >>> Date: Fri Oct 2 18:26:41 2020 > >>> New Revision: 366372 > >>> URL: https://svnweb.freebsd.org/changeset/base/366372 > >>> > >>> Log: > >>> linuxkpi: Add backlight support > >>> > >>> Add backlight function to linuxkpi. > >>> Graphics drivers expose the backlight of the panel > >>> directly so > >>> allow them > >>> to use the backlight subsystem so > >>> user can use backlight(8) to configure them. > >>> > >>> Reviewed by: hselasky > >>> Relnotes: yes > >>> Differential Revision: The FreeBSD Foundation > >>> > >>> Added: > >>> > >>> head/sys/compat/linuxkpi/common/include/linux/backlight.h > >>> (contents, > >>> props changed) > >>> Modified: > >>> > >>> head/sys/compat/linuxkpi/common/include/linux/device.h > >>> head/sys/compat/linuxkpi/common/src/linux_kmod.c > >>> head/sys/compat/linuxkpi/common/src/linux_pci.c > >>> head/sys/conf/kmod.mk > >>> > >>> It breaks the build for me with > >>> > >>> /usr/home/melifaro/free/head/sys/compat/linuxkpi/common/src/linux_pci.c:70:10: > >>> fatal error: 'backlight_if.h' file not found > >>> > >>> > >>> How do you build? Doesn't break over here. > >>> > >>> GENERIC + COMPAT_LINUXKPI. > >>> > >>> > >>> > >>> Try adding: > >>> > >>> options backlight > >>> > >>> To the kernel config. > >>> > >>> Yep, thank you! > >>> Maybe it's worth considering adding static assert with the message > >>> describing this dependency? > >> > >> > >> Yes. It likely is worth doing something to highlight this issue. > >> > >> Warner > >> > > > > I think we just need to slap the two core backlight files with an ` | > > compat_linux` so that they simply get pulled in if you specify > > COMPAT_LINUX. config(8) handles this terribly, configng must have a > > better provides/requires/implies/whatever functionality so we can > > specify that compat_linux implies backlight and not do crud like this > > where it becomes more complicated to see what any given option really > > entails. > > > > Thanks, > > COMPAT_LINUX can't be right. Isn't that the linuxolator? > Regards > -- > Niclas Zeising
Whoops, s/LINUX/LINUXKPI/ :-) _______________________________________________ svn-src-head@freebsd.org mailing list https://lists.freebsd.org/mailman/listinfo/svn-src-head To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"