Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932984AbbD1CXH (ORCPT ); Mon, 27 Apr 2015 22:23:07 -0400 Received: from mail-wg0-f49.google.com ([74.125.82.49]:33383 "EHLO mail-wg0-f49.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932813AbbD1CXF (ORCPT ); Mon, 27 Apr 2015 22:23:05 -0400 MIME-Version: 1.0 In-Reply-To: <553EB0B3.5000707@linux.intel.com> References: <1430146908-27919-1-git-send-email-octavian.purdila@intel.com> <1430146908-27919-4-git-send-email-octavian.purdila@intel.com> <64714.10.254.87.235.1430149342.squirrel@linux.intel.com> <553EB0B3.5000707@linux.intel.com> Date: Tue, 28 Apr 2015 05:23:03 +0300 Message-ID: Subject: Re: [RFC PATCH 3/3] iio: derive the mounting matrix from ACPI _PLD objects From: Octavian Purdila To: sathyanarayanan.kuppuswamy@linux.intel.com Cc: Jonathan Cameron , Lars-Peter Clausen , Peter Meerwald , Robert Moore , Rafael J Wysocki , lenb@kernel.org, linux-api@vger.kernel.org, lkml , "linux-iio@vger.kernel.org" , "linux-acpi@vger.kernel.org" , Srinivas Pandruvada Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1977 Lines: 44 On Tue, Apr 28, 2015 at 12:57 AM, sathyanarayanan kuppuswamy wrote: > Hi > > On 04/27/2015 08:54 AM, Octavian Purdila wrote: >> >> On Mon, Apr 27, 2015 at 6:42 PM, Kuppuswamy Sathyanarayanan >> wrote: >>> >>> Since Acpi framework already exports this info to user space, Why not do >>> this derivation in user space code ? Why do we need new ABI, if the same >>> can be derived from existing one. >>> >> The ABI was added in the previous patch so that we can present the >> sensor orientation information to userspace even in the case of device >> tree. > > If the main reason for implementing a new ABI is to support DT platforms, > Why not implement a version of _PLD for device tree ? Don't you think it > would be much better than adding a new ABI to export redundant information ? > IMO the mounting matrix is more consistent with the IIO ABIs. Although I have no issue with repicating _PLD for device tree if people agree that it is better. > Also the location information of the device is not just specific to iio > drivers. You should consider that we would have similar requirements for > devices implemented as input or platform drivers. The upstream standard for those sensors where the orientation matters (accelerometer, gyro, compass) is IIO. Granted, there are other device types for which the orientation information may be useful (e.g. camera). However the actual interpretation and action to be taken is different for each subsystem (e.g. in the camera case do the correction via V4L2_CID_HFLIP / V4L2_CID_VFLIP) so I think it is better to expose it at the subsystem level in a way consistent with the subsystem's ABIs. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/