Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S934865AbbDQOuF (ORCPT ); Fri, 17 Apr 2015 10:50:05 -0400 Received: from mail-vn0-f53.google.com ([209.85.216.53]:41426 "EHLO mail-vn0-f53.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932603AbbDQNak (ORCPT ); Fri, 17 Apr 2015 09:30:40 -0400 MIME-Version: 1.0 In-Reply-To: <1427985376-20420-3-git-send-email-k.kozlowski@samsung.com> References: <1427985376-20420-1-git-send-email-k.kozlowski@samsung.com> <1427985376-20420-3-git-send-email-k.kozlowski@samsung.com> Date: Fri, 17 Apr 2015 22:30:38 +0900 X-Google-Sender-Auth: gvWJrrwc_-UYIvgtZG8ta8myo64 Message-ID: Subject: Re: [PATCH 2/2] ARM: dts: Fix pinctrl settings for S2MPS11 RTC alarm IRQ on Arndale Octa From: Krzysztof Kozlowski To: Kukjin Kim Cc: Sangbeom Kim , Krzysztof Kozlowski , Samuel Ortiz , Lee Jones , linux-kernel@vger.kernel.org, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-samsung-soc@vger.kernel.org, Javier Martinez Canillas Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2325 Lines: 63 2015-04-02 23:36 GMT+09:00 Krzysztof Kozlowski : > On Arndale Octa the S2MPS11 RTC alarm interrupt was not handled at all > because of wrong configuration of interrupt and gpx3-2. > 1. Interrupt is signaled by falling edge. > 2. This GPIO line is hard-wired on the board to PVDD_APIO_1V8 through a > resistor so pull-up/down must be disabled. > > Signed-off-by: Krzysztof Kozlowski > --- > arch/arm/boot/dts/exynos5420-arndale-octa.dts | 13 ++++++++++++- > 1 file changed, 12 insertions(+), 1 deletion(-) Dear Kukjin, Any comments on this and other patches. A lot of emails waits for your opinion. Is there anything I could do to help you in smooth review or applying? Best regards, Krzysztof > > diff --git a/arch/arm/boot/dts/exynos5420-arndale-octa.dts b/arch/arm/boot/dts/exynos5420-arndale-octa.dts > index d78fcd997ce6..97346df31d41 100644 > --- a/arch/arm/boot/dts/exynos5420-arndale-octa.dts > +++ b/arch/arm/boot/dts/exynos5420-arndale-octa.dts > @@ -87,7 +87,9 @@ > s2mps11,buck4-ramp-enable = <1>; > > interrupt-parent = <&gpx3>; > - interrupts = <2 IRQ_TYPE_LEVEL_HIGH>; > + interrupts = <2 IRQ_TYPE_EDGE_FALLING>; > + pinctrl-names = "default"; > + pinctrl-0 = <&s2mps11_irq>; > > s2mps11_osc: clocks { > #clock-cells = <1>; > @@ -379,3 +381,12 @@ > clocks = <&clock CLK_RTC>, <&s2mps11_osc S2MPS11_CLK_AP>; > clock-names = "rtc", "rtc_src"; > }; > + > +&pinctrl_0 { > + s2mps11_irq: s2mps11-irq { > + samsung,pins = "gpx3-2"; > + samsung,pin-function = <0xf>; > + samsung,pin-pud = <0>; > + samsung,pin-drv = <0>; > + }; > +}; > -- > 1.9.1 > > > _______________________________________________ > linux-arm-kernel mailing list > linux-arm-kernel@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel -- 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/