Re: [PATCH v5 09/15] ARM: dts: aspeed: anacapa: evt2: Add shunt resistor values for HSC monitors

[email protected]
Newsgroups org.kernel.vger.linux-devicetree
Message-ID <[email protected]>
Thank you for your contribution! Sashiko AI review found 1 potential issue(s) to consider:
- [Low] Missing Signed-off-by from the patch author.
--

--- Patch [9]: [PATCH v5 09/15] ARM: dts: aspeed: anacapa: evt2: Add shunt resistor values for HSC monitors ---
commit ed139ca587bbe5fe1cadfdeab9c5f4d6712eff09
Author: Colin Huang <[email protected]>

ARM: dts: aspeed: anacapa: evt2: Add shunt resistor values for HSC monitors

This commit adds shunt resistor configurations for LTC4287 hot-swap
controllers on the Anacapa EVT2 platform. It configures the values to
150 micro-ohms or 250 micro-ohms depending on the rail to ensure accurate
current and power measurement by the driver.

> Author: Colin Huang <[email protected]>
> Date:   Thu Aug 13 11:59:39 2026 +0000
> 
>     ARM: dts: aspeed: anacapa: evt2: Add shunt resistor values for HSC monitors
>     
>     Add shunt resistor configuration for LTC4287 hot-swap controllers on
>     Anacapa EVT2 platform.
[ ... ]
>     Without this configuration, the reported readings would be incorrect.
>     
>     Signed-off-by: Peter Shen <[email protected]>

[Severity: Low]
This isn't a bug, but there appears to be a mismatch between the patch
author and the provided sign-offs. Is a Signed-off-by tag from the original
author missing here to satisfy the Developer's Certificate of Origin rules
for kernel submissions?

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20260813-anacapa-devlop-phase-devicetree-v5-0-c7058f63dc67@gmail.com?part=9
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.