From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752259AbcAFTxm (ORCPT ); Wed, 6 Jan 2016 14:53:42 -0500 Received: from comal.ext.ti.com ([198.47.26.152]:60386 "EHLO comal.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751777AbcAFTxj (ORCPT ); Wed, 6 Jan 2016 14:53:39 -0500 Subject: Re: [PATCH 1/3] ARM: dts: omap5-board-common: enable rtc and charging of backup battery To: Rob Herring References: <568C5451.8070709@ti.com> <568CCCA2.4040004@nvidia.com> <568D2657.6050400@ti.com> CC: Laxman Dewangan , "H. Nikolaus Schaller" , =?UTF-8?Q?Beno=c3=aet_Cousson?= , Tony Lindgren , Pawel Moll , Mark Rutland , Ian Campbell , Kumar Gala , Russell King , linux-omap , "devicetree@vger.kernel.org" , LKML , Marek Belisko From: Nishanth Menon Message-ID: <568D70A3.2070007@ti.com> Date: Wed, 6 Jan 2016 13:53:07 -0600 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Icedove/38.5.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 01/06/2016 01:34 PM, Rob Herring wrote: > On Wed, Jan 6, 2016 at 8:36 AM, Nishanth Menon wrote: >> On 01/06/2016 02:13 AM, Laxman Dewangan wrote: >>> >>> On Wednesday 06 January 2016 01:12 PM, H. Nikolaus Schaller wrote: >>>> Hi, >>>> >>>> Am 06.01.2016 um 00:40 schrieb Nishanth Menon : >>>> >>>>> On 01/05/2016 06:01 AM, H. Nikolaus Schaller wrote: >>>>>> + rtc { >>>>>> + compatible = "ti,palmas-rtc"; >>>>>> + interrupt-parent = <&palmas>; >>>>>> + interrupts = <8 IRQ_TYPE_NONE>; >>>>> IRQ_TYPE_NONE is not correct here -> it should have some polarity - if >>>>> it had none, there'd be no interrupt, right? >>>> Well, it just translates IRQ_TYPE_NONE through >>>> >>>> Linux/include/dt-bindings/interrupt-controller/irq.h >>>> >>>> to >>>> >>>> interrupts = <8 0>; >>>> >>>> which is given as an example in >>>> >>>> Documentation//devicetree/bindings/rtc/rtc-palmas.txt >>>> >>>> Since I don't know anything about the rtc driver beyond the bindings >>>> documentation I assume it is correct... >>>> I have added Laxman Dewangan because he introduced this interrupts = >>>> <8 0>; >>>> >>> >>> As this is for palmas interrupt controller, it does not use the second >>> field for interrupt from RTC. >>> So there is no really any polarity. It can be set to 0. >>> >>> The second argument will be used for GPIOs mainly. However, support need >>> to be added on GPIO driver for rising/falling configuration. >> >> >> Device tree represents the hardware - not to reflect how the driver >> works. if the driver is wrong, fix it. the interrupt polarity needs to >> be described in DT. based on palmas like designs, you should probably >> use IRQ_TYPE_EDGE_FALLING because that is the default signaling inside >> the SoC as it reaches Secondary interrupt handlers(SIH) registers. > > If the trigger type is not programmable, then not setting the trigger > type in the DT is fine. Internal connections are often not documented. > Weird, that is not what I got feedback when I send https://patchwork.ozlabs.org/patch/381125/ If this is the new norm, I retract my objection. -- Regards, Nishanth Menon From mboxrd@z Thu Jan 1 00:00:00 1970 From: Nishanth Menon Subject: Re: [PATCH 1/3] ARM: dts: omap5-board-common: enable rtc and charging of backup battery Date: Wed, 6 Jan 2016 13:53:07 -0600 Message-ID: <568D70A3.2070007@ti.com> References: <568C5451.8070709@ti.com> <568CCCA2.4040004@nvidia.com> <568D2657.6050400@ti.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Rob Herring Cc: Laxman Dewangan , "H. Nikolaus Schaller" , =?UTF-8?Q?Beno=c3=aet_Cousson?= , Tony Lindgren , Pawel Moll , Mark Rutland , Ian Campbell , Kumar Gala , Russell King , linux-omap , "devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , LKML , Marek Belisko List-Id: devicetree@vger.kernel.org On 01/06/2016 01:34 PM, Rob Herring wrote: > On Wed, Jan 6, 2016 at 8:36 AM, Nishanth Menon wrote: >> On 01/06/2016 02:13 AM, Laxman Dewangan wrote: >>> >>> On Wednesday 06 January 2016 01:12 PM, H. Nikolaus Schaller wrote: >>>> Hi, >>>> >>>> Am 06.01.2016 um 00:40 schrieb Nishanth Menon : >>>> >>>>> On 01/05/2016 06:01 AM, H. Nikolaus Schaller wrote: >>>>>> + rtc { >>>>>> + compatible = "ti,palmas-rtc"; >>>>>> + interrupt-parent = <&palmas>; >>>>>> + interrupts = <8 IRQ_TYPE_NONE>; >>>>> IRQ_TYPE_NONE is not correct here -> it should have some polarity - if >>>>> it had none, there'd be no interrupt, right? >>>> Well, it just translates IRQ_TYPE_NONE through >>>> >>>> Linux/include/dt-bindings/interrupt-controller/irq.h >>>> >>>> to >>>> >>>> interrupts = <8 0>; >>>> >>>> which is given as an example in >>>> >>>> Documentation//devicetree/bindings/rtc/rtc-palmas.txt >>>> >>>> Since I don't know anything about the rtc driver beyond the bindings >>>> documentation I assume it is correct... >>>> I have added Laxman Dewangan because he introduced this interrupts = >>>> <8 0>; >>>> >>> >>> As this is for palmas interrupt controller, it does not use the second >>> field for interrupt from RTC. >>> So there is no really any polarity. It can be set to 0. >>> >>> The second argument will be used for GPIOs mainly. However, support need >>> to be added on GPIO driver for rising/falling configuration. >> >> >> Device tree represents the hardware - not to reflect how the driver >> works. if the driver is wrong, fix it. the interrupt polarity needs to >> be described in DT. based on palmas like designs, you should probably >> use IRQ_TYPE_EDGE_FALLING because that is the default signaling inside >> the SoC as it reaches Secondary interrupt handlers(SIH) registers. > > If the trigger type is not programmable, then not setting the trigger > type in the DT is fine. Internal connections are often not documented. > Weird, that is not what I got feedback when I send https://patchwork.ozlabs.org/patch/381125/ If this is the new norm, I retract my objection. -- Regards, Nishanth Menon -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html