Re: [i-g-t,1/2] lib/igt_kms: Added library functions for LOBF status

["Joshi, Kunal1" <kunal1.joshi@intel.com>]

[-- Attachment #1: Type: text/plain, Size: 2099 bytes --]

Hello Jeevan,

On 6/27/2024 12:45 AM, Jeevan B wrote:
> Added helper functions to check LOBF status to check weather
> it is enabled or disabled.
>
> Signed-off-by: Jeevan B<jeevan.b@intel.com>
> ---
>   lib/igt_kms.c | 23 +++++++++++++++++++++++
>   lib/igt_kms.h |  1 +
>   2 files changed, 24 insertions(+)
>
> diff --git a/lib/igt_kms.c b/lib/igt_kms.c
> index 174bbe6c4..1f0243eb6 100644
> --- a/lib/igt_kms.c
> +++ b/lib/igt_kms.c
> @@ -6071,6 +6071,29 @@ void igt_dump_crtcs_fd(int drmfd)
>   	drmModeFreeResources(mode_resources);
>   }
>   
> +/**
> + * igt_get_i915_edp_lobf_info
> + * @drmfd: A drm file descriptor
> + * @connector_name: Name of the libdrm connector we're going to use
> + *
> + * Return: True if its enabled.
> + */
> +bool igt_get_i915_edp_lobf_info(int drmfd, char *connector_name)
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
> +{
> +	char buf[24];
> +	int fd, res;
> +
> +	fd = igt_debugfs_connector_dir(drmfd, connector_name, O_RDONLY);
> +	igt_assert(fd >= 0);
> +
> +	res = igt_debugfs_simple_read(fd, "i915_edp_lobf_info", buf, sizeof(buf));
> +	igt_require(res > 0);
> +
> +	close(fd);
> +
> +	return strstr(buf, "LOBF status: enabled");
> +}
> +
>   /**
>    * igt_get_output_max_bpc:
>    * @drmfd: A drm file descriptor
> diff --git a/lib/igt_kms.h b/lib/igt_kms.h
> index 01604dac9..5b1591631 100644
> --- a/lib/igt_kms.h
> +++ b/lib/igt_kms.h
> @@ -1195,6 +1195,7 @@ void igt_require_pipe(igt_display_t *display,
>   void igt_dump_connectors_fd(int drmfd);
>   void igt_dump_crtcs_fd(int drmfd);
>   bool igt_override_all_active_output_modes_to_fit_bw(igt_display_t *display);
> +bool igt_get_i915_edp_lobf_info(int drmfd, char *connector_name);
>   unsigned int igt_get_output_max_bpc(int drmfd, char *connector_name);
>   unsigned int igt_get_pipe_current_bpc(int drmfd, enum pipe pipe);
>   void igt_assert_output_bpc_equal(int drmfd, enum pipe pipe,

[-- Attachment #2: Type: text/html, Size: 2505 bytes --]