Received: by 2002:a05:6a10:c7c6:0:0:0:0 with SMTP id h6csp1985866pxy; Mon, 2 Aug 2021 16:01:52 -0700 (PDT) X-Google-Smtp-Source: ABdhPJwbrE33T/DJY+e0iRK7hf1+tzx7hNHZYgyrK334/H9o2uToqrGJYZGYvPzYJ3vFK1OWIVAf X-Received: by 2002:a05:6602:1814:: with SMTP id t20mr776912ioh.204.1627945312100; Mon, 02 Aug 2021 16:01:52 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1627945312; cv=none; d=google.com; s=arc-20160816; b=Ap7L7IswA/8qcTkn/hcV+oEtw+ykOhKO0tN5CbVbBDftPzycXzMY0ENQSKhboYwSDV oGLoL0qr85tWYb/dj9bhLtFVaLMJdzWsM2le/V+JBXZIXI8GPUWiL3yzcGaCmbwluA+p KdZFanrtFFp5NhA8wpxtvReO59l8XXdWqLbGUwRzK4TI6IMKbnv/SU/Mot4jJ5RAjzmB JAEpXNxAYnajHqKsneIGPqfjT/E2LgaVfCEcnfNCsPUkZ/Z8MKGr8E+38V0Ink4FaG6B kualXxbQLqKzBv3Gr9ivKnaoHuR7fLbEpiN8leFshb4n/ZuG5e6vRvZnxhbnLZfrK18P ZFng== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:cc:to:subject :message-id:date:from:in-reply-to:references:mime-version :dkim-signature; bh=UZbteSPL5sOXkEiDidS2VuyhalUVOzoGSJZIcuWENCs=; b=BvfRcCsY+CmeX7cMwRkpA3uo94vf4ejbfw4sBbvcSzFaVWqpEZ+dSLBJ48CkPDI7Wj EIYHOJ10h88w28CyU2btllxuszoHVJIW1YsIcbYt3xzKb1BBiSWXAjHMUG80awysiCmh ewuog5EjaotmiKgEWgltHgr8pyFN49G4kcnXofOcTmf98rzEq8qtqm1PEKzpNGdVDtSJ VOQgCyD4Gww97gSJQvMsaOwZoARWmUJMXdLCyi4GpQHsWrwdGMcXq20M8szEAVC9Y1Ql dfuTMSV+3wAR8gTRDDV6yykH17QJBglwwC/7G4gCM7VErC6aSbDIDmkIkvv0OpXsh37C R7mQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=JxOIrHwD; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id n9si3020770jam.54.2021.08.02.16.01.39; Mon, 02 Aug 2021 16:01:52 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=JxOIrHwD; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S232259AbhHBXBB (ORCPT + 99 others); Mon, 2 Aug 2021 19:01:01 -0400 Received: from mail.kernel.org ([198.145.29.99]:53720 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231126AbhHBXBA (ORCPT ); Mon, 2 Aug 2021 19:01:00 -0400 Received: by mail.kernel.org (Postfix) with ESMTPSA id 6557D60D07; Mon, 2 Aug 2021 23:00:50 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1627945250; bh=e822VuDx6z8sMZ4u2JoN2zmm7TSpCXSrlQ2PRMcE4RA=; h=References:In-Reply-To:From:Date:Subject:To:Cc:From; b=JxOIrHwDEsSnb0iq0rIpsYlOLoG24P4miwqyD1FG7n4FWHzwDNfy/NHjYdDMu5Qji kXPSkX44FzNpdxEJtmmIARGgMRDYC9iIdUV2TyjtD0Sq+Kg/QNdoUV6RLsJm3/acl7 25pdODPscvgAAJXGUOgPzTBKvkfiDzgA/ato56Vb671pWbF2IJGQzrlpL81XqgcdW3 pRcpknNaL3LOwPZKRRkwGd2LdUK2o8yDLvrlVmcOBcoyoiz7VZp8vqJysP6tbh3Cse 8+95COQy5cbA3qCXkRhDTUyY/YfbD6G1rfhrCL41k18t2xk1f7jCngE/ya9GL8S5ER FfHzzKXcHrclw== Received: by mail-ed1-f52.google.com with SMTP id n2so26585463eda.10; Mon, 02 Aug 2021 16:00:50 -0700 (PDT) X-Gm-Message-State: AOAM530l68U0ARElBsVYDOR4IUYkvt9fboaELtpiTOxi8cRAB/Wc5w1A BB3DeG/PyyO3L7DJ6pP2iVW/SsozrZpLVy8o7A== X-Received: by 2002:a05:6402:718:: with SMTP id w24mr21831525edx.49.1627945249057; Mon, 02 Aug 2021 16:00:49 -0700 (PDT) MIME-Version: 1.0 References: <1626762588-6020-1-git-send-email-yongqiang.niu@mediatek.com> <1626762588-6020-2-git-send-email-yongqiang.niu@mediatek.com> In-Reply-To: <1626762588-6020-2-git-send-email-yongqiang.niu@mediatek.com> From: Chun-Kuang Hu Date: Tue, 3 Aug 2021 07:00:37 +0800 X-Gmail-Original-Message-ID: Message-ID: Subject: Re: [PATCH v5] drm/mediatek: add dither 6 setting To: Yongqiang Niu Cc: Chun-Kuang Hu , Rob Herring , Matthias Brugger , Philipp Zabel , David Airlie , Daniel Vetter , Jassi Brar , Fabien Parent , Dennis YC Hsieh , DTML , Linux ARM , "moderated list:ARM/Mediatek SoC support" , linux-kernel , DRI Development , Project_Global_Chrome_Upstream_Group@mediatek.com, Hsin-Yi Wang Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi, Yongqiang: Yongqiang Niu =E6=96=BC 2021=E5=B9=B47=E6=9C= =8820=E6=97=A5 =E9=80=B1=E4=BA=8C =E4=B8=8B=E5=8D=882:30=E5=AF=AB=E9=81=93= =EF=BC=9A > > dither 6 setting is missed in a6b7c98afdca > bit 1 is lfsr_en( "Enables LFSR-type dithering"), need enable > bit 2 is rdither_en(Enables running order dithering), need disable > > Fixes: a6b7c98afdca(drm/mediatek: add mtk_dither_set_common()) > Signed-off-by: Yongqiang Niu > --- > drivers/gpu/drm/mediatek/mtk_drm_ddp_comp.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/drivers/gpu/drm/mediatek/mtk_drm_ddp_comp.c b/drivers/gpu/dr= m/mediatek/mtk_drm_ddp_comp.c > index 99cbf44..7dd8e05 100644 > --- a/drivers/gpu/drm/mediatek/mtk_drm_ddp_comp.c > +++ b/drivers/gpu/drm/mediatek/mtk_drm_ddp_comp.c > @@ -26,6 +26,7 @@ > #define DISP_OD_CFG 0x0020 > #define DISP_OD_SIZE 0x0030 > #define DISP_DITHER_5 0x0114 > +#define DISP_DITHER_6 0x0118 > #define DISP_DITHER_7 0x011c > #define DISP_DITHER_15 0x013c > #define DISP_DITHER_16 0x0140 > @@ -135,6 +136,7 @@ void mtk_dither_set_common(void __iomem *regs, struct= cmdq_client_reg *cmdq_reg, > > if (bpc >=3D MTK_MIN_BPC) { > mtk_ddp_write(cmdq_pkt, 0, cmdq_reg, regs, DISP_DITHER_5)= ; > + mtk_ddp_write(cmdq_pkt, 0x3002, cmdq_reg, regs, DISP_DITH= ER_6); Symbolized 0x3002. BIT(1) is lfsr_en. Regards, Chun-Kuang. > mtk_ddp_write(cmdq_pkt, 0, cmdq_reg, regs, DISP_DITHER_7)= ; > mtk_ddp_write(cmdq_pkt, > DITHER_LSB_ERR_SHIFT_R(MTK_MAX_BPC - bpc) | > -- > 1.8.1.1.dirty >