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
next prev parent 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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox