Re: [ptxdist] [PATCH] udev: Use ptxdist kernel headers
Am 16.08.2011 15:56, schrieb Michael Olbrich: On Mon, Aug 15, 2011 at 07:34:58PM +0200, bernh...@bwalle.de wrote: From: Bernhard Walle wa...@corscience.de When a toolchain without kernel headers (such as the Lite version from Codesourcery) or with very old kernel header is used, udev doesn't build because videodev.h is missing. This patch adds the $(KERNEL_HEADERS_INCLUDE_DIR) to the include path of udev like it's done for busybox or barebox. Tested on ARM with kernel headers 2.6.36.4 (the version OSELAS.Toolchain() is currently using). You need to select KERNEL_HEADERS as well otherwise udev might be build before the kernel headers are available. But how to express the fact that we don't want to force the kernel headers to be built. But if they are built, they should be built before udev. Does select KERNEL_HEADERS if KERNEL_HEADERS work? Regards, Bernhard -- ptxdist mailing list ptxdist@pengutronix.de
Re: [ptxdist] [PATCH] udev: Use ptxdist kernel headers
On Wed, Aug 17, 2011 at 09:17:02AM +0200, Bernhard Walle wrote: Am 16.08.2011 15:56, schrieb Michael Olbrich: On Mon, Aug 15, 2011 at 07:34:58PM +0200, bernh...@bwalle.de wrote: From: Bernhard Walle wa...@corscience.de When a toolchain without kernel headers (such as the Lite version from Codesourcery) or with very old kernel header is used, udev doesn't build because videodev.h is missing. This patch adds the $(KERNEL_HEADERS_INCLUDE_DIR) to the include path of udev like it's done for busybox or barebox. Tested on ARM with kernel headers 2.6.36.4 (the version OSELAS.Toolchain() is currently using). You need to select KERNEL_HEADERS as well otherwise udev might be build before the kernel headers are available. But how to express the fact that we don't want to force the kernel headers to be built. But if they are built, they should be built before udev. Does select KERNEL_HEADERS if KERNEL_HEADERS work? hmmm, different idea: make kernel-headers a 'base' package. That way it's built before any non-base package. menuconfig KERNEL_HEADER tristate select BASE [...] Michael -- Pengutronix e.K. | | Industrial Linux Solutions | http://www.pengutronix.de/ | Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0| Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917- | -- ptxdist mailing list ptxdist@pengutronix.de
Re: [ptxdist] [PATCH] udev: Use ptxdist kernel headers
On Mon, Aug 15, 2011 at 07:34:58PM +0200, bernh...@bwalle.de wrote: From: Bernhard Walle wa...@corscience.de When a toolchain without kernel headers (such as the Lite version from Codesourcery) or with very old kernel header is used, udev doesn't build because videodev.h is missing. This patch adds the $(KERNEL_HEADERS_INCLUDE_DIR) to the include path of udev like it's done for busybox or barebox. Tested on ARM with kernel headers 2.6.36.4 (the version OSELAS.Toolchain() is currently using). You need to select KERNEL_HEADERS as well otherwise udev might be build before the kernel headers are available. Michael Signed-off-by: Bernhard Walle wa...@corscience.de --- rules/udev.make |5 + 1 files changed, 5 insertions(+), 0 deletions(-) diff --git a/rules/udev.make b/rules/udev.make index ac2f0d4..b57bd01 100644 --- a/rules/udev.make +++ b/rules/udev.make @@ -47,6 +47,11 @@ $(UDEV_SOURCE): # # autoconf + +UDEV_CONF_ENV := \ + $(CROSS_ENV) \ + CPPFLAGS=-I$(KERNEL_HEADERS_INCLUDE_DIR) $(CROSS_CPPFLAGS) + # # Note: explicit sbindir to avoid ${exec_prefix}/sbin/ on the target. UDEV_AUTOCONF := \ -- 1.7.6 -- ptxdist mailing list ptxdist@pengutronix.de -- Pengutronix e.K. | | Industrial Linux Solutions | http://www.pengutronix.de/ | Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0| Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917- | -- ptxdist mailing list ptxdist@pengutronix.de
[ptxdist] [PATCH] udev: Use ptxdist kernel headers
From: Bernhard Walle wa...@corscience.de When a toolchain without kernel headers (such as the Lite version from Codesourcery) or with very old kernel header is used, udev doesn't build because videodev.h is missing. This patch adds the $(KERNEL_HEADERS_INCLUDE_DIR) to the include path of udev like it's done for busybox or barebox. Tested on ARM with kernel headers 2.6.36.4 (the version OSELAS.Toolchain() is currently using). Signed-off-by: Bernhard Walle wa...@corscience.de --- rules/udev.make |5 + 1 files changed, 5 insertions(+), 0 deletions(-) diff --git a/rules/udev.make b/rules/udev.make index ac2f0d4..b57bd01 100644 --- a/rules/udev.make +++ b/rules/udev.make @@ -47,6 +47,11 @@ $(UDEV_SOURCE): # # autoconf + +UDEV_CONF_ENV := \ + $(CROSS_ENV) \ + CPPFLAGS=-I$(KERNEL_HEADERS_INCLUDE_DIR) $(CROSS_CPPFLAGS) + # # Note: explicit sbindir to avoid ${exec_prefix}/sbin/ on the target. UDEV_AUTOCONF := \ -- 1.7.6 -- ptxdist mailing list ptxdist@pengutronix.de