[PATCH] drm/radeon: don't mess with hot plug detect for eDP or LVDS connector v2

[Alex Deucher]

On Wed, May 9, 2012 at 10:23 AM, Jerome Glisse <j.glisse at gmail.com> wrote:
> On Wed, May 9, 2012 at 9:40 AM, Alex Deucher <alexdeucher at gmail.com> wrote:
>> On Fri, May 4, 2012 at 11:06 AM,  <j.glisse at gmail.com> wrote:
>>> From: Jerome Glisse <jglisse at redhat.com>
>>>
>>> It seems imac pannel doesn't like whe we change the hot plug setup
>>> and then refuse to work. This help but doesn't fully fix:
>>> https://bugzilla.redhat.com/show_bug.cgi?id=726143
>>
>> How does it help?  Does it fix the the aux problems, but the monitor
>> still doesn't train?  What's the working value of the relevant
>> DC_HPD*_CONTROL register?
>>
>> Alex
>
> Don't have the hw but somehow the way we program this reg completely
> disable the panel, after that the panel doesn't answer to anything
> (nor i2c nor any aux transaction). Without programming that link
> training is successfull but panel stays black. I can ask to get the
> value before and after.

Patch seems reasonable in general (we don't really need hpd to be
explicitly enabled for lvds or edp) so:

Reviewed-by: Alex Deucher <alexander.deucher at amd.com>

>
> Cheers,
> Jerome