From: Daniel Vetter <daniel@ffwll.ch>
To: "Ville Syrjälä" <ville.syrjala@linux.intel.com>
Cc: "intel-gfx@lists.freedesktop.org" <intel-gfx@lists.freedesktop.org>
Subject: Re: [PATCH 6/6] drm/i915/bxt: Fix irq_port for eDP
Date: Thu, 10 Sep 2015 16:51:51 +0200 [thread overview]
Message-ID: <20150910145151.GA2767@phenom.ffwll.local> (raw)
In-Reply-To: <20150910133557.GG29811@intel.com>
On Thu, Sep 10, 2015 at 04:35:57PM +0300, Ville Syrjälä wrote:
> On Thu, Sep 10, 2015 at 01:07:30PM +0000, R, Durgadoss wrote:
> > Hi Rodrigo,
> >
> > I had to add this to get HDMI hotplug working on BXT.
> > As you might already know, we have the HPD pins A & B
> > swapped in BXT. And, we are using HPD_PORT_A as
> > 'intel_encoder->hpd_pin' for HDMI on port B.
>
> People keep saying that, but according to the spec it's not
> even true. A and B are not swapped, but instead all the pins
> are sort of shifted by one position.
>
> HPD A = DDI0 / port B
> HPD B = DDI1 / port C
> HPD C = DDI2 / port A
>
> If the code actually did the remapping in a way that matches
> the spec, things would be a lot less confusing.
And we even have the logic to allow that remapping, so I guess the real
job is to figure out where that's not done correctly. Not throwing more
hacks on top.
-Daniel
--
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
prev parent reply other threads:[~2015-09-10 14:49 UTC|newest]
Thread overview: 74+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-09-04 13:26 [PATCH 0/6] HDMI optimization series Sonika Jindal
2015-09-04 13:26 ` [PATCH 1/6] drm/i915: add attached connector to hdmi container Sonika Jindal
2015-09-09 18:54 ` Rodrigo Vivi
2015-09-10 15:24 ` Daniel Vetter
2015-09-04 13:26 ` [PATCH 2/6] drm/i915: Add HDMI probe function Sonika Jindal
2015-09-04 14:48 ` Daniel Vetter
2015-09-09 18:55 ` Rodrigo Vivi
2015-09-11 10:45 ` Jindal, Sonika
2015-09-14 8:30 ` Daniel Vetter
2015-09-14 9:55 ` Sharma, Shashank
2015-09-14 13:07 ` Daniel Vetter
2015-09-14 13:45 ` Sharma, Shashank
2015-09-28 8:56 ` [PATCH] drm/i915: Add hot_plug hook for hdmi encoder Sonika Jindal
2015-09-28 13:34 ` Daniel Vetter
2015-09-29 4:13 ` Jindal, Sonika
2015-09-29 9:04 ` Daniel Vetter
2015-10-05 11:13 ` [PATCH 1/2] drm/i915: Call encoder hotplug for init and resume cases Sonika Jindal
2015-10-05 11:13 ` [PATCH 2/2] drm/i915: Add hot_plug hook for hdmi encoder Sonika Jindal
2015-10-06 8:28 ` Daniel Vetter
2015-12-10 4:30 ` Sonika Jindal
2015-12-10 4:45 ` [PATCH] " Sonika Jindal
2015-12-10 8:29 ` Daniel Vetter
2015-12-10 8:35 ` Jindal, Sonika
2015-12-10 8:53 ` Daniel Vetter
2015-12-10 9:04 ` Jindal, Sonika
2015-10-08 13:35 ` [PATCH 1/2] drm/i915: Call encoder hotplug for init and resume cases Ville Syrjälä
2015-10-08 14:38 ` Jani Nikula
2015-10-08 19:54 ` Daniel Vetter
2015-10-09 4:31 ` Jindal, Sonika
2015-12-10 4:27 ` Sonika Jindal
2015-10-12 12:24 ` Sharma, Shashank
2015-10-15 1:32 ` Jindal, Sonika
2015-09-04 13:26 ` [PATCH 3/6] drm/i915: Make intel_digital_port_connected global Sonika Jindal
2015-09-09 18:57 ` Rodrigo Vivi
2015-09-11 11:28 ` [PATCH] drm/i915/bxt: Use intel_encoder->hpd_pin to check live status Sonika Jindal
2015-09-11 18:00 ` Rodrigo Vivi
2015-09-12 12:02 ` Jindal, Sonika
2015-09-14 8:34 ` Daniel Vetter
2015-09-14 8:38 ` Jindal, Sonika
2015-09-14 9:15 ` Daniel Vetter
2015-09-04 13:26 ` [PATCH 4/6] drm/i915: drm/i915: Check live status before reading edid Sonika Jindal
2015-09-04 14:49 ` Daniel Vetter
2015-09-07 5:02 ` [PATCH] " Sonika Jindal
2015-09-08 11:21 ` Sonika Jindal
2015-09-09 19:11 ` [PATCH 4/6] drm/i915: " Rodrigo Vivi
2015-09-11 11:26 ` [PATCH] " Sonika Jindal
2015-09-11 17:56 ` Rodrigo Vivi
2015-09-14 8:42 ` Daniel Vetter
2015-09-14 9:14 ` Jindal, Sonika
2015-09-14 9:36 ` Daniel Vetter
2015-09-15 4:14 ` Sonika Jindal
2015-09-23 8:18 ` Daniel Vetter
2016-03-08 21:03 ` Chris Wilson
2016-03-09 5:52 ` Jindal, Sonika
2016-03-09 5:55 ` Sharma, Shashank
2016-03-10 11:37 ` Sharma, Shashank
2015-09-04 13:26 ` [PATCH 5/6] drm/i915: drm/i915: Process hpd only for hdmi inside hotplug_work_func Sonika Jindal
2015-09-04 14:47 ` Daniel Vetter
2015-09-06 4:31 ` Jindal, Sonika
2015-09-07 5:04 ` [PATCH] drm/i915: Call encoder hot_plug hook only for hdmi Sonika Jindal
2015-09-07 16:26 ` Daniel Vetter
2015-09-08 4:42 ` Jindal, Sonika
2015-09-08 11:38 ` Jindal, Sonika
2015-09-09 15:17 ` Daniel Vetter
2015-09-10 1:07 ` Jindal, Sonika
2015-09-10 14:47 ` Daniel Vetter
2015-09-09 19:20 ` [PATCH 5/6] drm/i915: drm/i915: Process hpd only for hdmi inside hotplug_work_func Rodrigo Vivi
2015-09-09 23:37 ` Jindal, Sonika
2015-09-04 13:26 ` [PATCH 6/6] drm/i915/bxt: Fix irq_port for eDP Sonika Jindal
2015-09-09 19:24 ` Rodrigo Vivi
2015-09-10 10:49 ` Jindal, Sonika
2015-09-10 13:07 ` R, Durgadoss
2015-09-10 13:35 ` Ville Syrjälä
2015-09-10 14:51 ` Daniel Vetter [this message]
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=20150910145151.GA2767@phenom.ffwll.local \
--to=daniel@ffwll.ch \
--cc=intel-gfx@lists.freedesktop.org \
--cc=ville.syrjala@linux.intel.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 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.