On Tue, 26 Jul 2011 09:24:39 +0200, Daniel Vetter <daniel at 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 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/20110726/1b607231/attachment-0001.pgp>
