linux-stable.git/drivers/gpu/drm/mgag200, branch linux-3.17.y

PCI: Remove DEFINE_PCI_DEVICE_TABLE macro use

2014-08-12T18:15:14+00:00

We should prefer `struct pci_device_id` over `DEFINE_PCI_DEVICE_TABLE` to
meet kernel coding style guidelines.  This issue was reported by checkpatch.

A simplified version of the semantic patch that makes this change is as
follows (http://coccinelle.lip6.fr/):

// 

@@
identifier i;
declarer name DEFINE_PCI_DEVICE_TABLE;
initializer z;
@@

- DEFINE_PCI_DEVICE_TABLE(i)
+ const struct pci_device_id i[]
= z;

// 

[bhelgaas: add semantic patch]
Signed-off-by: Benoit Taine 
Signed-off-by: Bjorn Helgaas

drm/ttm: Fix a few sparse warnings

2014-07-22T00:58:21+00:00

The final parameter to ttm_bo_reserve() is a pointer, therefore callers
should use NULL instead of 0.

Fixes a bunch of sparse warnings of this type:

	warning: Using plain integer as NULL pointer

Signed-off-by: Thierry Reding 
Reviewed-by: Alex Deucher 
Signed-off-by: Dave Airlie

drm/mgag200: use helpers

2014-07-18T04:24:39+00:00

Signed-off-by: Rob Clark 
Signed-off-by: Dave Airlie

drm: Introduce drm_fb_helper_prepare()

2014-07-08T01:31:28+00:00

To implement hotplug detection in a race-free manner, drivers must call
drm_kms_helper_poll_init() before hotplug events can be triggered. Such
events can be triggered right after any of the encoders or connectors
are initialized. At the same time, if the drm_fb_helper_hotplug_event()
helper is used by a driver, then the poll helper requires some parts of
the FB helper to be initialized to prevent a crash.

At the same time, drm_fb_helper_init() requires information that is not
necessarily available at such an early stage (number of CRTCs and
connectors), so it cannot be used yet.

Add a new helper, drm_fb_helper_prepare(), that initializes the bare
minimum needed to allow drm_kms_helper_poll_init() to execute and any
subsequent hotplug events to be processed properly.

Reviewed-by: Daniel Vetter 
Signed-off-by: Thierry Reding 
Signed-off-by: Dave Airlie

drm: Constify struct drm_fb_helper_funcs

2014-07-08T01:31:15+00:00

There's no need for this to be modifiable. Make it const so that it can
be put into the .rodata section.

Reviewed-by: Daniel Vetter 
Signed-off-by: Thierry Reding 
Acked-by: Russell King 
Signed-off-by: Dave Airlie

drm: add register and unregister functions for connectors

2014-06-19T06:55:28+00:00

Introduce generic functions to register and unregister connectors. This
provides a common place to add and remove associated user space
interfaces.

Signed-off-by: Thomas Wood 
Reviewed-by: David Herrmann 
Signed-off-by: Daniel Vetter

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