Re: [PATCH v8 2/2] drm/bridge: Add Lontium LT9611C(EX/UXD) MIPI DSI to HDMI driver
Dmitry Baryshkov <[email protected]> Wed, 29 Jul 2026 17:05:15 +0300
| Newsgroups | dev.linux.lists.sashiko-reviews,org.freedesktop.lists.dri-devel,org.kernel.vger.linux-devicetree |
|---|---|
| Message-ID | <5xpvwwovgwuxltjqjnlchamwcvppdc4376fcujeqovm7fr3uhw@6r55ztlq47ob> |
On Tue, Jul 28, 2026 at 11:26:14AM +0000, [email protected] wrote: > Thank you for your contribution! Sashiko AI review found 12 potential issue(s) to consider: > - [Critical] Recursive locking deadlock in `lt9611c_hdmi_audio_prepare`. > - [High] The IRQ is enabled before the hardware reset is completed during resume. > - [High] Premature manual teardown in remove() conflicts with devm_* APIs, leading to UAF and timeouts. > - [High] The IRQ handler incorrectly clears unhandled interrupts by writing an inverted bitmask. > - [High] Early registration of the DRM bridge exposes it before resource initialization is complete, risking use-after-free on probe failure. > - [Medium] The return value of devm_drm_bridge_add() is ignored. > - [Medium] Selecting REGMAP_I2C without a dependency on I2C causes a Kconfig warning and build failure. > - [Medium] request_firmware() is called while holding the ocm_lock mutex. > - [Medium] Missing `regmap_read()` error handling causes uninitialized memory usage and broken firmware upgrade logic. > - [Medium] Incomplete error recovery in suspend callback leaves the device in a permanently broken state. > - [Medium] Synchronous firmware request during probe may permanently fail if the root filesystem is not yet mounted. > - [Low] Firmware version is not updated in the driver state after a successful sysfs upgrade. > -- Please respond to Sashiko findings. -- With best wishes Dmitry