From: lawrence rust <lawrence@softsystem.co.uk>
To: Benjamin Herrenschmidt <benh@kernel.crashing.org>
Cc: linuxppc-dev@lists.ozlabs.org
Subject: Re: [PATCH] Add cmd64x IDE driver to default pmac32 config
Date: Thu, 08 Jul 2010 08:18:27 +0200 [thread overview]
Message-ID: <1278569907.1411.19.camel@gagarin> (raw)
In-Reply-To: <1278565221.28659.74.camel@pasglop>
On Thu, 2010-07-08 at 15:00 +1000, Benjamin Herrenschmidt wrote:
> On Sat, 2010-07-03 at 20:21 +0200, lawrence rust wrote:
> > The Blue/White Apple PowerMac G3 and early G4's use a cmd64x compatible
> > IDE disk controller. E.g. lspci shows...
> >
> > 01:01.0 IDE interface: Silicon Image, Inc. PCI0646 (rev 07)
> >
> > Unfortunately the default pmac32 configuration does not include this
> > driver and so PowerMac G3's can't load a root filesystem. This is an
> > issue on a least Ubuntu since version 9.04, which uses the default
> > config as a starting point.
> >
> > Signed-off-by: Lawrence Rust <lawrence at softsystem.co.uk>
>
> Shouldn't we just switch the whole thing to libata now anyways ?
Sure. it would be preferable but unfortunately the PowerMac on-board IDE
controller (CONFIG_BLK_DEV_IDE_PMAC), used for the DVD drive on B/W
G3's, doesn't have a PATA equivalent. So it's pragmatic (until the IDE
code is removed) to use the IDE cmd64x driver to minimise kernel code
size.
A minor correction to my previous post, the first version of Ubuntu to
suffer from this problem was 9.10.
-- Lawrence
>
> Cheers,
> Ben.
>
> > diff -uprN a/arch/powerpc/configs/pmac32_defconfig b/arch/powerpc/configs/pmac32_defconfig
> > --- a/arch/powerpc/configs/pmac32_defconfig 2010-05-16 23:17:36.000000000 +0200
> > +++ b/arch/powerpc/configs/pmac32_defconfig 2010-07-03 20:11:10.000000000 +0200
> > @@ -738,7 +738,7 @@ CONFIG_BLK_DEV_IDEDMA_PCI=y
> > # CONFIG_BLK_DEV_AEC62XX is not set
> > # CONFIG_BLK_DEV_ALI15X3 is not set
> > # CONFIG_BLK_DEV_AMD74XX is not set
> > -# CONFIG_BLK_DEV_CMD64X is not set
> > +CONFIG_BLK_DEV_CMD64X=y
> > # CONFIG_BLK_DEV_TRIFLEX is not set
> > # CONFIG_BLK_DEV_CS5520 is not set
> > # CONFIG_BLK_DEV_CS5530 is not set
> >
> >
> >
> >
> > _______________________________________________
> > Linuxppc-dev mailing list
> > Linuxppc-dev@lists.ozlabs.org
> > https://lists.ozlabs.org/listinfo/linuxppc-dev
>
>
--
-- Lawrence Rust
next prev parent reply other threads:[~2010-07-08 6:27 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-07-03 18:21 [PATCH] Add cmd64x IDE driver to default pmac32 config lawrence rust
2010-07-08 5:00 ` Benjamin Herrenschmidt
2010-07-08 6:18 ` lawrence rust [this message]
2010-07-08 6:30 ` Benjamin Herrenschmidt
2010-07-08 9:01 ` lawrence rust
2010-07-08 10:49 ` Benjamin Herrenschmidt
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=1278569907.1411.19.camel@gagarin \
--to=lawrence@softsystem.co.uk \
--cc=benh@kernel.crashing.org \
--cc=linuxppc-dev@lists.ozlabs.org \
/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.