[PATCH 20/34] drm/doc: Repleace LOCKING kerneldoc sections in drm_modes.c

2014-03-23 Thread Daniel Vetter
On Sat, Mar 22, 2014 at 7:45 AM, Ben Widawsky wrote: > On Thu, Mar 20, 2014 at 11:31:26AM +1000, Dave Airlie wrote: >> On Tue, Mar 11, 2014 at 8:30 PM, Daniel Vetter >> wrote: >> > There's not really any value in stating that no locking is needed. And >> > even if the comment is useful, a check

[PATCH 20/34] drm/doc: Repleace LOCKING kerneldoc sections in drm_modes.c

2014-03-21 Thread Ben Widawsky
On Thu, Mar 20, 2014 at 11:31:26AM +1000, Dave Airlie wrote: > On Tue, Mar 11, 2014 at 8:30 PM, Daniel Vetter > wrote: > > There's not really any value in stating that no locking is needed. And > > even if the comment is useful, a check for the right mutex at the > > beginning of the function is

[PATCH 20/34] drm/doc: Repleace LOCKING kerneldoc sections in drm_modes.c

2014-03-20 Thread Dave Airlie
On Tue, Mar 11, 2014 at 8:30 PM, Daniel Vetter wrote: > There's not really any value in stating that no locking is needed. And > even if the comment is useful, a check for the right mutex at the > beginning of the function is better since that can't be ingored as > easily as a bit of documentatio

[PATCH 20/34] drm/doc: Repleace LOCKING kerneldoc sections in drm_modes.c

2014-03-11 Thread Daniel Vetter
There's not really any value in stating that no locking is needed. And even if the comment is useful, a check for the right mutex at the beginning of the function is better since that can't be ingored as easily as a bit of documentation. Note that drm_mode_probed_add in drm_crtc.c is also changed,