From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S967269AbcCQR6A (ORCPT ); Thu, 17 Mar 2016 13:58:00 -0400 Received: from hqemgate16.nvidia.com ([216.228.121.65]:17536 "EHLO hqemgate16.nvidia.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932954AbcCQR5z (ORCPT ); Thu, 17 Mar 2016 13:57:55 -0400 X-PGP-Universal: processed; by hqnvupgp08.nvidia.com on Thu, 17 Mar 2016 10:57:03 -0700 Message-ID: <56EAED01.4070107@nvidia.com> Date: Thu, 17 Mar 2016 23:14:33 +0530 From: Laxman Dewangan User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.4.0 MIME-Version: 1.0 To: Rob Herring CC: Markus Pargmann , Stephen Warren , , , , , , , , Benoit Parrot , Alexandre Courbot Subject: Re: [PATCH 4/5] gpio: of: Add support to have multiple gpios in gpio-hog References: <1457438528-29054-1-git-send-email-ldewangan@nvidia.com> <56E05AA9.8000503@wwwdotorg.org> <56E11D34.60401@nvidia.com> <2337414.ntEtcfKnX0@hermes> <56E16053.60802@nvidia.com> <20160317154656.GA9444@rob-hp-laptop> In-Reply-To: <20160317154656.GA9444@rob-hp-laptop> X-Originating-IP: [10.19.65.30] X-ClientProxiedBy: BGMAIL104.nvidia.com (10.25.59.13) To bgmail102.nvidia.com (10.25.59.11) Content-Type: text/plain; charset="ISO-8859-1"; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thursday 17 March 2016 09:16 PM, Rob Herring wrote: > On Thu, Mar 10, 2016 at 05:23:55PM +0530, Laxman Dewangan wrote: >>>> On this case, we have already property "line-name" and passed the name >>>> of the gpio via this property. >>>> The property names is "line-name" which is good for one string. We can >>>> support other property "line-names" with multiple string per GPIO index. >>>> >>>> line-names = "wlan-reset", "wlan-enable"; > Then what happens when someone wants to selectively disable gpio hogs? > > status = "okay", "disabled", "okay"; > > While I often push things to fewer nodes and more compact descriptions, > I don't think that is the right direction in this case. I dont think we need to support the individual gpio to be enable/disable by status. We need to support the status property at node level only. if individual gpio need to be enabled/disabled by status then it need to break in different hog nodes. This is same as like we have in pincontrol where we can provide the list of pin names for some configuration under same node. >>> There is currently a discussion about the future bindings for subnodes in GPIO >>> controller nodes. Please have a look at these two mail threads: >>> >>> "Device tree binding documentation for gpio-switch" >>> "gpio: of: Add support to have multiple gpios in gpio-hog" >> Second one is this patch only. Is it by intention? >> >> The binding details about the gpio-switch and names are given by property >> "lable". I think property "label" is standard way of going forward i.e. I >> post similar patch for gpio-keys device name from DT after got review >> comment. >> >> So here, we can have the gpio names under property "label" or "labels". > label is standard. labels you just made up. Yaah, lables for plural only. Otherwise no issue with "label". > >> Or am I missing anything? > The point is the more one off changes I see that are all inter-related, > the less willing I am to accept any that don't consider all the cases. > The inter-relationship here is how do we describe gpio lines that don't > otherwise have a connection to another node and how to deal with them if > that changes.