Message ID | 20210120090148.30598-1-prabhakar.mahadev-lad.rj@bp.renesas.com (mailing list archive) |
---|---|
State | Accepted, archived |
Delegated to: | Sakari Ailus |
Headers |
Received: from vger.kernel.org ([23.128.96.18]) by www.linuxtv.org with esmtp (Exim 4.92) (envelope-from <linux-media-owner@vger.kernel.org>) id 1l29P9-005bC5-RU; Wed, 20 Jan 2021 09:03:56 +0000 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1728167AbhATJDa (ORCPT <rfc822;mkrufky@linuxtv.org> + 1 other); Wed, 20 Jan 2021 04:03:30 -0500 Received: from relmlor2.renesas.com ([210.160.252.172]:47617 "EHLO relmlie6.idc.renesas.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1726928AbhATJCy (ORCPT <rfc822;linux-media@vger.kernel.org>); Wed, 20 Jan 2021 04:02:54 -0500 X-IronPort-AV: E=Sophos;i="5.79,360,1602514800"; d="scan'208";a="69371153" Received: from unknown (HELO relmlir6.idc.renesas.com) ([10.200.68.152]) by relmlie6.idc.renesas.com with ESMTP; 20 Jan 2021 18:01:51 +0900 Received: from localhost.localdomain (unknown [10.226.36.204]) by relmlir6.idc.renesas.com (Postfix) with ESMTP id 2A13741F8151; Wed, 20 Jan 2021 18:01:49 +0900 (JST) From: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com> To: Sakari Ailus <sakari.ailus@linux.intel.com>, Mauro Carvalho Chehab <mchehab@kernel.org>, Laurent Pinchart <laurent.pinchart@ideasonboard.com>, Jacopo Mondi <jacopo+renesas@jmondi.org>, linux-media@vger.kernel.org Cc: linux-kernel@vger.kernel.org, Biju Das <biju.das.jz@bp.renesas.com>, Prabhakar <prabhakar.csengg@gmail.com> Subject: [PATCH] media: i2c/Kconfig: Select FWNODE for OV772x sensor Date: Wed, 20 Jan 2021 09:01:48 +0000 Message-Id: <20210120090148.30598-1-prabhakar.mahadev-lad.rj@bp.renesas.com> X-Mailer: git-send-email 2.17.1 Precedence: bulk List-ID: <linux-media.vger.kernel.org> X-Mailing-List: linux-media@vger.kernel.org X-LSpam-Score: -2.4 (--) X-LSpam-Report: No, score=-2.4 required=5.0 tests=BAYES_00=-1.9,HEADER_FROM_DIFFERENT_DOMAINS=0.5,MAILING_LIST_MULTI=-1 autolearn=ham autolearn_force=no |
Series |
media: i2c/Kconfig: Select FWNODE for OV772x sensor
|
|
Commit Message
Prabhakar Mahadev Lad
Jan. 20, 2021, 9:01 a.m. UTC
Fix OV772x build breakage by selecting V4L2_FWNODE config:
ia64-linux-ld: drivers/media/i2c/ov772x.o: in function `ov772x_probe':
ov772x.c:(.text+0x1ee2): undefined reference to `v4l2_fwnode_endpoint_alloc_parse'
ia64-linux-ld: ov772x.c:(.text+0x1f12): undefined reference to `v4l2_fwnode_endpoint_free'
ia64-linux-ld: ov772x.c:(.text+0x2212): undefined reference to `v4l2_fwnode_endpoint_alloc_parse'
Fixes: 8a10b4e3601e ("media: i2c: ov772x: Parse endpoint properties")
Reported-by: kernel test robot <lkp@intel.com>
Signed-off-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>
---
drivers/media/i2c/Kconfig | 1 +
1 file changed, 1 insertion(+)
Comments
Hi Prabhakar, On Wed, Jan 20, 2021 at 09:01:48AM +0000, Lad Prabhakar wrote: > Fix OV772x build breakage by selecting V4L2_FWNODE config: > > ia64-linux-ld: drivers/media/i2c/ov772x.o: in function `ov772x_probe': > ov772x.c:(.text+0x1ee2): undefined reference to `v4l2_fwnode_endpoint_alloc_parse' > ia64-linux-ld: ov772x.c:(.text+0x1f12): undefined reference to `v4l2_fwnode_endpoint_free' > ia64-linux-ld: ov772x.c:(.text+0x2212): undefined reference to `v4l2_fwnode_endpoint_alloc_parse' > > Fixes: 8a10b4e3601e ("media: i2c: ov772x: Parse endpoint properties") > Reported-by: kernel test robot <lkp@intel.com> > Signed-off-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com> You beat me to it, I was about to hit send for the same patch :) Thanks! Acked-by: Jacopo Mondi <jacopo+renesas@jmondi.org> > --- > drivers/media/i2c/Kconfig | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/media/i2c/Kconfig b/drivers/media/i2c/Kconfig > index eddb10220953..bb1b5a340431 100644 > --- a/drivers/media/i2c/Kconfig > +++ b/drivers/media/i2c/Kconfig > @@ -1013,6 +1013,7 @@ config VIDEO_OV772X > tristate "OmniVision OV772x sensor support" > depends on I2C && VIDEO_V4L2 > select REGMAP_SCCB > + select V4L2_FWNODE > help > This is a Video4Linux2 sensor driver for the OmniVision > OV772x camera. > -- > 2.17.1 >
Hi Lad, On 20/01/2021 09:01, Lad Prabhakar wrote: > Fix OV772x build breakage by selecting V4L2_FWNODE config: > > ia64-linux-ld: drivers/media/i2c/ov772x.o: in function `ov772x_probe': > ov772x.c:(.text+0x1ee2): undefined reference to `v4l2_fwnode_endpoint_alloc_parse' > ia64-linux-ld: ov772x.c:(.text+0x1f12): undefined reference to `v4l2_fwnode_endpoint_free' > ia64-linux-ld: ov772x.c:(.text+0x2212): undefined reference to `v4l2_fwnode_endpoint_alloc_parse' > > Fixes: 8a10b4e3601e ("media: i2c: ov772x: Parse endpoint properties") > Reported-by: kernel test robot <lkp@intel.com> > Signed-off-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com> I see this driver uses subdev API too. Should the driver also select VIDEO_V4L2_SUBDEV_API? Or is that covered sufficiently already on any platforms that would use the driver? Reviewed-by: Kieran Bingham <kieran.bingham+renesas@ideasonboard.com> > --- > drivers/media/i2c/Kconfig | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/media/i2c/Kconfig b/drivers/media/i2c/Kconfig > index eddb10220953..bb1b5a340431 100644 > --- a/drivers/media/i2c/Kconfig > +++ b/drivers/media/i2c/Kconfig > @@ -1013,6 +1013,7 @@ config VIDEO_OV772X > tristate "OmniVision OV772x sensor support" > depends on I2C && VIDEO_V4L2 > select REGMAP_SCCB > + select V4L2_FWNODE > help > This is a Video4Linux2 sensor driver for the OmniVision > OV772x camera. >
On Wed, Jan 20, 2021 at 10:17:14AM +0000, Kieran Bingham wrote: > Hi Lad, > > On 20/01/2021 09:01, Lad Prabhakar wrote: > > Fix OV772x build breakage by selecting V4L2_FWNODE config: > > > > ia64-linux-ld: drivers/media/i2c/ov772x.o: in function `ov772x_probe': > > ov772x.c:(.text+0x1ee2): undefined reference to `v4l2_fwnode_endpoint_alloc_parse' > > ia64-linux-ld: ov772x.c:(.text+0x1f12): undefined reference to `v4l2_fwnode_endpoint_free' > > ia64-linux-ld: ov772x.c:(.text+0x2212): undefined reference to `v4l2_fwnode_endpoint_alloc_parse' > > > > Fixes: 8a10b4e3601e ("media: i2c: ov772x: Parse endpoint properties") > > Reported-by: kernel test robot <lkp@intel.com> > > Signed-off-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com> > > I see this driver uses subdev API too. > > Should the driver also select VIDEO_V4L2_SUBDEV_API? Yes, it should. Another patch? This one fixes a compilation problem. > > Or is that covered sufficiently already on any platforms that would use > the driver? > > Reviewed-by: Kieran Bingham <kieran.bingham+renesas@ideasonboard.com> > > > --- > > drivers/media/i2c/Kconfig | 1 + > > 1 file changed, 1 insertion(+) > > > > diff --git a/drivers/media/i2c/Kconfig b/drivers/media/i2c/Kconfig > > index eddb10220953..bb1b5a340431 100644 > > --- a/drivers/media/i2c/Kconfig > > +++ b/drivers/media/i2c/Kconfig > > @@ -1013,6 +1013,7 @@ config VIDEO_OV772X > > tristate "OmniVision OV772x sensor support" > > depends on I2C && VIDEO_V4L2 > > select REGMAP_SCCB > > + select V4L2_FWNODE > > help > > This is a Video4Linux2 sensor driver for the OmniVision > > OV772x camera. > > >
Hi Sakari and Kieran, On Wed, Jan 20, 2021 at 10:36 AM Sakari Ailus <sakari.ailus@linux.intel.com> wrote: > > On Wed, Jan 20, 2021 at 10:17:14AM +0000, Kieran Bingham wrote: > > Hi Lad, > > > > On 20/01/2021 09:01, Lad Prabhakar wrote: > > > Fix OV772x build breakage by selecting V4L2_FWNODE config: > > > > > > ia64-linux-ld: drivers/media/i2c/ov772x.o: in function `ov772x_probe': > > > ov772x.c:(.text+0x1ee2): undefined reference to `v4l2_fwnode_endpoint_alloc_parse' > > > ia64-linux-ld: ov772x.c:(.text+0x1f12): undefined reference to `v4l2_fwnode_endpoint_free' > > > ia64-linux-ld: ov772x.c:(.text+0x2212): undefined reference to `v4l2_fwnode_endpoint_alloc_parse' > > > > > > Fixes: 8a10b4e3601e ("media: i2c: ov772x: Parse endpoint properties") > > > Reported-by: kernel test robot <lkp@intel.com> > > > Signed-off-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com> > > > > I see this driver uses subdev API too. > > > > Should the driver also select VIDEO_V4L2_SUBDEV_API? > > Yes, it should. Another patch? This one fixes a compilation problem. > Do agree, I will post an incremental patch on top of this. Cheers, Prabhakar > > > > Or is that covered sufficiently already on any platforms that would use > > the driver? > > > > Reviewed-by: Kieran Bingham <kieran.bingham+renesas@ideasonboard.com> > > > > > --- > > > drivers/media/i2c/Kconfig | 1 + > > > 1 file changed, 1 insertion(+) > > > > > > diff --git a/drivers/media/i2c/Kconfig b/drivers/media/i2c/Kconfig > > > index eddb10220953..bb1b5a340431 100644 > > > --- a/drivers/media/i2c/Kconfig > > > +++ b/drivers/media/i2c/Kconfig > > > @@ -1013,6 +1013,7 @@ config VIDEO_OV772X > > > tristate "OmniVision OV772x sensor support" > > > depends on I2C && VIDEO_V4L2 > > > select REGMAP_SCCB > > > + select V4L2_FWNODE > > > help > > > This is a Video4Linux2 sensor driver for the OmniVision > > > OV772x camera. > > > > > > > -- > Sakari Ailus
On 20/01/2021 10:36, Sakari Ailus wrote: > On Wed, Jan 20, 2021 at 10:17:14AM +0000, Kieran Bingham wrote: >> Hi Lad, >> >> On 20/01/2021 09:01, Lad Prabhakar wrote: >>> Fix OV772x build breakage by selecting V4L2_FWNODE config: >>> >>> ia64-linux-ld: drivers/media/i2c/ov772x.o: in function `ov772x_probe': >>> ov772x.c:(.text+0x1ee2): undefined reference to `v4l2_fwnode_endpoint_alloc_parse' >>> ia64-linux-ld: ov772x.c:(.text+0x1f12): undefined reference to `v4l2_fwnode_endpoint_free' >>> ia64-linux-ld: ov772x.c:(.text+0x2212): undefined reference to `v4l2_fwnode_endpoint_alloc_parse' >>> >>> Fixes: 8a10b4e3601e ("media: i2c: ov772x: Parse endpoint properties") >>> Reported-by: kernel test robot <lkp@intel.com> >>> Signed-off-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com> >> >> I see this driver uses subdev API too. >> >> Should the driver also select VIDEO_V4L2_SUBDEV_API? > > Yes, it should. Another patch? This one fixes a compilation problem. Yes, it's probably another patch, because indeed this is a specific fix. I wonder if that means the builders haven't been able to construct a combination without VIDEO_V4L2_SUBDEV_API... -- Kieran >> Or is that covered sufficiently already on any platforms that would use >> the driver? >> >> Reviewed-by: Kieran Bingham <kieran.bingham+renesas@ideasonboard.com> >> >>> --- >>> drivers/media/i2c/Kconfig | 1 + >>> 1 file changed, 1 insertion(+) >>> >>> diff --git a/drivers/media/i2c/Kconfig b/drivers/media/i2c/Kconfig >>> index eddb10220953..bb1b5a340431 100644 >>> --- a/drivers/media/i2c/Kconfig >>> +++ b/drivers/media/i2c/Kconfig >>> @@ -1013,6 +1013,7 @@ config VIDEO_OV772X >>> tristate "OmniVision OV772x sensor support" >>> depends on I2C && VIDEO_V4L2 >>> select REGMAP_SCCB >>> + select V4L2_FWNODE >>> help >>> This is a Video4Linux2 sensor driver for the OmniVision >>> OV772x camera. >>> >> >
On Wed, Jan 20, 2021 at 11:09 AM Kieran Bingham <kieran.bingham+renesas@ideasonboard.com> wrote: > > On 20/01/2021 10:36, Sakari Ailus wrote: > > On Wed, Jan 20, 2021 at 10:17:14AM +0000, Kieran Bingham wrote: > >> Hi Lad, > >> > >> On 20/01/2021 09:01, Lad Prabhakar wrote: > >>> Fix OV772x build breakage by selecting V4L2_FWNODE config: > >>> > >>> ia64-linux-ld: drivers/media/i2c/ov772x.o: in function `ov772x_probe': > >>> ov772x.c:(.text+0x1ee2): undefined reference to `v4l2_fwnode_endpoint_alloc_parse' > >>> ia64-linux-ld: ov772x.c:(.text+0x1f12): undefined reference to `v4l2_fwnode_endpoint_free' > >>> ia64-linux-ld: ov772x.c:(.text+0x2212): undefined reference to `v4l2_fwnode_endpoint_alloc_parse' > >>> > >>> Fixes: 8a10b4e3601e ("media: i2c: ov772x: Parse endpoint properties") > >>> Reported-by: kernel test robot <lkp@intel.com> > >>> Signed-off-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com> > >> > >> I see this driver uses subdev API too. > >> > >> Should the driver also select VIDEO_V4L2_SUBDEV_API? > > > > Yes, it should. Another patch? This one fixes a compilation problem. > > Yes, it's probably another patch, because indeed this is a specific fix. > > I wonder if that means the builders haven't been able to construct a > combination without VIDEO_V4L2_SUBDEV_API... > Thats because v4l2-subdev.o is built irrespective of VIDEO_V4L2_SUBDEV_API enabled/disabled and there are empty fillers in v4l2-subdev.c when VIDEO_V4L2_SUBDEV_API is disabled. Cheers, Prabhakar > -- > Kieran > > > >> Or is that covered sufficiently already on any platforms that would use > >> the driver? > >> > >> Reviewed-by: Kieran Bingham <kieran.bingham+renesas@ideasonboard.com> > >> > >>> --- > >>> drivers/media/i2c/Kconfig | 1 + > >>> 1 file changed, 1 insertion(+) > >>> > >>> diff --git a/drivers/media/i2c/Kconfig b/drivers/media/i2c/Kconfig > >>> index eddb10220953..bb1b5a340431 100644 > >>> --- a/drivers/media/i2c/Kconfig > >>> +++ b/drivers/media/i2c/Kconfig > >>> @@ -1013,6 +1013,7 @@ config VIDEO_OV772X > >>> tristate "OmniVision OV772x sensor support" > >>> depends on I2C && VIDEO_V4L2 > >>> select REGMAP_SCCB > >>> + select V4L2_FWNODE > >>> help > >>> This is a Video4Linux2 sensor driver for the OmniVision > >>> OV772x camera. > >>> > >> > > >
diff --git a/drivers/media/i2c/Kconfig b/drivers/media/i2c/Kconfig index eddb10220953..bb1b5a340431 100644 --- a/drivers/media/i2c/Kconfig +++ b/drivers/media/i2c/Kconfig @@ -1013,6 +1013,7 @@ config VIDEO_OV772X tristate "OmniVision OV772x sensor support" depends on I2C && VIDEO_V4L2 select REGMAP_SCCB + select V4L2_FWNODE help This is a Video4Linux2 sensor driver for the OmniVision OV772x camera.