* [PATCH] sata-via: enable magic transmission fix for vt6420
@ 2010-11-14 12:31 Peter Zijlstra
2010-11-14 12:37 ` Peter Zijlstra
` (2 more replies)
0 siblings, 3 replies; 5+ messages in thread
From: Peter Zijlstra @ 2010-11-14 12:31 UTC (permalink / raw)
To: Tejun Heo; +Cc: linux-kernel, Joseph Chan
I was able to reproduce and test the transmission error on vt6420
hardware with the 2G WD drives. Adding the controller ID to this fix
seems to result in a properly working system.
Signed-off-by: Peter Zijlstra <a.p.zijlstra@chello.nl>
---
drivers/ata/sata_via.c | 4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/ata/sata_via.c b/drivers/ata/sata_via.c
index 4730c42..2822492 100644
--- a/drivers/ata/sata_via.c
+++ b/drivers/ata/sata_via.c
@@ -577,7 +577,7 @@ static void svia_configure(struct pci_dev *pdev)
}
/*
- * vt6421 has problems talking to some drives. The following
+ * vt642[01] has problems talking to some drives. The following
* is the fix from Joseph Chan <JosephChan@via.com.tw>.
*
* When host issues HOLD, device may send up to 20DW of data
@@ -597,7 +597,7 @@ static void svia_configure(struct pci_dev *pdev)
* https://bugzilla.kernel.org/show_bug.cgi?id=15173
* http://article.gmane.org/gmane.linux.ide/46352
*/
- if (pdev->device == 0x3249) {
+ if (pdev->device == 0x3249 || pdev->device == 0x3149) {
pci_read_config_byte(pdev, 0x52, &tmp8);
tmp8 |= 1 << 2;
pci_write_config_byte(pdev, 0x52, tmp8);
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] sata-via: enable magic transmission fix for vt6420
2010-11-14 12:31 [PATCH] sata-via: enable magic transmission fix for vt6420 Peter Zijlstra
@ 2010-11-14 12:37 ` Peter Zijlstra
2010-11-15 4:02 ` Jeff Garzik
2010-11-15 8:28 ` Tejun Heo
2 siblings, 0 replies; 5+ messages in thread
From: Peter Zijlstra @ 2010-11-14 12:37 UTC (permalink / raw)
To: Tejun Heo; +Cc: linux-kernel, Joseph Chan
On Sun, 2010-11-14 at 13:31 +0100, Peter Zijlstra wrote:
> hardware with the 2G WD drives. Adding the controller ID to this fix
Clearly that was meant to say 2T drives :-)
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] sata-via: enable magic transmission fix for vt6420
2010-11-14 12:31 [PATCH] sata-via: enable magic transmission fix for vt6420 Peter Zijlstra
2010-11-14 12:37 ` Peter Zijlstra
@ 2010-11-15 4:02 ` Jeff Garzik
2010-11-15 9:10 ` Peter Zijlstra
2010-11-15 8:28 ` Tejun Heo
2 siblings, 1 reply; 5+ messages in thread
From: Jeff Garzik @ 2010-11-15 4:02 UTC (permalink / raw)
To: Peter Zijlstra; +Cc: Tejun Heo, linux-kernel, Joseph Chan
On 11/14/2010 07:31 AM, Peter Zijlstra wrote:
> I was able to reproduce and test the transmission error on vt6420
> hardware with the 2G WD drives. Adding the controller ID to this fix
> seems to result in a properly working system.
>
> Signed-off-by: Peter Zijlstra<a.p.zijlstra@chello.nl>
Please CC linux-ide and myself on all libata patches, as MAINTAINERS and
drivers/ata/sata_via.c both indicate.
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] sata-via: enable magic transmission fix for vt6420
2010-11-14 12:31 [PATCH] sata-via: enable magic transmission fix for vt6420 Peter Zijlstra
2010-11-14 12:37 ` Peter Zijlstra
2010-11-15 4:02 ` Jeff Garzik
@ 2010-11-15 8:28 ` Tejun Heo
2 siblings, 0 replies; 5+ messages in thread
From: Tejun Heo @ 2010-11-15 8:28 UTC (permalink / raw)
To: Peter Zijlstra; +Cc: linux-kernel, Joseph Chan, linux-ide, Jeff Garzik
Hello, Peter.
On 11/14/2010 01:31 PM, Peter Zijlstra wrote:
> I was able to reproduce and test the transmission error on vt6420
> hardware with the 2G WD drives. Adding the controller ID to this fix
> seems to result in a properly working system.
Yeah, there was another report on bugzilla and I contacted Joseph a
few days ago for confirmation. There are multiple PCI IDs for vt6420
and I think we'll probably have to apply the workaround to all of
them. Joseph, can you please confirm this?
Thanks.
--
tejun
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] sata-via: enable magic transmission fix for vt6420
2010-11-15 4:02 ` Jeff Garzik
@ 2010-11-15 9:10 ` Peter Zijlstra
0 siblings, 0 replies; 5+ messages in thread
From: Peter Zijlstra @ 2010-11-15 9:10 UTC (permalink / raw)
To: Jeff Garzik; +Cc: Tejun Heo, linux-kernel, Joseph Chan
On Sun, 2010-11-14 at 23:02 -0500, Jeff Garzik wrote:
> On 11/14/2010 07:31 AM, Peter Zijlstra wrote:
> > I was able to reproduce and test the transmission error on vt6420
> > hardware with the 2G WD drives. Adding the controller ID to this fix
> > seems to result in a properly working system.
> >
> > Signed-off-by: Peter Zijlstra<a.p.zijlstra@chello.nl>
>
> Please CC linux-ide and myself on all libata patches, as MAINTAINERS and
> drivers/ata/sata_via.c both indicate.
Oh, I thought I had CC'ed you, you're in the signoff trail of the
initial patch, I must have fudged it. Sorry about that.
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2010-11-15 9:09 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2010-11-14 12:31 [PATCH] sata-via: enable magic transmission fix for vt6420 Peter Zijlstra
2010-11-14 12:37 ` Peter Zijlstra
2010-11-15 4:02 ` Jeff Garzik
2010-11-15 9:10 ` Peter Zijlstra
2010-11-15 8:28 ` Tejun Heo
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).