Re: [PATCH] dt-bindings: media: i2c: imx219: Remove redundant description of data-lanes

From: Krzysztof Kozlowski
Date: Wed Apr 02 2025 - 06:29:35 EST


On 02/04/2025 11:57, Niklas Söderlund wrote:
>>
>>> Support four-lane operation") the driver errored out if not 2 lanes
>>> where used, and after it if not 2 or 4 lanes where used.
>>
>> Then... fix the driver?
>>
>> This property describes hardware, not driver. Why current driver
>> implementation, e.g. 1 year ago or now, would change the hardware (so
>> the bindings)?
>
> I agree, I thought that here we have a case where the bindings predate
> the standardisation. The driver do not match the bindings, in fact it
> breaks if the imx219 specific instructions are followed. So the risk of
> breaking stuff is likely low. And this was an opportunity to align the
> imx219 with video-interfaces.yaml.

I am sorry, but what breaks exactly?

Is the device supporting two and four lanes setups? If yes, then the
binding is correct.


Best regards,
Krzysztof