All of lore.kernel.org
 help / color / mirror / Atom feed
From: Krzysztof Nowicki <krzysztof.a.nowicki@gmail.com>
To: dri-devel@lists.freedesktop.org
Subject: Re: [PATCH 3/5] drm/dp/mst: change MST detection scheme
Date: Wed, 25 Jan 2017 18:16:06 +0100	[thread overview]
Message-ID: <2549859.X1rmbOG6Wp@cruinn> (raw)
In-Reply-To: <CAPM=9tysB7NxEW8w_AKAtJF12Fqb7YE1gbzn7HB-tWZU0XX+1A@mail.gmail.com>

Hi,

On środa, 15 czerwca 2016 14:14:01 CET  wrote:
> On 15 June 2016 at 13:49, Dave Airlie <airlied at gmail.com> wrote:
> > Excuse me for top posting.
> > 
> > So I finally got back to this patch, still don't like it.
> > 
> > Apart from the doing 3 things in once which is just annoying, current
> > userspace for tiled monitors
> > relies on the behaviour that the cached edids are retrieved before the
> > connector is show to userspace.
> > 
> > This is so that for tiled monitors both connectors are shown with
> > tiling properties so userspace can pick the correct monitor size.
> > 
> > Now I realise this isn't great, since at least for the two cable 5K
> > monitors you won't ever plug them in at once.
> > 
> > So we should probably discuss more what userspace should do in the
> > presence of a tiled monitor where it can only see one tile,
> > 
> > If we go ahead with something like this, things will look kinda ugly
> > as you plug in a 4k monitor and it'll try and set a mode
> > on the first tile when it gets the hotplug (or it could be racing and
> > notice the connectors before the hotplug). So it then
> > sets the 1920x1080 mode on one connector and gets another hotplug to
> > set the tiled mode up.
> 
> Though I think the fix is to leave the cached edid test in add port,
> 
> And maybe the rest of the patch is okay then.
> 
> Dave.

Is this patch still being considered for pulling in?

Recently I've been fighting with a HP UltraSlim docking station with a MST hub 
inside in order to get it working with the amdgpu DAL/DC code. I couldn't get 
it to work until I stumbled upon this patch. Applying it solved the issue 
instantly.

What I have found during debugging is that the MST hub would return SST sink 
port numbers < 8 which resulted in no EDID being retrieved and in turn the 
status of the outputs was 'disconnected'.

Regards
Chris
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

  reply	other threads:[~2017-01-25 17:16 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-01-22 22:07 [PATCH 0/5] Fixes for MST (daisy-chain and 4k tiles) Harry Wentland
2016-01-22 22:07 ` [PATCH 1/5] drm: Add drm_fixp_from_fraction and drm_fixp2int_ceil Harry Wentland
2016-01-22 22:07 ` [PATCH 2/5] drm/dp/mst: Calculate MST PBN with 31.32 fixed point Harry Wentland
2016-01-22 22:07 ` [PATCH 3/5] drm/dp/mst: change MST detection scheme Harry Wentland
2016-02-17  5:45   ` Dave Airlie
2016-02-18  4:53     ` Lysenko, Mykola
2016-02-23  3:09     ` Lysenko, Mykola
2016-05-12 22:41       ` Grodzovsky, Andrey
2016-06-15  3:49         ` Dave Airlie
2016-06-15  4:14           ` Dave Airlie
2017-01-25 17:16             ` Krzysztof Nowicki [this message]
2017-01-25 17:24               ` Alex Deucher
2017-01-25 18:32                 ` Harry Wentland
2017-01-25 22:46                   ` Alex Deucher
2017-01-25 23:07                     ` Harry Wentland
2017-01-27 18:21                 ` Krzysztof Nowicki
2016-01-22 22:07 ` [PATCH 4/5] drm/dp/mst: move GUID storage from mgr, port to only mst branch Harry Wentland
2016-01-22 22:07 ` [PATCH 5/5] drm/dp/mst: Reverse order of MST enable and clearing VC payload table Harry Wentland
2016-01-25  4:53 ` [PATCH 0/5] Fixes for MST (daisy-chain and 4k tiles) Dave Airlie
2016-01-25 18:55   ` Wentland, Harry
2016-02-02 21:20     ` Alex Deucher
2016-02-04  0:35       ` Dave Airlie

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=2549859.X1rmbOG6Wp@cruinn \
    --to=krzysztof.a.nowicki@gmail.com \
    --cc=dri-devel@lists.freedesktop.org \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.