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 Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id B08AAC04E69 for ; Wed, 2 Aug 2023 14:26:02 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S232680AbjHBO0B (ORCPT ); Wed, 2 Aug 2023 10:26:01 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:60748 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S232560AbjHBOZ7 (ORCPT ); Wed, 2 Aug 2023 10:25:59 -0400 Received: from mail-ej1-x62c.google.com (mail-ej1-x62c.google.com [IPv6:2a00:1450:4864:20::62c]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 574D62706 for ; Wed, 2 Aug 2023 07:25:57 -0700 (PDT) Received: by mail-ej1-x62c.google.com with SMTP id a640c23a62f3a-99bded9d93dso171424666b.1 for ; Wed, 02 Aug 2023 07:25:57 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ffwll.ch; s=google; t=1690986356; x=1691591156; h=in-reply-to:content-disposition:mime-version:references :mail-followup-to:message-id:subject:cc:to:from:date:from:to:cc :subject:date:message-id:reply-to; bh=FsVcsCDvamjl31gifaKwLNv2n2Xspa6emd5tT3prZvw=; b=eq9qvw/au3DtjNiInHv+/Ks+sX1tL4hda1Ymw/mKWe9H1Dn/0GH3Lkc2XHr2w4hnqh szW59lqw7i9lIfZLBs86jn+JmO2jiioTa1/OThHRi93xkN7nQo8rqnOgCYtBmvg7+wtd WF753BEgN/emMxve58EPL3p3mFOmrc8E1KGkg= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1690986356; x=1691591156; h=in-reply-to:content-disposition:mime-version:references :mail-followup-to:message-id:subject:cc:to:from:date :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=FsVcsCDvamjl31gifaKwLNv2n2Xspa6emd5tT3prZvw=; b=JGwYQziUss1HB3fSAf2ME4sv7z5d6HF8kSJyvN2MKF+Y3Y/eWijCoicFJaEqd0qvVO 74WWf+DDJ40trKWFRoaA40nene6+WXoMyjnL0XWKYndJhmeC8N+JcOikIYR8hgSvjdaD PiAKYXYFsYTLJn15eu5FogP1j8WCjY160P3GYfM/UH/V5OwWVyCj+rD+EzmQnGHlAJjw nxQpKEtShqkpBrbgms9mLnQLwqWKHFYglDTpyhaARyFz+BIFXfvbHy8fowVgVCGZDWma zVjdFSlWNeo37C0uoEcBsgL6TtDKEsgQRy5ELX0/Q7PjuwYfx3TozJWy8uMhQCBhgaCD H5FQ== X-Gm-Message-State: ABy/qLaO2Jp5MoNiBrdkt88xxgJlpHh5kQnmtfJCMbOYWVFUF6hRQgcA 30wo/nXcZBNlFHkCvFBBRSISKA== X-Google-Smtp-Source: APBJJlEGuIJd0GEks9/vYG2PGf+sSGozgW3aVzw1hHeGDBgQvjzANdpy3n5m6RtZqxVwQKbbxW+z3w== X-Received: by 2002:a17:906:73dd:b0:99b:d594:8f89 with SMTP id n29-20020a17090673dd00b0099bd5948f89mr10192169ejl.0.1690986355776; Wed, 02 Aug 2023 07:25:55 -0700 (PDT) Received: from phenom.ffwll.local ([2a02:168:57f4:0:efd0:b9e5:5ae6:c2fa]) by smtp.gmail.com with ESMTPSA id si15-20020a170906cecf00b00992e265495csm9261942ejb.212.2023.08.02.07.25.54 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 02 Aug 2023 07:25:55 -0700 (PDT) Date: Wed, 2 Aug 2023 16:25:53 +0200 From: Daniel Vetter To: Michael Riesch Cc: Rob Herring , Krzysztof Kozlowski , Conor Dooley , Neil Armstrong , Sam Ravnborg , David Airlie , Daniel Vetter , Maxime Ripard , Miquel Raynal , Sebastian Reichel , Gerald Loacker , devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org Subject: Re: [PATCH 0/4] drm/panel: sitronix-st7789v: add support for partial mode Message-ID: Mail-Followup-To: Michael Riesch , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Neil Armstrong , Sam Ravnborg , David Airlie , Maxime Ripard , Miquel Raynal , Sebastian Reichel , Gerald Loacker , devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org References: <20230718-feature-lcd-panel-v1-0-e9a85d5374fd@wolfvision.net> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20230718-feature-lcd-panel-v1-0-e9a85d5374fd@wolfvision.net> X-Operating-System: Linux phenom 6.3.0-2-amd64 Precedence: bulk List-ID: X-Mailing-List: devicetree@vger.kernel.org On Tue, Jul 18, 2023 at 05:31:49PM +0200, Michael Riesch wrote: > Hi all, > > This series adds support for the partial display mode to the Sitronix > ST7789V panel driver. This is useful for panels that are partially > occluded by design, such as the Jasonic JT240MHQS-HWT-EK-E3. Support > for this particular panel is added as well. > > Note: This series is already based on > https://lore.kernel.org/lkml/20230714013756.1546769-1-sre@kernel.org/ > > Looking forward to your comments! Summary of my take from a fairly long (and kinda still on-going) irc discussion: - Where we do know the exact overscan, the kernel should expose correct modes and adjust the display pipeline to match if needed when programming the hardware. Meaning the approach in this patch series. - For hdmi overscan there's a lot of automagic overscan happening by default. Drivers can mostly fix this by setting all the right infoframes, but unfortuantely a very big pile of infoframes is needed. Assuming drivers actually use the helpers I think only i915 gets them all, so intel_hdmi_compute_config() at the bottom would be the example to follow, and maybe some more code to extract from and share. - That /should/ only leave the really old analog TV and similar horrors leftover. For those we simply can't even guess the right amount of overscan (because back then no one cared back then about really seeing everything), and so that's the only case where we should rely on the overscan properties. And that case only works when the compositor stack passes these properties all the way to the user, since only they can check when the settings are good. The overscan properties should _not_ be used to fix issues of the previous kind, that really should all work out of the box as much as possible. Cheers, Sima > > --- > Michael Riesch (4): > dt-bindings: vendor-prefixes: add jasonic > dt-bindings: display: st7789v: add jasonic jt240mhqs-hwt-ek-e3 display > drm/panel: sitronix-st7789v: add support for partial mode > drm/panel: sitronix-st7789v: add jasonic jt240mhqs-hwt-ek-e3 support > > .../bindings/display/panel/sitronix,st7789v.yaml | 1 + > .../devicetree/bindings/vendor-prefixes.yaml | 2 + > drivers/gpu/drm/panel/panel-sitronix-st7789v.c | 67 +++++++++++++++++++++- > 3 files changed, 68 insertions(+), 2 deletions(-) > --- > base-commit: b43dae411767f34288aa347f26b5ed2dade39469 > change-id: 20230718-feature-lcd-panel-26d9f29a7830 > > Best regards, > -- > Michael Riesch > -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch