From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tushar Behera Subject: Re: [PATCH 1/5] ARM: dts: exynos5250-arndale: Add node entry for gpio-buttons Date: Thu, 07 Feb 2013 15:10:27 +0530 Message-ID: <5113768B.90102@linaro.org> References: <1360214129-4096-1-git-send-email-tushar.behera@linaro.org> <51134907.1050803@linaro.org> <5113501D.5060603@linaro.org> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: Sender: linux-samsung-soc-owner@vger.kernel.org To: Girish KS Cc: Manish Badarkhe , "kgene.kim@samsung.com" , "patches@linaro.org" , "devicetree-discuss@lists.ozlabs.org" , "linux-samsung-soc@vger.kernel.org" , "Kumar, Anil" , "linux-arm-kernel@lists.infradead.org" , david@protonic.nl, dtor@mail.ru List-Id: devicetree@vger.kernel.org +CC: David Jander +CC: Dmitry Torokhov On 02/07/2013 01:13 PM, Girish KS wrote: >>>>>> + >>>>>> + gpio_keys { >>>>>> + compatible = "gpio-keys"; >>>>>> + #address-cells = <1>; >>>>>> + #size-cells = <0>; >>>>> >>>>> Just want to understand why these properties are here? >>>>> As these properties are for child dt node. But have not seen >>>>> anyone is using here. >>>>> >>>> >>>> That is how gpio_keys node entries are defined in other .dts files. >>> >>> I have gone through example for address-cells and size-cells in following link: >>> http://devicetree.org/mediawiki/index.php?title=Device_Tree_Usage&stable=1#CPU_addressing >>> >>> which indicates that these fields are for child "reg". >>> I think, here in child node there is no "reg". so there is no use >>> of address-cells and size-cells propeties. >>> >> >> Please check Documentation/devicetree/bindings/gpio/gpio_keys.txt >> >> And whether these properties are required or not, I will let device tree >> experts to comment on that. >> >> As such, currently all node entries for gpio_keys use these properties. > you can just verify by a simple test. > delete the 2 lines. address-cells and size cells. > this will have no affect. > now with these 2 lines deleted lines add a dummy reg property with > address and cell. dtb compiler will warn. > So I think what manish reffered is right. > There is also a reference of smdk4210 for kepads. Right. Compilation is ok even after removing the address-cell and size-cell properties. But since this is used across all the instances, I would like to know the view of the authors on this. David, Dimitry, Can you please let us know your opinion on this? If these properties are not required, then we can remove them from the documentation and from node entries in several other dts files. >> >> -- >> Tushar Behera >> >> _______________________________________________ >> linux-arm-kernel mailing list >> linux-arm-kernel@lists.infradead.org >> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel -- Tushar Behera From mboxrd@z Thu Jan 1 00:00:00 1970 From: tushar.behera@linaro.org (Tushar Behera) Date: Thu, 07 Feb 2013 15:10:27 +0530 Subject: [PATCH 1/5] ARM: dts: exynos5250-arndale: Add node entry for gpio-buttons In-Reply-To: References: <1360214129-4096-1-git-send-email-tushar.behera@linaro.org> <51134907.1050803@linaro.org> <5113501D.5060603@linaro.org> Message-ID: <5113768B.90102@linaro.org> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org +CC: David Jander +CC: Dmitry Torokhov On 02/07/2013 01:13 PM, Girish KS wrote: >>>>>> + >>>>>> + gpio_keys { >>>>>> + compatible = "gpio-keys"; >>>>>> + #address-cells = <1>; >>>>>> + #size-cells = <0>; >>>>> >>>>> Just want to understand why these properties are here? >>>>> As these properties are for child dt node. But have not seen >>>>> anyone is using here. >>>>> >>>> >>>> That is how gpio_keys node entries are defined in other .dts files. >>> >>> I have gone through example for address-cells and size-cells in following link: >>> http://devicetree.org/mediawiki/index.php?title=Device_Tree_Usage&stable=1#CPU_addressing >>> >>> which indicates that these fields are for child "reg". >>> I think, here in child node there is no "reg". so there is no use >>> of address-cells and size-cells propeties. >>> >> >> Please check Documentation/devicetree/bindings/gpio/gpio_keys.txt >> >> And whether these properties are required or not, I will let device tree >> experts to comment on that. >> >> As such, currently all node entries for gpio_keys use these properties. > you can just verify by a simple test. > delete the 2 lines. address-cells and size cells. > this will have no affect. > now with these 2 lines deleted lines add a dummy reg property with > address and cell. dtb compiler will warn. > So I think what manish reffered is right. > There is also a reference of smdk4210 for kepads. Right. Compilation is ok even after removing the address-cell and size-cell properties. But since this is used across all the instances, I would like to know the view of the authors on this. David, Dimitry, Can you please let us know your opinion on this? If these properties are not required, then we can remove them from the documentation and from node entries in several other dts files. >> >> -- >> Tushar Behera >> >> _______________________________________________ >> linux-arm-kernel mailing list >> linux-arm-kernel at lists.infradead.org >> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel -- Tushar Behera