linux-toradex.git/drivers/video/pxa168fb.c, branch v3.13.8

video: pxa168fb: use dev_get_platdata()

2013-09-20T09:22:01+00:00

Use the wrapper function for retrieving the platform data instead of
accessing dev->platform_data directly. This is a cosmetic change
to make the code simpler and enhance the readability.

Signed-off-by: Jingoo Han 
Signed-off-by: Tomi Valkeinen

Drivers: video: remove __dev* attributes.

2013-01-03T23:57:01+00:00

CONFIG_HOTPLUG is going away as an option.  As a result, the __dev*
markings need to be removed.

This change removes the use of __devinit, __devexit_p, __devinitdata,
__devinitconst, and __devexit from these drivers.

Based on patches originally written by Bill Pemberton, but redone by me
in order to handle some of the coding style issues better, by hand.

Cc: Bill Pemberton 
Signed-off-by: Greg Kroah-Hartman

drivers/video/pxa168fb.c: use devm_ functions

2012-02-24T00:49:59+00:00

The various devm_ functions allocate memory that is released when a driver
detaches.  This patch uses these functions for data that is allocated in
the probe function of a platform device and is only freed in the remove
function.

By using devm_ioremap, it also removes a potential memory leak, because
there was no call to iounmap in the probe function.

Signed-off-by: Julia Lawall 
Signed-off-by: Florian Tobias Schandinat

video: convert drivers/video/* to use module_platform_driver()

2011-12-03T22:08:42+00:00

This patch converts the drivers in drivers/video/* to use the
module_platform_driver() macro which makes the code smaller and a bit
simpler.

Cc: Ben Dooks 
Cc: Manuel Lauss 
Signed-off-by: Axel Lin 
Acked-by: Wan ZongShun 
Acked-by: Sascha Hauer 
Acked-by: Lennert Buytenhek 
Acked-by: Alexey Charkov 
Acked-by: Damian Hobson-Garcia 
Acked-by: Jingoo Han 
Signed-off-by: Florian Tobias Schandinat

treewide: Convert uses of struct resource to resource_size(ptr)

2011-06-10T12:55:36+00:00

Several fixes as well where the +1 was missing.

Done via coccinelle scripts like:

@@
struct resource *ptr;
@@

- ptr->end - ptr->start + 1
+ resource_size(ptr)

and some grep and typing.

Mostly uncompiled, no cross-compilers.

Signed-off-by: Joe Perches 
Signed-off-by: Jiri Kosina

drivers/video/pxa168fb.c: add missing clk_put

2011-06-02T08:07:46+00:00

Add a label for error-handling code in the case where only clk_get has
succeeded.  Rename the label failed to be consistent with the rest.

A simplified version of the semantic match that finds the missing clk_put
is as follows: (http://coccinelle.lip6.fr/)

// 
@r exists@
expression e1,e2;
statement S;
@@

e1 = clk_get@p1(...);
... when != e1 = e2
    when != clk_put(e1)
    when any
if (...) { ... when != clk_put(e1)
               when != if (...) { ... clk_put(e1) ... }
* return@p3 ...;
 } else S
// 

Signed-off-by: Julia Lawall 
Signed-off-by: Paul Mundt

video: pxa168fb: remove a redundant pxa168fb_check_var call

2011-01-25T06:09:19+00:00

Current implementation calls pxa168fb_check_var twice in pxa168fb_probe.

Signed-off-by: Axel Lin 
Signed-off-by: Paul Mundt

ARM: pxa168fb: add .remove function

2010-10-09T09:07:23+00:00

The pxa168fb driver is missing .remove function so the framebuffer isn't
correctly shut down when the module is removed.

Signed-off-by: Marek Vasut 
Signed-off-by: Haojian Zhuang 
Signed-off-by: Eric Miao

ARM: pxa168fb: clear enable bit when not active

2010-09-25T01:47:04+00:00

While fb isn't active, we should clear CFG_GRA_ENA bit. The existing code
can't clear this bit.

Signed-off-by: Haojian Zhuang 
Acked-by: Marek Vasut 
Signed-off-by: Eric Miao

ARM: pxa168fb: fix section mismatch

2010-08-30T01:59:44+00:00

Signed-off-by: Marek Vasut 
Acked-by: Haojian Zhuang 
Signed-off-by: Eric Miao