Re: [PATCH V2 1/6] dt-bindings: power: supply: sgm41542: document sgm41542

[sashiko-bot@kernel.org]

Thank you for your contribution! Sashiko AI review found 1 potential issue(s) to consider:
- [Medium] The `$id` field in the YAML binding does not match the file path, which breaks the schema cross-reference system.
--

commit e462413669a11b133f4d59f276a9c7d14b146226
Author: Chris Morgan <macromorgan@hotmail.com>

dt-bindings: power: supply: sgm41542: document sgm41542

This commit documents the device tree bindings for the SG Micro sgm41542
single-cell battery charger and boost controller. It establishes properties
for the hardware's input limits, interrupts, regulators, and monitored
battery values.

> diff --git a/Documentation/devicetree/bindings/power/supply/sgmicro,sgm41542.yaml b/Documentation/devicetree/bindings/power/supply/sgmicro,sgm41542.yaml
> new file mode 100644
> index 0000000000000..fa0d31e496f79
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/power/supply/sgmicro,sgm41542.yaml
> @@ -0,0 +1,99 @@
> +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
> +%YAML 1.2
> +---
> +$id: http://devicetree.org/schemas/power/supply/sgmicro,sgm41542-charger.yaml#

Does the $id path need to exactly match the file name? The file is named
sgmicro,sgm41542.yaml, but the $id includes an extra -charger suffix.

A mismatch here can break the schema cross-reference system and cause
dt_binding_check to silently skip validation.

[ ... ]

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20260515221947.299229-1-macroalpha82@gmail.com?part=1