diff options
| author | Dan Carpenter <dan.carpenter@linaro.org> | 2024-11-12 11:23:03 +0100 |
|---|---|---|
| committer | Liu Ying <victor.liu@nxp.com> | 2025-01-15 04:27:08 +0100 |
| commit | ef84aee151bd6c2c9223c8b430cae21d57b5e1c1 (patch) | |
| tree | 59a4ab149112d9ca30fd9ff61a2cb8bddd316060 /drivers | |
| parent | drm/v3d: Ensure job pointer is set to NULL after job completion (diff) | |
| download | linux-ef84aee151bd6c2c9223c8b430cae21d57b5e1c1.tar.xz linux-ef84aee151bd6c2c9223c8b430cae21d57b5e1c1.zip | |
drm/bridge: ite-it6263: Prevent error pointer dereference in probe()
If devm_i2c_new_dummy_device() fails then we were supposed to return an
error code, but instead the function continues and will crash on the next
line. Add the missing return statement.
Fixes: 049723628716 ("drm/bridge: Add ITE IT6263 LVDS to HDMI converter")
Signed-off-by: Dan Carpenter <dan.carpenter@linaro.org>
Reviewed-by: Biju Das <biju.das.jz@bp.renesas.com>
Reviewed-by: Liu Ying <victor.liu@nxp.com>
Signed-off-by: Liu Ying <victor.liu@nxp.com>
Link: https://patchwork.freedesktop.org/patch/msgid/804a758b-f2e7-4116-b72d-29bc8905beed@stanley.mountain
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/gpu/drm/bridge/ite-it6263.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/gpu/drm/bridge/ite-it6263.c b/drivers/gpu/drm/bridge/ite-it6263.c index cbabd4e20d3e..5f138a5692c7 100644 --- a/drivers/gpu/drm/bridge/ite-it6263.c +++ b/drivers/gpu/drm/bridge/ite-it6263.c @@ -845,8 +845,8 @@ static int it6263_probe(struct i2c_client *client) it->lvds_i2c = devm_i2c_new_dummy_device(dev, client->adapter, LVDS_INPUT_CTRL_I2C_ADDR); if (IS_ERR(it->lvds_i2c)) - dev_err_probe(it->dev, PTR_ERR(it->lvds_i2c), - "failed to allocate I2C device for LVDS\n"); + return dev_err_probe(it->dev, PTR_ERR(it->lvds_i2c), + "failed to allocate I2C device for LVDS\n"); it->lvds_regmap = devm_regmap_init_i2c(it->lvds_i2c, &it6263_lvds_regmap_config); |