Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932176AbbGPOQN (ORCPT ); Thu, 16 Jul 2015 10:16:13 -0400 Received: from mail-pa0-f50.google.com ([209.85.220.50]:34035 "EHLO mail-pa0-f50.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755466AbbGPOQL (ORCPT ); Thu, 16 Jul 2015 10:16:11 -0400 Message-ID: <55A7BCA3.2010607@linaro.org> Date: Thu, 16 Jul 2015 19:46:03 +0530 From: Vaibhav Hiremath User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.7.0 MIME-Version: 1.0 To: Linus Walleij CC: "linux-arm-kernel@lists.infradead.org" , Rob Herring , Lee Jones , Will Deacon , Tony Lindgren , "devicetree@vger.kernel.org" , "linux-kernel@vger.kernel.org" , "linux-gpio@vger.kernel.org" Subject: Re: [RFC PATCH] pinctrl-single: Use of pinctrl-single for external device over I2C References: <1435855992-10312-1-git-send-email-vaibhav.hiremath@linaro.org> In-Reply-To: Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1294 Lines: 37 On Thursday 16 July 2015 06:28 PM, Linus Walleij wrote: > On Thu, Jul 2, 2015 at 6:53 PM, Vaibhav Hiremath > wrote: > >> In some usecases, the external device (in my case it PMIC over I2C) >> does support pin in multiple configuration, we may need to control/configure them >> during boot or runtime. > > What is missing from this patch is an explanation why pinctrl-single should > be used for this PMIC. > > Repeat: pinctrl-single: when you have exactly one register per pin, and > the description of the pins is delivered in an opaque ASIC-specific format > without developers knowing much about how the hardware actually works, > just "set these magic values". > > I strongly prefer that this PMIC has its own pin controller using standard > bindings with groups and functions, see eg. drivers/pinctrl/nomadik/pinctrl-ab* > for a PMIC pinctrl driver. > Thanks for the reference, good to know this. Let me check this driver before commenting anything further on this. Thanks, Vaibhav > Yours, > Linus Walleij > -- 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/