On Fri, 10 Dec 2021, Kees Cook <keescook@xxxxxxxxxxxx> 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. -- Jani Nikula, Intel Open Source Graphics Center
