Messages in this thread | | | From | Keith Packard <> | Subject | Re: [PATCH 19/21] drm/i915: Asynchronous eDP panel power off | Date | Fri, 30 Sep 2011 13:57:39 -0700 |
| |
On Fri, 30 Sep 2011 20:55:24 +0200, Daniel Vetter <daniel@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@intel.com [unhandled content-type:application/pgp-signature] | |