linux-toradex.git/drivers/video/s3c2410fb.c, branch v3.8.13

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

video: s3c2410: fix checkpatch warnings

2012-09-22T21:37:56+00:00

This patch fixes the checkpatch warnings listed below:

WARNING: usleep_range should not use min == max args; see Documentation/timers/timers-howto.txt
WARNING: quoted string split across lines

Signed-off-by: Jingoo Han 
Signed-off-by: Florian Tobias Schandinat

video: s3c2410: Use pr_* and dev_* instead of printk

2012-09-22T21:37:56+00:00

printk calls are replaced by pr_* and dev_* calls to silence
checkpatch warnings.

Signed-off-by: Sachin Kamat 
Signed-off-by: Jingoo Han 
Signed-off-by: Florian Tobias Schandinat

video: s3c2410: fix checkpatch error and warnings

2011-12-03T22:10:43+00:00

This patch fixes the checkpatch errors listed below:

ERROR: do not initialise statics to 0 or NULL
WARNING: Use #include  instead of 
WARNING: braces {} are not necessary for single statement blocks
WARNING: braces {} are not necessary for any arm of this statement
WARNING: static char array declaration should probably be static const char
WARNING: line over 80 characters
WARNING: msleep < 20ms can sleep for up to 20ms; see Documentation/timers/timers-howto.txt

Signed-off-by: Jingoo Han 
Signed-off-by: Florian Tobias Schandinat

video: irq: Remove IRQF_DISABLED

2011-10-03T15:52:22+00:00

Since commit [e58aa3d2: genirq: Run irq handlers with interrupts disabled],
We run all interrupt handlers with interrupts disabled
and we even check and yell when an interrupt handler
returns with interrupts enabled (see commit [b738a50a:
genirq: Warn when handler enables interrupts]).

So now this flag is a NOOP and can be removed.

Signed-off-by: Yong Zhang 
Acked-by: David Brown 
Signed-off-by: Florian Tobias Schandinat

video: s3c2410: remove unused variable

2011-07-13T07:53:17+00:00

Fixes:

drivers/video/s3c2410fb.c: In function 's3c2410fb_cpufreq_transition':
drivers/video/s3c2410fb.c:770:24: warning: unused variable 'freqs'

Signed-off-by: Wolfram Sang 
Cc: Paul Mundt 
Signed-off-by: Paul Mundt

drivers/video/s3c2410fb.c: Convert release_resource to release_mem_region

2011-05-24T07:09:03+00:00

Request_mem_region should be used with release_mem_region, not
release_resource.

The semantic match that finds this problem is as follows:
(http://coccinelle.lip6.fr/)

// 
@s exists@
expression e1,e2,e3,e4,e;
@@

*e4 = request_mem_region(e1,e2,e3)
... when != e4 = e
*release_resource(e4);
// 

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

s3c2410fb: don't treat NULL clk as an error

2011-01-12T06:00:31+00:00

clk_get() returns a struct clk cookie to the driver and some platforms
may return NULL if they only support a single clock.  clk_get() has only
failed if it returns a ERR_PTR() encoded pointer.

Cc: Vincent Sanders 
Cc: linux-fbdev@vger.kernel.org
Signed-off-by: Jamie Iles 
Signed-off-by: Paul Mundt

s3c2410fb: fix sections

2010-05-25T15:07:09+00:00

Since the drivers probe calls were changed from .init.text to
.devinit.text in commit c2e13037e6794bd0d9de3f9ecabf5615f15c160b
("platform-drivers: move probe to .devinit.text in drivers/video") all the
function from .init.text should be moved to .devinit.text, too.

The drivers remove calls can also be move from .text to .devexit.text.

Signed-off-by: Henrik Kretzschmar 
Cc: Vincent Sanders 
Acked-by: Arnaud Patard 
Tested-by: Arnaud Patard 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

platform-drivers: move probe to .devinit.text in drivers/video

2010-03-08T01:04:50+00:00

A pointer to a probe callback is passed to the core via
platform_driver_register and so the function must not disappear when the
.init sections are discarded.  Otherwise (if also having HOTPLUG=y)
unbinding and binding a device to the driver via sysfs will result in an
oops as does a device being registered late.

An alternative to this patch is using platform_driver_probe instead of
platform_driver_register plus removing the pointer to the probe function
from the struct platform_driver.

Signed-off-by: Uwe Kleine-König 
Cc: Adrian Bunk 
Cc: Alberto Mardegan 
Cc: Andrew Morton 
Cc: Andriy Skulysh 
Cc: Antonino Daplas 
Cc: Anton Vorontsov 
Cc: Ben Dooks 
Cc: Chandramouli Narayanan 
Cc: Christoph Hellwig 
Cc: Frans Pop 
Cc: Geert Uytterhoeven 
Cc: Greg Kroah-Hartman 
Cc: Helge Deller 
Cc: Huang Ying 
Cc: Ian Molton 
Cc: Joshua Kinard 
Cc: Kaj-Michael Lang 
Cc: Krzysztof Helt 
Cc: linux-fbdev-devel@lists.sourceforge.net
Cc: Maciej W. Rozycki 
Cc: Magnus Damm 
Cc: Martin Michlmayr 
Cc: Matthias Kaehlcke 
Cc: Paul Mundt 
Cc: Pavel Machek 
Cc: Philipp Zabel 
Cc: Richard Purdie 
Cc: Roel Kluin 
Cc: Roland Stigge 
Cc: Russell King 
Cc: Thomas Bogendoerfer 
Cc: Vincent Sanders 
Cc: Yoichi Yuasa 
Acked-by: Ralf Baechle 
Acked-by: Arnaud Patard 
Acked-by: James Simmons 
Acked-by: Peter Jones 
Acked-by: Jaya Kumar 
Signed-off-by: Greg Kroah-Hartman