Re: SATA status report updated

From: Jeff Garzik
Date: Fri Aug 19 2005 - 13:34:30 EST


Rainer Koenig wrote:
Hi Simon,

Simon Oosthoek <simon.oosthoek@xxxxxxxxx> writes:


I'm wondering how the support for the SIS 182 controller is doing, I
noticed they have a GPL driver on their website for kernel 2.6.10,
which is not a drop in replacement for sata_sis.c in 2.6.12.5, I
haven't tried compiling it as an add-on module outside the tree,
though...


I tried the sources from the SiS website (that seem to add more
details than my simple patch that just adds the device ID) as a drop
in for the Fedora installation kernel 2.6.11-1.1369_FC4, but the
kernel build process ran into an error at the sata_sis module. The
problem is that the source from SiS has a conditional code that
depends on the definition of a symbol "KERN_2_6_10" which is defined
by their "outside build makefile", but not in the standard kernel
build process. I added a #define KERN_2_6_10 to the source and then it
compiled also inside the kernel build process.


Adding the 0x182 identifier to the 180 driver does compile (duh!), but
I haven't tried it on hardware.


Working at a PC manufacturer I have access to hardware and I tried out
a lot and didn't run into any problem so far.


As a temporary measure, there was a patch posted to this list [1] a
while ago, would it be a good idea to include this while full support
is being worked on?


Seeing that the source from the SiS website is much more going into the
details than my simple adding of the device ID (of course SiS has hopefully
a much deeper knowledge of their hardware than I have ;-) I would rather
go for integrating the SiS source in the current kernel.

Yes, that's why I have resisted the "just add the PCI ID" patches that have cropped up.

SiS submitted patches that duplicated portions of libata inside their driver, rather than simply fixing libata as would be proper.

So we are stuck in the middle :(

Someone needs to work with the SiS submission until it's kosher with the upstream kernel, then everybody will be happy.

Jeff


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