From mboxrd@z Thu Jan 1 00:00:00 1970 From: Laxman Dewangan Subject: Re: [PATCH 6/7] pinctrl: tegra: Add DT binding for io pads control Date: Fri, 15 Apr 2016 23:19:45 +0530 Message-ID: <571129B9.7050602@nvidia.com> References: <1460473007-11535-1-git-send-email-ldewangan@nvidia.com> <1460473007-11535-7-git-send-email-ldewangan@nvidia.com> <5710F7A4.5070902@nvidia.com> <5710F6CA.6060700@nvidia.com> <57110560.80004@nvidia.com> <57110558.8010209@nvidia.com> <57110CA4.6050903@nvidia.com> <571119C6.6000107@nvidia.com> <5711288D.7060701@nvidia.com> Mime-Version: 1.0 Content-Type: text/plain; charset="windows-1252"; format=flowed Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <5711288D.7060701-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org> Sender: linux-tegra-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Jon Hunter , swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org, thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org, linus.walleij-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org, gnurou-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org, robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org, mark.rutland-5wv7dgnIgG8@public.gmane.org Cc: linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-gpio-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-Id: linux-tegra@vger.kernel.org On Friday 15 April 2016 11:14 PM, Jon Hunter wrote: > On 15/04/16 17:41, Laxman Dewangan wrote: >> On Friday 15 April 2016 09:15 PM, Jon Hunter wrote: >>> On 15/04/16 16:14, Laxman Dewangan wrote: >>>> I used pins as this is the property from pincon generic so that I can >>>> use the generic implementation. >>>> >>>> Here, I will not go to the pin level control as HW does not support pin >>>> level control. >>>> >>>> I will say the unit should be interface level. Should we say >>>> IO_GROUP_CSIA, IO_GROUP_CSIB etc? >>> So we need to reflect the hardware in device-tree and although yes the >>> power-down for the CSI_x_xxx pads are all controlled together as a >>> single group, it does not feel right that we add a pseudo pin called >>> csix to represent these. >>> >>> The CSI_x_xxx pads are already in device-tree and so why not add a >>> property to each of these pads which has the IO rail information for >>> power-down and voltage-select? >> Which dt binding docs have these? >> I looked for nvidia,tegra210-pinmux.txt and not able to find csi_xxx. > For CSI you are right they are not included by the current DT binding > docs, however, the sdmmc1/3 pads are. So that makes things a bit more > messy as some are and some are not. Yaah and so lets have the names in new dt files. Names may be same but define all possible names f groups in dt binding and need not to refer from other file which does not have all. >> Here I dont want to refer the individual pins as control should be as >> group. > I understand, however, at least for power-down control I don't see why > we cannot refer to the individual pins and once all are inactive then > the rail can be powered down. > > For switching the voltage it is a bit more complex, but may be we could > still look-up the IO rail based upon the pads the device uses. > Yes, it can be done with ref count also for power down. But interfaces are complex. As a client, it is easy to say power down SDMMC1 IO interface rather than saying power down 10 pins (names) of that group. We need to provide all these from DT for dynamic and static configuration and listing all pins of groups are complicate then the pad names. From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751259AbcDOSBJ (ORCPT ); Fri, 15 Apr 2016 14:01:09 -0400 Received: from hqemgate15.nvidia.com ([216.228.121.64]:4286 "EHLO hqemgate15.nvidia.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750776AbcDOSBH (ORCPT ); Fri, 15 Apr 2016 14:01:07 -0400 X-PGP-Universal: processed; by hqnvupgp07.nvidia.com on Fri, 15 Apr 2016 10:58:23 -0700 Message-ID: <571129B9.7050602@nvidia.com> Date: Fri, 15 Apr 2016 23:19:45 +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: Jon Hunter , , , , , , CC: , , , Subject: Re: [PATCH 6/7] pinctrl: tegra: Add DT binding for io pads control References: <1460473007-11535-1-git-send-email-ldewangan@nvidia.com> <1460473007-11535-7-git-send-email-ldewangan@nvidia.com> <5710F7A4.5070902@nvidia.com> <5710F6CA.6060700@nvidia.com> <57110560.80004@nvidia.com> <57110558.8010209@nvidia.com> <57110CA4.6050903@nvidia.com> <571119C6.6000107@nvidia.com> <5711288D.7060701@nvidia.com> In-Reply-To: <5711288D.7060701@nvidia.com> X-Originating-IP: [10.19.65.30] X-ClientProxiedBy: DRHKMAIL103.nvidia.com (10.25.59.17) To bgmail102.nvidia.com (10.25.59.11) Content-Type: text/plain; charset="windows-1252"; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Friday 15 April 2016 11:14 PM, Jon Hunter wrote: > On 15/04/16 17:41, Laxman Dewangan wrote: >> On Friday 15 April 2016 09:15 PM, Jon Hunter wrote: >>> On 15/04/16 16:14, Laxman Dewangan wrote: >>>> I used pins as this is the property from pincon generic so that I can >>>> use the generic implementation. >>>> >>>> Here, I will not go to the pin level control as HW does not support pin >>>> level control. >>>> >>>> I will say the unit should be interface level. Should we say >>>> IO_GROUP_CSIA, IO_GROUP_CSIB etc? >>> So we need to reflect the hardware in device-tree and although yes the >>> power-down for the CSI_x_xxx pads are all controlled together as a >>> single group, it does not feel right that we add a pseudo pin called >>> csix to represent these. >>> >>> The CSI_x_xxx pads are already in device-tree and so why not add a >>> property to each of these pads which has the IO rail information for >>> power-down and voltage-select? >> Which dt binding docs have these? >> I looked for nvidia,tegra210-pinmux.txt and not able to find csi_xxx. > For CSI you are right they are not included by the current DT binding > docs, however, the sdmmc1/3 pads are. So that makes things a bit more > messy as some are and some are not. Yaah and so lets have the names in new dt files. Names may be same but define all possible names f groups in dt binding and need not to refer from other file which does not have all. >> Here I dont want to refer the individual pins as control should be as >> group. > I understand, however, at least for power-down control I don't see why > we cannot refer to the individual pins and once all are inactive then > the rail can be powered down. > > For switching the voltage it is a bit more complex, but may be we could > still look-up the IO rail based upon the pads the device uses. > Yes, it can be done with ref count also for power down. But interfaces are complex. As a client, it is easy to say power down SDMMC1 IO interface rather than saying power down 10 pins (names) of that group. We need to provide all these from DT for dynamic and static configuration and listing all pins of groups are complicate then the pad names.