Re: [Intel-gfx] [PATCH] drm/i915: Hold struct_mutex during hotplug processing

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!