linux-stable.git/drivers/gpu/drm/mgag200, branch v3.16.2

drm/mgag200: Remove unecessary NULL check in gem_free

2014-04-22T13:39:04+00:00

The ->gem_free_object never gets called with a NULL pointer, the check
is redundant. Also checking after the upcast allows compilers to elide
it anyway.

Spotted by coverity.

Reviewed-by: David Herrmann 
Reviewed-by: Ian Romanick 
Signed-off-by: Daniel Vetter

drm/mgag200: Remove unecessary NULL check in bo_unref

2014-04-22T13:39:03+00:00

ttm_bo_unref unconditionally calls kref_put on it's argument, so the
thing can't be NULL without already causing Oopses.

Spotted by coverity.

Reviewed-by: David Herrmann 
Reviewed-by: Ian Romanick 
Signed-off-by: Daniel Vetter

drm: Replace crtc fb with primary plane fb (v3)

2014-04-02T00:18:28+00:00

Now that CRTC's have a primary plane, there's no need to track the
framebuffer in the CRTC.  Replace all references to the CRTC fb with the
primary plane's fb.

This patch was generated by the Coccinelle semantic patching tool using
the following rules:

        @@ struct drm_crtc C; @@
        -   (C).fb
        +   C.primary->fb

        @@ struct drm_crtc *C; @@
        -   (C)->fb
        +   C->primary->fb

v3: Generate patch via coccinelle.  Actual removal of crtc->fb has been
    moved to a subsequent patch.

v2: Fixup several lingering crtc->fb instances that were missed in the
    first patch iteration.  [Rob Clark]

Signed-off-by: Matt Roper 
Reviewed-by: Rob Clark

drm: init TTM dev_mapping in ttm_bo_device_init()

2014-03-16T11:23:42+00:00

With dev->anon_inode we have a global address_space ready for operation
right from the beginning. Therefore, there is no need to do a delayed
setup with TTM. Instead, set dev_mapping during initialization in
ttm_bo_device_init() and remove any "if (dev_mapping)" conditions.

Cc: Dave Airlie 
Cc: Ben Skeggs 
Cc: Maarten Lankhorst 
Cc: Alex Deucher 
Cc: Thomas Hellstrom 
Signed-off-by: David Herrmann

drm: use anon-inode instead of relying on cdevs

2014-03-16T11:23:33+00:00

DRM drivers share a common address_space across all character-devices of a
single DRM device. This allows simple buffer eviction and mapping-control.
However, DRM core currently waits for the first ->open() on any char-dev
to mark the underlying inode as backing inode of the device. This delayed
initialization causes ugly conditions all over the place:
  if (dev->dev_mapping)
    do_sth();

To avoid delayed initialization and to stop reusing the inode of the
char-dev, we allocate an anonymous inode for each DRM device and reset
filp->f_mapping to it on ->open().

Signed-off-by: David Herrmann

drm/mgag200,ast,cirrus: fix regression with drm_can_sleep conversion

2014-02-06T01:39:03+00:00

I totally sign inverted my way out of this one.

Cc: stable@vger.kernel.org
Reported-by: "Sabrina Dubroca" 
Signed-off-by: Dave Airlie

drm/mgag200: fix typo causing bw limits to be ignored on some chips

2014-02-05T04:31:32+00:00

mode->mdev otherwise the bw limits never kick in.

Reported in RHEL testing.

Cc: stable@vger.kernel.org
Signed-off-by: Dave Airlie

drm: ast,cirrus,mgag200: use drm_can_sleep

2014-01-29T00:51:52+00:00

these 3 were checking in_interrupt but we have situations where
calling vunmap under this could cause a BUG to be hit in
smp_call_function_many. Use the drm_can_sleep macro instead,
which should stop this path from been taken in this case.

Cc: stable@vger.kernel.org
Signed-off-by: Dave Airlie

drm/mgag200: on cards with < 2MB VRAM default to 16-bit

2014-01-23T01:01:35+00:00

This aligns with what the userspace -mga driver does in
the same situation.

Signed-off-by: Dave Airlie

drm/mgag200: fix oops in cursor code.

2014-01-16T04:43:04+00:00

In some cases we enter the cursor code with file_priv = NULL causing an oops,
we also can try to unpin something that isn't pinned, and this is a good fix for it.

Cc: stable@vger.kernel.org
Signed-off-by: Dave Airlie