The comment is not necessary. The commit description gives a good rational.
>
> > + drm_dbg_kms(dev, "WAIT_VBLANK: timeout crtc=%d, last=%llu,
> curr=%llu\n",
> > + pipe, last, curr);
> > + } else {
> > + drm_dbg_kms(dev, "WAIT_VBLANK: completed crtc=%d,
> last=%llu, curr=%llu\n",
> > + pipe, last, curr);
> > + }
>
> Rather drop the else branch.
>
> Best regards
> Thomas
>
> >
> > drm_vblank_put(dev, pipe);
> > }
>
> Hi Thomas,
>
> Thanks a lot for your review and suggestions. I am just starting out here.
> I’ve sent a v2 addressing your comments.
>
> Best regards,
> Chintan
>
>
>
>
