Received: by 2002:a05:6a10:1287:0:0:0:0 with SMTP id d7csp665736pxv; Thu, 15 Jul 2021 12:55:06 -0700 (PDT) X-Google-Smtp-Source: ABdhPJwwtV/ThX7Cy5So+mNiICtBN1d72LhDlFLFaHafUSfxac2uskHuEABadhnGaHX8lqmzOTKE X-Received: by 2002:aa7:dd43:: with SMTP id o3mr9317353edw.302.1626378905968; Thu, 15 Jul 2021 12:55:05 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1626378905; cv=none; d=google.com; s=arc-20160816; b=wrvscBumi+CbKT1zc00mTYuE8fuWtEoO5tTBeYJ3tLNJGVcwLn2OusLnR91DLJbOSS hMMskr4fzOvxi7Ip8OGie+EzLlUlVBgzzq2icQ3qYJXB/Xhwle0qTJA6KJ73evgyrWbD J4pocFmSguKpACvn/aZ2Sux6SpGBfttXjDsZKvL6ZN0B8OXMzxhyr0TsMn5DhgJfU5KN mwLLSY1hK1eCdlJODh/Ch0C1j6j0vDcnmPuvNnfSEdW0SvUEY1L7KbIXfxT0yfSUuHmv ZpCF+pooW0eqF01Cp2EcK3LUeeZA65M49jL6farqi5VUgg6ZQEL3GkJ+qn09dFRvDEyF Q0lw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:cc:to:subject:message-id:date:from:in-reply-to :references:mime-version:dkim-signature; bh=ihOfb0tc3xVL5q8N3onToRfW8DW3UJeYLmddd1YjXNw=; b=iczH33JdHQuno9yHdJN2YyCkHuQSFOdyaUlw50EShG/e6nwIob9I5/2C603VvLOlcj eqBO4avnhmB98tBXzOzh2hPnYVqg88wyBxjKCcv95AFiy0/knhQOdL0R0r1eG4OCt8do St7b29Fyy+1EzJCTeIWLMNcnXpsfV6hKCC5DB2xoj5LHjhiD1GZAIrZELDVu9972EwIj gDFJs3pgzQmqQgJCVFhlf7ryfsCWYdR09fFk/EQRUr0NNcaYB2Nw9ZwGYO/2I319DFYt hieJOKQgsHzF7tgjlAx6PQqpiBJQPq3cv5/FCrz9rFSTWb34tLOvx67CUuYfbMR22pBB u3JQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@gmail.com header.s=20161025 header.b=TNgFfAfM; 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=QUARANTINE dis=NONE) header.from=gmail.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id bz24si8086566ejc.249.2021.07.15.12.54.42; Thu, 15 Jul 2021 12:55:05 -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=@gmail.com header.s=20161025 header.b=TNgFfAfM; 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=QUARANTINE dis=NONE) header.from=gmail.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S245202AbhGOT4n (ORCPT + 99 others); Thu, 15 Jul 2021 15:56:43 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:42130 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1346607AbhGOTvd (ORCPT ); Thu, 15 Jul 2021 15:51:33 -0400 Received: from mail-ej1-x62d.google.com (mail-ej1-x62d.google.com [IPv6:2a00:1450:4864:20::62d]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id B818BC014B1E for ; Thu, 15 Jul 2021 12:20:06 -0700 (PDT) Received: by mail-ej1-x62d.google.com with SMTP id qb4so11060871ejc.11 for ; Thu, 15 Jul 2021 12:20:06 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=ihOfb0tc3xVL5q8N3onToRfW8DW3UJeYLmddd1YjXNw=; b=TNgFfAfMluhQ3/TAFnzR108JbZSD2t2Cg2OhHV+y5IzE4VZgg1atgOSY+17DQV4w2c au925Ps4U0kh4odD9rUI8rvZ8i8ZVpvl5YuAvvi1UdsMvzHYryfDULnip10C99VCRPlP Z3Bd9lXqcSyqY11k16uH3Pyu5BSAfZ2ylIIIzQD7lxNYjEI6xTXO2Uoriff7B5obchay /uynkrtBj0a+NGRdJDqlyJRt4vgm8Stvy/RUVvzDwA6SaqTpF9rlpqCL/iJM5AAMfPws vK/Jz0snmMnTjnh6Q/EQhTW9IdDcJdk3Ezysmcrxz+8s5cfzdi1W0iY7vB6RrgSUeVlG Kb1A== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=ihOfb0tc3xVL5q8N3onToRfW8DW3UJeYLmddd1YjXNw=; b=OIcdxmJkygOBszrBnsdiaTxd7NO3dDhofP3ghe19V+Hmbc0P2KWXWfcGUbWa3fwaQH n+gF+nf/N4Tx0wB6Td9SR6DVf0yf9KxzaWkMVs4GOHAT+ogpuYk3pHJKg2yjzbKrUt6W GUDMHok0+NeBjoaeRby9q0wbh/GAsQY45HGfde//xg54cXCP1b6h/SXVZK8IoCprFvk0 Rkj/N0uLbEZgrF6BEQW12Mm/EunfTgFMpuRPSTYBZqEHh1T3829Wvb6kxtir+deAH8dv lygpuppw1E/GgLElPOnjOhj7fPcGq1cga7t1OE0vPz9U1AJxjRsyEW5FkumcQfS9rGHq K36Q== X-Gm-Message-State: AOAM531Og+4uoGWJLZkby3kyxKRhaMl+RK2Su2y4eTH5Q+tVmCiZUMVE WVP8VPehwCaEK1Onm8GZasgECDURVxhwVmNjalc= X-Received: by 2002:a17:906:4a8d:: with SMTP id x13mr7186041eju.349.1626376805330; Thu, 15 Jul 2021 12:20:05 -0700 (PDT) MIME-Version: 1.0 References: <20210714080652.113381-1-liviu@dudau.co.uk> In-Reply-To: From: Mario Kleiner Date: Thu, 15 Jul 2021 21:19:53 +0200 Message-ID: Subject: Re: [PATCH] drm/amd/display: Fix 10bit 4K display on CIK GPUs To: Alex Deucher Cc: Liviu Dudau , Harry Wentland , Leo Li , "Pan, Xinhui" , LKML , amd-gfx list , Maling list - DRI developers , Alex Deucher , =?UTF-8?Q?Christian_K=C3=B6nig?= Content-Type: text/plain; charset="UTF-8" Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, Jul 15, 2021 at 6:10 PM Alex Deucher wrote: > > On Wed, Jul 14, 2021 at 4:15 AM Liviu Dudau wrote: > > > > Commit 72a7cf0aec0c ("drm/amd/display: Keep linebuffer pixel depth at > > 30bpp for DCE-11.0.") doesn't seems to have fixed 10bit 4K rendering over > > DisplayPort for CIK GPUs. On my machine with a HAWAII GPU I get a broken > > image that looks like it has an effective resolution of 1920x1080 but > > scaled up in an irregular way. Reverting the commit or applying this > > patch fixes the problem on v5.14-rc1. > > > > Fixes: 72a7cf0aec0c ("drm/amd/display: Keep linebuffer pixel depth at 30bpp for DCE-11.0.") > > Signed-off-by: Liviu Dudau > > Harry or Mario any ideas? Maybe we need finer grained DCE version > checking? I don't remember all of the caveats of this stuff. DCE11 > and older is getting to be pretty old at this point. I can just apply > this if you don't have any insights. > > Alex > Hi Alex I'd be fine with applying this. As my original commit says, photometer measurements showed that increasing the line buffer depth was only needed for my DCN-1 RavenRidge, not for my DCE-11.2 Polaris11 or a DCE-8.3 cik, so this should probably not cause harm to the increased precision modes. Note that given the hardware and USB-C/DP-HDMI adapters i have, I only tested this on a 2560x1440@144 Hz DP monitor with DCN-1, DCE-11.2, and a 2560x1440@100 Hz HDMI monitor iirc with DCN-1, DCE-8.3, and i think on a 2880x1800@60 Hz MBP Retina eDP panel with DCE-11.2. These are the highest resolution/framerate monitors I have atm.I don't have access to any 4k monitors, so maybe the problem is somehow specific to such high resolutions? Maybe somewhere else in the code something would need to be adapted? Lacking actual hw docs, my coding here is by pattern matching against existing DC code, guessing and testing on my limited hw samples. Acked-by: Mario Kleiner -mario > > --- > > drivers/gpu/drm/amd/display/dc/core/dc_resource.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/drivers/gpu/drm/amd/display/dc/core/dc_resource.c b/drivers/gpu/drm/amd/display/dc/core/dc_resource.c > > index a6a67244a322e..1596f6b7fed7c 100644 > > --- a/drivers/gpu/drm/amd/display/dc/core/dc_resource.c > > +++ b/drivers/gpu/drm/amd/display/dc/core/dc_resource.c > > @@ -1062,7 +1062,7 @@ bool resource_build_scaling_params(struct pipe_ctx *pipe_ctx) > > * so use only 30 bpp on DCE_VERSION_11_0. Testing with DCE 11.2 and 8.3 > > * did not show such problems, so this seems to be the exception. > > */ > > - if (plane_state->ctx->dce_version != DCE_VERSION_11_0) > > + if (plane_state->ctx->dce_version > DCE_VERSION_11_0) > > pipe_ctx->plane_res.scl_data.lb_params.depth = LB_PIXEL_DEPTH_36BPP; > > else > > pipe_ctx->plane_res.scl_data.lb_params.depth = LB_PIXEL_DEPTH_30BPP; > > -- > > 2.32.0 > > > > _______________________________________________ > > amd-gfx mailing list > > amd-gfx@lists.freedesktop.org > > https://lists.freedesktop.org/mailman/listinfo/amd-gfx