[PATCH 07/12] drm: drop redundant drm_file->is_master

2014-07-28 Thread David Herrmann
Hi On Fri, Jul 25, 2014 at 9:56 AM, Daniel Vetter wrote: > On Thu, Jul 24, 2014 at 11:38:28PM +0200, David Herrmann wrote: >> On Thu, Jul 24, 2014 at 11:52 AM, Daniel Vetter wrote: >> > On Wed, Jul 23, 2014 at 05:26:42PM +0200, David Herrmann wrote: >> >> +static inline bool drm_is_master(const

[PATCH 07/12] drm: drop redundant drm_file->is_master

2014-07-25 Thread Daniel Vetter
On Thu, Jul 24, 2014 at 11:38:28PM +0200, David Herrmann wrote: > On Thu, Jul 24, 2014 at 11:52 AM, Daniel Vetter wrote: > > On Wed, Jul 23, 2014 at 05:26:42PM +0200, David Herrmann wrote: > >> +static inline bool drm_is_master(const struct drm_file *file) > >> +{ > > > > Hm, we don't have any

[PATCH 07/12] drm: drop redundant drm_file->is_master

2014-07-25 Thread David Herrmann
Hi On Thu, Jul 24, 2014 at 11:52 AM, Daniel Vetter wrote: > On Wed, Jul 23, 2014 at 05:26:42PM +0200, David Herrmann wrote: >> The drm_file->is_master field is redundant as it's equivalent to: >> drm_file->master && drm_file->master == drm_file->minor->master >> >> 1) "=>" >> Whenever we

[PATCH 07/12] drm: drop redundant drm_file->is_master

2014-07-24 Thread Daniel Vetter
On Wed, Jul 23, 2014 at 05:26:42PM +0200, David Herrmann wrote: > The drm_file->is_master field is redundant as it's equivalent to: > drm_file->master && drm_file->master == drm_file->minor->master > > 1) "=>" > Whenever we set drm_file->is_master, we also set: >

[PATCH 07/12] drm: drop redundant drm_file->is_master

2014-07-23 Thread David Herrmann
The drm_file->is_master field is redundant as it's equivalent to: drm_file->master && drm_file->master == drm_file->minor->master 1) "=>" Whenever we set drm_file->is_master, we also set: drm_file->minor->master = drm_file->master; Whenever we clear drm_file->is_master, we also