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=-6.9 required=3.0 tests=DKIM_SIGNED,DKIM_VALID, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI,SIGNED_OFF_BY, SPF_PASS,T_DKIMWL_WL_HIGH,UNPARSEABLE_RELAY,URIBL_BLOCKED autolearn=unavailable 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 42ECAC04A6B for ; Fri, 10 May 2019 03:31:42 +0000 (UTC) Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 143E72084A for ; Fri, 10 May 2019 03:31:42 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="qBz9+e0S" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 143E72084A Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=mediatek.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20170209; h=Sender: Content-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:References:In-Reply-To: Date:To:From:Subject:Message-ID:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=Sy+0QebSz05nF0MgcMMpe09mTYzOdkFXEirECt8jgrA=; b=qBz9+e0STAK/N8 EHOwSJoFAgvUL3eJS/Rom+T3T6X+34w27k7YRN3XSLbPSCoYb+wTp1cSXpfv5CR9+aGnd4/3sVwLb tHCFIlbKhCcMPQp6zRSAVos1cDg5pVzZes7bvGHvnbQsLjtbDfyt1xIpuWVsGOh+0VJxy1+Pn2l+m dKaD2lhsDGtv08k9wNeSYVOlKqzC50eGobw43MN9Uk7LV9ng2xOsxp43aoVGN296AKyzAly4zhn9O R4rSotWjGqzVLDbbp+xiXndBY+nWtZuiBGHQKpg09Zwe/mPlP+e8l8R6ALXh4lyoofiq/u6hTmrap W2Tu18xJzQEwweDob3xw==; Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.90_1 #2 (Red Hat Linux)) id 1hOwFz-00056f-Rk; Fri, 10 May 2019 03:31:35 +0000 Received: from mailgw02.mediatek.com ([216.200.240.185]) by bombadil.infradead.org with esmtps (Exim 4.90_1 #2 (Red Hat Linux)) id 1hOwFw-00055L-4L; Fri, 10 May 2019 03:31:33 +0000 X-UUID: 4659345febfb49c1a195b59bf684c97a-20190509 X-UUID: 4659345febfb49c1a195b59bf684c97a-20190509 Received: from mtkcas68.mediatek.inc [(172.29.94.19)] by mailgw02.mediatek.com (envelope-from ) (musrelay.mediatek.com ESMTP with TLS) with ESMTP id 1727774247; Thu, 09 May 2019 19:31:30 -0800 Received: from MTKMBS01N2.mediatek.inc (172.21.101.79) by MTKMBS62DR.mediatek.inc (172.29.94.18) with Microsoft SMTP Server (TLS) id 15.0.1395.4; Thu, 9 May 2019 20:31:28 -0700 Received: from mtkcas07.mediatek.inc (172.21.101.84) by mtkmbs01n2.mediatek.inc (172.21.101.79) with Microsoft SMTP Server (TLS) id 15.0.1395.4; Fri, 10 May 2019 11:31:21 +0800 Received: from [172.21.77.4] (172.21.77.4) by mtkcas07.mediatek.inc (172.21.101.73) with Microsoft SMTP Server id 15.0.1395.4 via Frontend Transport; Fri, 10 May 2019 11:31:21 +0800 Message-ID: <1557459081.29102.4.camel@mtksdaap41> Subject: Re: [PATCH v5 03/12] dt-binding: gce: add binding for gce subsys property From: Bibby Hsieh To: Rob Herring Date: Fri, 10 May 2019 11:31:21 +0800 In-Reply-To: <20190507174110.GA6767@bogus> References: <20190507081355.52630-1-bibby.hsieh@mediatek.com> <20190507081355.52630-4-bibby.hsieh@mediatek.com> <20190507174110.GA6767@bogus> X-Mailer: Evolution 3.10.4-0ubuntu2 MIME-Version: 1.0 X-TM-SNTS-SMTP: 2CE4E89CBFF18FFD220CA801469A51525C75B7E68B1C0A4BF46A2CE75B1E01422000:8 X-MTK: N X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20190509_203132_178833_553A5E25 X-CRM114-Status: GOOD ( 20.56 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: devicetree@vger.kernel.org, Nicolas Boichat , Philipp Zabel , srv_heupstream@mediatek.com, kendrick.hsu@mediatek.com, Daoyuan Huang , Sascha Hauer , Jassi Brar , linux-kernel@vger.kernel.org, Daniel Kurtz , YT Shen , CK HU , Dennis-YC Hsieh , linux-mediatek@lists.infradead.org, Houlong Wei , Sascha Hauer , Matthias Brugger , Jiaguang Zhang , Frederic Chen , linux-arm-kernel@lists.infradead.org, ginny.chen@mediatek.com Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org Hi, Rob, On Tue, 2019-05-07 at 12:41 -0500, Rob Herring wrote: > On Tue, May 07, 2019 at 04:13:46PM +0800, Bibby Hsieh wrote: > > tcmdq driver provide a function that get the relationship > > of sub system number from device node for client. > > add specification for #subsys-cells, mediatek,gce-subsys. > > > > Signed-off-by: Bibby Hsieh > > --- > > .../devicetree/bindings/mailbox/mtk-gce.txt | 15 ++++++++++++--- > > 1 file changed, 12 insertions(+), 3 deletions(-) > > > > diff --git a/Documentation/devicetree/bindings/mailbox/mtk-gce.txt b/Documentation/devicetree/bindings/mailbox/mtk-gce.txt > > index 1f7f8f2a3f49..8fd9479bc9f6 100644 > > --- a/Documentation/devicetree/bindings/mailbox/mtk-gce.txt > > +++ b/Documentation/devicetree/bindings/mailbox/mtk-gce.txt > > @@ -21,11 +21,19 @@ Required properties: > > priority: Priority of GCE thread. > > atomic_exec: GCE processing continuous packets of commands in atomic > > way. > > +- #subsys-cells: Should be 3. > > + <&phandle subsys_number start_offset size> > > + phandle: Label name of a gce node. > > + subsys_number: specify the sub-system id which is corresponding > > + to the register address. > > + start_offset: the start offset of register address that GCE can access. > > + size: the total size of register address that GCE can access. > > Like the #event-cells, do you need this if it isn't variable? > Yes, we need. Because the start_offset and size will be a fix number when the chip was designed out. Those two informations will help us to check if the same register address was wrote by different user. The checking mechanism is designing... > > > > Required properties for a client device: > > - mboxes: Client use mailbox to communicate with GCE, it should have this > > property and list of phandle, mailbox specifiers. > > -- mediatek,gce-subsys: u32, specify the sub-system id which is corresponding > > +Optional propertier for a client device: > > +- mediatek,gce-client-reg: u32, specify the sub-system id which is corresponding > > to the register address. > > This isn't a u32, but a phandle + 3 cells (or a list of those). How many > entries can there be? Ok, I will fix it. > > > > > Some vaules of properties are defined in 'dt-bindings/gce/mt8173-gce.h' > > @@ -40,6 +48,7 @@ Example: > > clocks = <&infracfg CLK_INFRA_GCE>; > > clock-names = "gce"; > > #mbox-cells = <3>; > > + #subsys-cells = <3>; > > }; > > > > Example for a client device: > > @@ -48,9 +57,9 @@ Example for a client device: > > compatible = "mediatek,mt8173-mmsys"; > > mboxes = <&gce 0 CMDQ_THR_PRIO_LOWEST 1>, > > <&gce 1 CMDQ_THR_PRIO_LOWEST 1>; > > - mediatek,gce-subsys = ; > > mutex-event-eof = > CMDQ_EVENT_MUTEX1_STREAM_EOF>; > > - > > + mediatek,gce-client-reg = <&gce SUBSYS_1400XXXX 0x3000 0x1000>, > > + <&gce SUBSYS_1401XXXX 0x2000 0x100>; > > ... > > }; > > -- > > 2.18.0 > > -- Bibby _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel