Received: by 2002:a05:7412:98c1:b0:fa:551:50a7 with SMTP id kc1csp65115rdb; Fri, 5 Jan 2024 03:00:03 -0800 (PST) X-Google-Smtp-Source: AGHT+IGLBuJQbY7HYE5L/L6q0/T2ManiDqeb0r8ICaD6YYsNU2FrVKeM9CMAM3zUtyeIT3BNeIIv X-Received: by 2002:a50:c112:0:b0:557:172c:d513 with SMTP id l18-20020a50c112000000b00557172cd513mr718384edf.76.1704452403750; Fri, 05 Jan 2024 03:00:03 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1704452403; cv=none; d=google.com; s=arc-20160816; b=mWZDHJ/jHek/gHLGngdHi0E8cAIoFLYEPvuSJIsf5mcTwJshT5OZTY5tQquVggK2SM fsfQr7bJhGlLzI5H1Xbazdz1teGaBQ/sQ3IVeRHmJO7nDhZcPtHpgPq3yOu5zqznSS3I qU6+HH8ulQy662qNONqFsAnux8Nov+MFCXiSYAv6K5cmeFU0hfnMOAgXa5XQVBLQ0KeS mfsBS7R6MqEQZ1kLKZnfTgMSu1sakq7LGYeTwYpgjYQYWaXC75Og7dNSXCir4g+mIjCK ftkNNlBMMtkGFWX3iX64mZ3+kfOmkB9tirGKVgmADv5A0WYuwY1s4dXUdPQX+R6tdc8j aOXg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:list-unsubscribe :list-subscribe:list-id:precedence:date:message-id:dkim-signature; bh=7Rpg1YBO3YN3stA5+Iw9eTT2absDZpy7ygnLlv2cEYU=; fh=3/1+EW24ECtgj/CESCErEsOck7XklGLnIPKdjtt1bmQ=; b=zf8TtipCR1tb7ci64kkPz3/NYBe5UtJ2lClUwOLm1Y312dADtZhuxsngrK0gGzjFrk sSksgJch169nHlrqnff+EPG4R0DdpcBnF6+E7hOGJOa2KksMAH6vaAwmXLSNDiiFzzAK rLHRzrTbkpfqT4jwgc3rppNKLRIGm3ApBFPdnCHznOqYxT6u5+9ySDC6r4OI+AXmnB14 Ve7XDNnN/FF141Z/nzOVynY5UDgBAq53SyK1tQSBLp/fNbqnQydEeYeecBpmxWX+2GCf 4ZVkhAba19ACSs7feTJKbQyqpCHSPjWAVcT5HTsv+rgD6t8eDtnzncwr6C+1Z3lwqgxY ZtFw== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@rock-chips.com header.s=default header.b=J49zP1kc; spf=pass (google.com: domain of linux-kernel+bounces-17777-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:4601:e00::3 as permitted sender) smtp.mailfrom="linux-kernel+bounces-17777-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=rock-chips.com Return-Path: Received: from am.mirrors.kernel.org (am.mirrors.kernel.org. [2604:1380:4601:e00::3]) by mx.google.com with ESMTPS id dc11-20020a056402310b00b0055410efe357si568865edb.199.2024.01.05.03.00.03 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 05 Jan 2024 03:00:03 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel+bounces-17777-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:4601:e00::3 as permitted sender) client-ip=2604:1380:4601:e00::3; Authentication-Results: mx.google.com; dkim=pass header.i=@rock-chips.com header.s=default header.b=J49zP1kc; spf=pass (google.com: domain of linux-kernel+bounces-17777-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:4601:e00::3 as permitted sender) smtp.mailfrom="linux-kernel+bounces-17777-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=rock-chips.com Received: from smtp.subspace.kernel.org (wormhole.subspace.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by am.mirrors.kernel.org (Postfix) with ESMTPS id 839BA1F23FD4 for ; Fri, 5 Jan 2024 11:00:03 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id CBA2828E27; Fri, 5 Jan 2024 10:59:57 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=rock-chips.com header.i=@rock-chips.com header.b="J49zP1kc" X-Original-To: linux-kernel@vger.kernel.org Received: from mail-m49216.qiye.163.com (mail-m49216.qiye.163.com [45.254.49.216]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id AE9092421C for ; Fri, 5 Jan 2024 10:59:50 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=rock-chips.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=rock-chips.com DKIM-Signature: a=rsa-sha256; b=J49zP1kcxyDLDuYjM1j/Nm85RnUYMNpETkXTFJtmfMkDN7noi608Tk7coSGxV/XODolyLuQGAfQwK5z9r8GBmQTRolRWVBxr3xENAIZ/q631aqmpzcv2t/h44mHU7k8zm6iScrfG66xhRB6ZXc+4CrxCcpc58vsJz6ASrURiAxE=; c=relaxed/relaxed; s=default; d=rock-chips.com; v=1; bh=7Rpg1YBO3YN3stA5+Iw9eTT2absDZpy7ygnLlv2cEYU=; h=date:mime-version:subject:message-id:from; Received: from [172.16.12.141] (unknown [58.22.7.114]) by mail-m12779.qiye.163.com (Hmail) with ESMTPA id 799D37802A7; Fri, 5 Jan 2024 17:04:56 +0800 (CST) Message-ID: Date: Fri, 5 Jan 2024 17:04:55 +0800 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH] drm/rockchip: vop2: Drop unused if_dclk_rate variable To: =?UTF-8?Q?Heiko_St=C3=BCbner?= , Sandy Huang , Maarten Lankhorst , Maxime Ripard , Thomas Zimmermann , David Airlie , Daniel Vetter , Cristian Ciocaltea Cc: dri-devel@lists.freedesktop.org, linux-arm-kernel@lists.infradead.org, linux-rockchip@lists.infradead.org, linux-kernel@vger.kernel.org, kernel@collabora.com References: <20240104143951.85219-1-cristian.ciocaltea@collabora.com> <20240104143951.85219-2-cristian.ciocaltea@collabora.com> <5867171.29KlJPOoH8@diego> Content-Language: en-US From: Andy Yan In-Reply-To: <5867171.29KlJPOoH8@diego> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit X-HM-Spam-Status: e1kfGhgUHx5ZQUpXWQgPGg8OCBgUHx5ZQUlOS1dZFg8aDwILHllBWSg2Ly tZV1koWUFDSUNOT01LS0k3V1ktWUFJV1kPCRoVCBIfWUFZQhlPTFZDH0lJTh0fQkJPTkJVEwETFh oSFyQUDg9ZV1kYEgtZQVlOQ1VJSVVMVUpKT1lXWRYaDxIVHRRZQVlPS0hVSk1PSU5IVUpLS1VKQk tLWQY+ X-HM-Tid: 0a8cd8de4dd6b24fkuuu799d37802a7 X-HM-MType: 1 X-HM-Sender-Digest: e1kMHhlZQR0aFwgeV1kSHx4VD1lBWUc6Kxg6Pww*MDwZST8KPjc*F09R UTAaCRxVSlVKTEtPT09OT0JMSEhMVTMWGhIXVRoVHwJVAhoVOwkUGBBWGBMSCwhVGBQWRVlXWRIL WUFZTkNVSUlVTFVKSk9ZV1kIAVlBT09IQzcG Hi: On 1/4/24 23:58, Heiko Stübner wrote: > Hi Christian, Andy, > > Am Donnerstag, 4. Januar 2024, 15:39:50 CET schrieb Cristian Ciocaltea: >> Commit 5a028e8f062f ("drm/rockchip: vop2: Add support for rk3588") >> introduced a variable which ended up being unused. Remove it. >> >> rockchip_drm_vop2.c:1688:23: warning: variable ‘if_dclk_rate’ set but not used [-Wunused-but-set-variable] >> >> Signed-off-by: Cristian Ciocaltea > > in general, please don't send non-series patches as replies to other patches. > It confuses tooling like b4 way too often, as this patch is not designated > as a 2/2 (similar to the first one not being 1/2). > >> --- >> drivers/gpu/drm/rockchip/rockchip_drm_vop2.c | 2 -- >> 1 file changed, 2 deletions(-) >> >> diff --git a/drivers/gpu/drm/rockchip/rockchip_drm_vop2.c b/drivers/gpu/drm/rockchip/rockchip_drm_vop2.c >> index 44508c2dd614..923985d4161b 100644 >> --- a/drivers/gpu/drm/rockchip/rockchip_drm_vop2.c >> +++ b/drivers/gpu/drm/rockchip/rockchip_drm_vop2.c >> @@ -1685,7 +1685,6 @@ static unsigned long rk3588_calc_cru_cfg(struct vop2_video_port *vp, int id, >> unsigned long dclk_core_rate = v_pixclk >> 2; >> unsigned long dclk_rate = v_pixclk; >> unsigned long dclk_out_rate; >> - unsigned long if_dclk_rate; >> unsigned long if_pixclk_rate; >> int K = 1; >> >> @@ -1700,7 +1699,6 @@ static unsigned long rk3588_calc_cru_cfg(struct vop2_video_port *vp, int id, >> } >> >> if_pixclk_rate = (dclk_core_rate << 1) / K; >> - if_dclk_rate = dclk_core_rate / K; >> /* >> * *if_pixclk_div = dclk_rate / if_pixclk_rate; >> * *if_dclk_div = dclk_rate / if_dclk_rate; >> */ > *if_pixclk_div = 2; > *if_dclk_div = 4; > > with the code continuing with those static constants but the comment > showing a forumula, I do hope Andy can provide a bit of insight into > what is happening here. > > I.e. I'd really like to understand if that really is just a remnant or > something different is needed. This is not a remnant, in my V1, I calculate all the div by formula, but Sascha prefer more for a constants value[0], so I keep this formula as comments to indicate how these value come from. [0]https://patchwork.kernel.org/project/linux-rockchip/patch/20231114112855.1771372-1-andyshrk@163.com/ > > > Heiko > >