Reviewed-by: Lyude Paul <[email protected]> On Wed, 2019-09-25 at 17:14 +0300, Ville Syrjala wrote: > From: Ville Syrjälä <[email protected]> > > Replace the nested ifs with a single if and a logical AND. > > Cc: Lyude Paul <[email protected]> > Signed-off-by: Ville Syrjälä <[email protected]> > --- > drivers/gpu/drm/drm_dp_mst_topology.c | 10 +++++----- > 1 file changed, 5 insertions(+), 5 deletions(-) > > diff --git a/drivers/gpu/drm/drm_dp_mst_topology.c > b/drivers/gpu/drm/drm_dp_mst_topology.c > index 97216099a718..e25597eb3ca1 100644 > --- a/drivers/gpu/drm/drm_dp_mst_topology.c > +++ b/drivers/gpu/drm/drm_dp_mst_topology.c > @@ -1123,11 +1123,11 @@ static void drm_dp_mst_put_payload_id(struct > drm_dp_mst_topology_mgr *mgr, > clear_bit(vcpi - 1, &mgr->vcpi_mask); > > for (i = 0; i < mgr->max_payloads; i++) { > - if (mgr->proposed_vcpis[i]) > - if (mgr->proposed_vcpis[i]->vcpi == vcpi) { > - mgr->proposed_vcpis[i] = NULL; > - clear_bit(i + 1, &mgr->payload_mask); > - } > + if (mgr->proposed_vcpis[i] && > + mgr->proposed_vcpis[i]->vcpi == vcpi) { > + mgr->proposed_vcpis[i] = NULL; > + clear_bit(i + 1, &mgr->payload_mask); > + } > } > mutex_unlock(&mgr->payload_lock); > } -- Cheers, Lyude Paul
_______________________________________________ dri-devel mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/dri-devel
