Re: [PATCH v2] drm/bridge: analogix_dp: Drop dead error check for analogix_dp_init_analog_func()

[sashiko-bot@kernel.org]

Thank you for your contribution! Sashiko AI review found 1 potential
issue to consider:
- [High] drm/bridge: analogix_dp: Signed-off-by uses handle instead of
  real name
--

commit b6a1a7f2eef3c9d11adbe1470d41eb8ca3821faa
Author: laixiangwu <laixiangwu@cmss.chinamobile.com>

> drm/bridge: analogix_dp: Drop dead error check for analogix_dp_init_analog_func()
>
> analogix_dp_init_analog_func() cannot fail: since commit 917c8d192ba6ad
> ("drm/bridge: analogix_dp: don't wait for PLL lock too early") removed
> the PLL lock polling loop, the function only performs register writes
> and always returns 0.  The error check on its return value in
> analogix_dp_set_bridge() is therefore dead code.
>
> Drop the dead check and make the function void.
>
> Signed-off-by: laixiangwu <laixiangwu@cmss.chinamobile.com>

[Severity: High]
The Author header and Signed-off-by trailer use a single-word account
handle ("laixiangwu") rather than a real human name (for example,
"Firstname Lastname"), which does not meet the Developer's Certificate of
Origin requirements in Documentation/process/submitting-patches.rst.

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20261111080149.1567-1-laixiangwu@cmss.chinamobile.com?part=1