Re: [PATCH v1] drm/msm/dp: Disable stream clock after video-ready timeout
최유호 <[email protected]> Mon, 3 Aug 2026 14:21:58 -0400
| Newsgroups | gmane.comp.freedesktop.xorg.drivers.freedreno,gmane.comp.video.dri.devel,gmane.linux.ports.arm.msm,gmane.linux.kernel |
|---|---|
| Message-ID | <CACrCO_UUX_A+Z7uCsT9=JkXqqkG0Tp=n0rXO3j+KAzNxGRuVQQ@mail.gmail.com> |
On Fri, 31 Jul 2026 at 10:05, Dmitry Baryshkov <[email protected]> wrote: > > On Tue, Jul 07, 2026 at 06:05:53PM +0000, [email protected] wrote: > > Thank you for your contribution! Sashiko AI review found 1 potential issue(s) to consider: > > - [High] Disabling the pixel clock while the DP controller is actively in the SEND_VIDEO state violates the teardown sequence and can lock up the hardware. > > -- > > Could you please comment on this Sashiko comment? Thanks, agreed. The posted patch disables the pixel clock before reverting the controller from SEND_VIDEO, and the bridge disable path may later issue PUSH_IDLE with the clock disabled. I will send the revised patch addressing the comment. I appreciate your time for the review. Thanks, Yuho