Current-Users archive

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

Re: adt7xxx drivers (Was Re: Device name length restriction?)

Hash: SHA1


On Sep 10, 2008, at 8:53 AM, Paul Goyette wrote:

On Wed, 10 Sep 2008, Michael Lorenz wrote:

I dimly remember that the ADM1030 doesn't have the chip/vendor ID
registers but at least a few registers were more or less identical to
the ADT7467. IIRC the main difference is less resolution in the fan
speed counter ( 8bit vs. 16bit or so )
So, many chips probably need the exact same conversion from register
value to whatever envsys wants.

The datasheet I grabbed yesterday for the ADM1030 indicates that it does indeed have Company_ID (0x3e), Device_ID (0x3d), and Revision (0x3f bits 0:3) registers.

You can tell it was a few years ago and I don't have the datasheet at hand right now ;) If you want the datasheets I used ( downloaded from Analog Devices when they still made the chips ) I'll mail them to you.

Anyone got any suggestions?  I'm not fond of "device tsfc" or
"device rtsfc".  :)

device dbcool?

That was my first thought, too, but many of the ADMxxxx are not specifically named "dBCOOL" by the vendor; also, dBCOOL is a registered trademark, and the trademark owner could require us to use it correctly, including proper capitalization. :)

True. adtxxxx? ;)

If you include all or most of the 'dBCool' family that seems to be
the least horrible name.

Yep, but as above, I'll also include non-dBCOOL(tm) family members.

Still, I think that's probably the best name, as long as there's no lawyerly objections.

I think there are examples in the code where drivers have names just as close to trademarks as this one, think radeonfb, voodoofb, unichromefb, xboxfb. On the other hand the companies who own the trademarks are probably not aware of our existence. We should probably take care to use the right capitalization in manpages as a matter of courtesy but IANAL.

have fun

Version: GnuPG v1.4.7 (Darwin)


Home | Main Index | Thread Index | Old Index