linux-toradex.git/drivers/gpu/drm/drm_drv.c, branch v4.2.6

drm: Always enable atomic API

2015-06-24T01:21:35+00:00

Now that the interface has been proven by a port of Weston (using all
atomic features including TEST_ONLY), remove the module parameter
guarding the atomic API from being exposed, and let it run free in the
wild.

Signed-off-by: Daniel Stone 
Acked-by: Daniel Vetter 
Signed-off-by: Dave Airlie

drm: simplify master cleanup

2015-05-05T07:46:38+00:00

In drm_master_destroy() we _free_ the master object. There is no reason to
hold any locks while dropping its static members, nor do we have to reset
it to 0.

Furthermore, kfree() already does NULL checks, so call it directly on
master->unique and drop the redundant reset-code.

Signed-off-by: David Herrmann 
Reviewed-by: Chris Wilson 
Signed-off-by: Daniel Vetter

drm: simplify authentication management

2015-05-05T07:45:57+00:00

The magic auth tokens we have are a simple map from cyclic IDs to drm_file
objects. Remove all the old bulk of code and replace it with a simple,
direct IDR.

The previous behavior is kept. Especially calling authmagic multiple times
on the same magic results in EINVAL except on the first call. The only
difference in behavior is that we never allocate IDs multiple times as
long as a client has its FD open.

v2:
 - Fix return code of GetMagic()
 - Use non-cyclic IDR allocator
 - fix off-by-one in "magic > INT_MAX" sanity check

v3:
 - drop redundant "magic > INT_MAX" check

Signed-off-by: David Herrmann 
Reviewed-by: Chris Wilson 
Signed-off-by: Daniel Vetter

drm: drop unused 'magicfree' list

2015-05-05T07:40:50+00:00

This list is write-only. It's never used for read-access, so no reason to
keep it around. Drop it!

Signed-off-by: David Herrmann 
Reviewed-by: Chris Wilson 
Signed-off-by: Daniel Vetter

drm: %pF is only for function pointers

2015-03-13T09:17:38+00:00

Use %pS for actual addresses, otherwise you'll get bad output
on arches like ppc64 where %pF expects a function descriptor.

Signed-off-by: Scott Wood 
Cc: dri-devel@lists.freedesktop.org
Signed-off-by: Daniel Vetter

drm: add atomic properties

2015-01-05T12:54:38+00:00

Once a driver is using atomic helpers for modeset, the next step is to
switch over to atomic properties.  To do this, make sure that any
modeset objects have their ->atomic_{get,set}_property() vfuncs suitably
populated if they have custom properties (you did already remember to
plug in atomic-helper func for the legacy ->set_property() vfuncs,
right?), and then set DRIVER_ATOMIC bit in driver_features flag.

A new cap is introduced, DRM_CLIENT_CAP_ATOMIC, for the purposes of
shielding legacy userspace from atomic properties.  Mostly for the
benefit of legacy DDX drivers that do silly things like getting/setting
each property at startup (since some of the new atomic properties will
be able to trigger modeset).

Signed-off-by: Rob Clark 
[danvet: Squash in fixup patch to check for DRM_MODE_PROP_ATOMIC
instaed of the CAP define when filtering properties. Reported by
Tvrtko Uruslin, acked by Rob.]
Signed-off-by: Daniel Vetter

drm: Document that drm_dev_alloc doesn't need a parent

2014-11-25T12:12:42+00:00

Possible for purely virtual debug devices.

Signed-off-by: Daniel Vetter

drm: drm_err: Remove unnecessary func argument

2014-10-12T18:57:16+00:00

Removing the unnecessary drm_err __func__ argument by using
the equivalent %pf and __builtin_return_address(0) makes the
code smaller for every use of the DRM_ERROR macro.

For instance: (allmodconfig)

$ size drivers/gpu/drm/i915/i915.o*
   text	   data	    bss	    dec	    hex	filename
 922447	 193257	 296736	1412440	 158d58	drivers/gpu/drm/i915/i915.o.new
 928111	 193257	 296736	1418104	 15a378	drivers/gpu/drm/i915/i915.o.old

Signed-off-by: Joe Perches 
Signed-off-by: Daniel Vetter

drm/core: use helper to check driver features

2014-10-03T08:38:56+00:00

The patch replaces direct access to driver_features field
by calls to helper function.

Signed-off-by: Andrzej Hajda 
Reviewed-by: David Herrmann 
Signed-off-by: Daniel Vetter

drm: change drm_err return type to void

2014-09-24T01:42:36+00:00

The return value is not used by callers of this function
nor by uses of the DRM_ERROR macro so change the function
to return void.

Signed-off-by: Joe Perches 
Reviewed-by: Jani Nikula 
Signed-off-by: Dave Airlie