From mboxrd@z Thu Jan 1 00:00:00 1970 From: Hans de Goede Date: Mon, 09 Apr 2007 12:34:35 +0000 Subject: Re: [lm-sensors] Proposal Message-Id: <461A32DB.3050209@hhs.nl> List-Id: References: <200704081630.17151.hjk@linutronix.de> In-Reply-To: <200704081630.17151.hjk@linutronix.de> MIME-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable To: lm-sensors@vger.kernel.org Jean Delvare wrote: > Hi Hans, >=20 > On Sun, 08 Apr 2007 19:36:47 +0200, Hans de Goede wrote: >> Hans-J=FCrgen Koch wrote: >>> Am Sonntag 08 April 2007 15:51 schrieb Hans de Goede: >>> >>>> The abituguru3 has a register which contains a motherboard ID. The cur= rent >>>> driver uses this ID, to look up info about the motherboard in a somewh= at >>>> lenght table in the driver.=20 >>> Can you elaborate your design decision a bit? My first idea would be to= have >>> a sysfs file that delivers that motherboard ID and then do the lookup in >>> user space. >=20 > I guess that the motherboard ID can be retrieved in user-space using > dmidecode, can't it? So it might not even be needed to export it. >=20 >> As I don't want the abituguru3 driver to create entries in sysfs for sen= sors=20 >> which aren't there, and as without the table in the driver I cannot be s= ure=20 >> wether to create an in / temp / fan device for a given sensor address. L= ast but=20 >> not least doing things this way allows me to always give a proper readin= g=20 >> without userspace needing to "guess" any further nescesarry calculations= to get=20 >> from the reading to an actual measurement. >=20 > This is absolutely not different from all other hardware monitoring > drivers. And all other drivers handle it in user-space, because that's > the right design. I see no valid reason why it would be different for > your abituguru3 driver. All you need is one configuration file per > motherboard. >=20 Oh and another thing I just came up with, last time I checked, it isn't=20 possible from userspace to change an inX_input to a tempX_input, or are you= =20 suggesting that I export all 48 sensors, 3 times, once as in, once as temp = and=20 once as fan and them make user space ignore temp1 and fan1, and use in0, as= =20 sensor 0/1 actually is an in sensor? Regards, Hans _______________________________________________ lm-sensors mailing list lm-sensors@lm-sensors.org http://lists.lm-sensors.org/mailman/listinfo/lm-sensors