[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CA+V-a8u7NC5+2k051EP_V_HLn638X5Fzwj_UcM83ZV24qES0uQ@mail.gmail.com>
Date: Sun, 5 Sep 2021 22:41:34 +0100
From: "Lad, Prabhakar" <prabhakar.csengg@...il.com>
To: Randy Dunlap <rdunlap@...radead.org>
Cc: LKML <linux-kernel@...r.kernel.org>,
kernel test robot <lkp@...el.com>,
Hans Verkuil <hverkuil-cisco@...all.nl>,
Sakari Ailus <sakari.ailus@...ux.intel.com>,
Ezequiel Garcia <ezequiel@...labora.com>,
Mauro Carvalho Chehab <mchehab@...nel.org>,
linux-media <linux-media@...r.kernel.org>
Subject: Re: [PATCH] media: i2c: ths8200 needs V4L2_ASYNC
Hi Randy,
Thank you for the patch.
On Sun, Sep 5, 2021 at 12:28 AM Randy Dunlap <rdunlap@...radead.org> wrote:
>
> Fix the build errors reported by the kernel test robot by
> selecting V4L2_ASYNC:
>
> mips-linux-ld: drivers/media/i2c/ths8200.o: in function `ths8200_remove':
> ths8200.c:(.text+0x1ec): undefined reference to `v4l2_async_unregister_subdev'
> mips-linux-ld: drivers/media/i2c/ths8200.o: in function `ths8200_probe':
> ths8200.c:(.text+0x404): undefined reference to `v4l2_async_register_subdev'
>
> Fixes: ed29f89497006 ("media: i2c: ths8200: support asynchronous probing")
> Signed-off-by: Randy Dunlap <rdunlap@...radead.org>
> Reported-by: kernel test robot <lkp@...el.com>
> Cc: "Lad, Prabhakar" <prabhakar.csengg@...il.com>
> Cc: Hans Verkuil <hverkuil-cisco@...all.nl>
> Cc: Sakari Ailus <sakari.ailus@...ux.intel.com>
> Cc: Ezequiel Garcia <ezequiel@...labora.com>
> Cc: Mauro Carvalho Chehab <mchehab@...nel.org>
> Cc: linux-media@...r.kernel.org
> ---
> drivers/media/i2c/Kconfig | 1 +
> 1 file changed, 1 insertion(+)
>
Reviewed-by: Lad Prabhakar <prabhakar.csengg@...il.com>
Cheers,
Prabhakar
> --- linux-next-20210903.orig/drivers/media/i2c/Kconfig
> +++ linux-next-20210903/drivers/media/i2c/Kconfig
> @@ -597,6 +597,7 @@ config VIDEO_AK881X
> config VIDEO_THS8200
> tristate "Texas Instruments THS8200 video encoder"
> depends on VIDEO_V4L2 && I2C
> + select V4L2_ASYNC
> help
> Support for the Texas Instruments THS8200 video encoder.
>
Powered by blists - more mailing lists