[bug report] drm/msm/dsi: Move PHY operations out of host
Dan Carpenter
dan.carpenter at oracle.com
Sat Feb 11 05:17:35 UTC 2017
Hello Hai Li,
This is a semi-automatic email about new static checker warnings.
The patch b62aa70a98c5: "drm/msm/dsi: Move PHY operations out of
host" from Jan 7, 2017, leads to the following Smatch complaint:
./drivers/gpu/drm/msm/dsi/dsi_manager.c:174 dsi_mgr_phy_enable()
error: we previously assumed 'mdsi' could be null (see line 157)
./drivers/gpu/drm/msm/dsi/dsi_manager.c
156 */
157 if (IS_DUAL_DSI() && mdsi && sdsi) {
^^^^
Check for NULL.
158 if (!mdsi->phy_enabled && !sdsi->phy_enabled) {
159 msm_dsi_host_reset_phy(mdsi->host);
160 msm_dsi_host_reset_phy(sdsi->host);
161
162 ret = enable_phy(mdsi, src_pll_id,
163 &shared_timings[DSI_CLOCK_MASTER]);
164 if (ret)
165 return ret;
166 ret = enable_phy(sdsi, src_pll_id,
167 &shared_timings[DSI_CLOCK_SLAVE]);
168 if (ret) {
169 msm_dsi_phy_disable(mdsi->phy);
170 return ret;
171 }
172 }
173 } else {
174 msm_dsi_host_reset_phy(mdsi->host);
^^^^^^^^^^
Unchecked dereference.
175 ret = enable_phy(msm_dsi, src_pll_id, &shared_timings[id]);
176 if (ret)
regards,
dan carpenter
More information about the dri-devel
mailing list