> > - not clear how power_state and runtime pm interact - do we in fact need > > power state ? > > power_state turns chip on or off. In off state runtime pm is used to > handle bookkeeping and also handling chip state transitions including > regulators. Other ideas than separate power_state for this kind of > control? Ok that makes sense. > > > > - lux0_input range being fixed seems inconvenient, the sensors I've got > > queued here use lux0_input as you do but also provide a read (and > > optionally writable) range limit in lux > > > > static DEVICE_ATTR(lux0_sensor_range, S_IRUGO | S_IWUSR, > > als_sensing_range_show, als_sensing_range_store); > > > > and in your case you could just return 65535 > > > > I'll add that Cool. I'll bash our sensor queue to fit this API -- To unsubscribe from this list: send the line "unsubscribe linux-i2c" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html