From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-7.0 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, INCLUDES_PATCH,MAILING_LIST_MULTI,SIGNED_OFF_BY,SPF_PASS autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id CAE3FC43441 for ; Mon, 12 Nov 2018 11:01:42 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 9EDE2223D8 for ; Mon, 12 Nov 2018 11:01:42 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 9EDE2223D8 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=denx.de Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1729328AbeKLUyY (ORCPT ); Mon, 12 Nov 2018 15:54:24 -0500 Received: from mail-out.m-online.net ([212.18.0.9]:41819 "EHLO mail-out.m-online.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1728431AbeKLUyY (ORCPT ); Mon, 12 Nov 2018 15:54:24 -0500 Received: from frontend01.mail.m-online.net (unknown [192.168.8.182]) by mail-out.m-online.net (Postfix) with ESMTP id 42tnrJ6ZjYz1qxkP; Mon, 12 Nov 2018 12:01:36 +0100 (CET) Received: from localhost (dynscan1.mnet-online.de [192.168.6.70]) by mail.m-online.net (Postfix) with ESMTP id 42tnrJ4jVqz1qtfB; Mon, 12 Nov 2018 12:01:36 +0100 (CET) X-Virus-Scanned: amavisd-new at mnet-online.de Received: from mail.mnet-online.de ([192.168.8.182]) by localhost (dynscan1.mail.m-online.net [192.168.6.70]) (amavisd-new, port 10024) with ESMTP id BQzW1qUgt6F8; Mon, 12 Nov 2018 12:01:33 +0100 (CET) X-Auth-Info: G678UU9TOq8Q408K3Ln/IjO/ECaTHSwt0H4zy9PbhvE= Received: from antares.denx.de (unknown [62.91.23.180]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.mnet-online.de (Postfix) with ESMTPSA; Mon, 12 Nov 2018 12:01:33 +0100 (CET) Cc: pn@denx.de, tglx@linutronix.de, jason@lakedaemon.net, marc.zyngier@arm.com, mark.rutland@arm.com, afaerber@suse.de, catalin.marinas@arm.com, will.deacon@arm.com, manivannan.sadhasivam@linaro.org, linux-kernel@vger.kernel.org, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, sravanhome@gmail.com, thomas.liau@actions-semi.com, mp-cs@actions-semi.com, linux@cubietech.com, edgar.righi@lsitec.org.br, laisa.costa@lsitec.org.br, guilherme.simoes@lsitec.org.br, mkzuffo@lsi.usp.br Subject: Re: [PATCH v2 1/3] dt-bindings: interrupt-controller: Actions external interrupt controller To: Rob Herring References: <20180812122215.1079590-1-pn@denx.de> <20180812122215.1079590-2-pn@denx.de> <20180813194453.GA30702@rob-hp-laptop> From: Parthiban Nallathambi Message-ID: <728e5f10-e09f-3fb9-022b-1e74731f774f@denx.de> Date: Mon, 12 Nov 2018 12:01:33 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.2.1 MIME-Version: 1.0 In-Reply-To: <20180813194453.GA30702@rob-hp-laptop> Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 8/13/18 9:44 PM, Rob Herring wrote: > On Sun, Aug 12, 2018 at 02:22:13PM +0200, Parthiban Nallathambi wrote: >> Actions Semi OWL family SoC's provides support for external interrupt >> controller to be connected and controlled using SIRQ pins. S500, S700 >> and S900 provides 3 SIRQ lines and works independently for 3 external >> interrupt controllers. >> >> Signed-off-by: Parthiban Nallathambi >> Signed-off-by: Saravanan Sekar >> --- >> .../interrupt-controller/actions,owl-sirq.txt | 46 ++++++++++++++++++++++ >> 1 file changed, 46 insertions(+) >> create mode 100644 Documentation/devicetree/bindings/interrupt-controller/actions,owl-sirq.txt >> >> diff --git a/Documentation/devicetree/bindings/interrupt-controller/actions,owl-sirq.txt b/Documentation/devicetree/bindings/interrupt-controller/actions,owl-sirq.txt >> new file mode 100644 >> index 000000000000..4b8437751331 >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/interrupt-controller/actions,owl-sirq.txt >> @@ -0,0 +1,46 @@ >> +Actions Semi Owl SoCs SIRQ interrupt controller >> + >> +S500, S700 and S900 SoC's from Actions provides 3 SPI's from GIC, >> +in which external interrupt controller can be connected. 3 SPI's >> +45, 46, 47 from GIC are directly exposed as SIRQ. It has >> +the following properties: >> + >> +- inputs three interrupt signal from external interrupt controller >> + >> +Required properties: >> + >> +- compatible: should be "actions,owl-sirq" >> +- reg: physical base address of the controller and length of memory mapped. >> +- interrupt-controller: identifies the node as an interrupt controller >> +- #interrupt-cells: specifies the number of cells needed to encode an interrupt >> + source, should be 2. > >> +- actions,sirq-shared-reg: Applicable for S500 and S700 where SIRQ register >> + details are maintained at same offset/register. >> +- actions,sirq-offset: register offset for SIRQ interrupts. When registers are >> + shared, all the three offsets will be same (S500 and S700). > > You should have more specific compatible strings if there are > differences and these settings can be implied by them. This to meant to get the register offset because s500, s700 uses the same external interrupt controller register to provide three or more interrupt line. But this is not the case for s900. So should it be "actions,sirq-offset-reg"? > >> +- actions,sirq-clk-sel: external interrupt controller can be either >> + connected to 32Khz or 24Mhz external/internal clock. This needs >> + to be configured for per SIRQ line. Failing defaults to 32Khz clock. > > What are the valid values? > >> + >> +Example for S900: >> + >> +sirq: interrupt-controller@e01b0000 { >> + compatible = "actions,owl-sirq"; >> + reg = <0 0xe01b0000 0 0x1000>; >> + interrupt-controller; >> + #interrupt-cells = <2>; >> + actions,sirq-clk-sel = <0 0 0>; > > If 0 is 32khz, then having this is pointless. But I can't tell what the > values correspond to. Thanks, clock selection will be removed and defaults to 24MHz. > >> + actions,sirq-offset = <0x200 0x528 0x52c>; >> +}; >> + >> +Example for S500 and S700: >> + >> +sirq: interrupt-controller@e01b0000 { >> + compatible = "actions,owl-sirq"; >> + reg = <0 0xe01b0000 0 0x1000>; >> + interrupt-controller; >> + #interrupt-cells = <2>; >> + actions,sirq-shared-reg; >> + actions,sirq-clk-sel = <0 0 0>; >> + actions,sirq-offset = <0x200 0x200 0x200>; >> +}; >> -- >> 2.14.4 >> > -- Thanks, Parthiban N DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-22 Fax: (+49)-8142-66989-80 Email: pn@denx.de