From mboxrd@z Thu Jan 1 00:00:00 1970 From: Mark Rutland Subject: Re: [RFC v2 PATCH 11/14] Documentation: Add st_magn binding documentation Date: Mon, 21 Oct 2013 12:16:32 +0100 Message-ID: <20131021111631.GB8144@kartoffel> References: <1380299538-22047-1-git-send-email-l.czerwinski@samsung.com> <1380299538-22047-12-git-send-email-l.czerwinski@samsung.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Return-path: Content-Disposition: inline In-Reply-To: <1380299538-22047-12-git-send-email-l.czerwinski-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org> Sender: linux-iio-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Lukasz Czerwinski Cc: "jic23-KWPb1pKIrIJaa/9Udqfwiw@public.gmane.org" , "denis.ciocca-qxv4g6HH51o@public.gmane.org" , "linux-iio-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , "lee.jones-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org" , "devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" List-Id: devicetree@vger.kernel.org On Fri, Sep 27, 2013 at 05:32:15PM +0100, Lukasz Czerwinski wrote: > This patch adds the document for STMicroeletronics Magnetic Sensors driver under > Documentation/devicetree/bindings/iio/. > > Signed-off-by: Lukasz Czerwinski > Signed-off-by: Kyungmin Park > --- > .../bindings/iio/magnetometer/st_magnometer.txt | 33 ++++++++++++++++++++ > 1 file changed, 33 insertions(+) > create mode 100644 Documentation/devicetree/bindings/iio/magnetometer/st_magnometer.txt > > diff --git a/Documentation/devicetree/bindings/iio/magnetometer/st_magnometer.txt b/Documentation/devicetree/bindings/iio/magnetometer/st_magnometer.txt > new file mode 100644 > index 0000000..fb4f473 > --- /dev/null > +++ b/Documentation/devicetree/bindings/iio/magnetometer/st_magnometer.txt > @@ -0,0 +1,33 @@ > +STMicroelectronics Magnetic Sensors > + > +Required properties: > + > + - compatible : value should be one of the following: s/should be/should contain/ -- we might have a future variant that's compatible. > + (a) "st,lsm303dlhc" for magnetometer in LSM330DLHC > + (b) "st,lsm303dlm" for magnetometer in LIS3DH > + (c) "st,lis3mdl" for magnetometer in LSM330 I'd drop the (a), (b), (c) here, it'll just make it more painful to add future variants. It's probably better to use "*" instead. > + > + - reg : the I2C address of the magnetometer > + > +Optional properties: > + > + - st,drdy-int-pin : redirect DRDY on pin INT1 (1) or pin INT2 (2) (u8) Could you elaborate on this? What does this property mena? What values are valid? > + > + - interrupts : Interrupt numbers for the ST accelerometers, as an array > + in case the magnetometer have more interrupt lines: > + , > + ; > + > + - interrupt-names : Array of strings associated with the interrupt numbers Nit: Interrupts are described by interrupt-specifiers. Please describe the names explicitly, otherwise there's no point having them at all, as no-one knows what they are... How about something like: - interrupts: a list of interrupt-specifiers, one for each entry in interrupt-names - interrupt names: a list of strings. Should contain * "drdy" for the ???? interrupt * "event" for the ???? interrupt > + > +Example: > + > +lis3mdl@1c { > + compatible = "st,lis3mdl"; > + reg = <0x1C>; > + > + st,drdy-int-pin = /bits/ 8 <1>; The use of /bits/ is remarkably uncommon, why do you need it here? Thanks, Mark. > + interrupt-parent = <&gpf0>; > + interrupts = <5 0>, <6 0>; > + interrupt-names = "drdy-int", "event-int"; > +}; > -- > 1.7.9.5 > > -- > To unsubscribe from this list: send the line "unsubscribe devicetree" in > the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org > More majordomo info at http://vger.kernel.org/majordomo-info.html >