Re: [PATCH v15 0/9] Enable Adaptive Sync SDP Support for DP

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, 01 Mar 2024, Mitul Golani <mitulkumar.ajitkumar.golani@xxxxxxxxx> wrote:
> An Adaptive-Sync-capable DP protocol converter indicates its
> support by setting the related bit in the DPCD register.

Please stop sending new series all the time!

Let the review come to a natural stop, with all the review comments
gathered, and issues fixed, before sending a new version. Don't send
another version until *all* patches in the series have gotten either R-b
or actionable review comments.

Sending new versions in short succession actually harms your series
getting merged instead of helps, because it scatters the review comments
and discussion to multiple threads that are difficult to follow.

You also haven't added the Reviewed-by's for the patches that actually
have been reviewed. Whoever applies this would have to go through the
old threads to find out which patches got reviewed, and if they match
the latest thread. And any potential reviewer might not know some of the
patches have already been reviewed, and might do duplicate work.

For a series of this size, please send at most 1-2 new revisions per
*week*, unless you're sure all the pathes have gotten review and the
series should be ready to merge. Make each new revision count.


BR,
Jani.


-- 
Jani Nikula, Intel



[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux