From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jonathan Cameron Subject: Re: [PATCH] iio: document bindings for mounting matrixes Date: Mon, 15 Aug 2016 15:58:48 +0100 Message-ID: References: <1469355434-17043-1-git-send-email-linus.walleij@linaro.org> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: Sender: linux-iio-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Linus Walleij , "linux-iio-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" Cc: Gregor Boirie , Sebastian Reichel , Samu Onkalo , "devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" List-Id: devicetree@vger.kernel.org On 11/08/16 12:33, Linus Walleij wrote: > On Sun, Jul 24, 2016 at 12:17 PM, Linus Walleij > wrote: > >> The mounting matrix for sensors was introduced in >> commit dfc57732ad38 ("iio:core: mounting matrix support") >> >> However the device tree bindings are very terse and since this is >> a widely applicable property, we need a proper binding for it >> that the other bindings can reference. This will also be useful >> for other operating systems and sensor engineering at large. >> >> I think all 3D sensors should support it, the current situation >> is probably that the mounting information is confined in magic >> userspace components rather than using the mounting matrix, which >> is not good for portability and reuse. >> >> Cc: Gregor Boirie >> Cc: Sebastian Reichel >> Cc: Samu Onkalo >> Cc: devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org >> Signed-off-by: Linus Walleij > > AFAICT there is no comments on this patch per se, just chit chat > about misc, and we have Rob's ACK. > > Can it be merged? > I failed to actually look at it properly so far... I'd also like input from Gregor as his usecase is somewhat different and drove the sysfs docs. They need to agree and I haven't actually verified they do as yet! Also for trivial comments, plural of matrix is matrices. Isn't English weird sometimes ;) Jonathan > Yours, > Linus Walleij > From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from saturn.retrosnub.co.uk ([178.18.118.26]:41339 "EHLO saturn.retrosnub.co.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752488AbcHOO6v (ORCPT ); Mon, 15 Aug 2016 10:58:51 -0400 Subject: Re: [PATCH] iio: document bindings for mounting matrixes To: Linus Walleij , "linux-iio@vger.kernel.org" References: <1469355434-17043-1-git-send-email-linus.walleij@linaro.org> Cc: Gregor Boirie , Sebastian Reichel , Samu Onkalo , "devicetree@vger.kernel.org" From: Jonathan Cameron Message-ID: Date: Mon, 15 Aug 2016 15:58:48 +0100 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8 Sender: linux-iio-owner@vger.kernel.org List-Id: linux-iio@vger.kernel.org On 11/08/16 12:33, Linus Walleij wrote: > On Sun, Jul 24, 2016 at 12:17 PM, Linus Walleij > wrote: > >> The mounting matrix for sensors was introduced in >> commit dfc57732ad38 ("iio:core: mounting matrix support") >> >> However the device tree bindings are very terse and since this is >> a widely applicable property, we need a proper binding for it >> that the other bindings can reference. This will also be useful >> for other operating systems and sensor engineering at large. >> >> I think all 3D sensors should support it, the current situation >> is probably that the mounting information is confined in magic >> userspace components rather than using the mounting matrix, which >> is not good for portability and reuse. >> >> Cc: Gregor Boirie >> Cc: Sebastian Reichel >> Cc: Samu Onkalo >> Cc: devicetree@vger.kernel.org >> Signed-off-by: Linus Walleij > > AFAICT there is no comments on this patch per se, just chit chat > about misc, and we have Rob's ACK. > > Can it be merged? > I failed to actually look at it properly so far... I'd also like input from Gregor as his usecase is somewhat different and drove the sysfs docs. They need to agree and I haven't actually verified they do as yet! Also for trivial comments, plural of matrix is matrices. Isn't English weird sometimes ;) Jonathan > Yours, > Linus Walleij >