<html xmlns:v="urn:schemas-microsoft-com:vml" xmlns:o="urn:schemas-microsoft-com:office:office" xmlns:w="urn:schemas-microsoft-com:office:word" xmlns:m="http://schemas.microsoft.com/office/2004/12/omml" xmlns="http://www.w3.org/TR/REC-html40">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
<meta name="Generator" content="Microsoft Word 15 (filtered medium)">
<style><!--
/* Font Definitions */
@font-face
{font-family:"Cambria Math";
panose-1:2 4 5 3 5 4 6 3 2 4;}
@font-face
{font-family:Calibri;
panose-1:2 15 5 2 2 2 4 3 2 4;}
@font-face
{font-family:Aptos;}
@font-face
{font-family:Consolas;
panose-1:2 11 6 9 2 2 4 3 2 4;}
/* Style Definitions */
p.MsoNormal, li.MsoNormal, div.MsoNormal
{margin:0cm;
font-size:12.0pt;
font-family:"Aptos",sans-serif;}
a:link, span.MsoHyperlink
{mso-style-priority:99;
color:blue;
text-decoration:underline;}
pre
{mso-style-priority:99;
mso-style-link:"HTML Preformatted Char";
margin:0cm;
font-size:10.0pt;
font-family:"Courier New";}
span.HTMLPreformattedChar
{mso-style-name:"HTML Preformatted Char";
mso-style-priority:99;
mso-style-link:"HTML Preformatted";
font-family:Consolas;}
span.EmailStyle21
{mso-style-type:personal-reply;
font-family:"Aptos",sans-serif;
color:windowtext;}
.MsoChpDefault
{mso-style-type:export-only;
font-size:10.0pt;
mso-ligatures:none;}
@page WordSection1
{size:612.0pt 792.0pt;
margin:72.0pt 72.0pt 72.0pt 72.0pt;}
div.WordSection1
{page:WordSection1;}
--></style><!--[if gte mso 9]><xml>
<o:shapedefaults v:ext="edit" spidmax="1026" />
</xml><![endif]--><!--[if gte mso 9]><xml>
<o:shapelayout v:ext="edit">
<o:idmap v:ext="edit" data="1" />
</o:shapelayout></xml><![endif]-->
</head>
<body lang="EN-US" link="blue" vlink="purple" style="word-wrap:break-word">
<div class="WordSection1">
<p class="MsoNormal"><span style="font-size:11.0pt"><o:p> </o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt"><o:p> </o:p></span></p>
<div style="border:none;border-left:solid blue 1.5pt;padding:0cm 0cm 0cm 4.0pt">
<div>
<div style="border:none;border-top:solid #E1E1E1 1.0pt;padding:3.0pt 0cm 0cm 0cm">
<p class="MsoNormal"><b><span style="font-size:11.0pt;font-family:"Calibri",sans-serif">From:</span></b><span style="font-size:11.0pt;font-family:"Calibri",sans-serif"> Joshi, Kunal1 <kunal1.joshi@intel.com>
<br>
<b>Sent:</b> Wednesday, July 17, 2024 10:15 AM<br>
<b>To:</b> B, Jeevan <jeevan.b@intel.com>; igt-dev@lists.freedesktop.org<br>
<b>Cc:</b> Manna, Animesh <animesh.manna@intel.com><br>
<b>Subject:</b> Re: [i-g-t,1/2] lib/igt_kms: Added library functions for LOBF status<o:p></o:p></span></p>
</div>
</div>
<p class="MsoNormal"><o:p> </o:p></p>
<p>Hello Jeevan,<o:p></o:p></p>
<div>
<p class="MsoNormal">On 6/27/2024 12:45 AM, Jeevan B wrote:<o:p></o:p></p>
</div>
<blockquote style="margin-top:5.0pt;margin-bottom:5.0pt">
<pre>Added helper functions to check LOBF status to check weather<o:p></o:p></pre>
<pre>it is enabled or disabled.<o:p></o:p></pre>
<pre><o:p> </o:p></pre>
<pre>Signed-off-by: Jeevan B <a href="mailto:jeevan.b@intel.com"><jeevan.b@intel.com></a><o:p></o:p></pre>
<pre>---<o:p></o:p></pre>
<pre> lib/igt_kms.c | 23 +++++++++++++++++++++++<o:p></o:p></pre>
<pre> lib/igt_kms.h | 1 +<o:p></o:p></pre>
<pre> 2 files changed, 24 insertions(+)<o:p></o:p></pre>
<pre><o:p> </o:p></pre>
<pre>diff --git a/lib/igt_kms.c b/lib/igt_kms.c<o:p></o:p></pre>
<pre>index 174bbe6c4..1f0243eb6 100644<o:p></o:p></pre>
<pre>--- a/lib/igt_kms.c<o:p></o:p></pre>
<pre>+++ b/lib/igt_kms.c<o:p></o:p></pre>
<pre>@@ -6071,6 +6071,29 @@ void igt_dump_crtcs_fd(int drmfd)<o:p></o:p></pre>
<pre> drmModeFreeResources(mode_resources);<o:p></o:p></pre>
<pre> }<o:p></o:p></pre>
<pre> <o:p></o:p></pre>
<pre>+/**<o:p></o:p></pre>
<pre>+ * igt_get_i915_edp_lobf_info<o:p></o:p></pre>
<pre>+ * @drmfd: A drm file descriptor<o:p></o:p></pre>
<pre>+ * @connector_name: Name of the libdrm connector we're going to use<o:p></o:p></pre>
<pre>+ *<o:p></o:p></pre>
<pre>+ * Return: True if its enabled.<o:p></o:p></pre>
<pre>+ */<o:p></o:p></pre>
<pre>+bool igt_get_i915_edp_lobf_info(int drmfd, char *connector_name)<o:p></o:p></pre>
</blockquote>
<p class="MsoNormal">Small nitpick, can this be igt_get_i915_edp_lobf_status, since we are not reading other info. Also do we need to use aux wake alpm / aux less alpm status or that doesn't matter? Animesh any thoughts from you here? Thanks and Regards Kunal
Joshi<o:p></o:p></p>
<p class="MsoNormal"><span style="font-size:11.0pt"><o:p> </o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt">I also feel aux-less and aux-wake info can be optional for igt, as a debug info we are publishing through the same lobf-debugfs entry from kernel.<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt">Lobf-status maybe better though do not have any strong objection on lobf-info.<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt"><o:p> </o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt">Regards,<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt">Animesh<o:p></o:p></span></p>
<blockquote style="margin-top:5.0pt;margin-bottom:5.0pt">
<pre><o:p> </o:p></pre>
<pre><o:p> </o:p></pre>
<pre>+{<o:p></o:p></pre>
<pre>+ char buf[24];<o:p></o:p></pre>
<pre>+ int fd, res;<o:p></o:p></pre>
<pre>+<o:p></o:p></pre>
<pre>+ fd = igt_debugfs_connector_dir(drmfd, connector_name, O_RDONLY);<o:p></o:p></pre>
<pre>+ igt_assert(fd >= 0);<o:p></o:p></pre>
<pre>+<o:p></o:p></pre>
<pre>+ res = igt_debugfs_simple_read(fd, "i915_edp_lobf_info", buf, sizeof(buf));<o:p></o:p></pre>
<pre>+ igt_require(res > 0);<o:p></o:p></pre>
<pre>+<o:p></o:p></pre>
<pre>+ close(fd);<o:p></o:p></pre>
<pre>+<o:p></o:p></pre>
<pre>+ return strstr(buf, "LOBF status: enabled");<o:p></o:p></pre>
<pre>+}<o:p></o:p></pre>
<pre>+<o:p></o:p></pre>
<pre> /**<o:p></o:p></pre>
<pre> * igt_get_output_max_bpc:<o:p></o:p></pre>
<pre> * @drmfd: A drm file descriptor<o:p></o:p></pre>
<pre>diff --git a/lib/igt_kms.h b/lib/igt_kms.h<o:p></o:p></pre>
<pre>index 01604dac9..5b1591631 100644<o:p></o:p></pre>
<pre>--- a/lib/igt_kms.h<o:p></o:p></pre>
<pre>+++ b/lib/igt_kms.h<o:p></o:p></pre>
<pre>@@ -1195,6 +1195,7 @@ void igt_require_pipe(igt_display_t *display,<o:p></o:p></pre>
<pre> void igt_dump_connectors_fd(int drmfd);<o:p></o:p></pre>
<pre> void igt_dump_crtcs_fd(int drmfd);<o:p></o:p></pre>
<pre> bool igt_override_all_active_output_modes_to_fit_bw(igt_display_t *display);<o:p></o:p></pre>
<pre>+bool igt_get_i915_edp_lobf_info(int drmfd, char *connector_name);<o:p></o:p></pre>
<pre> unsigned int igt_get_output_max_bpc(int drmfd, char *connector_name);<o:p></o:p></pre>
<pre> unsigned int igt_get_pipe_current_bpc(int drmfd, enum pipe pipe);<o:p></o:p></pre>
<pre> void igt_assert_output_bpc_equal(int drmfd, enum pipe pipe,<o:p></o:p></pre>
</blockquote>
</div>
</div>
</body>
</html>