<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=us-ascii">
</head>
<body>
<div dir="auto">Hi</div>
<div dir="auto"><br>
</div>
<div dir="auto">I think so far with my trials port clock changes in sync with phys. As pmdemand atomic check is called many times, I don't think optimizing is bad.</div>
<div dir="auto"><br>
</div>
<div dir="auto">Br</div>
<div dir="auto">Vinod</div>
<div id="mail-editor-reference-message-container" dir="auto"><br>
<hr style="display:inline-block;width:98%" tabindex="-1">
<div id="divRplyFwdMsg" style="font-size: 11pt;"><strong>From:</strong> Sousa, Gustavo <gustavo.sousa@intel.com><br>
<strong>Sent:</strong> Thursday, 25 May 2023, 17:58<br>
<strong>To:</strong> Govindapillai, Vinod <vinod.govindapillai@intel.com>; intel-gfx@lists.freedesktop.org <intel-gfx@lists.freedesktop.org><br>
<strong>Cc:</strong> Syrjala, Ville <ville.syrjala@intel.com>; Lisovskiy, Stanislav <stanislav.lisovskiy@intel.com>; Kahola, Mika <mika.kahola@intel.com>; Saarinen, Jani <jani.saarinen@intel.com><br>
<strong>Subject:</strong> Re: [PATCH v8 7/7] drm/i915/mtl: Add support for PM DEMAND<br>
</div>
<br>
<meta name="Generator" content="Microsoft Exchange Server">
<!-- converted from text --><font size="2"><span style="font-size:11pt;">
<div class="PlainText">Quoting Govindapillai, Vinod (2023-05-25 11:31:05-03:00)<br>
> Hi<br>
><br>
> (...)<br>
><br>
> Can only the phys change without impacting other parameters here?<br>
<br>
Not sure to be honest, but, if so, I think we would covering cases like<br>
that with the proposed change. IMO, if we depend on PHYs being<br>
active/non-active, we should track that to in pmdemand_needs_update() be<br>
safe.<br>
<br>
One scenario I could think of is an update to switch connectors while<br>
keeping current pipe configuration.<br>
<br>
--<br>
Gustavo Sousa<br>
</div>
</span></font><br>
</div>
</body>
</html>