On Tue, Sep 13, 2022 at 06:21:10PM +0900, Takumi Sueda wrote:
> Signed-off-by: Takumi Sueda <puhit...@gmail.com>
> ---
>  drivers/video/simplefb.c | 6 +++++-
>  1 file changed, 5 insertions(+), 1 deletion(-)
> 
> diff --git a/drivers/video/simplefb.c b/drivers/video/simplefb.c
> index 2b0d8835e3..235ec761f7 100644
> --- a/drivers/video/simplefb.c
> +++ b/drivers/video/simplefb.c
> @@ -43,7 +43,11 @@ static int simple_video_probe(struct udevice *dev)
>  
>       uc_priv->xsize = fdtdec_get_uint(blob, node, "width", 0);
>       uc_priv->ysize = fdtdec_get_uint(blob, node, "height", 0);
> -     uc_priv->rot = 0;
> +     uc_priv->rot = fdtdec_get_uint(blob, node, "rot", 0);
> +     if (uc_priv->rot > 3) {
> +             log_debug("%s: invalid rot\n", __func__);
> +             return log_msg_ret("rot", -EINVAL);
> +     }
>  
>       format = fdt_getprop(blob, node, "format", NULL);
>       debug("%s: %dx%d@%s\n", __func__, uc_priv->xsize, uc_priv->ysize, 
> format);
> -- 
> 2.20.1
> 

Uh oh, I forgot to add the description. I'll send the v3.
Sorry for sending an incomplete patch.

Takumi

Reply via email to