[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