linux-media.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] staging: media: tegra-video: include video.h header
@ 2023-07-20  7:20 Hans Verkuil
  2023-07-20 13:44 ` Luca Ceresoli
  0 siblings, 1 reply; 2+ messages in thread
From: Hans Verkuil @ 2023-07-20  7:20 UTC (permalink / raw)
  To: Linux Media Mailing List; +Cc: Luca Ceresoli, Dmitry Osipenko

This tells sparse that tegra_vip_driver is actually used,
and so avoids this warning:

drivers/staging/media/tegra-video/vip.c:280:31: warning: 'tegra_vip_driver' defined but not used [-Wunused-variable]
  280 | static struct platform_driver tegra_vip_driver = {
      |                               ^~~~~~~~~~~~~~~~

Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl>
---
 drivers/staging/media/tegra-video/vip.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/staging/media/tegra-video/vip.c b/drivers/staging/media/tegra-video/vip.c
index a1ab886acc18..bc5fc82a7702 100644
--- a/drivers/staging/media/tegra-video/vip.c
+++ b/drivers/staging/media/tegra-video/vip.c
@@ -20,6 +20,7 @@
 #include <media/v4l2-fwnode.h>

 #include "vip.h"
+#include "video.h"

 static inline struct tegra_vip *host1x_client_to_vip(struct host1x_client *client)
 {
-- 
2.40.1


^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH] staging: media: tegra-video: include video.h header
  2023-07-20  7:20 [PATCH] staging: media: tegra-video: include video.h header Hans Verkuil
@ 2023-07-20 13:44 ` Luca Ceresoli
  0 siblings, 0 replies; 2+ messages in thread
From: Luca Ceresoli @ 2023-07-20 13:44 UTC (permalink / raw)
  To: Hans Verkuil; +Cc: Linux Media Mailing List, Dmitry Osipenko

Hi Hans,

On Thu, 20 Jul 2023 09:20:03 +0200
Hans Verkuil <hverkuil@xs4all.nl> wrote:

> This tells sparse that tegra_vip_driver is actually used,
> and so avoids this warning:
> 
> drivers/staging/media/tegra-video/vip.c:280:31: warning: 'tegra_vip_driver' defined but not used [-Wunused-variable]
>   280 | static struct platform_driver tegra_vip_driver = {
>       |                               ^~~~~~~~~~~~~~~~
> 
> Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl>


Reviewed-by: Luca Ceresoli <luca.ceresoli@bootlin.com>

-- 
Luca Ceresoli, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2023-07-20 13:44 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-07-20  7:20 [PATCH] staging: media: tegra-video: include video.h header Hans Verkuil
2023-07-20 13:44 ` Luca Ceresoli

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).