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

Jani Nikula jani.nikula at intel.com
Fri Mar 1 09:28:18 UTC 2024


On Fri, 01 Mar 2024, Mitul Golani <mitulkumar.ajitkumar.golani at intel.com> 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


More information about the dri-devel mailing list