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