On 25/08/2026 16:09, Luca Ceresoli wrote:
> 
>> +static struct i2c_device_id lt9611c_id[] = {
>> +    { "lontium,lt9611c",   (kernel_ulong_t)&lt9611c_chip_data[CHIP_LT9611C] 
>>   },
>> +    { "lontium,lt9611ex",  
>> (kernel_ulong_t)&lt9611c_chip_data[CHIP_LT9611EX]  },
>> +    { "lontium,lt9611uxd", 
>> (kernel_ulong_t)&lt9611c_chip_data[CHIP_LT9611UXD] },
> 
> AFAIK there should be no vendor prefix for the i2c_device_id. Have a look
> at the docs and other drivers for the best practice.

This is because of bugs in the driver, which I reported at v6. Code had
NULL ptr and to solve it, instead of fixing it, Lontium added such prefixes.

Obviously this is a no-go and they MUST fix the null pointer in the
first place.

Best regards,
Krzysztof

Reply via email to