Re: drm_do_probe_ddc_edid ENXIO check too aggressive?

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Dec 18, 2013 at 2:18 PM, Daniel Drake <drake@xxxxxxxxxxxx> wrote:
> Yes, this looks very similar to the approach I tried earlier. I guess
> the patch was written for the same reasons as well.
> Sean, any objections to me taking your patch and sending it upstream?
>
> http://git.chromium.org/gitweb/?p=chromiumos/third_party/kernel-next.git;a=commitdiff;h=77aed71acf4b84e722ead24c820d8ad059121e5b

Seems easier said than done. The patch looks innocent but it touches
on other stuff that has changed a lot.

For example, it adds cancel_delayed_work_sync() in the hdmi_poweroff()
path. With my naive backport to vanilla-3.8, hdmi_poweroff() gets
called within the context of that work item, and a work item trying to
cancel_delayed_work_sync() on itself hangs. Reproduced just by
unplugging the display once.

I see that things have changed substantially in the ChromiumOS tree,
for example this patch probably has some effect:

commit 29ae0c6096395a96a597453271946fc7d8442b6e
Author: Sean Paul <seanpaul@xxxxxxxxxxxx>
Date:   Thu Jun 20 17:24:12 2013 -0400

    drm/exynos: Consolidate suspend/resume in drm_drv


As this job seems bigger than anticipated and I don't have any Exynos
hardware that can boot mainline, I think I might have to stop here,
and hope that ChromiumOS guys upstream all this soon?

Daniel
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/dri-devel




[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux