On Sun, Jan 26, 2025 at 04:46:49PM +, Chris Bainbridge wrote:
> Hardware is HP Pavilion Aero 13 laptop with Dell WD19 dock and three
> external monitors. I get a warning with recent kernel builds when
> enabling the external monitors with xrandr after initial boot:
>
> 16:57:49 kernel: WARNING
On Thu, Dec 19, 2024 at 11:34:01PM +0200, Jani Nikula wrote:
> Add ACT handling for 128b/132b SST.
>
> This is preparation for enabling 128b/132b SST. This path is not
> reachable yet.
>
> Signed-off-by: Jani Nikula
> ---
> drivers/gpu/drm/i915/display/intel_ddi.c | 13 +
> 1 file c
gt; dsc_needed = true;
> }
> @@ -3148,8 +3155,13 @@ intel_dp_compute_config(struct intel_encoder *encoder,
> pipe_config->limited_color_range =
> intel_dp_limited_color_range(pipe_config, conn_state);
>
> - pipe_config-
On Thu, Dec 19, 2024 at 11:34:04PM +0200, Jani Nikula wrote:
> 128b/1232b SST will have mst_master_transcoder set and matching
> cpu_transcoder. Ensure disable also for 128b/132b SST.
>
> Co-developed-by: Imre Deak
> Signed-off-by: Imre Deak
> Signed-off-by: Jani Nikula
Revi
28b/132b SST path (Imre)
>
> Signed-off-by: Jani Nikula
Reviewed-by: Imre Deak
> ---
> drivers/gpu/drm/i915/display/intel_ddi.c | 19 +--
> 1 file changed, 17 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/display/intel_ddi.c
&
ware state, i.e. intel_dp->is_mst. It should be fine
> for the state checker, but for hardware takeover at probe, we'll have to
> trust the GOP has only enabled SST.
>
> TODO: Not sure how this *or* our current code handles 128b/132b enabled
> by GOP.
>
> Cc: Imre Deak
On Thu, Jan 02, 2025 at 12:52:38PM +0200, Jani Nikula wrote:
> On Tue, 31 Dec 2024, Imre Deak wrote:
> > On Thu, Dec 19, 2024 at 11:33:59PM +0200, Jani Nikula wrote:
> >> Write the payload allocation table for 128b/132b SST. Use VCPID 1 and
> >> start from s
On Thu, Jan 02, 2025 at 12:30:34PM +0200, Jani Nikula wrote:
> On Tue, 31 Dec 2024, Imre Deak wrote:
> > On Thu, Dec 19, 2024 at 11:33:56PM +0200, Jani Nikula wrote:
> >> Handle 128b/132b SST in intel_dp_mtp_tu_compute_config(). The remote
> >> bandwidth overhead and ti
On Thu, Jan 02, 2025 at 11:39:07AM +0200, Jani Nikula wrote:
> On Tue, 31 Dec 2024, Imre Deak wrote:
> > On Thu, Dec 19, 2024 at 11:34:00PM +0200, Jani Nikula wrote:
> >> Write the DP2 specific VFREQ registers.
> >>
> >> This is preparation for enab
On Thu, Dec 19, 2024 at 11:34:00PM +0200, Jani Nikula wrote:
> Write the DP2 specific VFREQ registers.
>
> This is preparation for enabling 128b/132b SST. This path is not
> reachable yet.
>
> Signed-off-by: Jani Nikula
> ---
> drivers/gpu/drm/i915/display/intel_ddi.c | 12
> 1 fil
TRANS_DP2_VFREQ_PIXEL_CLOCK(crtc_clock_hz &
> 0xff));
> + }
> +
Nit: This could be in a helper, used by the MST encoder as well. Either way:
Reviewed-by: Imre Deak
> intel_ddi_enable_transcoder_func(encoder, crtc_state);
>
> /* Enable/Disable DP2.0 SDP split config before transcoder */
> --
> 2.39.5
>
->aux, 1, 0,
> crtc_state->dp_m_n.tu);
I would handle the error by sending a modeset retry uevent. Either way:
Reviewed-by: Imre Deak
> + }
> +
> if (!is_mst)
> intel_dsc_dp_pps_write(encoder, crtc_state);
> }
> @@ -2808,6 +2814,11 @@ s
t
> reachable yet.
>
> Signed-off-by: Jani Nikula
Reviewed-by: Imre Deak
> ---
> drivers/gpu/drm/i915/display/intel_ddi.c | 6 --
> 1 file changed, 4 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/display/intel_ddi.c
> b/drivers/gpu/drm/i
to also set transport select (Imre)
>
> Signed-off-by: Jani Nikula
Reviewed-by: Imre Deak
> ---
> drivers/gpu/drm/i915/display/intel_ddi.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/i915/display/intel_ddi.c
> b/dri
* pbn_div.full;
> -
> - drm_WARN_ON(display->drm, remote_tu < crtc_state->dp_m_n.tu);
> - crtc_state->dp_m_n.tu = remote_tu;
> + if (intel_dp->is_mst) {
> + int remote_bw_overhead;
> +
load_bw(crtc_state->port_clock,
> + crtc_state->lane_count);
Nit: looking this up from drm_dp_mst_topology_state (once SST and MST is
separated done only in the MST case) instead of recalculating the same
would be better. Either way:
Reviewed-by:
On Thu, Dec 19, 2024 at 11:33:54PM +0200, Jani Nikula wrote:
> The crtc_state->pbn member is only used as a temporary variable within
> mst_stream_find_vcpi_slots_for_bpp(). Remove it as unnecessary.
>
> Suggested-by: Imre Deak
> Signed-off-by: Jani Nikula
Revi
ead of slots from mst_stream_find_vcpi_slots_for_bpp() for success,
> and simplify the callers.
>
> There's a pointless ret local variable that we can drop in the process.
>
> Signed-off-by: Jani Nikula
Reviewed-by: Imre Deak
> ---
> drivers/gpu/drm/i915/display/int
On Thu, Dec 19, 2024 at 11:33:52PM +0200, Jani Nikula wrote:
> intel_dp_mst_compute_m_n() doesn't need the connector. Remove the
> parameter.
>
> Signed-off-by: Jani Nikula
Reviewed-by: Imre Deak
> ---
> drivers/gpu/drm/i915/display/intel_dp_mst.c | 3 +--
> 1 file
On Thu, Dec 19, 2024 at 11:33:51PM +0200, Jani Nikula wrote:
> intel_dp_mst_bw_overhead() doesn't need the connector. Remove the
> parameter.
>
> Signed-off-by: Jani Nikula
Reviewed-by: Imre Deak
> ---
> drivers/gpu/drm/i915/display/intel_dp_mst.c | 5 ++---
> 1
be used in non-MST contexts without the
> topology manager.
>
> Cc: Imre Deak
> Cc: Lyude Paul
> Signed-off-by: Jani Nikula
> ---
> [...]
> diff --git a/drivers/gpu/drm/tests/drm_dp_mst_helper_test.c
> b/drivers/gpu/drm/tests/drm_dp_mst_helper_test.c
>
On Thu, Dec 12, 2024 at 01:03:17AM +0200, Imre Deak wrote:
The patchset at [2] is pushed now to drm-misc-next, thanks for the
reviews, acks and ideas.
While merging, I fixed the typos in patch 3, 5-7 and removed references
to 'patches', 'patchsets' in the commit logs of patc
Hi Harry, Leo, Alex, Wayne,
could you please ack this change as well?
Thanks,
Imre
A typo below in the commit log, can fix it while merging the patch here and in
the i915/nouveau patches.
On Thu, Dec 12, 2024 at 01:03:23AM +0200, Imre Deak wrote:
> After a connector is added to
Hi Harry, Leo, Alex, Wayne,
could you please ack this change?
Thanks,
Imre
On Thu, Dec 12, 2024 at 01:03:21AM +0200, Imre Deak wrote:
> MST connectors should be initialized/registered by calling
> drm_connector_dynamic_init()/drm_connector_dynamic_register(). The
> previous patch add
Hi Thomas, Maxime, Maarten,
are you ok to merge patches 1-9 to drm-misc-next? Patches 10,11 could be
merged then to drm-intel-next after back-merging drm-misc-next to
drm-intel-next.
Thanks,
Imre
On Thu, Dec 12, 2024 at 01:03:17AM +0200, Imre Deak wrote:
> This is v3 of [1], with the follow
On Mon, Dec 16, 2024 at 01:03:42PM +0200, Jani Nikula wrote:
> On Fri, 13 Dec 2024, Imre Deak wrote:
> > On Thu, Dec 12, 2024 at 12:12:15PM +0200, Jani Nikula wrote:
> >> On Thu, 12 Dec 2024, Imre Deak wrote:
> >> > MST connectors should be ini
On Thu, Dec 12, 2024 at 12:12:15PM +0200, Jani Nikula wrote:
> On Thu, 12 Dec 2024, Imre Deak wrote:
> > MST connectors should be initialized/registered by calling
> > drm_connector_dynamic_init()/drm_connector_dynamic_register(). The
> > previous patch adding these functio
On Thu, Dec 12, 2024 at 12:06:12PM +0200, Jani Nikula wrote:
> On Thu, 12 Dec 2024, Imre Deak wrote:
> > The connectors enumerated by the GETRESOURCES ioctl may not be fully
> > initialized yet wrt. to the state set up during connector registration
> > (for instance the conn
On Thu, Dec 12, 2024 at 12:25:38PM +0200, Jani Nikula wrote:
> On Thu, 12 Dec 2024, Imre Deak wrote:
> > Follow the canonical way in intel_dp_mst.c, referencing a connector only
> > via a struct intel_connector pointer and naming this pointer 'connector'
> > instead
On Thu, Dec 12, 2024 at 12:10:58PM +0200, Jani Nikula wrote:
> [...]
> On Thu, 12 Dec 2024, Imre Deak wrote:
> > @@ -863,9 +866,14 @@ EXPORT_SYMBOL(drm_connector_dynamic_register);
> > * drm_connector_unregister - unregister a connector
> > * @connector: the
On Thu, Dec 12, 2024 at 12:04:17PM +0200, Jani Nikula wrote:
> On Thu, 12 Dec 2024, Imre Deak wrote:
> > Atm when the connector is added to the drm_mode_config::connector_list,
> > the connector may not be fully initialized yet. This is not a problem
> > for static connect
which doesn't add the connector to
the list - and registering it with drm_connector_dynamic_register() -
which adds the connector to the list - after the initialization is
complete.
Cc: Karol Herbst
Cc: Lyude Paul
Cc: Danilo Krummrich
Signed-off-by: Imre Deak
---
drivers/gpu/drm/nouvea
After an error during adding an MST connector the MST port and the
intel_connector object could be leaked, fix this up.
Reviewed-by: Rodrigo Vivi
Signed-off-by: Imre Deak
---
drivers/gpu/drm/i915/display/intel_dp_mst.c | 17 +
1 file changed, 9 insertions(+), 8 deletions
which doesn't add the connector to
the list - and registering it with drm_connector_dynamic_register() -
which adds the connector to the list - after the initialization is
complete.
Cc: Harry Wentland
Cc: Leo Li
Cc: Wayne Lin
Cc: Alex Deucher
Cc: Lyude Paul
Signed-off-by: Imre Deak
---
d
to WARN in such
cases.
Cc: Lyude Paul
Cc: Harry Wentland
Cc: Leo Li
Cc: Wayne Lin
Cc: Alex Deucher
Cc: Karol Herbst
Cc: Danilo Krummrich
Signed-off-by: Imre Deak
---
drivers/gpu/drm/display/drm_dp_mst_topology.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu
nnector, calling
the drm_connector pointer _connector.
Suggested-by: Jani Nikula
Signed-off-by: Imre Deak
---
drivers/gpu/drm/i915/display/intel_dp_mst.c | 180 ++--
1 file changed, 88 insertions(+), 92 deletions(-)
diff --git a/drivers/gpu/drm/i915/display/intel_dp_mst.c
b/drivers/
Add kunit tests for
drm_connector_dynamic_init()/drm_connector_dynamic_register() added in
this patchset.
Suggested-by: Maxime Ripard
Signed-off-by: Imre Deak
---
drivers/gpu/drm/tests/drm_connector_test.c | 463 +
1 file changed, 463 insertions(+)
diff --git a/drivers/gpu
ons incorrectly).
Signed-off-by: Imre Deak
---
drivers/gpu/drm/drm_connector.c | 6 +-
1 file changed, 1 insertion(+), 5 deletions(-)
diff --git a/drivers/gpu/drm/drm_connector.c b/drivers/gpu/drm/drm_connector.c
index c322dbf6e3161..1bd7407223fbb 100644
--- a/drivers/gpu/drm/drm_connector.c
++
en calling
drm_connector_dynamic_register().
Cc: Lyude Paul
Reviewed-by: Rodrigo Vivi (v1)
Signed-off-by: Imre Deak
---
drivers/gpu/drm/i915/display/intel_dp_mst.c | 27 +++--
1 file changed, 9 insertions(+), 18 deletions(-)
diff --git a/drivers/gpu/drm/i915/display/intel_dp_mst.c
b/drive
which should be a nop
for them and hence are scheduled to be removed. Update the function
documentation for these functions accordingly.
Signed-off-by: Imre Deak
---
drivers/gpu/drm/drm_connector.c | 22 +++---
1 file changed, 15 insertions(+), 7 deletions(-)
diff --git a/drivers
r
Signed-off-by: Imre Deak
---
drivers/gpu/drm/drm_mode_config.c | 9 +
1 file changed, 9 insertions(+)
diff --git a/drivers/gpu/drm/drm_mode_config.c
b/drivers/gpu/drm/drm_mode_config.c
index 37d2e0a4ef4be..8642a2fb25a90 100644
--- a/drivers/gpu/drm/drm_mode_config.c
+++ b
the
commit log according to the above changes.
- Update the commit log describing the problematic scenario during
connector detection. (Maxime)
Cc: Jani Nikula
Cc: Simona Vetter
Cc: Maxime Ripard
Reviewed-by: Rodrigo Vivi (v1)
Signed-off-by: Imre Deak
---
drivers
Nikula
Cc: Simona Vetter
Cc: Maxime Ripard
Cc: Lyude Paul
Cc: Harry Wentland
Cc: Leo Li
Cc: Wayne Lin
Cc: Alex Deucher
Cc: Karol Herbst
Cc: Danilo Krummrich
[1] https://lore.kernel.org/all/20241126161859.1858058-1-imre.d...@intel.com
Imre Deak (11):
drm/connector: Add a way to init
On Wed, Dec 04, 2024 at 03:30:24PM +, Patchwork wrote:
> == Series Details ==
>
> Series: drm/dp_mst: Fix a few side-band message handling issues (rev3)
> URL : https://patchwork.freedesktop.org/series/142057/
> State : success
Thanks for the review, patchset is pushed to drm-misc-fixes.
>
ff-by: Imre Deak
---
drivers/gpu/drm/display/drm_dp_mst_topology.c | 24 ++-
1 file changed, 18 insertions(+), 6 deletions(-)
diff --git a/drivers/gpu/drm/display/drm_dp_mst_topology.c
b/drivers/gpu/drm/display/drm_dp_mst_topology.c
index 895c78806f0c5..7a0e757b712c7 100644
--- a/dr
ter verify_rx_request_type() fails.
Cc: Lyude Paul
Signed-off-by: Imre Deak
---
drivers/gpu/drm/display/drm_dp_mst_topology.c | 12 +---
1 file changed, 9 insertions(+), 3 deletions(-)
diff --git a/drivers/gpu/drm/display/drm_dp_mst_topology.c
b/drivers/gpu/drm/display/drm_dp_mst_topolog
/use-after-free of mst_primary in
drm_dp_mst_handle_up_req().
Avoid the above by holding a reference for mst_primary in
drm_dp_mst_handle_up_req() while it's used.
Cc: Lyude Paul
Signed-off-by: Imre Deak
---
drivers/gpu/drm/display/drm_dp_mst_topology.c | 23 ++-
1 file ch
Simplify the error return path in drm_dp_mst_handle_down_rep(),
preparing for the next patch.
While at it use reset_msg_rx_state() instead of open-coding it.
Cc: Lyude Paul
Signed-off-by: Imre Deak
---
drivers/gpu/drm/display/drm_dp_mst_topology.c | 7 +--
1 file changed, 1 insertion
After an out-of-memory error the reception state should be reset, so
that the next attempt receiving a message doesn't fail (due to getting a
start-of-message packet, while the reception state has already the
start-of-message flag set).
Cc: Lyude Paul
Signed-off-by: Imre Deak
---
driver
This patchset fixes a few issues related to MST side-band message
handling reported by IGT CI (patch 1), by a user (patch 2) and ones I
noticed during debugging (patch 4-6).
Cc: Lyude Paul
Imre Deak (7):
drm/dp_mst: Fix resetting msg rx state after topology removal
drm/dp_mst: Verify
freed txmsg, hence leading to a use-after-free in
drm_dp_mst_handle_down_rep().
Prevent the above by holding the drm_dp_mst_topology_mgr::qlock in
drm_dp_mst_handle_down_rep() for the whole duration txmsg is looked up
from the request list and dereferenced.
Cc: Lyude Paul
Signed-off-by: Imre Deak
Use reset_msg_rx_state() in drm_dp_mst_handle_up_req() instead of
open-coding it.
Cc: Lyude Paul
Signed-off-by: Imre Deak
---
drivers/gpu/drm/display/drm_dp_mst_topology.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/display/drm_dp_mst_topology.c
b
804
Signed-off-by: Imre Deak
---
drivers/gpu/drm/display/drm_dp_mst_topology.c | 31 +++
1 file changed, 31 insertions(+)
diff --git a/drivers/gpu/drm/display/drm_dp_mst_topology.c
b/drivers/gpu/drm/display/drm_dp_mst_topology.c
index 1475aa95ab6b2..bcf3a33123be1 100644
--
nect")
Closes: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/13056
Signed-off-by: Imre Deak
---
drivers/gpu/drm/display/drm_dp_mst_topology.c | 21 +--
include/drm/display/drm_dp_mst_helper.h | 7 +++
2 files changed, 26 insertions(+), 2 deletions(-)
dif
On Mon, Dec 02, 2024 at 04:07:56PM +0100, Maxime Ripard wrote:
> On Mon, Dec 02, 2024 at 03:24:43PM +0200, Imre Deak wrote:
> > On Mon, Dec 02, 2024 at 02:07:36PM +0200, Jani Nikula wrote:
> > > On Mon, 02 Dec 2024, Maxime Ripard wrote:
> > > > It's not about wh
On Mon, Dec 02, 2024 at 05:35:34PM +0100, Simona Vetter wrote:
> On Tue, Nov 26, 2024 at 06:18:56PM +0200, Imre Deak wrote:
> > Atm when the connector is added to the drm_mode_config::connector_list,
> > the connector may not be fully initialized yet. This is not a problem
>
On Mon, Dec 02, 2024 at 02:07:36PM +0200, Jani Nikula wrote:
> On Mon, 02 Dec 2024, Maxime Ripard wrote:
> > It's not about whether we have a problem or not: you introduce new
> > framework functions, you need to have kunit tests to check their
> > behaviour.
>
> I don't fundamentally disagree wi
the port
> number of one nibble.
>
> [How]
> Adjust the code by:
> - RAD array items are valuable only for LCT >= 1.
> - Use 0xF as the mask to replace BIT_MASK(4)
>
> Fixes: 2f015ec6eab6 ("drm/dp_mst: Add sideband down request tracing +
> selftests")
On Fri, Nov 29, 2024 at 03:46:28PM +0100, Maxime Ripard wrote:
> On Fri, Nov 29, 2024 at 04:26:01PM +0200, Imre Deak wrote:
> > Adding more people from DRM core domain. Any comment, objection to this
> > change?
> >
> > On Tue, Nov 26, 2024 at 06:18:56PM +0200, Imre Dea
Adding more people from DRM core domain. Any comment, objection to this
change?
On Tue, Nov 26, 2024 at 06:18:56PM +0200, Imre Deak wrote:
> Atm when the connector is added to the drm_mode_config::connector_list,
> the connector may not be fully initialized yet. This is not a problem
>
7;t have a
dependency.
v2: (Jani)
- Let initing DDC as well via drm_connector_init_core().
- Rename __drm_connector_init to drm_connector_init_core_and_add().
Cc: Jani Nikula
Reviewed-by: Rodrigo Vivi (v1)
Signed-off-by: Imre Deak
---
drivers/gpu/drm/drm_connector.c | 111 ++-
After a connector is added to the drm_mode_config::connector_list, it's
visible to any in-kernel users looking up connectors via the above list.
Make sure that the connector is properly initialized before such
look-ups.
Reviewed-by: Rodrigo Vivi
Signed-off-by: Imre Deak
---
drivers/gp
nnector, calling
the drm_connector pointer _connector.
Suggested-by: Jani Nikula
Signed-off-by: Imre Deak
---
drivers/gpu/drm/i915/display/intel_dp_mst.c | 184 ++--
1 file changed, 90 insertions(+), 94 deletions(-)
diff --git a/drivers/gpu/drm/i915/display/intel_dp_mst.c
b/drivers/
After an error during adding an MST connector the MST port and the
intel_connector object could be leaked, fix this up.
Reviewed-by: Rodrigo Vivi
Signed-off-by: Imre Deak
---
drivers/gpu/drm/i915/display/intel_dp_mst.c | 17 +
1 file changed, 9 insertions(+), 8 deletions
5-1-imre.d...@intel.com
Imre Deak (4):
drm/dp: Add a way to init/add a connector in separate steps
drm/i915/dp_mst: Expose a connector to kernel users after it's
properly initialized
drm/i915/dp_mst: Fix error handling while adding a connector
drm/i915/dp_mst: Use intel_connector vs
On Mon, Nov 25, 2024 at 10:04:46PM +, Patchwork wrote:
> == Series Details ==
>
> Series: drm/dp_mst: Fix MST sideband message body length check
> URL : https://patchwork.freedesktop.org/series/141772/
> State : failure
Thanks for the review, patch is pushed to drm-misc-fixes.
The failure
d_irq_handle_event+0xc8/0x1580 [drm_display_helper]
Cc:
Cc: Lyude Paul
Signed-off-by: Imre Deak
---
drivers/gpu/drm/display/drm_dp_mst_topology.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/drivers/gpu/drm/display/drm_dp_mst_topology.c
b/drivers/gpu/drm/display/drm_dp_mst_topology.c
index ac901
On Fri, Nov 22, 2024 at 02:22:44PM +0200, Jani Nikula wrote:
> On Wed, 20 Nov 2024, Imre Deak wrote:
> > On Mon, Nov 18, 2024 at 05:14:52PM +0200, Jani Nikula wrote:
> >> SST with 128b/132b channel coding needs this too. Extract to a separate
> >> helper, independ
On Wed, Nov 20, 2024 at 04:59:43PM +0200, Imre Deak wrote:
> On Mon, Nov 18, 2024 at 05:14:52PM +0200, Jani Nikula wrote:
> > SST with 128b/132b channel coding needs this too. Extract to a separate
> > helper, independent of MST.
> >
> > Pass timeout in as a paramete
On Mon, Nov 18, 2024 at 05:14:54PM +0200, Jani Nikula wrote:
> SST with 128b/132b channel coding needs this too. Extract to a separate
> helper, independent of MST.
>
> Signed-off-by: Jani Nikula
Reviewed-by: Imre Deak
> ---
> drivers/gpu/drm/display/drm_dp_h
; + * drm_dp_dpcd_write_payload() - Write payload
Stg like "Write Virtual Channel Payload information to payload table"
instead?
Regardless of the above or the earlier mtp namespacing comment, patch
looks ok:
Reviewed-by: Imre Deak
> + * @aux: DisplayPort AUX channel
>
* for the ACT handled bit for up to @timeout_ms milliseconds, defaulting to
> + * 3000 ms if 0.
> + *
> + * Returns:
> + * 0 if the ACT was handled in time, negative error code on failure.
> + */
> +int drm_dp_dpcd_poll_act_handled(struct drm_dp_aux *aux, int timeout_ms)
I wonder if i
On Mon, Nov 18, 2024 at 02:12:14PM +0200, Jani Nikula wrote:
> On Fri, 15 Nov 2024, Imre Deak wrote:
> > After a connector is added to the drm_mode_config::connector_list, it's
> > visible to any in-kernel users looking up connectors via the above list.
> > Make
On Mon, Nov 18, 2024 at 02:09:29PM +0200, Jani Nikula wrote:
> On Fri, 15 Nov 2024, Imre Deak wrote:
> > Atm when the connector is added to the drm_mode_config::connector_list,
> > the connector may not be fully initialized yet. This is not a problem
> > for user spac
On Mon, Nov 18, 2024 at 11:10:18AM +0200, Jani Nikula wrote:
> On Fri, 15 Nov 2024, Imre Deak wrote:
> > On Fri, Nov 15, 2024 at 03:20:58PM -0500, Rodrigo Vivi wrote:
> >> On Fri, Nov 15, 2024 at 06:41:56PM +0200, Imre Deak wrote:
> >> > The connector initialization
On Mon, Nov 18, 2024 at 02:23:48PM +0200, Jani Nikula wrote:
> On Mon, 18 Nov 2024, Imre Deak wrote:
> > On Mon, Nov 18, 2024 at 02:12:14PM +0200, Jani Nikula wrote:
> >> On Fri, 15 Nov 2024, Imre Deak wrote:
> >> > After a connector is added to the drm_mod
On Fri, Nov 15, 2024 at 03:20:58PM -0500, Rodrigo Vivi wrote:
> On Fri, Nov 15, 2024 at 06:41:56PM +0200, Imre Deak wrote:
> > The connector initialization in intel_dp_add_mst_connector() depends on
> > the device pointer in connector to be valid, at least by connector
> >
After a connector is added to the drm_mode_config::connector_list, it's
visible to any in-kernel users looking up connectors via the above list.
Make sure that the connector is properly initialized before such
look-ups.
Signed-off-by: Imre Deak
---
drivers/gpu/drm/i915/display/intel_dp_
ULL device pointer
gracefully in DRM core.
Cc: Jani Nikula
Fixes: 529798bd786a ("drm/i915/mst: convert to struct intel_display")
Closes: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/12799
Signed-off-by: Imre Deak
---
drivers/gpu/drm/i915/display/intel_dp_mst.c | 10 ++
7;t have a
dependency.
Signed-off-by: Imre Deak
---
drivers/gpu/drm/drm_connector.c | 103 ++--
include/drm/drm_connector.h | 5 ++
2 files changed, 91 insertions(+), 17 deletions(-)
diff --git a/drivers/gpu/drm/drm_connector.c b/drivers/gpu/drm/drm_connector.c
index fc3
After an error during adding an MST connector the MST port and the
intel_connector object could be leaked, fix this up.
Signed-off-by: Imre Deak
---
drivers/gpu/drm/i915/display/intel_dp_mst.c | 17 +
1 file changed, 9 insertions(+), 8 deletions(-)
diff --git a/drivers/gpu/drm
On Fri, Nov 01, 2024 at 11:22:13AM +0200, Jani Nikula wrote:
> On Thu, 31 Oct 2024, Imre Deak wrote:
> > On Thu, Oct 31, 2024 at 05:12:45PM +0200, Abel Vesa wrote:
> >> According to the DisplayPort standard, LTTPRs have two operating
> >> modes:
> >> - non-tr
ed-off-by: Abel Vesa
Acked-by: Imre Deak
> ---
> drivers/gpu/drm/i915/display/intel_dp_link_training.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/i915/display/intel_dp_link_training.c
> b/drivers/gpu/drm/i915/display
On Thu, Oct 31, 2024 at 05:12:45PM +0200, Abel Vesa wrote:
> According to the DisplayPort standard, LTTPRs have two operating
> modes:
> - non-transparent - it replies to DPCD LTTPR field specific AUX
>requests, while passes through all other AUX requests
> - transparent - it passes through a
On Wed, Sep 25, 2024 at 07:25:34PM -0500, Benjamin Hoefs wrote:
> Hello,
>
> I am using a Dell WD19TB dock with a 3440x1440 monitor. Using it at
> 100Hz used to work but recently I tried it again and discovered it no longer
> did, specifically the modeset seems to silently fail with no error messa
On Tue, Sep 10, 2024 at 11:07:53PM +, Patchwork wrote:
> == Series Details ==
>
> Series: drm/dp_mst: Fix DSC decompression detection in Synaptics branch
> devices
> URL : https://patchwork.freedesktop.org/series/138419/
> State : failure
Patch is pushed to drm-misc-fixes, thanks for the r
rm/i915/kernel/-/issues/12047
Signed-off-by: Imre Deak
---
drivers/gpu/drm/display/drm_dp_mst_topology.c | 10 --
1 file changed, 8 insertions(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/display/drm_dp_mst_topology.c
b/drivers/gpu/drm/display/drm_dp_mst_topology.c
index a040d7dfced1
On Sun, Aug 04, 2024 at 06:45:43AM -0700, Manasi Navare wrote:
> On Mon, Jul 22, 2024 at 9:55 AM Imre Deak wrote:
> >
> > In the
> > if (old_ddps != port->ddps || !created)
> > if (port->ddps && !por
024-07-22 at 19:54 +0300, Imre Deak wrote:
> > Factor out a function to queue a work for probing the topology, also
> > used by the next patch.
> >
> > Cc: Lyude Paul
> > Cc: dri-devel@lists.freedesktop.org
> > Signed-off-by: Imre Deak
> > ---
> >
A follow up i915 patch will need to reprobe the MST topology after the
initial probing, add a helper for this.
Cc: Lyude Paul
Cc: dri-devel@lists.freedesktop.org
Signed-off-by: Imre Deak
---
drivers/gpu/drm/display/drm_dp_mst_topology.c | 27 +++
include/drm/display
r that the path resources are
always enumerated for an output port in the plugged state.
Cc: Lyude Paul
Cc: dri-devel@lists.freedesktop.org
Signed-off-by: Imre Deak
---
drivers/gpu/drm/display/drm_dp_mst_topology.c | 19 ---
1 file changed, 8 insertions(+), 11 deletions(-)
Factor out a function to queue a work for probing the topology, also
used by the next patch.
Cc: Lyude Paul
Cc: dri-devel@lists.freedesktop.org
Signed-off-by: Imre Deak
---
drivers/gpu/drm/display/drm_dp_mst_topology.c | 9 +++--
1 file changed, 7 insertions(+), 2 deletions(-)
diff --git
On Fri, Jul 19, 2024 at 02:43:27PM +0300, Imre Deak wrote:
> On Fri, Jul 19, 2024 at 02:39:11PM +0300, Imre Deak wrote:
> > On Fri, Jul 19, 2024 at 12:36:08PM +0200, Jocelyn Falempe wrote:
> > > This breaks build if DRM_PANIC is not enabled.
> > >
> > >
On Fri, Jul 19, 2024 at 02:39:11PM +0300, Imre Deak wrote:
> On Fri, Jul 19, 2024 at 12:36:08PM +0200, Jocelyn Falempe wrote:
> > This breaks build if DRM_PANIC is not enabled.
> >
> > Fixes: de338c754d40 ("drm/panic: Add missing static inline to
> > drm_panic_is_
On Fri, Jul 19, 2024 at 12:36:08PM +0200, Jocelyn Falempe wrote:
> This breaks build if DRM_PANIC is not enabled.
>
> Fixes: de338c754d40 ("drm/panic: Add missing static inline to
> drm_panic_is_enabled()")
> Signed-off-by: Jocelyn Falempe
Thanks for the quick fi
Hi,
On Wed, Jul 17, 2024 at 10:48:39AM +0200, Jocelyn Falempe wrote:
> It allows to check if the drm device supports drm_panic.
> Prepare the work to have better integration with fbcon and vtconsole.
>
> Signed-off-by: Jocelyn Falempe
> ---
> drivers/gpu/drm/drm_panic.c | 20 +++
On Tue, Jul 09, 2024 at 11:12:26AM +, Patchwork wrote:
> == Series Details ==
>
> Series: drm/i915/dp: Fix LTTPR detection (rev2)
> URL : https://patchwork.freedesktop.org/series/135711/
> State : success
Thanks for the reviews, pushed patches 1-4 to drm-intel-next, patch 5 to
drm-misc-next
Cc: dri-devel@lists.freedesktop.org
Cc: Ville Syrjälä
Signed-off-by: Imre Deak
---
drivers/gpu/drm/display/drm_dp_helper.c | 66 +
include/drm/display/drm_dp.h| 4 ++
include/drm/display/drm_dp_helper.h | 2 +
3 files changed, 62 insertions(+), 10
Add the DPCD register for the LTTPR PHY OUI. This will be used by a
later i915 patch to dump the descriptors for the detected LTTPR PHYs.
Cc: dri-devel@lists.freedesktop.org
Signed-off-by: Imre Deak
---
include/drm/display/drm_dp.h | 4
1 file changed, 4 insertions(+)
diff --git a/include
On Mon, Jul 01, 2024 at 02:28:31PM +0300, Jani Nikula wrote:
> On Fri, 28 Jun 2024, Imre Deak wrote:
> > This is v2 of [1], renaming the helpers from drm_x16 to fxp_q4 as
> > suggested by Jani.
> >
> > [1] https://lore.kernel.org/all/20240614173911.3743172-1-imre.d...
1 - 100 of 695 matches
Mail list logo
