From mboxrd@z Thu Jan 1 00:00:00 1970 From: Claude Code Review Bot To: dri-devel-reviews@example.com Subject: Claude review: phy: introduce phy_get_max_link_rate() helper for consumers Date: Tue, 10 Mar 2026 11:57:17 +1000 Message-ID: In-Reply-To: <20260309190842.927634-15-vladimir.oltean@nxp.com> References: <20260309190842.927634-1-vladimir.oltean@nxp.com> <20260309190842.927634-15-vladimir.oltean@nxp.com> X-Mailer: Claude Code Patch Reviewer Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit MIME-Version: 1.0 Patch Review Adds `phy_get_max_link_rate()` to replace direct `phy->attrs.max_link_rate` accesses in DRM and CAN drivers. NULL-tolerant (returns 0). The CAN driver simplification is nice: ```c - if (transceiver) - priv->can.bitrate_max = transceiver->attrs.max_link_rate; + priv->can.bitrate_max = phy_get_max_link_rate(transceiver); ``` Clean. Already has `Acked-by` from m_can maintainer. --- Generated by Claude Code Patch Reviewer