All of lore.kernel.org
 help / color / mirror / Atom feed
From: Sergei Shtylyov <sshtylyov@ru.mvista.com>
To: Olaf Hering <olaf@aepfle.de>
Cc: Andrew Morton <akpm@osdl.org>,
	linuxppc-dev@ozlabs.org, Paul Mackeras <paulus@samba.org>,
	Jeff Garzik <jeff@garzik.org>,
	linux-ide@vger.kernel.org
Subject: Re: [PATCH] use winbond libata instead of ide driver for pseries CD drives
Date: Fri, 16 Feb 2007 17:51:06 +0300	[thread overview]
Message-ID: <45D5C4DA.2040609@ru.mvista.com> (raw)
In-Reply-To: <45D5B720.4010308@ru.mvista.com>

Hello, I wrote:

>>Change the default for the built-in IDE on p610/p615/p630 from
>>ide to libata. libata has better error handling and the drive can
>>recover when hald does its CD media polling.

>>Signed-off-by: Olaf Hering <olaf@aepfle.de>

>>Index: linux-2.6.20/arch/powerpc/configs/ppc64_defconfig
>>===================================================================
>>--- linux-2.6.20.orig/arch/powerpc/configs/ppc64_defconfig
>>+++ linux-2.6.20/arch/powerpc/configs/ppc64_defconfig
>>@@ -500,7 +500,7 @@ CONFIG_BLK_DEV_AMD74XX=y
>> # CONFIG_BLK_DEV_PDC202XX_NEW is not set
>> # CONFIG_BLK_DEV_SVWKS is not set
>> # CONFIG_BLK_DEV_SIIMAGE is not set
>>-CONFIG_BLK_DEV_SL82C105=y
>>+# CONFIG_BLK_DEV_SL82C105 is not set
>> # CONFIG_BLK_DEV_SLC90E66 is not set
>> # CONFIG_BLK_DEV_TRM290 is not set
>> # CONFIG_BLK_DEV_VIA82CXXX is not set
>>@@ -646,7 +646,7 @@ CONFIG_SATA_SVW=y
>> # CONFIG_PATA_SIL680 is not set
>> # CONFIG_PATA_SIS is not set
>> # CONFIG_PATA_VIA is not set
>>-# CONFIG_PATA_WINBOND is not set
>>+CONFIG_PATA_WINBOND=y
>> 
>> #
>> # Multi-device support (RAID and LVM)
>>Index: linux-2.6.20/arch/powerpc/configs/pseries_defconfig
>>===================================================================
>>--- linux-2.6.20.orig/arch/powerpc/configs/pseries_defconfig
>>+++ linux-2.6.20/arch/powerpc/configs/pseries_defconfig
>>@@ -483,7 +483,7 @@ CONFIG_BLK_DEV_AMD74XX=y
>> # CONFIG_BLK_DEV_PDC202XX_NEW is not set
>> # CONFIG_BLK_DEV_SVWKS is not set
>> # CONFIG_BLK_DEV_SIIMAGE is not set
>>-CONFIG_BLK_DEV_SL82C105=y
>>+# CONFIG_BLK_DEV_SL82C105 is not set
>> # CONFIG_BLK_DEV_SLC90E66 is not set
>> # CONFIG_BLK_DEV_TRM290 is not set
>> # CONFIG_BLK_DEV_VIA82CXXX is not set
>>@@ -628,7 +628,7 @@ CONFIG_ATA=y
>> # CONFIG_PATA_SIL680 is not set
>> # CONFIG_PATA_SIS is not set
>> # CONFIG_PATA_VIA is not set
>>-# CONFIG_PATA_WINBOND is not set
>>+CONFIG_PATA_WINBOND=y

>     I'm sorry, how those 2 drivers can be equivalent?! The pata_winbond driver 

    Sorry indeed -- I didn't realize PATA_WINBOND correscponds to 
pata_sl82c105 (somewhat confusing, isn't it?).

WBR, Sergei

  reply	other threads:[~2007-02-16 14:51 UTC|newest]

Thread overview: 29+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-02-10 20:35 [PATCH] move variables in drivers/macintosh to bss Olaf Hering
2007-02-10 20:36 ` [PATCH] add delay around sl82c105_reset_engine calls Olaf Hering
2007-02-10 20:37   ` [PATCH] cleanup variable usage in mesh interrupt handler Olaf Hering
2007-02-10 20:38     ` [PATCH] mark winbond IDE PCI resources with start 0 as unassigned Olaf Hering
2007-02-10 20:40       ` [PATCH] correct AC Power: in /proc/pmu/info on ibook1 Olaf Hering
2007-02-15 23:09   ` [PATCH] add delay around sl82c105_reset_engine calls Jeff Garzik
2007-02-16  9:20     ` [PATCH] use winbond libata instead of ide driver for pseries CD drives Olaf Hering
2007-02-16 13:52       ` Sergei Shtylyov
2007-02-16 13:52         ` Sergei Shtylyov
2007-02-16 14:51         ` Sergei Shtylyov [this message]
2007-02-16 14:53         ` Bartlomiej Zolnierkiewicz
2007-02-16 14:53           ` Bartlomiej Zolnierkiewicz
2007-02-16 14:55           ` Sergei Shtylyov
2007-02-16 14:55             ` Sergei Shtylyov
2007-02-16 15:09             ` Bartlomiej Zolnierkiewicz
2007-02-16 15:09               ` Bartlomiej Zolnierkiewicz
2007-02-16 15:06               ` Sergei Shtylyov
2007-02-16 15:06                 ` Sergei Shtylyov
2007-02-16 15:11                 ` Sergei Shtylyov
2007-02-16 15:11                   ` Sergei Shtylyov
2007-02-16 21:41             ` Benjamin Herrenschmidt
2007-02-16 14:46       ` Bartlomiej Zolnierkiewicz
2007-02-16 14:46         ` Bartlomiej Zolnierkiewicz
2007-02-16 14:51         ` Bartlomiej Zolnierkiewicz
2007-02-16 14:51           ` Bartlomiej Zolnierkiewicz
2007-02-16 17:37         ` Olaf Hering
2007-02-16 17:37           ` Olaf Hering
2007-02-17 20:36       ` Jeff Garzik
2007-02-17 20:36         ` Jeff Garzik

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=45D5C4DA.2040609@ru.mvista.com \
    --to=sshtylyov@ru.mvista.com \
    --cc=akpm@osdl.org \
    --cc=jeff@garzik.org \
    --cc=linux-ide@vger.kernel.org \
    --cc=linuxppc-dev@ozlabs.org \
    --cc=olaf@aepfle.de \
    --cc=paulus@samba.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.