unofficial mirror of help-guix@gnu.org 
 help / color / mirror / Atom feed
* Missing platform_device.h in linux-libre-headers
@ 2018-04-08 15:18 Eleven
  2018-04-08 20:52 ` Mark H Weaver
  0 siblings, 1 reply; 2+ messages in thread
From: Eleven @ 2018-04-08 15:18 UTC (permalink / raw)
  To: help-guix

I’m trying to compile some extra kernel module for my guixsd, however the headers files needed, `modules.h` and `platform_device.h` was not found. After redefining the linux-libra-headers package, update the version to 4.15.13 which is the version of the kernel I’m currently running, I get `modules.h` but `platform_device.h` still missing. What’s the problem and how can I fix it?

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: Missing platform_device.h in linux-libre-headers
  2018-04-08 15:18 Missing platform_device.h in linux-libre-headers Eleven
@ 2018-04-08 20:52 ` Mark H Weaver
  0 siblings, 0 replies; 2+ messages in thread
From: Mark H Weaver @ 2018-04-08 20:52 UTC (permalink / raw)
  To: Eleven; +Cc: help-guix

Hi,

Eleven <qiaochu321@126.com> writes:
> I’m trying to compile some extra kernel module for my guixsd, however
> the headers files needed, `modules.h` and `platform_device.h` was not
> found. After redefining the linux-libra-headers package, update the
> version to 4.15.13 which is the version of the kernel I’m currently
> running, I get `modules.h` but `platform_device.h` still
> missing. What’s the problem and how can I fix it?

platform_device.h is not part of the user API, so it's not installed by
the "headers_install" make target in linux-libre.

I would recommend against changing the 'linux-libre-headers' package
itself.  The contents of 'linux-libre-headers' is included in our
default C headers used to build the entire system.  Updating it entails
a full system rebuild from the toolchain up.  Even if not for this
practical problem, I would argue that private kernel driver interfaces
do not belong in our default C headers.

The headers needed to build kernel modules should be a separate package,
maybe called 'linux-libre-kernel-headers' or
'linux-libre-internal-headers'.  It could be almost the same as
'linux-libre-headers', except that it should use the current kernel
version (using the %linux-libre-version and %linux-libre-hash variables)
and it should use different make targets in the 'build' and 'install'
phases, namely "headers_check_all" and "headers_install_all".

      Mark

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2018-04-08 20:53 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2018-04-08 15:18 Missing platform_device.h in linux-libre-headers Eleven
2018-04-08 20:52 ` Mark H Weaver

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).