[Concept,16/22] video: Tidy up the comment for sync()

Message ID 20251003165525.440173-17-sjg@u-boot.org
State New
Headers
Series video: Enhancements to support a pointer |

Commit Message

Simon Glass Oct. 3, 2025, 4:55 p.m. UTC
  From: Simon Glass <sjg@chromium.org>

Adjust this comment to match the style of other uclasses.

Signed-off-by: Simon Glass <sjg@chromium.org>
---

 include/video.h | 20 ++++++++++++++------
 1 file changed, 14 insertions(+), 6 deletions(-)
  

Patch

diff --git a/include/video.h b/include/video.h
index 116a10afc93..3ce603384dc 100644
--- a/include/video.h
+++ b/include/video.h
@@ -133,14 +133,22 @@  struct video_priv {
 
 /**
  * struct video_ops - structure for keeping video operations
- * @sync: Synchronize FB with device. Some device like SPI based LCD
- *	  displays needs synchronization when data in an FB is available.
- *	  For these devices implement sync hook to call a sync function.
- *	  vid is pointer to video device udevice. Function should return 0
- *	  on success and error code otherwise
  */
 struct video_ops {
-	int (*sync)(struct udevice *vid);
+	/**
+	 * @sync() - Synchronize FB with device
+	 *
+	 * Some devices like SPI-based LCD displays needs synchronization when
+	 * data in a framebuffer is available. These devices can implement this
+	 * method which is called whenever a video device is synced.
+	 *
+	 * Note that if CONFIG_VIDEO_DAMAGE is enabled, the driver can use this
+	 * to optimise the region to redraw.
+	 *
+	 * @dev: Video device
+	 * Return 0 on success, or -ve error code
+	 */
+	int (*sync)(struct udevice *dev);
 };
 
 #define video_get_ops(dev)        ((struct video_ops *)(dev)->driver->ops)