Messages in this thread | | | Date | Wed, 23 Sep 2009 09:26:07 +0200 | Subject | Re: [RFC] [PATCH 1/2] introduce ALS sysfs class | From | Corentin Chary <> |
| |
On Wed, Sep 23, 2009 at 9:18 AM, Pavel Machek <pavel@ucw.cz> wrote: > On Tue 2009-09-22 13:42:23, Jonathan Cameron wrote: >> Zhang Rui wrote: >> > Hi, Jonathan, >> > >> > this is the refresh ALS sysfs class driver. >> > I just introduced one sysfs attribute "illuminance", because >> > I didn't catch the exact meaning of the others like "???infrared". >> > So it would be great if you can generate an incremental patch >> > to introduce the other optional attributes needed, and update >> > the documentation as well. :) >> Will do, though may just leave it out of first pass of drivers >> (as it may be controversial and it would be nice to get something >> in place before the arguments begin!) >> >> All looks nice and clean. The only real question is whether >> we want to standardize naming of devices under sysfs (like hwmon does) >> or allow the individual drivers to do the naming? > > Allow the drivers to do the naming. Having useless name like "als0", > with als0/name telling me what the driver is is bad. > Pavel >
This is how rfkill works, is there a good reason for that ? Should we change that ?
-- Corentin Chary http://xf.iksaif.net -- 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/
| |