On Fri, 10 Dec 2021, Kees Cook keescook@chromium.org wrote:
On Fri, Dec 10, 2021 at 12:06:20PM +0200, Jani Nikula wrote:
Post Cursor2 was completely optional for the transmitter even before it was deprecated.
And now we'd be adding 5 bytes extra to all link status reads. To fix the only user of drm_dp_get_adjust_request_post_cursor() that apparently has never worked as intended. I'm just not convinced.
I was trying to look through the implications of DP_LINK_STATUS_SIZE increase, and at least drm_dp_dpcd_read_phy_link_status() comes across as something probably needing attention.
Okay, it sounds like you'd prefer the "make it tegra-specific" patch I proposed. I will work that up as a proper patch and send it.
Yes. Of course, I'd like to hear Thierry's view as well.
BR, Jani.