From: Arnd Bergmann <arnd@arndb.de>

The new driver fails to build if I2C is disabled:

drivers/media/i2c/lt6911uxe.c:703:1: error: data definition has no type or storage class [-Werror]
  703 | module_i2c_driver(lt6911uxe_i2c_driver);

or if I2C is on but V4L2_CCI_I2C is not:

ERROR: modpost: "cci_write" [drivers/media/i2c/lt6911uxe.ko] undefined!
ERROR: modpost: "cci_read" [drivers/media/i2c/lt6911uxe.ko] undefined!

For both by adding a dependency on I2C and selecting V4L2_CCI_I2C, which
follows the common practice for these.

Fixes: e49563c3be09 ("media: i2c: add lt6911uxe hdmi bridge driver")
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
---
 drivers/media/i2c/Kconfig | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/drivers/media/i2c/Kconfig b/drivers/media/i2c/Kconfig
index e576b213084d..b06365d02ef1 100644
--- a/drivers/media/i2c/Kconfig
+++ b/drivers/media/i2c/Kconfig
@@ -1149,8 +1149,9 @@ config VIDEO_ISL7998X
 
 config VIDEO_LT6911UXE
 	tristate "Lontium LT6911UXE decoder"
-	depends on ACPI && VIDEO_DEV
+	depends on ACPI && VIDEO_DEV && I2C
 	select V4L2_FWNODE
+	select V4L2_CCI_I2C
 	help
 	  This is a Video4Linux2 sensor-level driver for the Lontium
 	  LT6911UXE HDMI to MIPI CSI-2 bridge.
-- 
2.39.5

Hi Arnd,

On 3/14/2025 11:46 PM, Arnd Bergmann wrote:
> From: Arnd Bergmann <arnd@arndb.de>
> 
> The new driver fails to build if I2C is disabled:
> 
> drivers/media/i2c/lt6911uxe.c:703:1: error: data definition has no type or storage class [-Werror]
>   703 | module_i2c_driver(lt6911uxe_i2c_driver);
> 
> or if I2C is on but V4L2_CCI_I2C is not:
> 
> ERROR: modpost: "cci_write" [drivers/media/i2c/lt6911uxe.ko] undefined!
> ERROR: modpost: "cci_read" [drivers/media/i2c/lt6911uxe.ko] undefined!
> 
> For both by adding a dependency on I2C and selecting V4L2_CCI_I2C, which
> follows the common practice for these.
> 
> Fixes: e49563c3be09 ("media: i2c: add lt6911uxe hdmi bridge driver")
> Signed-off-by: Arnd Bergmann <arnd@arndb.de>
> ---
>  drivers/media/i2c/Kconfig | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/drivers/media/i2c/Kconfig b/drivers/media/i2c/Kconfig
> index e576b213084d..b06365d02ef1 100644
> --- a/drivers/media/i2c/Kconfig
> +++ b/drivers/media/i2c/Kconfig
> @@ -1149,8 +1149,9 @@ config VIDEO_ISL7998X
>  
>  config VIDEO_LT6911UXE
>  	tristate "Lontium LT6911UXE decoder"
> -	depends on ACPI && VIDEO_DEV
> +	depends on ACPI && VIDEO_DEV && I2C
>  	select V4L2_FWNODE
> +	select V4L2_CCI_I2C
>  	help
>  	  This is a Video4Linux2 sensor-level driver for the Lontium
>  	  LT6911UXE HDMI to MIPI CSI-2 bridge.

Thanks for your fix.

Lkp is a bit weird to me, because it tested a warning in patch v6 likes
below:

    kismet: WARNING: unmet direct dependencies detected for V4L2_CCI_I2C
when selected by VIDEO_LT6911UXE

So I remove this select flag and passed lkp build test in patch v7.
But now it encounters build error again, I'm curious why...

Thanks,
Dongcheng

On Mon, Mar 17, 2025, at 11:17, Yan, Dongcheng wrote:
> On 3/14/2025 11:46 PM, Arnd Bergmann wrote:
>> From: Arnd Bergmann <arnd@arndb.de>

>> @@ -1149,8 +1149,9 @@ config VIDEO_ISL7998X
>>  
>>  config VIDEO_LT6911UXE
>>  	tristate "Lontium LT6911UXE decoder"
>> -	depends on ACPI && VIDEO_DEV
>> +	depends on ACPI && VIDEO_DEV && I2C
>>  	select V4L2_FWNODE
>> +	select V4L2_CCI_I2C
>>  	help
>>  	  This is a Video4Linux2 sensor-level driver for the Lontium
>>  	  LT6911UXE HDMI to MIPI CSI-2 bridge.
>
> Thanks for your fix.
>
> Lkp is a bit weird to me, because it tested a warning in patch v6 likes
> below:
>
>     kismet: WARNING: unmet direct dependencies detected for V4L2_CCI_I2C
> when selected by VIDEO_LT6911UXE

What was the full warning? The only dependency I see in V4L2_CCI_I2C
is CONFIG_I2C itself, and that is what I add above.

> So I remove this select flag and passed lkp build test in patch v7.
> But now it encounters build error again, I'm curious why...

I don't currently get any more build errors with my patch
added in, and I think this should be sufficient. Can you forward
me the errors and config you get with my patch?

     Arnd

Hi all,

On 3/17/25 11:32, Arnd Bergmann wrote:
> On Mon, Mar 17, 2025, at 11:17, Yan, Dongcheng wrote:
>> On 3/14/2025 11:46 PM, Arnd Bergmann wrote:
>>> From: Arnd Bergmann <arnd@arndb.de>
> 
>>> @@ -1149,8 +1149,9 @@ config VIDEO_ISL7998X
>>>  
>>>  config VIDEO_LT6911UXE
>>>  	tristate "Lontium LT6911UXE decoder"
>>> -	depends on ACPI && VIDEO_DEV
>>> +	depends on ACPI && VIDEO_DEV && I2C
>>>  	select V4L2_FWNODE
>>> +	select V4L2_CCI_I2C
>>>  	help
>>>  	  This is a Video4Linux2 sensor-level driver for the Lontium
>>>  	  LT6911UXE HDMI to MIPI CSI-2 bridge.
>>
>> Thanks for your fix.
>>
>> Lkp is a bit weird to me, because it tested a warning in patch v6 likes
>> below:
>>
>>     kismet: WARNING: unmet direct dependencies detected for V4L2_CCI_I2C
>> when selected by VIDEO_LT6911UXE
> 
> What was the full warning? The only dependency I see in V4L2_CCI_I2C
> is CONFIG_I2C itself, and that is what I add above.
> 
>> So I remove this select flag and passed lkp build test in patch v7.
>> But now it encounters build error again, I'm curious why...
> 
> I don't currently get any more build errors with my patch
> added in, and I think this should be sufficient. Can you forward
> me the errors and config you get with my patch?

FYI: I plan to take Arnd's patch tomorrow to be in time for the merge window.

If more issues are found, then they can be on top later.

Regards,

	Hans

> 
>      Arnd
>

On Tue, Mar 18, 2025, at 14:51, Hans Verkuil wrote:
> On 3/17/25 11:32, Arnd Bergmann wrote:
>> On Mon, Mar 17, 2025, at 11:17, Yan, Dongcheng wrote:
>> 
>>> So I remove this select flag and passed lkp build test in patch v7.
>>> But now it encounters build error again, I'm curious why...
>> 
>> I don't currently get any more build errors with my patch
>> added in, and I think this should be sufficient. Can you forward
>> me the errors and config you get with my patch?
>
> FYI: I plan to take Arnd's patch tomorrow to be in time for the merge window.
>
> If more issues are found, then they can be on top later.

Not sure what happened: I see that my patch was in next-20250424
and earlier but is now missing again, so the link failure returned.

It was in git://linuxtv.org/media-ci/media-pending.git at the
time along with two more patches that are now also missing:

d51adf038ebe media: cec: tda9950: add back i2c dependency
118b34092e37 media: i2c: lt6911uxe: add two selects to Kconfig
0dce5b44bd38 media: platform: synopsys: VIDEO_SYNOPSYS_HDMIRX should depend on ARCH_ROCKCHIP

      Arnd

On 30/04/2025 13:21, Arnd Bergmann wrote:
> On Tue, Mar 18, 2025, at 14:51, Hans Verkuil wrote:
>> On 3/17/25 11:32, Arnd Bergmann wrote:
>>> On Mon, Mar 17, 2025, at 11:17, Yan, Dongcheng wrote:
>>>
>>>> So I remove this select flag and passed lkp build test in patch v7.
>>>> But now it encounters build error again, I'm curious why...
>>>
>>> I don't currently get any more build errors with my patch
>>> added in, and I think this should be sufficient. Can you forward
>>> me the errors and config you get with my patch?
>>
>> FYI: I plan to take Arnd's patch tomorrow to be in time for the merge window.
>>
>> If more issues are found, then they can be on top later.
> 
> Not sure what happened: I see that my patch was in next-20250424
> and earlier but is now missing again, so the link failure returned.
> 
> It was in git://linuxtv.org/media-ci/media-pending.git at the
> time along with two more patches that are now also missing:
> 
> d51adf038ebe media: cec: tda9950: add back i2c dependency
> 118b34092e37 media: i2c: lt6911uxe: add two selects to Kconfig
> 0dce5b44bd38 media: platform: synopsys: VIDEO_SYNOPSYS_HDMIRX should depend on ARCH_ROCKCHIP
> 
>       Arnd

I believe Mauro is working on preparing a PR for 6.15 with these patches.
They are here:

https://git.linuxtv.org/media-ci/media-pending.git/log/?h=fixes

Regards,

	Hans