From mboxrd@z Thu Jan 1 00:00:00 1970 From: Claude Code Review Bot To: dri-devel-reviews@example.com Subject: Claude review: drm/mxsfb/lcdif: simplify remote pointer management using __free Date: Sun, 12 Apr 2026 13:37:33 +1000 Message-ID: In-Reply-To: <20260407-drm-lcdif-dbanc-v4-1-247a16e61ef9@bootlin.com> References: <20260407-drm-lcdif-dbanc-v4-0-247a16e61ef9@bootlin.com> <20260407-drm-lcdif-dbanc-v4-1-247a16e61ef9@bootlin.com> X-Mailer: Claude Code Patch Reviewer Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit MIME-Version: 1.0 Patch Review Clean conversion to `__free(device_node)` for the `remote` pointer. The two `of_node_put(remote)` calls and the surrounding brace adjustments are correctly removed. ```c + struct device_node *remote __free(device_node) = + of_graph_get_remote_port_parent(ep); ``` Correct -- `of_graph_get_remote_port_parent()` returns a refcounted node, and the `__free` cleanup handles the put in all paths including `continue`. **Reviewed-by tags**: Has Liu Ying's R-by and two T-bys. Looks correct. No issues. --- --- Generated by Claude Code Patch Reviewer