From patchwork Sun Oct 9 07:49:10 2016 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Shawn Guo X-Patchwork-Id: 9368393 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork.web.codeaurora.org (Postfix) with ESMTP id 0404460487 for ; Sun, 9 Oct 2016 07:49:34 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id DA204290CA for ; Sun, 9 Oct 2016 07:49:33 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id BACC8290CE; Sun, 9 Oct 2016 07:49:33 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-4.2 required=2.0 tests=BAYES_00, RCVD_IN_DNSWL_MED autolearn=ham version=3.3.1 Received: from gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) (using TLSv1.2 with cipher DHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.wl.linuxfoundation.org (Postfix) with ESMTPS id 95039290CA for ; Sun, 9 Oct 2016 07:49:32 +0000 (UTC) Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id C09C46E052; Sun, 9 Oct 2016 07:49:28 +0000 (UTC) X-Original-To: dri-devel@lists.freedesktop.org Delivered-To: dri-devel@lists.freedesktop.org Received: from mail.kernel.org (mail.kernel.org [198.145.29.136]) by gabe.freedesktop.org (Postfix) with ESMTPS id BAD196E052 for ; Sun, 9 Oct 2016 07:49:27 +0000 (UTC) Received: from mail.kernel.org (localhost [127.0.0.1]) by mail.kernel.org (Postfix) with ESMTP id 7FAB620274; Sun, 9 Oct 2016 07:49:26 +0000 (UTC) Received: from tiger (unknown [180.106.218.147]) (using TLSv1.2 with cipher DHE-RSA-AES128-SHA (128/128 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id 4954F2026C; Sun, 9 Oct 2016 07:49:23 +0000 (UTC) Date: Sun, 9 Oct 2016 15:49:10 +0800 From: Shawn Guo To: Rob Herring Subject: Re: [PATCH v2 1/2] dt-bindings: add bindings doc for ZTE VOU display controller Message-ID: <20161009074910.GA15400@tiger> References: <1474727185-24180-1-git-send-email-shawn.guo@linaro.org> <1474727185-24180-2-git-send-email-shawn.guo@linaro.org> <20161003174429.GA26054@rob-hp-laptop> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20161003174429.GA26054@rob-hp-laptop> User-Agent: Mutt/1.5.21 (2010-09-15) X-Virus-Scanned: ClamAV using ClamSMTP Cc: Mark Rutland , devicetree@vger.kernel.org, Daniel Vetter , Baoyou Xie , dri-devel@lists.freedesktop.org, Jun Nie , linux-arm-kernel@lists.infradead.org X-BeenThere: dri-devel@lists.freedesktop.org X-Mailman-Version: 2.1.18 Precedence: list List-Id: Direct Rendering Infrastructure - Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" X-Virus-Scanned: ClamAV using ClamSMTP On Mon, Oct 03, 2016 at 12:44:29PM -0500, Rob Herring wrote: > > +Example: > > + > > +vou: vou@1440000 { > > + compatible = "zte,zx296718-vou"; > > + #address-cells = <1>; > > + #size-cells = <1>; > > + reg = <0x1440000 0x10000>; > > + ranges; > > You still have overlapping addresses. Explicitly list the sub ranges in > reg here used by the VOU driver if the driver usage doesn't overlap. If > there is overlap (2 drivers accessing the same range), then you need > some APIs between the components (or possibly regmap). The driver matching "zte,zx296718-vou" doesn't map or access the any 'reg' address. The 'reg' property here is more like a hint telling that the VOU block covers the address space of all child devices. I will simply drop the 'reg' property here. > Also, don't do an empty ranges here. Fill it in so the child nodes are > just offsets of 0x1440000 Okay. I thought that empty 'ranges' is fine as long as parent and child address spaces are identical (1:1 mapping). So with your suggestion, I made the changes below. Let me know if this is still not what you are asking for. Shawn -----8<----------------- dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel Acked-by: Rob Herring diff --git a/Documentation/devicetree/bindings/display/zte,vou.txt b/Documentation/devicetree/bindings/display/zte,vou.txt index d03ba4c4810c..6bb4ab2517ef 100644 --- a/Documentation/devicetree/bindings/display/zte,vou.txt +++ b/Documentation/devicetree/bindings/display/zte,vou.txt @@ -56,14 +56,13 @@ vou: vou@1440000 { compatible = "zte,zx296718-vou"; #address-cells = <1>; #size-cells = <1>; - reg = <0x1440000 0x10000>; - ranges; + ranges = <0 0x1440000 0x10000>; - dpc: dpc@1440000 { + dpc: dpc@0 { compatible = "zte,zx296718-dpc"; - reg = <0x1440000 0x1000>, <0x1441000 0x1000>, - <0x1445000 0x1000>, <0x1446000 0x1000>, - <0x144a000 0x1000>; + reg = <0x0000 0x1000>, <0x1000 0x1000>, + <0x5000 0x1000>, <0x6000 0x1000>, + <0xa000 0x1000>; reg-names = "osd", "timing_ctrl", "dtrc", "vou_ctrl", "otfppu"; @@ -74,9 +73,9 @@ vou: vou@1440000 { "main_wclk", "aux_wclk"; }; - hdmi: hdmi@144c000 { + hdmi: hdmi@c000 { compatible = "zte,zx296718-hdmi"; - reg = <0x144c000 0x4000>; + reg = <0xc000 0x4000>; interrupts = ; _______________________________________________