public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH] ata_piix: add Asus Eee 701 controller to short cable list
@ 2008-04-21  3:03 Dan McGee
  2008-04-23  2:09 ` Tejun Heo
                   ` (2 more replies)
  0 siblings, 3 replies; 4+ messages in thread
From: Dan McGee @ 2008-04-21  3:03 UTC (permalink / raw)
  To: htejun; +Cc: linux-kernel, Dan McGee

The drive is directly soldered to the controller, so there is no cable at
all. Remove the 40-wire assumption so the drive can operate at max speed.

Before patch:
$ dd if=/dev/sda of=/dev/null bs=2M count=64 iflag=direct
134217728 bytes (134 MB) copied, 5.29612 s, 25.3 MB/s

After patch:
$ dd if=/dev/sda of=/dev/null bs=2M count=64 iflag=direct
134217728 bytes (134 MB) copied, 3.94955 s, 34.0 MB/s

Signed-off-by: Dan McGee <dpmcgee@gmail.com>
---
 drivers/ata/ata_piix.c |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/drivers/ata/ata_piix.c b/drivers/ata/ata_piix.c
index b7c38ee..ea2c764 100644
--- a/drivers/ata/ata_piix.c
+++ b/drivers/ata/ata_piix.c
@@ -573,6 +573,7 @@ static const struct ich_laptop ich_laptop[] = {
 	{ 0x27DF, 0x1043, 0x1267 },	/* ICH7 on Asus W5F */
 	{ 0x27DF, 0x103C, 0x30A1 },	/* ICH7 on HP Compaq nc2400 */
 	{ 0x24CA, 0x1025, 0x0061 },	/* ICH4 on ACER Aspire 2023WLMi */
+	{ 0x2653, 0x1043, 0x82D8 },	/* ICH6M on Asus Eee 701 */
 	/* end marker */
 	{ 0, }
 };
-- 
1.5.5.1


^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH] ata_piix: add Asus Eee 701 controller to short cable list
  2008-04-21  3:03 [PATCH] ata_piix: add Asus Eee 701 controller to short cable list Dan McGee
@ 2008-04-23  2:09 ` Tejun Heo
  2008-04-24 14:48 ` Alan Cox
  2008-04-25  5:15 ` Jeff Garzik
  2 siblings, 0 replies; 4+ messages in thread
From: Tejun Heo @ 2008-04-23  2:09 UTC (permalink / raw)
  To: Dan McGee; +Cc: linux-kernel

Dan McGee wrote:
> The drive is directly soldered to the controller, so there is no cable at
> all. Remove the 40-wire assumption so the drive can operate at max speed.
> 
> Before patch:
> $ dd if=/dev/sda of=/dev/null bs=2M count=64 iflag=direct
> 134217728 bytes (134 MB) copied, 5.29612 s, 25.3 MB/s
> 
> After patch:
> $ dd if=/dev/sda of=/dev/null bs=2M count=64 iflag=direct
> 134217728 bytes (134 MB) copied, 3.94955 s, 34.0 MB/s
> 
> Signed-off-by: Dan McGee <dpmcgee@gmail.com>

Acked-by: Tejun Heo <htejun@gmail.com>

-- 
tejun

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] ata_piix: add Asus Eee 701 controller to short cable list
  2008-04-21  3:03 [PATCH] ata_piix: add Asus Eee 701 controller to short cable list Dan McGee
  2008-04-23  2:09 ` Tejun Heo
@ 2008-04-24 14:48 ` Alan Cox
  2008-04-25  5:15 ` Jeff Garzik
  2 siblings, 0 replies; 4+ messages in thread
From: Alan Cox @ 2008-04-24 14:48 UTC (permalink / raw)
  To: Dan McGee; +Cc: htejun, jeff, linux-kernel, Dan McGee

On Sun, 20 Apr 2008 22:03:27 -0500
Dan McGee <dpmcgee@gmail.com> wrote:

> The drive is directly soldered to the controller, so there is no cable at
> all. Remove the 40-wire assumption so the drive can operate at max speed.
> 
> Before patch:
> $ dd if=/dev/sda of=/dev/null bs=2M count=64 iflag=direct
> 134217728 bytes (134 MB) copied, 5.29612 s, 25.3 MB/s
> 
> After patch:
> $ dd if=/dev/sda of=/dev/null bs=2M count=64 iflag=direct
> 134217728 bytes (134 MB) copied, 3.94955 s, 34.0 MB/s
> 
> Signed-off-by: Dan McGee <dpmcgee@gmail.com>

Acked-by: Alan Cox <alan@redhat.com>

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] ata_piix: add Asus Eee 701 controller to short cable list
  2008-04-21  3:03 [PATCH] ata_piix: add Asus Eee 701 controller to short cable list Dan McGee
  2008-04-23  2:09 ` Tejun Heo
  2008-04-24 14:48 ` Alan Cox
@ 2008-04-25  5:15 ` Jeff Garzik
  2 siblings, 0 replies; 4+ messages in thread
From: Jeff Garzik @ 2008-04-25  5:15 UTC (permalink / raw)
  To: Dan McGee; +Cc: htejun, linux-kernel

Dan McGee wrote:
> The drive is directly soldered to the controller, so there is no cable at
> all. Remove the 40-wire assumption so the drive can operate at max speed.
> 
> Before patch:
> $ dd if=/dev/sda of=/dev/null bs=2M count=64 iflag=direct
> 134217728 bytes (134 MB) copied, 5.29612 s, 25.3 MB/s
> 
> After patch:
> $ dd if=/dev/sda of=/dev/null bs=2M count=64 iflag=direct
> 134217728 bytes (134 MB) copied, 3.94955 s, 34.0 MB/s
> 
> Signed-off-by: Dan McGee <dpmcgee@gmail.com>
> ---
>  drivers/ata/ata_piix.c |    1 +
>  1 files changed, 1 insertions(+), 0 deletions(-)
> 
> diff --git a/drivers/ata/ata_piix.c b/drivers/ata/ata_piix.c
> index b7c38ee..ea2c764 100644
> --- a/drivers/ata/ata_piix.c
> +++ b/drivers/ata/ata_piix.c
> @@ -573,6 +573,7 @@ static const struct ich_laptop ich_laptop[] = {
>  	{ 0x27DF, 0x1043, 0x1267 },	/* ICH7 on Asus W5F */
>  	{ 0x27DF, 0x103C, 0x30A1 },	/* ICH7 on HP Compaq nc2400 */
>  	{ 0x24CA, 0x1025, 0x0061 },	/* ICH4 on ACER Aspire 2023WLMi */
> +	{ 0x2653, 0x1043, 0x82D8 },	/* ICH6M on Asus Eee 701 */

applied



^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2008-04-25  5:15 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2008-04-21  3:03 [PATCH] ata_piix: add Asus Eee 701 controller to short cable list Dan McGee
2008-04-23  2:09 ` Tejun Heo
2008-04-24 14:48 ` Alan Cox
2008-04-25  5:15 ` Jeff Garzik

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox