Re: Sata_via problems in a Vintage2-AE1

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Jean Delvare wrote:
I've read the whole thread, the source code (quickly) and the patches to ahci.c and sata_via.c, and here are some comments:

It looks like support for the VT8251 was added to the ahci driver in kernel 2.6.18, and was then updated in 2.6.20. The code is different from the patch Leopold is using with 2.6.16. As I am not an expert in this area, I can't tell whether both versions are equivalent or not, but I'd guess so.

The differences between 2.6.16 and 2.6.20 in ahci are mostly things like detection fixes and hotplug handling fixes which shouldn't affect normal operations too much and irqpoll definitely won't fix the problem.

On the other hand, I do not see VT8251 support in the sata_via driver in the kernel, so I'm not surprised that it doesn't work properly for Leopold. But again I am no expert in the area, so maybe the sata_via driver in the kernel is supposed to work for VT8251-based systems. Jeff (cc'd) should know better.

vt8251 is an ahci controller so sata_via is irrelevant.

I also see that there were many changes to these drivers post-2.6.20, so maybe Leopold could give a try to Linus' latest kernel (2.6.20-git6 as I speak) and see if there are any improvements.

As for the VIA IRQ quirks, the version we have in 2.6.20 does not handle the VIA chips more recent than the VT8237A. If the VT8251 needs it, then we have to add it. Alan?

*If* the VT8251 needs the VIA IRQ quirk, then the attached patch may help. Leopold, can you give it a try?

This is where I have no clue.  :-)  Leopold, please test Jean's patch.

Thanks a lot.

--
tejun
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]
  Powered by Linux