linux-next.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Stephen Rothwell <sfr@canb.auug.org.au>
To: Geert Uytterhoeven <geert@linux-m68k.org>
Cc: Matthew Wilcox <matthew@wil.cx>, linux-next@vger.kernel.org
Subject: Re: linux-next: m68k/semaphore build failure
Date: Wed, 28 May 2008 16:04:06 +1000	[thread overview]
Message-ID: <20080528160406.4fceb036.sfr@canb.auug.org.au> (raw)
In-Reply-To: <Pine.LNX.4.64.0805271809520.4081@anakin>

On Tue, 27 May 2008 18:10:51 +0200 (CEST) Geert Uytterhoeven <geert@linux-m68k.org> wrote:
>
> On Tue, 27 May 2008, Stephen Rothwell wrote:
> > Today's linux-next build (m68k multi_defconfig) failed like this:
> > 
> > drivers/input/keyboard/hil_kbd.c:336: error: 'PCI_VENDOR_ID_HP' undeclared (first use in this function)
> > drivers/input/keyboard/hil_kbd.c:336: error: (Each undeclared identifier is reported only once
> > drivers/input/keyboard/hil_kbd.c:336: error: for each function it appears in.)
> > 
> > This is caused by commit fd7c1a9773fd107674216b63a752f21ba6495045 ("Add
> > missing semaphore.h includes") from the semaphore tree.  That commit adds
> > a linux/semaphore.h include but removes the linux/pci_ids.h include.
> 
> Thx!
> 
> I'd swear I reported this to Willy the first time it appeared, but according
> to my mail logs, I never did...

I have added the following patch to linux-next for today, but I am
assuming the Willy will fix his semaphore tree shortly.

-- 
Cheers,
Stephen Rothwell                    sfr@canb.auug.org.au
http://www.canb.auug.org.au/~sfr/

>From 4307d699fe862f594d27fc3e877de5628142aeb7 Mon Sep 17 00:00:00 2001
From: Stephen Rothwell <sfr@canb.auug.org.au>
Date: Wed, 28 May 2008 15:47:53 +1000
Subject: [PATCH] hil_kbd: restore pci_ids.h

This was removed by a semaphore patch (presumable by accident).

Signed-off-by: Stephen Rothwell <sfr@canb.auug.org.au>
---
 drivers/input/keyboard/hil_kbd.c |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/drivers/input/keyboard/hil_kbd.c b/drivers/input/keyboard/hil_kbd.c
index c0336ff..724f5b4 100644
--- a/drivers/input/keyboard/hil_kbd.c
+++ b/drivers/input/keyboard/hil_kbd.c
@@ -38,6 +38,7 @@
 #include <linux/module.h>
 #include <linux/init.h>
 #include <linux/slab.h>
+#include <linux/pci_ids.h>
 #include <linux/semaphore.h>
 
 #define PREFIX "HIL KEYB: "
-- 
1.5.5.1


  reply	other threads:[~2008-05-28  6:04 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-05-27 10:10 linux-next: m68k/semaphore build failure Stephen Rothwell
2008-05-27 16:10 ` Geert Uytterhoeven
2008-05-28  6:04   ` Stephen Rothwell [this message]
2008-05-31 17:20     ` Matthew Wilcox
2008-06-01  2:18       ` Stephen Rothwell

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20080528160406.4fceb036.sfr@canb.auug.org.au \
    --to=sfr@canb.auug.org.au \
    --cc=geert@linux-m68k.org \
    --cc=linux-next@vger.kernel.org \
    --cc=matthew@wil.cx \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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 NNTP newsgroup(s).