Messages in this thread | | | From | Keith Packard <> | Subject | Re: [Intel-gfx] [PATCH] drm/i915: Hold struct_mutex during hotplug processing | Date | Tue, 26 Jul 2011 08:23:13 -0700 |
| |
On Tue, 26 Jul 2011 09:24:39 +0200, Daniel Vetter <daniel@ffwll.ch> wrote: > Two things I've noticed:
> - Why not dev->mode_config.mutex?
You're right, of course. I noticed that just after posting that version and updated it; the updated version is on my drm-intel-fixes branch already (having been reviewed by Jesse).
> - And a nitpick: Why the dev_priv->dev indirection, when dev is > already lying around?
All nicely cleaned up by using &mode_config->mutex instead :-)
Thanks for looking it over!
-- keith.packard@intel.com [unhandled content-type:application/pgp-signature] | |