Re: [PATCH v5 6/6] ARM: dts: qcom: Add Samsung Galaxy S4

Thank you for your contribution! Sashiko AI review found 1 potential issue(=
s) to consider:
- [Low] The `muic` node contains invalid properties `safeout1-supply` and `=
safeout2-supply` which violate the device tree schema.
--

--- Patch [6]: [PATCH v5 6/6] ARM: dts: qcom: Add Samsung Galaxy S4 ---
commit 71d3822fee0022c24c538ba38400b342e55f2d78
Author: Alexandre MINETTE <[email protected]>

ARM: dts: qcom: Add Samsung Galaxy S4

This commit adds initial device tree support for the Samsung Galaxy S4 (jfl=
te).
It covers UART, USB peripheral mode, networking, front LED, and physical bu=
ttons.

> diff --git a/arch/arm/boot/dts/qcom/qcom-apq8064-samsung-jflte.dts b/arch=
/arm/boot/dts/qcom/qcom-apq8064-samsung-jflte.dts
> new file mode 100644
> index 0000000000000..75ae19af96e54
> --- /dev/null
> +++ b/arch/arm/boot/dts/qcom/qcom-apq8064-samsung-jflte.dts

[ ... ]

> @@ -115,14 +115,14 @@
> +			interrupts =3D <55 IRQ_TYPE_EDGE_FALLING>;
> +			pinctrl-0 =3D <&muic_int_default_state>;
> +			pinctrl-names =3D "default";
> +
> +			muic: muic {
> +				compatible =3D "maxim,max77693-muic";
> +				safeout1-supply =3D <&esafeout1_reg>;
> +				safeout2-supply =3D <&esafeout2_reg>;

[Severity: Low]
Are safeout1-supply and safeout2-supply supported by the maxim,max77693-muic
device tree binding?

The schema validation fails here because the binding does not define these
properties. This causes a validation failure during make dtbs_check.

> +			};
> +
> +			regulators {
> +				esafeout1_reg: ESAFEOUT1 {

[ ... ]

--=20
Sashiko AI review =C2=B7 https://sashiko.dev/#/patchset/20260804-mainline-s=
[email protected]?part=3D6