All of lore.kernel.org
 help / color / mirror / Atom feed
From: Bartlomiej Zolnierkiewicz <bzolnier@gmail.com>
To: Tony Breeds <tony@bakeyournoodle.com>
Cc: Bret Towe <magnade@gmail.com>,
	Linux Kernel Mailing List <linux-kernel@vger.kernel.org>
Subject: Re: linux-2.6.23-rc4 ppc build failure
Date: Wed, 5 Sep 2007 22:17:56 +0200	[thread overview]
Message-ID: <200709052217.56590.bzolnier@gmail.com> (raw)
In-Reply-To: <20070831033528.GN10345@bakeyournoodle.com>

On Friday 31 August 2007, Tony Breeds wrote:
> On Thu, Aug 30, 2007 at 02:56:50PM -0700, Bret Towe wrote:
> > On 8/29/07, Bret Towe <magnade@gmail.com> wrote:
> > > while trying to build a fresh kernel for my mini after upgrading from gutsy
> > > (and forgetting to save my .config) I hit the below build error
> > > 2.6.23-rc3 I did work under feisty but that is with a different .config
> > > so not sure if that makes any difference or not
> > > I've not bisected cause it takes so long on this computer...
> > > attached is the config I'm working off now
> 
> So how did you generate the new .config?  Did you start with a defconfig
> of some sort and then alter to personal taste?

The problem might be related with switching CONFIG_IDE to use menuconfig.

> I ask because if we have other users our there that have working system
> with BLK_DEV_IDE=m it'd be nice to have make oldconfig fix them.  If
> it's just a matter of not letting people get into that state then
> something like the patch below will probably help.

Unlikely that there are working BLK_DEV_IDE=m setups - these missing
symbols (pmac_ide_get_base and friends) have never been exported (IIRC).

> Ensure that BLK_DEV_IDE is built-in befoer allowing BLK_DEV_IDE_PMAC to
> be selected.
> 
> Signed-off-by: Tony Breeds <tony@bakeyournoodle.com>

applied with s/befoer/before/ as spotted by Bret

> ---
> 
>  drivers/ide/Kconfig |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/drivers/ide/Kconfig b/drivers/ide/Kconfig
> index e049f65..e55be26 100644
> --- a/drivers/ide/Kconfig
> +++ b/drivers/ide/Kconfig
> @@ -780,7 +780,7 @@ endif
>  
>  config BLK_DEV_IDE_PMAC
>  	bool "Builtin PowerMac IDE support"
> -	depends on PPC_PMAC && IDE=y
> +	depends on PPC_PMAC && IDE=y && BLK_DEV_IDE=y
>  	help
>  	  This driver provides support for the built-in IDE controller on
>  	  most of the recent Apple Power Macintoshes and PowerBooks.

      parent reply	other threads:[~2007-09-05 20:42 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-08-29 16:41 linux-2.6.23-rc4 ppc build failure Bret Towe
2007-08-29 17:00 ` Felipe Balbi
2007-08-29 19:32   ` Bret Towe
2007-08-30 10:41     ` Felipe Balbi
2007-08-30 21:56 ` Bret Towe
2007-08-31  3:35   ` Tony Breeds
2007-08-31  5:26     ` Bret Towe
2007-09-05 20:17     ` Bartlomiej Zolnierkiewicz [this message]

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=200709052217.56590.bzolnier@gmail.com \
    --to=bzolnier@gmail.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=magnade@gmail.com \
    --cc=tony@bakeyournoodle.com \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.