linux-toradex.git/drivers/video/fbdev/amba-clcd.c, branch v5.5

video: amba-clcd: Spout an error if of_get_display_timing() gives an error

2019-07-26T14:32:28+00:00

In the patch ("video: of: display_timing: Don't yell if no timing node
is present") we'll stop spouting an error directly in
of_get_display_timing() if no node is present.  Presumably amba-clcd
should take charge of spouting its own error now.

NOTE: we'll print two errors if the node was present but there were
problems parsing the timing node (one in of_parse_display_timing() and
this new one).  Since this is a fatal error for the driver's probe
(and presumably someone will be debugging), this should be OK.

Signed-off-by: Douglas Anderson 
Reviewed-by: Sam Ravnborg 
Acked-by: Linus Walleij 
Cc: Thierry Reding 
Cc: David Airlie 
Cc: Philipp Zabel 
Cc: Tomi Valkeinen 
Cc: Laurent Pinchart 
Cc: Russell King 
Cc: Daniel Vetter 
Signed-off-by: Bartlomiej Zolnierkiewicz 
Link: https://patchwork.freedesktop.org/patch/msgid/20190722182439.44844-5-dianders@chromium.org

video: amba-clcd: Decomission Versatile and Nomadik

2019-04-11T17:25:12+00:00

These board families are now handled in the DRM subsystem
where we can have reusable panel drivers and some other
stuff. The PL111 there is now the driver used in the
defconfig for Versatile and Nomadik so no need to keep
this code around.

There are a few minor machines in arch/arm/ such as
mach-netx still using the old driver, so we need to keep
the core fbdev driver around for some time.

Signed-off-by: Linus Walleij 
Cc: Russell King 
Signed-off-by: Bartlomiej Zolnierkiewicz

video: ARM CLCD: Improve a size determination in clcdfb_probe()

2018-03-28T14:34:29+00:00

Replace the specification of a data structure by a pointer dereference
as the parameter for the operator "sizeof" to make the corresponding size
determination a bit safer according to the Linux coding style convention.

This issue was detected by using the Coccinelle software.

Signed-off-by: Markus Elfring 
Cc: Russell King 
Signed-off-by: Bartlomiej Zolnierkiewicz

video: ARM CLCD: Delete an error message for a failed memory allocation in clcdfb_probe()

2018-03-28T14:34:29+00:00

Omit an extra message for a memory allocation failure in this function.

This issue was detected by using the Coccinelle software.

Signed-off-by: Markus Elfring 
Cc: Russell King 
Signed-off-by: Bartlomiej Zolnierkiewicz

video: ARM CLCD: constify amba_id

2017-09-04T14:00:49+00:00

amba_id are not supposed to change at runtime. All functions
working with const amba_id. So mark the non-const structs as const.

Signed-off-by: Arvind Yadav 
Cc: Russell King 
Signed-off-by: Bartlomiej Zolnierkiewicz

video: ARM CLCD: fix dma allocation size

2017-04-07T15:03:24+00:00

This patch forces the frambuffer size to be aligned on kernel pages.

During the board startup, the splash screed did appear;
the "ts_test" program or our application were not able to start.

The following error message was reported:
error: failed to map framebuffer device to memory.
LinuxFB: driver cannot connect

The issue was discovered, on the LPC32xx platform, during the migration
of the LCD definition from the board file to the device tree.

Signed-off-by: Liam Beguin 
Signed-off-by: Sylvain Lemieux 
Cc: Vladimir Zapolskiy 
Cc: Russell King 
Signed-off-by: Bartlomiej Zolnierkiewicz

video: ARM CLCD: use panel device node for getting backlight and mode

2017-01-30T16:39:48+00:00

Having obtained panel device node in clcdfb_of_init_display() it allows to
generalize and simplify two more helper functions clcdfb_of_get_backlight()
and clcdfb_of_get_mode().

Cc: Tomi Valkeinen 
Cc: Russell King 
Cc: Linus Walleij 
Cc: Arnd Bergmann 
Signed-off-by: Vladimir Zapolskiy 
Signed-off-by: Bartlomiej Zolnierkiewicz

video: ARM CLCD: use panel device node for panel initialization

2017-01-30T16:39:48+00:00

There is no necessity to pass an endpoint device node to custom panel
initialization functions, because a child panel device node should be
sufficient, note that the existing init_panel() callback declaration from
linux/amba/clcd.h already prompts to use the panel device node here.

Cc: Tomi Valkeinen 
Cc: Russell King 
Cc: Linus Walleij 
Cc: Arnd Bergmann 
Signed-off-by: Vladimir Zapolskiy 
Signed-off-by: Bartlomiej Zolnierkiewicz

video: ARM CLCD: sort included headers out alphabetically

2017-01-30T16:39:48+00:00

To simplify the task of looking through the list of included headers sort
them out alphabetically, some of the truly redundant headers are removed
from the list.

Cc: Tomi Valkeinen 
Cc: Russell King 
Cc: Linus Walleij 
Cc: Arnd Bergmann 
Signed-off-by: Vladimir Zapolskiy 
Signed-off-by: Bartlomiej Zolnierkiewicz

video: ARM CLCD: fix endpoint lookup logic

2016-08-30T08:31:22+00:00

The addition of the Nomadik support in this driver introduced
a bug in clcdfb_of_init_display(), which now calls init_panel
with an uninitialized 'endpoint' pointer, as "gcc -Wmaybe-uninitialized"
warns:

drivers/video/fbdev/amba-clcd.c: In function 'clcdfb_of_init_display':
drivers/video/fbdev/amba-clcd.c:785:5: error: 'endpoint' may be used uninitialized in this function [-Werror=maybe-uninitialized]

This reverts the broken part of the function to what it was before
the patch, which is the best guess I have to what it should be.
I assume this was left over from an attempted rework of the
code that was partially backed out.

Signed-off-by: Arnd Bergmann 
Fixes: 046ad6cdeb3f ("video: ARM CLCD: support Nomadik variant")
Cc: Linus Walleij 
Reviewed-by: Linus Walleij 
Reported-by: Colin Ian King 
Signed-off-by: Tomi Valkeinen