On 08/05/2026 15:40, [email protected] wrote:
> +
> +static int lt9611c_probe(struct i2c_client *client)
> +{
> +     const struct i2c_device_id *id = i2c_client_get_device_id(client);
> +     struct lt9611c *lt9611c;
> +     struct device *dev = &client->dev;
> +     bool fw_updated = false;
> +     int ret;
> +
> +     crc8_populate_msb(lt9611c_crc8_table, LT9611C_CRC_POLYNOMIAL);
> +
> +     if (!i2c_check_functionality(client->adapter, I2C_FUNC_I2C))
> +             return dev_err_probe(dev, -ENODEV, "device doesn't support 
> I2C\n");
> +
> +     lt9611c = devm_drm_bridge_alloc(dev, struct lt9611c, bridge, 
> &lt9611c_bridge_funcs);
> +     if (IS_ERR(lt9611c))
> +             return dev_err_probe(dev, PTR_ERR(lt9611c), "drm bridge alloc 
> failed.\n");
> +
> +     lt9611c->dev = dev;
> +     lt9611c->client = client;
> +     lt9611c->chip_type = id->driver_data;

Also, NULL pointer exception here. With simple look I found at least two
easy to reproduce and trigger bugs which makes me questioning how did
you test it.


Best regards,
Krzysztof

Reply via email to