On Wed, Dec 18, 2013 at 5:07 PM, Daniel Drake <drake@xxxxxxxxxxxx> wrote: > 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? > Hey Daniel, The set is posted, but I still owe a new version to Tomasz. It's on my TODO list, I'm hoping to get to it soon. Sean > Daniel _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/dri-devel
