[PATCH 4/8] include: video: Reserve video using blob

[Devarsh Thakkar]

Hi Nikhil,

Thanks for the patch.

On 11/05/23 15:29, Nikhil M Jain wrote:
> Add method to reserve video using blob.

reserve memory for video using blob
received from previous stage.
> 

Mention what info is updated.

> Signed-off-by: Nikhil M Jain <n-jain1 at ti.com>
> ---
>  drivers/video/video-uclass.c | 12 ++++++++++++
>  include/video.h              |  9 +++++++++
>  2 files changed, 21 insertions(+)
> 
> diff --git a/drivers/video/video-uclass.c b/drivers/video/video-uclass.c
> index 8396bdfb11..1264ad1101 100644
> --- a/drivers/video/video-uclass.c
> +++ b/drivers/video/video-uclass.c
> @@ -142,6 +142,18 @@ int video_reserve(ulong *addrp)
>  	return 0;
>  }
>  
> +int video_reserve_from_blob(struct video_handoff *ho)
> +{

video_reserve_from_bloblist would be better name imho as bloblist is used
elsewhere too.

> +#if CONFIG_IS_ENABLED(VIDEO)

This CONFIG may not be required as file is video-uclass, so I assume
CONFIG_VIDEO is always enabled.

Regards
Devarsh

> +	gd->video_bottom = ho->fb;
> +	gd->fb_base = ho->fb;
> +	gd->video_top = ho->fb + ho->size;
> +	debug("Reserving %luk for video using blob at: %08x\n",
> +	      ((unsigned long)ho->size) >> 10, (u32)ho->fb);
> +#endif
> +	return 0;
> +}
> +
>  int video_fill(struct udevice *dev, u32 colour)
>  {
>  	struct video_priv *priv = dev_get_uclass_priv(dev);
> diff --git a/include/video.h b/include/video.h
> index 18ed159b8d..13460adc45 100644
> --- a/include/video.h
> +++ b/include/video.h
> @@ -389,4 +389,13 @@ int bmp_display(ulong addr, int x, int y);
>   */
>  int bmp_info(ulong addr);
>  
> +/*
> + * video_reserve_from_blob()- Reserve frame-buffer memory for video devices
> + * using blobs.
> + *
> + * @ho: video information passed from SPL
> + * Returns: 0 (always)
> + */
> +int video_reserve_from_blob(struct video_handoff *ho);
> +
>  #endif