Re: 2.6.10-mm1, class_simple_* and GPL addition

From: Norbert Preining
Date: Wed Oct 27 2004 - 12:22:18 EST


Hi all!

On Mit, 27 Okt 2004, Arjan van de Ven wrote:
> > for class_simple_* makes the nvidia module useless as it uses several:
>
> is this the module as downloaded from nvidia, or as hacked by some
> distro ?
>

On Mit, 27 Okt 2004, Greg KH wrote:
> I think these changes are only in the Gentoo modified version of the
> driver, right? I don't think that nvidia wrote the driver that way.

Yes they did. I downloaded the original NVIDIA-Linux-x86-1.0-6111-pkg1.run,
extracted the source and found in nv.c many things like:
#ifdef NV_CLASS_SIMPLE_PRESENT
class_simple_device_remove(MKDEV(NV_MAJOR_DEVICE_NUMBER, 255));
for (i = 0; i < num_nv_devices; i++)
class_simple_device_remove(MKDEV(NV_MAJOR_DEVICE_NUMBER, i));
class_simple_destroy(class_nvidia);
#endif

and in conftest.sh the check for class_simple_present by checking for
struct class_simple.

> > I don't want to start a flame war and long discussion, just want to ask
> > wether this change (to _GPL) was intended,
>
> Yes it was.

Ok. I can live with that.

> > and if yes, if there is a way to fix nvidia kernel modules (or others)
> > using this device management interface.
>
> Get them to change the license on their code.

Well, funny answer, but not useful for now.

Best wishes

Norbert

-------------------------------------------------------------------------------
Norbert Preining <preining AT logic DOT at> Technische Universität Wien
gpg DSA: 0x09C5B094 fp: 14DF 2E6C 0307 BE6D AD76 A9C0 D2BF 4AA3 09C5 B094
-------------------------------------------------------------------------------
BERY POMEROY
1. The shape of a gourmet's lips. 2. The droplet of saliva which hangs
from them.
--- Douglas Adams, The Meaning of Liff
-
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/