Re: SATA status report updated

[Jeff Garzik <jgarzik@pobox.com>]

Rainer Koenig wrote:
> Hi Simon,
>
> Simon Oosthoek <simon.oosthoek@ti-wmc.nl> 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@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/