On Thu, Mar 12, 2015 at 03:20:33PM +0200, Kalle Valo wrote: > Rajkumar Manoharan <rmanohar@xxxxxxxxxxxxxxxx> writes: > > > On Tue, Mar 10, 2015 at 05:39:46PM +0200, Kalle Valo wrote: > >> Rajkumar Manoharan <rmanohar@xxxxxxxxxxxxxxxx> writes: > >> > >> > Add symbloic link "thermal_sensors" for hwmon devices for better > >> > readability. > >> > > >> > /sys/class/ieee80211/phyX/device/thermal_sensors -> > >> > sys/class/ieee80211/phyX/device/hwmon/hwmonY > >> > >> I don't get it, how does that improve readability? Also I don't see any > >> other driver doing this, so isn't this just some private hack and not a > >> proper generic interface? > > > > The patch is providing same level access to monitoring device similar to > > cooling device. > > > > cooling_device -> ../../../../../virtual/thermal/cooling_deviceX > > thermal_sensors -> hwmon/hwmonX > > Is this documented somewhere? We cannot just start inventing sysfs > interfaces on our own. > Agree. this is not a standard interface. I added symlink just to provide user readable name instead of hwmon. Please drop this change, if it does not add any values to existing interface. -Rajkumar -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html