mirror of
https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git
synced 2024-09-05 02:19:51 +00:00
drm/vc4: simplify exit path of a failed allocation of dsi_connector
If dsi_connector fails to allocate, the exit path via label 'fail' checks if connector is null, which it always is, so the cleanup that destroys connector is never going to be called. Hence the failure path can be more optimally performed by removing this and just returning ERR_PTR(-ENOMEM). This also removes the need to initialize connector to NULL, and we can also remove ret too. Detected by CoverityScan, CID#1399504 ("Logicall Dead Code") Signed-off-by: Colin Ian King <colin.king@canonical.com> Reviewed-by: Eric Anholt <eric@anholt.net> Reviewed-by: Gustavo Padovan <gustavo.padovan@collabora.com> Link: http://patchwork.freedesktop.org/patch/msgid/20170203195624.7189-1-colin.king@canonical.com
This commit is contained in:
parent
64b0e1d6c6
commit
fce6a7bc22
1 changed files with 4 additions and 12 deletions
|
@ -771,16 +771,14 @@ static const struct drm_connector_helper_funcs vc4_dsi_connector_helper_funcs =
|
|||
static struct drm_connector *vc4_dsi_connector_init(struct drm_device *dev,
|
||||
struct vc4_dsi *dsi)
|
||||
{
|
||||
struct drm_connector *connector = NULL;
|
||||
struct drm_connector *connector;
|
||||
struct vc4_dsi_connector *dsi_connector;
|
||||
int ret = 0;
|
||||
|
||||
dsi_connector = devm_kzalloc(dev->dev, sizeof(*dsi_connector),
|
||||
GFP_KERNEL);
|
||||
if (!dsi_connector) {
|
||||
ret = -ENOMEM;
|
||||
goto fail;
|
||||
}
|
||||
if (!dsi_connector)
|
||||
return ERR_PTR(-ENOMEM);
|
||||
|
||||
connector = &dsi_connector->base;
|
||||
|
||||
dsi_connector->dsi = dsi;
|
||||
|
@ -796,12 +794,6 @@ static struct drm_connector *vc4_dsi_connector_init(struct drm_device *dev,
|
|||
drm_mode_connector_attach_encoder(connector, dsi->encoder);
|
||||
|
||||
return connector;
|
||||
|
||||
fail:
|
||||
if (connector)
|
||||
vc4_dsi_connector_destroy(connector);
|
||||
|
||||
return ERR_PTR(ret);
|
||||
}
|
||||
|
||||
static void vc4_dsi_encoder_destroy(struct drm_encoder *encoder)
|
||||
|
|
Loading…
Reference in a new issue