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=-0.6 required=3.0 tests=FROM_EXCESS_BASE64, HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SPF_PASS,URIBL_BLOCKED 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 E577FC6778A for ; Fri, 29 Jun 2018 19:07:45 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 9C34D27CC6 for ; Fri, 29 Jun 2018 19:07:45 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 9C34D27CC6 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=siol.net 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 S936627AbeF2THm convert rfc822-to-8bit (ORCPT ); Fri, 29 Jun 2018 15:07:42 -0400 Received: from mailoutvs43.siol.net ([185.57.226.234]:43723 "EHLO mail.siol.net" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S934996AbeF2THj (ORCPT ); Fri, 29 Jun 2018 15:07:39 -0400 Received: from localhost (localhost [127.0.0.1]) by mail.siol.net (Postfix) with ESMTP id AED415229C6; Fri, 29 Jun 2018 21:07:36 +0200 (CEST) X-Virus-Scanned: amavisd-new at psrvmta11.zcs-production.pri Received: from mail.siol.net ([127.0.0.1]) by localhost (psrvmta11.zcs-production.pri [127.0.0.1]) (amavisd-new, port 10032) with ESMTP id KRgP9pkKUU0T; Fri, 29 Jun 2018 21:07:36 +0200 (CEST) Received: from mail.siol.net (localhost [127.0.0.1]) by mail.siol.net (Postfix) with ESMTPS id 06458522FFA; Fri, 29 Jun 2018 21:07:36 +0200 (CEST) Received: from jernej-laptop.localnet (unknown [194.152.15.144]) (Authenticated sender: 031275009) by mail.siol.net (Postfix) with ESMTPA id 073015229C6; Fri, 29 Jun 2018 21:07:35 +0200 (CEST) From: Jernej =?utf-8?B?xaBrcmFiZWM=?= To: Maxime Ripard Cc: Chen-Yu Tsai , Rob Herring , David Airlie , Gustavo Padovan , Maarten Lankhorst , Sean Paul , Mark Rutland , dri-devel , devicetree , linux-arm-kernel , linux-kernel , linux-clk , linux-sunxi Subject: Re: [PATCH v3 10/24] drm/sun4i: tcon: Generalize engine search algorithm Date: Fri, 29 Jun 2018 21:06:09 +0200 Message-ID: <3257762.ZQzN8rrT6c@jernej-laptop> In-Reply-To: <20180628182543.yyja6hiry7rx7fu2@flea> References: <20180625120304.7543-1-jernej.skrabec@siol.net> <1812731.FQOhD8MfmX@jernej-laptop> <20180628182543.yyja6hiry7rx7fu2@flea> MIME-Version: 1.0 Content-Transfer-Encoding: 8BIT Content-Type: text/plain; charset="UTF-8" Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Dne četrtek, 28. junij 2018 ob 20:25:43 CEST je Maxime Ripard napisal(a): > On Thu, Jun 28, 2018 at 06:48:50AM +0200, Jernej Škrabec wrote: > > Dne četrtek, 28. junij 2018 ob 04:06:52 CEST je Chen-Yu Tsai napisal(a): > > > On Mon, Jun 25, 2018 at 8:02 PM, Jernej Skrabec > > > > > > > wrote: > > > > Current "old" method to find engine worked pretty well for DE2. > > > > However, > > > > it doesn't work when TCON TOP is between mixer (engine) and TCON. > > > > TCON > > > > TOP has multiple input ports, but current engine search algorithm > > > > expects only one. > > > > > > > > This can be fixed by first looking for output port id and selecting > > > > matching input by subtracting 1 for the next round. This work even if > > > > there is only one input and output. > > > > > > > > Signed-off-by: Jernej Skrabec > > > > --- > > > > > > > > drivers/gpu/drm/sun4i/sun4i_tcon.c | 22 ++++++++++++++++++---- > > > > 1 file changed, 18 insertions(+), 4 deletions(-) > > > > > > > > diff --git a/drivers/gpu/drm/sun4i/sun4i_tcon.c > > > > b/drivers/gpu/drm/sun4i/sun4i_tcon.c index 08747fc3ee71..264bcc43da11 > > > > 100644 > > > > --- a/drivers/gpu/drm/sun4i/sun4i_tcon.c > > > > +++ b/drivers/gpu/drm/sun4i/sun4i_tcon.c > > > > @@ -791,12 +791,14 @@ static int sun4i_tcon_init_regmap(struct device > > > > *dev, > > > > > > > > */ > > > > > > > > static struct sunxi_engine * > > > > sun4i_tcon_find_engine_traverse(struct sun4i_drv *drv, > > > > > > > > - struct device_node *node) > > > > + struct device_node *node, > > > > + u32 port_id) > > > > > > > > { > > > > > > > > struct device_node *port, *ep, *remote; > > > > struct sunxi_engine *engine = ERR_PTR(-EINVAL); > > > > > > > > + u32 reg = 0; > > > > > > > > - port = of_graph_get_port_by_id(node, 0); > > > > + port = of_graph_get_port_by_id(node, port_id); > > > > > > > > if (!port) > > > > > > > > return ERR_PTR(-EINVAL); > > > > > > > > @@ -826,8 +828,20 @@ sun4i_tcon_find_engine_traverse(struct sun4i_drv > > > > *drv, > > > > > > > > if (remote == engine->node) > > > > > > > > goto out_put_remote; > > > > > > > > + /* > > > > + * According to device tree binding input ports have even id > > > > + * number and output ports have odd id. Since component with > > > > + * more than one input and one output (TCON TOP) exits, > > > > correct > > > > + * remote input id has to be calculated by subtracting 1 from > > > > + * remote output id. If this for some reason can't be done, 0 > > > > + * is used as input port id. > > > > + */ > > > > > > You need to call > > > > > > of_node_put(port); > > > > > > to drop the reference to the original port. > > > > Thanks for noticing it. I guess I should send fix patch, since patches > > from > > drm-misc-next can't be dropped. > > Yeah, please send additional patches for all the issues pointed out by > Chen-Yu. Of course. I hope this can be resolved till the end of the next week. After that, I will be away from PC for 2 weeks. Feel free to drop DT patches if you think that it will come too close to merge window. Best regards, Jernej