Re: [PATCH 4/4] video: amba-clcd: Spout an error if of_get_display_timing() gives an error

From: Linus Walleij
Date: Tue Jul 23 2019 - 04:27:45 EST


On Mon, Jul 22, 2019 at 8:25 PM Douglas Anderson <dianders@xxxxxxxxxxxx> wrote:

> 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 <dianders@xxxxxxxxxxxx>

Acked-by: Linus Walleij <linus.walleij@xxxxxxxxxx>

Yours,
Linus Walleij