linux.git/drivers/video/ffb.c, branch v2.6.24

[FFB]: source cleanup

2007-10-14T04:53:08+00:00

This patch cleans up the driver a bit. It contains
coding style fixes (pointed by Lindent and checkpatch)
and white space clean ups.

Signed-off-by: Krzysztof Helt 
Signed-off-by: David S. Miller

[VIDEO]: Fix OOPS in all SBUS framebuffer drivers.

2007-07-30T07:27:33+00:00

All of these drivers use a silly:

struct all_info {
	struct fb_info info;
	struct foo_par par;
};

struct all_info *all = kzalloc(sizeof(*all), GFP_KERNEL);
all->info.par = &all->par;

etc. etc. code sequence, basically replicating the provided
framebuffer_alloc()/framebuffer_release(), and doing it badly.

Not only is this massive code duplication, it also caused a
bug in that we weren't setting the fb_info->device pointer
which results in an OOPS when fb_is_primary_device() runs.

Fix all of this by using framebuffer_{alloc,release}() and
passing in "&of_device->dev" as the device pointer.

Signed-off-by: David S. Miller

[VIDEO] ffb: The pseudo_palette is only 16 elements long

2007-06-05T20:14:33+00:00

The pseudo_palette is only 16 elements long.

Signed-off-by: Antonino Daplas 
Signed-off-by: David S. Miller

[VIDEO] ffb: Fix two DAC handling bugs.

2007-03-27T06:18:09+00:00

The determination of whether the DAC has inverted cursor logic is
broken, import the version checks the X.org driver uses to fix this.

Next, when we change the timing generator, borrow code from X.org that
does 10 NOP reads of the timing generator register afterwards to make
sure the video-enable transition occurs cleanly.

Finally, use macros for the DAC registers and fields in order to
provide documentation for the next person who reads this code.

Signed-off-by: David S. Miller

[SPARC64]: Fix of_iounmap() region release.

2006-12-31T22:06:05+00:00

We need to pass in the resource otherwise we cannot
release the region properly.  We must know whether it is
an I/O or MEM resource.

Spotted by Eric Brower.

Signed-off-by: David S. Miller

[PATCH] ioremap balanced with iounmap for drivers/video/ffb

2006-12-08T16:29:04+00:00

ioremap must be balanced by an iounmap and failing to do so can result in a
memory leak.

Signed-off-by: Amol Lad 
Cc: "Antonino A. Daplas" 
Cc: David S. Miller 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

[SPARC]: Convert all FB SBUS drivers to of_driver framework.

2006-06-29T23:37:18+00:00

Signed-off-by: David S. Miller

BUG_ON() Conversion in drivers/video/

2006-03-24T17:53:18+00:00

this changes if() BUG(); constructs to BUG_ON() which is
cleaner, contains unlikely() and can better optimized away.

Signed-off-by: Eric Sesterhenn 
Signed-off-by: Adrian Bunk

[PATCH] fbdev: Sanitize ->fb_mmap prototype

2006-01-15T02:27:15+00:00

No need for a file argument.  If we'd really need it it's in vma->vm_file
already.  gbefb and sgivwfb used to set vma->vm_file to the file argument, but
the kernel alrady did that.

Signed-off-by: Christoph Hellwig 
Signed-off-by: Antonino Daplas 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

[PATCH] fbdev: Sanitize ->fb_ioctl prototype

2006-01-15T02:27:14+00:00

The ioctl and file arguments to ->fb_mmap are totally unused and there's not
reason a driver should need them.

Also update the ->fb_compat_ioctl prototype to be the same as ->fb_mmap.

Signed-off-by: Antonino Daplas 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds