Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751988AbdHAPwB (ORCPT ); Tue, 1 Aug 2017 11:52:01 -0400 Received: from mout02.posteo.de ([185.67.36.66]:51956 "EHLO mout02.posteo.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751607AbdHAPuS (ORCPT ); Tue, 1 Aug 2017 11:50:18 -0400 Subject: Re: [PATCH] iio: accel: Bugfix to enbale and allow different events to work parallely. To: Harinath Nampally Cc: Jonathan Cameron , knaack.h@gmx.de, lars@metafoo.de, Peter Meerwald-Stadler , Greg KH , linux-iio@vger.kernel.org, linux-kernel@vger.kernel.org, Alison Schofield References: <1501499858-4476-1-git-send-email-harinath922@gmail.com> <653B8888-D699-4B8B-93C2-6FF3308932AF@posteo.de> From: Martin Kepplinger Message-ID: Date: Tue, 1 Aug 2017 17:50:11 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.2.1 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1441 Lines: 30 On 2017-08-01 05:08, Harinath Nampally wrote: >> Thanks for doing that work. I have had it on my list for a long time >> and you seem to fix it. Although I'd happily review and possibly test >> it, unfortunately I can't do so before the week of August 21st. >> >> If this might go in quick, nothing will stop me from reviewing either, >> so, whatever. Thanks again! > Sure no problem, looking forward to your review comments. > Actually I am planning to add Orientation events for FXLS8471Q, for > that is it good idea to overload existing > IIO_ROT channel type? Also thinking of adding 4 channel modifiers i.e > portrait up/down, landscape left/right. > Any suggestions are welcome. Thank you. > My only suggestion for adding all these chips' orientation features, is to start the discussion independently from this driver. Are there other device series that provide such an orientation interrupt? Is it worth finding a representation in iio? Additionally to portait up/down, landscape left/right there is back/front facing, so you'd have 8 new channel modifiers. If IIO_ROT is a current userspace "standard" to read for rotating the screen, it may be worth discussing how to fit this in without new modifiers. Would you have to make up fake angle values? Anything else userspace already uses for getting the orientation? But again, instead of replying here and going off topic, write up a proposal and post it independently.