From mboxrd@z Thu Jan  1 00:00:00 1970
Return-path: <ptxdist-bounces@pengutronix.de>
Received: from mediacenter.hi.pengutronix.de ([2001:6f8:1178:2::65])
	by metis.ext.pengutronix.de with esmtp (Exim 4.72)
	(envelope-from <mol@pengutronix.de>) id 1R5wC0-00013x-CS
	for ptxdist@pengutronix.de; Tue, 20 Sep 2011 11:00:40 +0200
Received: from mol by mediacenter.hi.pengutronix.de with local (Exim 4.72)
	(envelope-from <mol@pengutronix.de>) id 1R5wC0-0007B8-Bd
	for ptxdist@pengutronix.de; Tue, 20 Sep 2011 11:00:40 +0200
Date: Tue, 20 Sep 2011 11:00:40 +0200
From: Michael Olbrich <m.olbrich@pengutronix.de>
Message-ID: <20110920090040.GT27632@pengutronix.de>
References: <20110817095613.GU20524@pengutronix.de>
	<1313579150-8834-1-git-send-email-bernhard@bwalle.de>
	<CAMwGMjwRTGc8FbmhrEzBh=MVmhRG_iaj_W5D0Cu2ZS74zbo9=w@mail.gmail.com>
MIME-Version: 1.0
Content-Disposition: inline
In-Reply-To: <CAMwGMjwRTGc8FbmhrEzBh=MVmhRG_iaj_W5D0Cu2ZS74zbo9=w@mail.gmail.com>
Subject: Re: [ptxdist] [PATCH 1/2] udev: Use ptxdist kernel headers
Reply-To: ptxdist@pengutronix.de
List-Id: PTXdist Development Mailing List <ptxdist.pengutronix.de>
List-Unsubscribe: <http://metis.pengutronix.de/cgi-bin/mailman/options/ptxdist>, 
	<mailto:ptxdist-request@pengutronix.de?subject=unsubscribe>
List-Archive: <http://metis.pengutronix.de/pipermail/ptxdist>
List-Post: <mailto:ptxdist@pengutronix.de>
List-Help: <mailto:ptxdist-request@pengutronix.de?subject=help>
List-Subscribe: <http://metis.pengutronix.de/cgi-bin/mailman/listinfo/ptxdist>, 
	<mailto:ptxdist-request@pengutronix.de?subject=subscribe>
Content-Type: text/plain; charset="iso-8859-1"
Content-Transfer-Encoding: quoted-printable
Sender: ptxdist-bounces@pengutronix.de
Errors-To: ptxdist-bounces@pengutronix.de
To: ptxdist@pengutronix.de

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>
> > ---
> > =A0rules/udev.make | =A0 =A05 +++++
> > =A01 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):
> >
> > =A0#
> > =A0# autoconf
> > +
> > +UDEV_CONF_ENV :=3D \
> > + =A0 =A0 =A0 $(CROSS_ENV) \
> > + =A0 =A0 =A0 CPPFLAGS=3D"-I$(KERNEL_HEADERS_INCLUDE_DIR) $(CROSS_CPPFL=
AGS)"
> > +
> =

> 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