drm/msm/dp: Fix incorrect NULL check kbot warnings in DP driver
[ Upstream commit 7d649cfe0314aad2ba18042885ab9de2f13ad809 ] Fix an incorrect NULL check reported by kbot in the MSM DP driver smatch warnings: drivers/gpu/drm/msm/dp/dp_hpd.c:37 dp_hpd_connect() error: we previously assumed 'hpd_priv->dp_cb' could be null (see line 37) Reported-by: kernel test robot <lkp@intel.com> Reported-by: Dan Carpenter <dan.carpenter@oracle.com> Signed-off-by: Abhinav Kumar <abhinavk@codeaurora.org> Reviewed-by: Stephen Boyd <swboyd@chromium.org> Link: https://lore.kernel.org/r/1614971839-2686-2-git-send-email-abhinavk@codeaurora.org Signed-off-by: Rob Clark <robdclark@chromium.org> Signed-off-by: Sasha Levin <sashal@kernel.org>
This commit is contained in:

committed by
Greg Kroah-Hartman

parent
4066456bb2
commit
573bbb6f80
@@ -34,8 +34,8 @@ int dp_hpd_connect(struct dp_usbpd *dp_usbpd, bool hpd)
|
|||||||
|
|
||||||
dp_usbpd->hpd_high = hpd;
|
dp_usbpd->hpd_high = hpd;
|
||||||
|
|
||||||
if (!hpd_priv->dp_cb && !hpd_priv->dp_cb->configure
|
if (!hpd_priv->dp_cb || !hpd_priv->dp_cb->configure
|
||||||
&& !hpd_priv->dp_cb->disconnect) {
|
|| !hpd_priv->dp_cb->disconnect) {
|
||||||
pr_err("hpd dp_cb not initialized\n");
|
pr_err("hpd dp_cb not initialized\n");
|
||||||
return -EINVAL;
|
return -EINVAL;
|
||||||
}
|
}
|
||||||
|
Reference in New Issue
Block a user