linux.git/drivers/video, branch v6.5-rc4

fbdev: Explicitly include correct DT includes

2023-07-20T05:56:30+00:00

The DT of_device.h and of_platform.h date back to the separate
of_platform_bus_type before it as merged into the regular platform bus.
As part of that merge prepping Arm DT support 13 years ago, they
"temporarily" include each other. They also include platform_device.h
and of.h. As a result, there's a pretty much random mix of those include
files used throughout the tree. In order to detangle these headers and
replace the implicit includes with struct declarations, users need to
explicitly include the correct includes.

Reviewed-by: Thomas Zimmermann 
Signed-off-by: Rob Herring 
Signed-off-by: Helge Deller

fbdev: ep93xx-fb: fix return value check in ep93xxfb_probe

2023-07-20T05:52:55+00:00

function clk_prepare_enable may fail in ep93xxfb_probe, therefore,
add a return value check to clk_prepare_enable and handle the
error.

Signed-off-by: Yuanjun Gong 
Signed-off-by: Helge Deller

fbdev: au1200fb: Fix missing IRQ check in au1200fb_drv_probe

2023-07-20T05:52:54+00:00

This func misses checking for platform_get_irq()'s call and may passes the
negative error codes to request_irq(), which takes unsigned IRQ #,
causing it to fail with -EINVAL, overriding an original error code.

Fix this by stop calling request_irq() with invalid IRQ #s.

Fixes: 1630d85a8312 ("au1200fb: fix hardcoded IRQ")
Signed-off-by: Zhang Shurong 
Signed-off-by: Helge Deller

fbdev: kyro: make some const read-only arrays static and reduce type size

2023-07-20T05:52:54+00:00

Don't populate the const read-only arrays on the stack but instead
make them static const. Use smaller types to use less storage for
the arrays.  Also makes the object code a little smaller.

Signed-off-by: Colin Ian King 
Signed-off-by: Helge Deller

fbcon: remove unused display (p) from fbcon_redraw()

2023-07-20T05:43:11+00:00

The parameter is unused.

Signed-off-by: "Jiri Slaby (SUSE)" 
Cc: Daniel Vetter 
Cc: Helge Deller 
Cc: linux-fbdev@vger.kernel.org
Cc: dri-devel@lists.freedesktop.org
Signed-off-by: Helge Deller

sticon: make sticon_set_def_font() void and remove op parameter

2023-07-20T05:43:11+00:00

sticon_set_def_font() always returns 0, so make it void. And remove an
unused 'op' parameter.

Signed-off-by: "Jiri Slaby (SUSE)" 
Cc: "James E.J. Bottomley" 
Cc: Helge Deller 
Cc: linux-parisc@vger.kernel.org
Cc: linux-fbdev@vger.kernel.org
Cc: dri-devel@lists.freedesktop.org
Signed-off-by: Helge Deller

vgacon: cache vc_cell_height in vgacon_cursor()

2023-07-20T05:43:11+00:00

There are many places c->vc_cell_height is used in the code of
vgacon_cursor(). Caching the value to a local variable makes the code
much easier to follow.

Signed-off-by: "Jiri Slaby (SUSE)" 
Cc: Helge Deller 
Cc: linux-fbdev@vger.kernel.org
Cc: dri-devel@lists.freedesktop.org
Signed-off-by: Helge Deller

vgacon: let vgacon_doresize() return void

2023-07-20T05:43:11+00:00

The return value is neither used, nor vgacon_doresize() returns an
error. So change the reurn type to void.

Signed-off-by: "Jiri Slaby (SUSE)" 
Cc: Helge Deller 
Cc: linux-fbdev@vger.kernel.org
Cc: dri-devel@lists.freedesktop.org
Signed-off-by: Helge Deller

vgacon: remove unused xpos from vgacon_set_cursor_size()

2023-07-20T05:43:11+00:00

xpos is unused, remove it.

Signed-off-by: "Jiri Slaby (SUSE)" 
Cc: Helge Deller 
Cc: linux-fbdev@vger.kernel.org
Cc: dri-devel@lists.freedesktop.org
Signed-off-by: Helge Deller

vgacon: remove unneeded forward declarations

2023-07-20T05:43:11+00:00

Most of the forward declarations in vgacon are not needed. Drop them.

Signed-off-by: "Jiri Slaby (SUSE)" 
Cc: Helge Deller 
Cc: linux-fbdev@vger.kernel.org
Cc: dri-devel@lists.freedesktop.org
Signed-off-by: Helge Deller