Re: [PATCH] media: i2c: lt6911uxe: Fix Kconfig dependencies:

[Hans Verkuil]

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@xxxxxxxx>
> 
>>> @@ -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
>