[PATCH v2 2/5] video: Let video_sync to return error value

[Simon Glass]

On Mon, 14 Dec 2020 at 01:38, Michal Simek <michal.simek at xilinx.com> wrote:
>
> This patch is preparation for follow up one to support cases where
> synchronization can fail.
>
> Suggested-by: Simon Glass <sjg at chromium.org>
> Signed-off-by: Michal Simek <michal.simek at xilinx.com>
>
> ---
>
> Changes in v2:
> - New patch is series
>
>  drivers/video/vidconsole-uclass.c | 40 ++++++++++++++++++++++---------
>  drivers/video/video-uclass.c      | 12 +++++++---
>  include/video.h                   |  4 +++-
>  3 files changed, 41 insertions(+), 15 deletions(-)

Reviewed-by: Simon Glass <sjg at chromium.org>

[..]


> diff --git a/include/video.h b/include/video.h
> index 7313b17f7ce8..1bfe6843a805 100644
> --- a/include/video.h
> +++ b/include/video.h
> @@ -155,11 +155,13 @@ int video_clear(struct udevice *dev);
>   * @force:     True to force a sync even if there was one recently (this is
>   *             very expensive on sandbox)
>   *
> + * @return: 0 always
> + *
>   * Some frame buffers are cached or have a secondary frame buffer. This
>   * function syncs these up so that the current contents of the U-Boot frame
>   * buffer are displayed to the user.
>   */
> -void video_sync(struct udevice *vid, bool force);
> +int video_sync(struct udevice *vid, bool force);
>
>  /**
>   * video_sync_all() - Sync all devices' frame buffers with there hardware
> --
> 2.29.2
>