Messages in this thread | | | From | Dave Airlie <> | Date | Wed, 4 Sep 2019 07:42:16 +1000 | Subject | Re: [PATCH v2 15/27] drm/dp_mst: Cleanup drm_dp_send_link_address() a bit |
| |
On Wed, 4 Sep 2019 at 06:48, Lyude Paul <lyude@redhat.com> wrote: > > Declare local pointer to the drm_dp_link_address_ack_reply struct > instead of constantly dereferencing it through the union in > txmsg->reply. Then, invert the order of conditionals so we don't have to > do the bulk of the work inside them, and can wrap lines even less. Then > finally, rearrange variable declarations a bit. > > Cc: Juston Li <juston.li@intel.com> > Cc: Imre Deak <imre.deak@intel.com> > Cc: Ville Syrjälä <ville.syrjala@linux.intel.com> > Cc: Harry Wentland <hwentlan@amd.com> > Cc: Daniel Vetter <daniel.vetter@ffwll.ch> > Signed-off-by: Lyude Paul <lyude@redhat.com>
Reviewed-by: Dave Airlie <airlied@redhat.com>
| |