On Wed, Sep 04, 2013 at 04:11:59PM +0100, Lee Jones wrote: > > Cheers for producing a binding. > > > > On Wed, Sep 04, 2013 at 02:50:55PM +0100, Lee Jones wrote: > > > LPS001WP is a Pressure and Temperature sensor. > > > > > > Signed-off-by: Lee Jones <lee.jones@xxxxxxxxxx> > > > --- > > > Documentation/devicetree/bindings/i2c/trivial-devices.txt | 1 + > > > 1 file changed, 1 insertion(+) > > > > > > diff --git a/Documentation/devicetree/bindings/i2c/trivial-devices.txt b/Documentation/devicetree/bindings/i2c/trivial-devices.txt > > > index ad6a738..6038807e6 100644 > > > --- a/Documentation/devicetree/bindings/i2c/trivial-devices.txt > > > +++ b/Documentation/devicetree/bindings/i2c/trivial-devices.txt > > > @@ -57,6 +57,7 @@ st-micro,24c256 i2c serial eeprom (24cxx) > > > stm,m41t00 Serial Access TIMEKEEPER > > > stm,m41t62 Serial real-time clock (RTC) with alarm > > > stm,m41t80 M41T80 - SERIAL ACCESS RTC WITH ALARMS > > > +stm,lps001wp Pressure and Temperature sensor > > > > As I hinted at in the other thread [1], I don't think this can be > > documented as a trivial binding -- the driver expects a couple of > > regulators which should be described in the binding (or the driver > > shouldn't be using them...). > > Surely all devices need to be powered and would subsequently require a > regulator of one description or other? We don't really *need* to > manually enable it on my platform but that's because it's 'always > on', but we do anyway for completeness. They may or may not, that's a grey area. Regardless of whether you actually need it, the driver is currently looking for two undocumented entries in the DT. If you want to be able to use them, they must be documented so that people know they exist, what format they are expected to be, and what they actually represent. We should not allow for undocumented ABIs. > > Would a (regulator: vdd) note in this patch be suitable? It seems a > shame to have to produce a boilerplate binding entry for the sake of a > regulator. I thought this could have two regulators (vdd and vms)? I also note that the device can also be attached to SPI. Do we have any other devices which may be attached to either? Do we handle that, and if so, how (do we have the same compatible string for both interfaces?)? Thanks, Mark. -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html