diff mbox

[media] vsp1_drm.h: add missing prototypes

Message ID 18922936dc2817488ebba985c5aaf3498f2ef96d.1455883689.git.mchehab@osg.samsung.com (mailing list archive)
State Accepted
Delegated to: Geert Uytterhoeven
Headers show

Commit Message

Mauro Carvalho Chehab Feb. 19, 2016, 12:08 p.m. UTC
drivers/media/platform/vsp1/vsp1_drm.c:47:5: warning: no previous prototype for 'vsp1_du_init' [-Wmissing-prototypes]
 int vsp1_du_init(struct device *dev)
     ^
drivers/media/platform/vsp1/vsp1_drm.c:76:5: warning: no previous prototype for 'vsp1_du_setup_lif' [-Wmissing-prototypes]
 int vsp1_du_setup_lif(struct device *dev, unsigned int width,
     ^
drivers/media/platform/vsp1/vsp1_drm.c:221:6: warning: no previous prototype for 'vsp1_du_atomic_begin' [-Wmissing-prototypes]
 void vsp1_du_atomic_begin(struct device *dev)
      ^
drivers/media/platform/vsp1/vsp1_drm.c:273:5: warning: no previous prototype for 'vsp1_du_atomic_update' [-Wmissing-prototypes]
 int vsp1_du_atomic_update(struct device *dev, unsigned int rpf_index,
     ^
drivers/media/platform/vsp1/vsp1_drm.c:451:6: warning: no previous prototype for 'vsp1_du_atomic_flush' [-Wmissing-prototypes]
 void vsp1_du_atomic_flush(struct device *dev)
      ^

Signed-off-by: Mauro Carvalho Chehab <mchehab@osg.samsung.com>
---
 drivers/media/platform/vsp1/vsp1_drm.h | 11 +++++++++++
 1 file changed, 11 insertions(+)

Comments

Laurent Pinchart Feb. 20, 2016, 1:20 a.m. UTC | #1
Hi Mauro,

Thank you for the patch.

On Friday 19 February 2016 10:08:21 Mauro Carvalho Chehab wrote:
> drivers/media/platform/vsp1/vsp1_drm.c:47:5: warning: no previous prototype
> for 'vsp1_du_init' [-Wmissing-prototypes] int vsp1_du_init(struct device
> *dev)
>      ^
> drivers/media/platform/vsp1/vsp1_drm.c:76:5: warning: no previous prototype
> for 'vsp1_du_setup_lif' [-Wmissing-prototypes] int vsp1_du_setup_lif(struct
> device *dev, unsigned int width,
>      ^
> drivers/media/platform/vsp1/vsp1_drm.c:221:6: warning: no previous prototype
> for 'vsp1_du_atomic_begin' [-Wmissing-prototypes] void
> vsp1_du_atomic_begin(struct device *dev)
>       ^
> drivers/media/platform/vsp1/vsp1_drm.c:273:5: warning: no previous prototype
> for 'vsp1_du_atomic_update' [-Wmissing-prototypes] int
> vsp1_du_atomic_update(struct device *dev, unsigned int rpf_index, ^
> drivers/media/platform/vsp1/vsp1_drm.c:451:6: warning: no previous prototype
> for 'vsp1_du_atomic_flush' [-Wmissing-prototypes] void
> vsp1_du_atomic_flush(struct device *dev)

The functions are declared in include/media/vsp1.h, but the file isn't 
included in vsp1_drm.c due to a typo. I've included <linux/vsp1.h> instead of 
<media/vsp1.h>. I'll send a fix (the DRM patches don't depend on it, so it's 
not urgent), sorry about that.

> Signed-off-by: Mauro Carvalho Chehab <mchehab@osg.samsung.com>
> ---
>  drivers/media/platform/vsp1/vsp1_drm.h | 11 +++++++++++
>  1 file changed, 11 insertions(+)
> 
> diff --git a/drivers/media/platform/vsp1/vsp1_drm.h
> b/drivers/media/platform/vsp1/vsp1_drm.h index 7704038c3add..f68056838319
> 100644
> --- a/drivers/media/platform/vsp1/vsp1_drm.h
> +++ b/drivers/media/platform/vsp1/vsp1_drm.h
> @@ -35,4 +35,15 @@ int vsp1_drm_init(struct vsp1_device *vsp1);
>  void vsp1_drm_cleanup(struct vsp1_device *vsp1);
>  int vsp1_drm_create_links(struct vsp1_device *vsp1);
> 
> +int vsp1_du_init(struct device *dev);
> +int vsp1_du_setup_lif(struct device *dev, unsigned int width,
> +		      unsigned int height);
> +void vsp1_du_atomic_begin(struct device *dev);
> +int vsp1_du_atomic_update(struct device *dev, unsigned int rpf_index,
> +			  u32 pixelformat, unsigned int pitch,
> +			  dma_addr_t mem[2], const struct v4l2_rect *src,
> +			  const struct v4l2_rect *dst);
> +void vsp1_du_atomic_flush(struct device *dev);
> +
> +
>  #endif /* __VSP1_DRM_H__ */
diff mbox

Patch

diff --git a/drivers/media/platform/vsp1/vsp1_drm.h b/drivers/media/platform/vsp1/vsp1_drm.h
index 7704038c3add..f68056838319 100644
--- a/drivers/media/platform/vsp1/vsp1_drm.h
+++ b/drivers/media/platform/vsp1/vsp1_drm.h
@@ -35,4 +35,15 @@  int vsp1_drm_init(struct vsp1_device *vsp1);
 void vsp1_drm_cleanup(struct vsp1_device *vsp1);
 int vsp1_drm_create_links(struct vsp1_device *vsp1);
 
+int vsp1_du_init(struct device *dev);
+int vsp1_du_setup_lif(struct device *dev, unsigned int width,
+		      unsigned int height);
+void vsp1_du_atomic_begin(struct device *dev);
+int vsp1_du_atomic_update(struct device *dev, unsigned int rpf_index,
+			  u32 pixelformat, unsigned int pitch,
+			  dma_addr_t mem[2], const struct v4l2_rect *src,
+			  const struct v4l2_rect *dst);
+void vsp1_du_atomic_flush(struct device *dev);
+
+
 #endif /* __VSP1_DRM_H__ */