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 gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 0003FCD98DA for ; Mon, 15 Jun 2026 17:00:17 +0000 (UTC) Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id 4E61C10E566; Mon, 15 Jun 2026 17:00:17 +0000 (UTC) Authentication-Results: gabe.freedesktop.org; dkim=pass (2048-bit key; unprotected) header.d=intel.com header.i=@intel.com header.b="H6ifrwii"; dkim-atps=neutral Received: from mgamail.intel.com (mgamail.intel.com [198.175.65.18]) by gabe.freedesktop.org (Postfix) with ESMTPS id 1EE6A10E4C6; Mon, 15 Jun 2026 17:00:16 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1781542817; x=1813078817; h=date:from:to:cc:subject:message-id:references: mime-version:content-transfer-encoding:in-reply-to; bh=G4RW0DOAXCOiFUvbjF7GDJDVbr5prJgzFp1ayrsFn6Y=; b=H6ifrwiizCqCafkSJqJEewmOXC5Olpz0nsSV0257HpWjqBzPtPbNLzv+ ZzyvGTLlEId4kCWoRLIEpJ3Q9TlMZIQ3XXnKAp4grlr96IzQBHLpdEaoK EQ7zG0Bsu9VqegXwaxerKSsxZXjl6KdKfMDOX7pakQY7I1Gcs8Db6zk/k Ee9x23E52/Uwnw9sQfO1WHuUFBZPn8RWG6/+ohHWciXGKQgn8Q6sH4gwA eXLbtLTl2KPCxOELesL9Y9395IeCC52uD0m3ia5QYTPFCuXwoYezTwhWV kQPDoEwUROSH8U2+7ZsjFS+A6/qQRzBczqze3D9O5ZshuSE42Wk5A4H71 w==; X-CSE-ConnectionGUID: 5C5dtPgzQae4PxUSfSA3WA== X-CSE-MsgGUID: 1A0aSn/8TDSNrfQFlVBNzA== X-IronPort-AV: E=McAfee;i="6800,10657,11818"; a="82354607" X-IronPort-AV: E=Sophos;i="6.24,206,1774335600"; d="scan'208";a="82354607" Received: from fmviesa002.fm.intel.com ([10.60.135.142]) by orvoesa110.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 15 Jun 2026 10:00:16 -0700 X-CSE-ConnectionGUID: yDx0XeIoSxOdERLXxeoxAQ== X-CSE-MsgGUID: fMdObLw+S16sLcSZDfWrtQ== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="6.24,206,1774335600"; d="scan'208";a="271235747" Received: from conormcd-mobl2.ger.corp.intel.com (HELO localhost) ([10.245.244.114]) by fmviesa002-auth.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 15 Jun 2026 10:00:11 -0700 Date: Mon, 15 Jun 2026 20:00:07 +0300 From: Ville =?iso-8859-1?Q?Syrj=E4l=E4?= To: Michel =?iso-8859-1?Q?D=E4nzer?= Cc: intel-gfx@lists.freedesktop.org, intel-xe@lists.freedesktop.org, dri-devel@lists.freedesktop.org, wayland-devel@lists.freedesktop.org Subject: Re: [PATCH 0/4] drm/i915: Work harder to enable VRR based refresh rate changes on eDP Message-ID: References: <20260612144203.31715-1-ville.syrjala@linux.intel.com> <3d441831-71bc-49fd-823f-3af443e55b20@mailbox.org> <31da350f-adfc-4b2c-a7c5-5ed884ffd9ca@mailbox.org> <7402a175-a1d9-4428-8536-e37f06c1e186@mailbox.org> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <7402a175-a1d9-4428-8536-e37f06c1e186@mailbox.org> X-Patchwork-Hint: comment Organization: Intel Finland Oy - BIC 0357606-4 - c/o Alberga Business Park, 6 krs Bertel Jungin Aukio 5, 02600 Espoo, Finland X-BeenThere: intel-xe@lists.freedesktop.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Intel Xe graphics driver List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-xe-bounces@lists.freedesktop.org Sender: "Intel-xe" On Mon, Jun 15, 2026 at 03:30:00PM +0200, Michel Dänzer wrote: > On 6/15/26 15:06, Ville Syrjälä wrote: > > On Mon, Jun 15, 2026 at 11:08:59AM +0200, Michel Dänzer wrote: > >> On 6/15/26 11:06, Michel Dänzer wrote: > >>> On 6/12/26 16:41, Ville Syrjala wrote: > >>>> From: Ville Syrjälä > >>>> > >>>> Tweak the eDP fixed mode selection algorithm to allow > >>>> userspace to do refresh rate changes on VRR capable > >>>> eDP panels without full modesets. > >>>> > >>>> Ville Syrjälä (4): > >>>> drm/modes: Add DRM_MODE_MATCH_TIMINGS_VRR > >>>> drm/i915: Pass the full atomic state to .compute_config() > >>>> drm/i915/panel: Adjust intel_panel_compute_config() calling convention > >>>> drm/i915/panel: Attempt VRR based refresh rate change for > >>>> !allow_modeset > >>> > >>> What's the motivation for this approach? > >>> > >>> Per https://gitlab.gnome.org/GNOME/mutter/-/merge_requests/5091#note_2784749 , it comes as a bit of a surprise. The approach we've been discussing at display hackfests instead is to add properties for controlling the maximum & minimum refresh rates. > > > > This has nothing to do with limiting the VRR range. What we're doing > > here is selecting the actual timings to drive an internal laptop panel, > > given some random cooked up modeline from userspace. > > This use case would be covered by setting the same values for both properties. It's all irrelevant here. We might not even have VRR enabled in this case. All we want is to set the mode (or something close enough) to what userspace has requested. > > (There are other use cases where changing mode alone isn't enough though, e.g. involving the compositor setting a narrow range between maximum & minimum refresh rate) > > > > For non-VRR panels we just pick the fixed mode whose refresh rate is closest to the > > user specified mode, and reject the commit if it's not close enough (<= 1 Hz). > > Sounds like that wouldn't be good enough for some video use cases I'm afraid. > > > >>> While the approach in this series could be considered an alternative for the maximum, AFAICT it doesn't allow enforcing a minimum refresh rate which differs from the maximum and default minimum. > > > > The timings specify the absolute max refresh rate you can achieve. So > > a separate max VRR refresh rate knob would be mostly redundant, but as > > we've discussed before, it could have its uses for the non-integer > > vtotal use cases (CMRR in Intel parlance). > > None of that addresses the lack of control of the minimum refresh range. We've been over this before. Yes, a new property would be needed to limit the min refresh rate if someone wants to do that. -- Ville Syrjälä Intel