From patchwork Mon Oct 27 07:44:22 2014 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Yingjoe Chen X-Patchwork-Id: 5157951 Return-Path: X-Original-To: patchwork-linux-arm@patchwork.kernel.org Delivered-To: patchwork-parsemail@patchwork1.web.kernel.org Received: from mail.kernel.org (mail.kernel.org [198.145.19.201]) by patchwork1.web.kernel.org (Postfix) with ESMTP id 6DDFE9F318 for ; Mon, 27 Oct 2014 07:46:56 +0000 (UTC) Received: from mail.kernel.org (localhost [127.0.0.1]) by mail.kernel.org (Postfix) with ESMTP id 8748B20211 for ; Mon, 27 Oct 2014 07:46:55 +0000 (UTC) Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.9]) (using TLSv1.2 with cipher DHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id B351C20176 for ; Mon, 27 Oct 2014 07:46:54 +0000 (UTC) Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.80.1 #2 (Red Hat Linux)) id 1Xiez2-0001Ay-0n; Mon, 27 Oct 2014 07:44:56 +0000 Received: from [210.61.82.183] (helo=mailgw01.mediatek.com) by bombadil.infradead.org with esmtps (Exim 4.80.1 #2 (Red Hat Linux)) id 1Xieyy-00015R-Js for linux-arm-kernel@lists.infradead.org; Mon, 27 Oct 2014 07:44:53 +0000 X-Listener-Flag: 11101 Received: from mtkhts09.mediatek.inc [(172.21.101.70)] by mailgw01.mediatek.com (envelope-from ) (mhqrelay.mediatek.com ESMTP with TLS) with ESMTP id 466245573; Mon, 27 Oct 2014 15:44:23 +0800 Received: from [172.21.77.4] (172.21.77.4) by mtkhts09.mediatek.inc (172.21.101.73) with Microsoft SMTP Server id 14.3.181.6; Mon, 27 Oct 2014 15:44:23 +0800 Subject: Re: [PATCH v4 0/7] ARM: mediatek: Add support for interrupt polarity From: Yingjoe Chen To: Matthias Brugger In-Reply-To: <1414163209.14830.614.camel@mtksdaap41> References: <1414079595-26129-1-git-send-email-yingjoe.chen@mediatek.com> <1414163209.14830.614.camel@mtksdaap41> Date: Mon, 27 Oct 2014 15:44:22 +0800 Message-ID: <1414395862.32399.9.camel@mtksdaap41> MIME-Version: 1.0 X-Mailer: Evolution 2.28.3 X-MTK: N X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20141027_004452_936105_1404DA06 X-CRM114-Status: GOOD ( 20.90 ) X-Spam-Score: 1.3 (+) Cc: Mark Rutland , Benjamin Herrenschmidt , Yingjoe Chen , Boris BREZILLON , Russell King , Arnd Bergmann , Yuhau Chen , arm@kernel.org, Nathan Chung , Grant Likely , "devicetree@vger.kernel.org" , Jason Cooper , Pawel Moll , Ian Campbell , Marc Zyngier , Hans de Goede , Rob Herring , Thomas Gleixner , huang eddie , "linux-arm-kernel@lists.infradead.org" , srv_heupstream , Hsien-Chun Yen , "linux-kernel@vger.kernel.org" , Kumar Gala , Olof Johansson , Jiang Liu X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.18-1 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+patchwork-linux-arm=patchwork.kernel.org@lists.infradead.org X-Spam-Status: No, score=-2.5 required=5.0 tests=BAYES_00, RCVD_IN_DNSWL_NONE, RP_MATCHES_RCVD, UNPARSEABLE_RELAY autolearn=unavailable version=3.3.1 X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on mail.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP Hi Matthias, On Fri, 2014-10-24 at 23:06 +0800, Yingjoe Chen wrote: > On Fri, 2014-10-24 at 16:59 +0200, Matthias Brugger wrote: > > Hi Joe, > > > > 2014-10-23 17:53 GMT+02:00 Yingjoe Chen : > > > This series is 4th version of interrupt polarity support for MediaTek SoCs. > > > This is based on Jiang's hierarchy irqdomain v2 [1] and my mediatek SoC > > > basic support [2]. > > > > > > This version addressed comments from previous discussion[3]. I removed > > > arm,hierarchy-irq-domain property. When GIC is probed by DT, it will > > > support hierarchy irqdomain. I still keep the first patch because this is > > > still base on hierarchy irqdomain v2. > > > > Is this set based on v3.17-rc1? > > Is it based on one of Jiangs trees [0]? > > > > I tried to use your patch set with v3.18-rc1 using Jiangs tree but the > > kernel crashed on mt6589 with a null-pointer exception [1]. A test > > with mt6592 didn't work either. > > > > It would be convenient if you rebase your patches to v3.18-rc1. > > It is based on Jiang's tree[0], which is based on v3.17-rc1. So yes, > this series is still on v3.17-rc1. > > All I need from Jiang's series are the first 2 patches, I'll bring them > to v3.18-rc1 to test next week. I cherry-pick/rebase my series with Jiang's 2 patches to v3.18-rc1, there are 2 conflicts but they are quite easy to fix so I won't repost this series now. I tested it on mt8127 with Eddie's uart patch and it boot to shell successfully. About crash on mt6589, I made a mistake in mt6589.dtsi. Please help to test with the following change: --- a/arch/arm/boot/dts/mt6589.dtsi +++ b/arch/arm/boot/dts/mt6589.dtsi @@ -86,7 +86,7 @@ interrupt-controller; #interrupt-cells = <3>; interrupt-parent = <&gic>; - reg = <0 0x10200100 0 0x1c>; + reg = <0x10200100 0x1c>; Joe.C