public inbox for igt-dev@lists.freedesktop.org
 help / color / mirror / Atom feed
From: "Joshi, Kunal1" <kunal1.joshi@intel.com>
To: Louis Chauvet <louis.chauvet@bootlin.com>, igt-dev@lists.freedesktop.org
Cc: thomas.petazzoni@bootlin.com, luca.ceresoli@bootlin.com,
	kory.maincent@bootlin.com, markyacoub@google.com,
	khaled.almahallawy@intel.com
Subject: Re: [i-g-t,v7,26/48] lib/unigraf: Add plug/unplug helpers
Date: Tue, 3 Mar 2026 23:46:52 +0530	[thread overview]
Message-ID: <af094f53-97fe-475a-9d08-3332a4daa7dd@intel.com> (raw)
In-Reply-To: <20260223-unigraf-integration-v7-26-73dc9143c8c1@bootlin.com>

Hello Louis,

On 23-02-2026 17:39, Louis Chauvet wrote:
> The unigraf can emulate a full cable plug/unplug, so add helper to emulate
> this.
>
> Reviewed-by: Kory Maincent <kory.maincent@bootlin.com>
> Reviewed-by: Luca Ceresoli <luca.ceresoli@bootlin.com>
> Signed-off-by: Louis Chauvet <louis.chauvet@bootlin.com>
> ---
>   lib/unigraf/unigraf.c | 28 +++++++++++++++++++++++++++-
>   lib/unigraf/unigraf.h |  4 ++++
>   2 files changed, 31 insertions(+), 1 deletion(-)
>
> diff --git a/lib/unigraf/unigraf.c b/lib/unigraf/unigraf.c
> index 5812f46fe6ae..f148d4fc099f 100644
> --- a/lib/unigraf/unigraf.c
> +++ b/lib/unigraf/unigraf.c
> @@ -307,7 +307,9 @@ void unigraf_require_device(int drm_fd)
>    * and link parameters to default values.
>    */
>   void unigraf_reset(void)
> -{}
> +{
> +	unigraf_plug();
> +}
>   
>   /**
>    * unigraf_hpd_assert() - Assert Hot Plug Detect signal
> @@ -345,3 +347,27 @@ void unigraf_hpd_deassert(void)
>   {
>   	unigraf_write_u32(TSI_FORCE_HOT_PLUG_STATE_W, 0);
>   }
> +
> +/**
> + * unigraf_plug() - Emulate a cable unplug
> + *
> + * This function will emulate a full cable unplug (not a simple HPD line change)
> + */
> +void unigraf_unplug(void)
> +{
> +	int d = 2 << 2;
Let's avoid magic numbers
> +
> +	unigraf_write_u32(TSI_DPRX_HPD_FORCE, d);
> +}
> +
> +/**
> + * unigraf_plug() - Emulate a cable plug
> + *
> + * This function will emulate a full cable plug (not a simple HPD line change)
> + */
> +void unigraf_plug(void)
> +{
> +	int d = 3 << 2;
Same here

Thanks and Regards
Kunal Joshi
> +
> +	unigraf_write_u32(TSI_DPRX_HPD_FORCE, d);
> +}
> diff --git a/lib/unigraf/unigraf.h b/lib/unigraf/unigraf.h
> index 77fba4e53912..ae1f0d8955ab 100644
> --- a/lib/unigraf/unigraf.h
> +++ b/lib/unigraf/unigraf.h
> @@ -48,4 +48,8 @@ void unigraf_hpd_pulse(int duration);
>   
>   void unigraf_hpd_assert(void);
>   
> +void unigraf_plug(void);
> +
> +void unigraf_unplug(void);
> +
>   #endif // UNIGRAF_H

  reply	other threads:[~2026-03-03 18:17 UTC|newest]

Thread overview: 82+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-02-23 12:09 [PATCH i-g-t v7 00/48] Unigraf integration Louis Chauvet
2026-02-23 12:09 ` [PATCH i-g-t v7 01/48] lib/igt_kms: Add a detect timeout value Louis Chauvet
2026-02-23 13:21   ` Kory Maincent
2026-02-23 12:09 ` [PATCH i-g-t v7 02/48] lib/igt_kms: Add helper to wait for a specific status on a connector Louis Chauvet
2026-02-23 13:23   ` Kory Maincent
2026-02-23 12:09 ` [PATCH i-g-t v7 03/48] lib/igt_kms: Add function to list connected connectors Louis Chauvet
2026-02-23 13:41   ` Kory Maincent
2026-02-26  8:35     ` Kamil Konieczny
2026-02-23 12:09 ` [PATCH i-g-t v7 04/48] lib/igt_kms: Add helper to obtain a connector by its name or MST path Louis Chauvet
2026-02-23 12:09 ` [PATCH i-g-t v7 05/48] lib/igt_kms: Add helper to wait for new connectors Louis Chauvet
2026-02-23 14:00   ` Kory Maincent
2026-02-26 16:17   ` Kamil Konieczny
2026-02-23 12:09 ` [PATCH i-g-t v7 06/48] lib/igt_kms: Add helper to get a pipe from a connector Louis Chauvet
2026-03-03 18:00   ` [i-g-t,v7,06/48] " Joshi, Kunal1
2026-02-23 12:09 ` [PATCH i-g-t v7 07/48] lib/igt_kms: Expose dump_connector_attrs Louis Chauvet
2026-02-23 12:09 ` [PATCH i-g-t v7 08/48] lib/igt_kms: Expose reset_connectors_at_exit Louis Chauvet
2026-02-23 12:09 ` [PATCH i-g-t v7 09/48] lib/igt_kms: Expose connector_attr_set and igt_connector_attr_set Louis Chauvet
2026-02-23 12:09 ` [PATCH i-g-t v7 10/48] lib/igt_debugfs: Move debugfs helpers to the proper location Louis Chauvet
2026-02-23 12:09 ` [PATCH i-g-t v7 11/48] lib/igt_debugfs: Add const when make sense Louis Chauvet
2026-02-23 12:09 ` [PATCH i-g-t v7 12/48] lib/igt_amd: " Louis Chauvet
2026-02-23 12:09 ` [PATCH i-g-t v7 13/48] lib/igt_kms: " Louis Chauvet
2026-02-23 12:09 ` [PATCH i-g-t v7 14/48] lib/monitor_edids: Add helper functions for using monitor_edid objects Louis Chauvet
2026-02-23 14:07   ` Kory Maincent
2026-02-23 12:09 ` [PATCH i-g-t v7 15/48] lib/monitor_edids: Add helper to get an EDID by its name Louis Chauvet
2026-02-23 12:09 ` [PATCH i-g-t v7 16/48] lib/monitor_edids: Add helper to print all available EDID names Louis Chauvet
2026-02-23 12:09 ` [PATCH i-g-t v7 17/48] lib/unigraf: Add used defines for TSI_Types Louis Chauvet
2026-02-23 12:09 ` [PATCH i-g-t v7 18/48] lib/unigraf: Add TSI.h Louis Chauvet
2026-02-23 12:09 ` [PATCH i-g-t v7 19/48] lib/unigraf: Initial Unigraf support Louis Chauvet
2026-02-23 14:12   ` Kory Maincent
2026-02-23 12:09 ` [PATCH i-g-t v7 20/48] lib/igt_kms: Automatically connect unigraf on display require Louis Chauvet
2026-02-23 14:13   ` Kory Maincent
2026-02-23 12:09 ` [PATCH i-g-t v7 21/48] lib/unigraf: Introduce device configuration Louis Chauvet
2026-02-23 14:22   ` Kory Maincent
2026-02-23 12:09 ` [PATCH i-g-t v7 22/48] lib/unigraf: Introduce role configuration Louis Chauvet
2026-02-23 12:09 ` [PATCH i-g-t v7 23/48] lib/unigraf: Introduce input configuration Louis Chauvet
2026-02-23 14:26   ` Kory Maincent
2026-02-23 12:09 ` [PATCH i-g-t v7 24/48] lib/unigraf: Add reset function Louis Chauvet
2026-02-23 14:26   ` Kory Maincent
2026-02-23 12:09 ` [PATCH i-g-t v7 25/48] lib/unigraf: Add unigraf assert and deassert helpers Louis Chauvet
2026-02-23 14:31   ` Kory Maincent
2026-02-23 12:09 ` [PATCH i-g-t v7 26/48] lib/unigraf: Add plug/unplug helpers Louis Chauvet
2026-03-03 18:16   ` Joshi, Kunal1 [this message]
2026-02-23 12:09 ` [PATCH i-g-t v7 27/48] lib/unigraf: Allows sst/mst configuration Louis Chauvet
2026-02-23 14:34   ` Kory Maincent
2026-02-23 12:09 ` [PATCH i-g-t v7 28/48] lib/unigraf: Add helpers to read and write edid Louis Chauvet
2026-02-23 14:37   ` Kory Maincent
2026-02-23 12:09 ` [PATCH i-g-t v7 29/48] lib/unigraf: Add connector and EDID configuration Louis Chauvet
2026-02-23 15:03   ` Kory Maincent
2026-02-23 12:09 ` [PATCH i-g-t v7 30/48] tests/unigraf: Add basic unigraf tests Louis Chauvet
2026-02-23 15:11   ` Kory Maincent
2026-02-23 12:09 ` [PATCH i-g-t v7 31/48] lib/unigraf: Add unigraf CRC capture Louis Chauvet
2026-02-23 12:09 ` [PATCH i-g-t v7 32/48] lib/unigraf: Add configuration for CRC usage Louis Chauvet
2026-02-23 12:09 ` [PATCH i-g-t v7 33/48] lib/unigraf: add unigraf_get_connector_by_stream Louis Chauvet
2026-02-23 15:13   ` Kory Maincent
2026-02-23 12:09 ` [PATCH i-g-t v7 34/48] lib/unigraf: Add helper to check timings received by unigraf Louis Chauvet
2026-02-23 12:09 ` [PATCH i-g-t v7 35/48] lib/igt_pipe_crc: Add unigraf crc calculation Louis Chauvet
2026-02-23 15:16   ` Kory Maincent
2026-02-23 12:09 ` [PATCH i-g-t v7 36/48] lib/i915/dp: Move DP-related function for i915 to proper folder Louis Chauvet
2026-02-23 15:43   ` Kory Maincent
2026-02-23 12:09 ` [PATCH i-g-t v7 37/48] lib/i915/dp: Rename functions to avoid confusion Louis Chauvet
2026-02-23 12:09 ` [PATCH i-g-t v7 38/48] lib/i915/dp: Add helper to get maximum supported rate Louis Chauvet
2026-02-23 15:45   ` Kory Maincent
2026-02-23 12:09 ` [PATCH i-g-t v7 39/48] lib/i915/dp: Properly check sscanf results Louis Chauvet
2026-02-23 12:09 ` [PATCH i-g-t v7 40/48] lib/i915/dp: Use igt_output_name instead of private field Louis Chauvet
2026-02-23 12:09 ` [PATCH i-g-t v7 41/48] lib/igt_dp: Create generic helpers for DP information Louis Chauvet
2026-02-23 15:56   ` Kory Maincent
2026-02-25 10:00     ` Louis Chauvet
2026-02-25 10:20       ` Kory Maincent
2026-02-26 15:56   ` Kamil Konieczny
2026-02-23 12:09 ` [PATCH i-g-t v7 42/48] lib/igt_kms: Add asserts to avoid null pointer dereference Louis Chauvet
2026-02-23 12:09 ` [PATCH i-g-t v7 43/48] lib/igt_kms: Add helper to get a CRTC from an output Louis Chauvet
2026-02-23 16:09   ` Kory Maincent
2026-02-23 12:09 ` [PATCH i-g-t v7 44/48] lib/unigraf: Add lane count configuration Louis Chauvet
2026-02-23 12:09 ` [PATCH i-g-t v7 45/48] docs/unigraf: Add unigraf documentation Louis Chauvet
2026-02-23 16:11   ` Kory Maincent
2026-02-23 12:09 ` [PATCH i-g-t v7 46/48] lib/unigraf: Add helpers to set maximum link rate Louis Chauvet
2026-02-23 16:14   ` Kory Maincent
2026-02-23 12:09 ` [PATCH i-g-t v7 47/48] lib/unigraf: Add helpers to get the current LT status Louis Chauvet
2026-02-23 12:09 ` [PATCH i-g-t v7 48/48] tests/unigraf/unigraf_lt: Add test for link training Louis Chauvet
2026-02-23 13:30 ` ✓ Xe.CI.BAT: success for Unigraf integration (rev6) Patchwork
2026-02-23 13:44 ` ✗ i915.CI.BAT: failure " Patchwork
2026-02-24  6:15 ` ✗ Xe.CI.FULL: " Patchwork

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=af094f53-97fe-475a-9d08-3332a4daa7dd@intel.com \
    --to=kunal1.joshi@intel.com \
    --cc=igt-dev@lists.freedesktop.org \
    --cc=khaled.almahallawy@intel.com \
    --cc=kory.maincent@bootlin.com \
    --cc=louis.chauvet@bootlin.com \
    --cc=luca.ceresoli@bootlin.com \
    --cc=markyacoub@google.com \
    --cc=thomas.petazzoni@bootlin.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox