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/bridge: analogix_dp: Rename and simplify is_rockchip() Date: Wed, 27 May 2026 13:55:46 +1000 Message-ID: In-Reply-To: <20260527024336.191433-9-damon.ding@rock-chips.com> References: <20260527024336.191433-1-damon.ding@rock-chips.com> <20260527024336.191433-9-damon.ding@rock-chips.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 rename from `is_rockchip()` to `analogix_dp_is_rockchip()` with conversion to switch-case: ```c -static inline bool is_rockchip(enum analogix_dp_devtype type) +static inline bool analogix_dp_is_rockchip(enum analogix_dp_devtype type) { - return type == RK3288_DP || type == RK3399_EDP || type == RK3588_EDP; + switch (type) { + case RK3288_DP: + case RK3399_EDP: + case RK3588_EDP: + return true; + default: + return false; + } } ``` All 9 call sites in `analogix_dp_core.c` and `analogix_dp_reg.c` are updated. The switch-case also benefits from compiler warnings on unhandled enum values when new entries are added. Good prep work for patch 9. No issues. --- Generated by Claude Code Patch Reviewer