* Re: [ptxdist] [PATCH 1/2] udev: Use ptxdist kernel headers
[not found] ` <1313579150-8834-1-git-send-email-bernhard@bwalle.de>
@ 2011-09-20 3:37 ` Jon Ringle
2011-09-20 5:53 ` Bernhard Walle
2011-09-20 9:00 ` Michael Olbrich
0 siblings, 2 replies; 3+ messages in thread
From: Jon Ringle @ 2011-09-20 3:37 UTC (permalink / raw)
To: ptxdist; +Cc: Bernhard Walle
On Wed, Aug 17, 2011 at 7:05 AM, <bernhard@bwalle.de> wrote:
> From: Bernhard Walle <walle@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 <walle@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)"
> +
This breaks udev for me. #include <scsi/scsi.h> includes the wrong header file:
--------------------
target: udev.compile
--------------------
make[1]: Entering directory
`${PTXDIST_WORKSPACE}/platform-initramfs-ISC/build-target/udev-162'
make --no-print-directory all-recursive
Making all in .
CC libudev/libudev.lo
CC libudev/libudev-list.lo
CC libudev/libudev-util.lo
CC libudev/libudev-device.lo
CC libudev/libudev-enumerate.lo
CC libudev/libudev-monitor.lo
CC libudev/libudev-queue.lo
CC libudev/libudev-ctrl.lo
CC libudev/libudev-util-private.lo
CC libudev/libudev-device-private.lo
CC libudev/libudev-queue-private.lo
CC extras/firmware/extras_firmware_firmware-firmware.o
CC extras/ata_id/ata_id.o
CC extras/cdrom_id/cdrom_id.o
CC extras/collect/collect.o
CC extras/edd_id/edd_id.o
CC extras/floppy/create_floppy_devices.o
In file included from extras/ata_id/ata_id.c:32:0:
${PTXDIST_WORKSPACE}/platform-initramfs-ISC/sysroot-target/kernel-headers/include/scsi/scsi.h:152:2:
error: expected specifier-qualifier-list before 'u8'
${PTXDIST_WORKSPACE}/platform-initramfs-ISC/sysroot-target/kernel-headers/include/scsi/scsi.h:
In function 'scsi_varlen_cdb_length':
${PTXDIST_WORKSPACE}/platform-initramfs-ISC/sysroot-target/kernel-headers/include/scsi/scsi.h:163:44:
error: 'struct scsi_varlen_cdb_hdr' has no member named
'additional_cdb_length'
make[3]: *** [extras/ata_id/ata_id.o] Error 1
make[3]: *** Waiting for unfinished jobs....
CC extras/input_id/input_id.o
make[2]: *** [all-recursive] Error 1
make[1]: Leaving directory
`${PTXDIST_WORKSPACE}/platform-initramfs-ISC/build-target/udev-162'
make[1]: *** [all] Error 2
make: *** [${PTXDIST_WORKSPACE}/platform-initramfs-ISC/state/udev.compile]
Error 2
Jon
--
ptxdist mailing list
ptxdist@pengutronix.de
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [ptxdist] [PATCH 1/2] udev: Use ptxdist kernel headers
2011-09-20 3:37 ` [ptxdist] [PATCH 1/2] udev: Use ptxdist kernel headers Jon Ringle
@ 2011-09-20 5:53 ` Bernhard Walle
2011-09-20 9:00 ` Michael Olbrich
1 sibling, 0 replies; 3+ messages in thread
From: Bernhard Walle @ 2011-09-20 5:53 UTC (permalink / raw)
To: ptxdist
Am 20.09.11 05:37, schrieb Jon Ringle:
>>
>> 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)"
>> +
>
> This breaks udev for me. #include <scsi/scsi.h> includes the wrong header file:
So why do you enable the option at all if your toolchain has up-to-date
kernel headers?
Regards,
Bernhard
--
ptxdist mailing list
ptxdist@pengutronix.de
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [ptxdist] [PATCH 1/2] udev: Use ptxdist kernel headers
2011-09-20 3:37 ` [ptxdist] [PATCH 1/2] udev: Use ptxdist kernel headers Jon Ringle
2011-09-20 5:53 ` Bernhard Walle
@ 2011-09-20 9:00 ` Michael Olbrich
1 sibling, 0 replies; 3+ messages in thread
From: Michael Olbrich @ 2011-09-20 9:00 UTC (permalink / raw)
To: ptxdist
On Mon, Sep 19, 2011 at 11:37:10PM -0400, Jon Ringle wrote:
> On Wed, Aug 17, 2011 at 7:05 AM, <bernhard@bwalle.de> wrote:
> > From: Bernhard Walle <walle@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 <walle@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)"
> > +
>
> This breaks udev for me. #include <scsi/scsi.h> includes the wrong header file:
This is a bug in the kernel (2.6.31-.34). scsi/scsi.h should not be
installed. Use a different version for the kernel headers or apply the
patch I mentioned[1] before.
Michael
[1] http://article.gmane.org/gmane.comp.embedded.ptxdist.devel/8046
--
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-5555 |
--
ptxdist mailing list
ptxdist@pengutronix.de
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2011-09-20 9:00 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <20110817095613.GU20524@pengutronix.de>
[not found] ` <1313579150-8834-1-git-send-email-bernhard@bwalle.de>
2011-09-20 3:37 ` [ptxdist] [PATCH 1/2] udev: Use ptxdist kernel headers Jon Ringle
2011-09-20 5:53 ` Bernhard Walle
2011-09-20 9:00 ` Michael Olbrich
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox