diff options
| author |   Imre Deak <imre.deak@intel.com> | 2016-01-29 14:44:28 +0200 |
|---|---|---|
| committer |   Dave Airlie <airlied@redhat.com> | 2016-02-05 15:23:48 +1000 |
| commit | 441388a8a73f905359e60518cfc18eca6aed2b6a (patch) | |
| tree | a94bca61937a9d7a6e9ee479ccb9ba7cb1a3f21d | |
| parent | drm: fix missing reference counting decrease (diff) | |
| download | linux-dev-441388a8a73f905359e60518cfc18eca6aed2b6a.tar.xz linux-dev-441388a8a73f905359e60518cfc18eca6aed2b6a.zip | |
drm/mst: Don't ignore the MST PBN self-test result
Otherwise this call would have no effect.
Caught by Coverity.
Signed-off-by: Imre Deak <imre.deak@intel.com>
Reviewed-by: David Weinehall <david.weinehall@intel.com>
Signed-off-by: Dave Airlie <airlied@redhat.com>
| -rw-r--r-- | drivers/gpu/drm/drm_dp_mst_topology.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/drivers/gpu/drm/drm_dp_mst_topology.c b/drivers/gpu/drm/drm_dp_mst_topology.c index fe273b681fe5..2b7c2f068834 100644 --- a/drivers/gpu/drm/drm_dp_mst_topology.c +++ b/drivers/gpu/drm/drm_dp_mst_topology.c @@ -2855,7 +2855,9 @@ int drm_dp_mst_topology_mgr_init(struct drm_dp_mst_topology_mgr *mgr, if (!mgr->proposed_vcpis) return -ENOMEM; set_bit(0, &mgr->payload_mask); - test_calc_pbn_mode(); + if (test_calc_pbn_mode() < 0) + DRM_ERROR("MST PBN self-test failed\n"); + return 0; } EXPORT_SYMBOL(drm_dp_mst_topology_mgr_init); |