[PATCH 01/13] dt-bindings: iio: adc: add mt6323 PMIC AUXADC
Roman Vivchar
rva333 at protonmail.com
Wed May 6 03:59:29 PDT 2026
On Wednesday, May 6th, 2026 at 10:56 AM, Krzysztof Kozlowski <krzk at kernel.org> wrote:
> On 04/05/2026 20:24, Roman Vivchar via B4 Relay wrote:
> > From: Roman Vivchar <rva333 at protonmail.com>
> >
> > The MediaTek mt6323 PMIC includes an AUXADC used for battery voltage,
> > temperature, and other internal measurements.
> >
...
> > +
> > +properties:
> > + compatible:
> > + const: mediatek,mt6323-auxadc
> > +
> > + "#io-channel-cells":
> > + const: 1
> > +
> > +required:
> > + - compatible
> > + - "#io-channel-cells"
>
>
> This is heavily incomplete node... or unnecessarily split. Why it cannot
> be part of parent binding? Or even parent node?
>
The MediaTek mt6359 AUXADC also has a very similar binding. The mt6323
cannot be merged there because it has a different hardware.
Should this go as a child for the mt6397 mfd like mediatek,mt6323-led?
It currently references only mediatek,mt6359-auxadc.yaml, so perhaps
the mt6359 should be merged to the mfd, and then use oneOf for the
compatible property?
Best regards,
Roman
More information about the linux-arm-kernel
mailing list