[PATCH 20/21] drm/dp: Fix a couple of whitespace issues

Thierry Reding thierry.reding at gmail.com
Mon Sep 2 11:10:39 UTC 2019


On Thu, Aug 08, 2019 at 03:18:47PM +0200, Philipp Zabel wrote:
> On Mon, 2019-08-05 at 14:23 +0200, Thierry Reding wrote:
> > From: Thierry Reding <treding at nvidia.com>
> > 
> > Remove a gratuituous blank line in one place and add a blank line in
> > another to improve readability.
> 
> Seems like the comment description is outdated here as well.

Indeed. I've had these patches in a development branch for a very long
time because the hardware that I used to test DP support on stopped
working and I only recently revived this on a different platform that
also has a DP output. Looks like that other blank line had been added by
some other commit in the meantime.

Fixed in v2.

Thanks,
Thierry

> 
> > Signed-off-by: Thierry Reding <treding at nvidia.com>
> > ---
> >  drivers/gpu/drm/drm_dp_helper.c | 1 -
> >  1 file changed, 1 deletion(-)
> > 
> > diff --git a/drivers/gpu/drm/drm_dp_helper.c b/drivers/gpu/drm/drm_dp_helper.c
> > index 136ee609f2ee..6b4431bade3e 100644
> > --- a/drivers/gpu/drm/drm_dp_helper.c
> > +++ b/drivers/gpu/drm/drm_dp_helper.c
> > @@ -216,7 +216,6 @@ static int drm_dp_dpcd_access(struct drm_dp_aux *aux, u8 request,
> >  		}
> >  
> >  		ret = aux->transfer(aux, &msg);
> > -
> >  		if (ret >= 0) {
> >  			native_reply = msg.reply & DP_AUX_NATIVE_REPLY_MASK;
> >  			if (native_reply == DP_AUX_NATIVE_REPLY_ACK) {
> 
> regards
> Philipp
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20190902/e5f9b426/attachment.sig>


More information about the dri-devel mailing list