On Fri, 30 Sep 2011 20:55:24 +0200, Daniel Vetter <daniel at ffwll.ch> wrote:
> If it's not too annoying to do, can you move this to the previous patch? > Dito for the s/panel_vdd_work/panel_work/. Yeah, that was a bit of sloppy patch mangling. > Same comment as in the previous patch: I think the > intel_dp->want_panel_power check belongs into the work queue. We don't > want to hide the fact that we properly handle that race ;-) Oh, I see your point. Yeah, either the check needs to be moved, or the function name changed. I'll move the check. -- keith.packard at intel.com -------------- next part -------------- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 189 bytes Desc: not available URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20110930/e51a4a29/attachment.pgp>