talos-op-linux/drivers/video/mb862xx, branch master

video: move fbdev to drivers/video/fbdev

2014-04-17T05:10:19+00:00

The drivers/video directory is a mess. It contains generic video related files, directories for backlight, console, linux logo, lots of fbdev device drivers, fbdev framework files. Make some order into the chaos by creating drivers/video/fbdev directory, and move all fbdev related files there. No functionality is changed, although I guess it is possible that some subtle Makefile build order related issue could be created by this patch. Signed-off-by: Tomi Valkeinen Acked-by: Laurent Pinchart Acked-by: Geert Uytterhoeven Acked-by: Rob Clark Acked-by: Jingoo Han Acked-by: Daniel Vetter

video: mb862xx: Remove redundant dev_set_drvdata

2013-09-26T10:49:09+00:00

Driver core sets driver data to NULL upon failure or remove. Signed-off-by: Sachin Kamat Acked-by: Anatolij Gustschin Signed-off-by: Tomi Valkeinen

video: mb862xxfb: remove unnecessary pci_set_drvdata()

2013-09-19T10:30:08+00:00

The driver core clears the driver data to NULL after device_release or on probe failure. Thus, it is not needed to manually clear the device driver data to NULL. 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/mb862xx/mb862xxfbdrv.c: fix error return code

2012-09-22T21:41:51+00:00

Convert a nonnegative error return code to a negative one, as returned elsewhere in the function. A simplified version of the semantic match that finds this problem is as follows: (http://coccinelle.lip6.fr/) // ( if@p1 (\(ret < 0\|ret != 0\)) { ... return ret; } | ret@p1 = 0 ) ... when != ret = e1 when != &ret *if(...) { ... when != ret = e2 when forall return ret; } // Signed-off-by: Peter Senna Tschudin Signed-off-by: Florian Tobias Schandinat

video: mb862xxfb: prevent divide by zero bug

2012-08-23T12:34:00+00:00

Do a sanity check on these before using them as divisors. Signed-off-by: Dan Carpenter Acked-by: Anatolij Gustschin Signed-off-by: Florian Tobias Schandinat

video: mb862xxfbdrv.c: local functions should not be exposed globally

2012-05-08T16:57:30+00:00

Functions not referenced outside of a source file should be marked static to prevent them from being exposed globally. Quiets the sparse warnings: warning: symbol 'mb862xx_intr' was not declared. Should it be static? Signed-off-by: H Hartley Sweeten Cc: Anatolij Gustschin Signed-off-by: Florian Tobias Schandinat

video: mb862xx-i2c: local functions should not be exposed globally

2012-05-08T16:57:23+00:00

Functions not referenced outside of a source file should be marked static to prevent them from being exposed globally. Quiets the sparse warnings: warning: symbol 'mb862xx_i2c_stop' was not declared. Should it be static? Signed-off-by: H Hartley Sweeten Cc: Anatolij Gustschin Signed-off-by: Florian Tobias Schandinat

video: Add module.h to drivers/video files who really use it.

2011-10-31T23:31:33+00:00

They were getting this implicitly by an include of module.h from device.h -- but we are going to clean that up and break that include chain, so include module.h explicitly now. [ with contributions from Axel Lin ] Signed-off-by: Paul Gortmaker

video: Add export.h for THIS_MODULE/EXPORT_SYMBOL to drivers/video

2011-10-31T23:31:33+00:00

With module.h being implicitly everywhere via device.h, the absence of explicitly including something for EXPORT_SYMBOL went unnoticed. Since we are heading to fix things up and clean module.h from the device.h file, we need to explicitly include these files now. Signed-off-by: Paul Gortmaker