Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751302AbaJRMOW (ORCPT ); Sat, 18 Oct 2014 08:14:22 -0400 Received: from saturn.retrosnub.co.uk ([178.18.118.26]:51251 "EHLO saturn.retrosnub.co.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750915AbaJRMOU (ORCPT ); Sat, 18 Oct 2014 08:14:20 -0400 Message-ID: <54425998.7060808@kernel.org> Date: Sat, 18 Oct 2014 13:14:16 +0100 From: Jonathan Cameron User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.2.0 MIME-Version: 1.0 To: Lars-Peter Clausen , "Ivan T. Ivanov" CC: Sachin Kamat , Srinivas Pandruvada , John Stultz , linux-iio@vger.kernel.org, svarbanov@mm-sol.com, linux-kernel@vger.kernel.org, linux-arm-msm@vger.kernel.org Subject: Re: [PATCH] iio: inkern: Add of_xlate function to struct iio_dev References: <1412253128-32165-1-git-send-email-iivanov@mm-sol.com> <5442522D.40203@kernel.org> <544253F3.9060002@metafoo.de> In-Reply-To: <544253F3.9060002@metafoo.de> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 18/10/14 12:50, Lars-Peter Clausen wrote: > On 10/18/2014 01:42 PM, Jonathan Cameron wrote: >> On 02/10/14 13:32, Ivan T. Ivanov wrote: >>> When #iio-cells is greater than '0', the driver could provide >>> a custom of_xlate function that reads the *args* and returns >>> the appropriate index in registered IIO channels array. >>> >>> Add simple translation function, suitable for the most 1:1 >>> mapped channels in IIO chips, and use it when driver did not >>> provide custom implementation. >>> >>> Signed-off-by: Ivan T. Ivanov >> Any more comments on this? Been sat a while and the discussions seems >> to have died out. >> >> As Ivan has pointed out, very similar approaches are used >> elsewhere (gpio for example). > > Looks good to me: > > Reviewed-by: Lars-Peter Clausen > > When we initially added the DT support to IIO I was hoping that we can get away > with just using the simple and generic xlate function for all devices. But it > looks as if some more complex devices need to overwrite it. We should be careful > about adding new driver specific xlate implementations and make sure that it is > actually needed. > > One thing we might want to consider though is instead of adding the xlate > callback to the iio_dev struct add it to the iio_info struct since it should be > the same for different device instances of the same driver. And this is also > where all the other callbacks are. Good point - would definitely prefer that. J > > - Lars -- 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/