[PATCH v2 1/4] dt-bindings: iio: adc: mediatek,mt6359-auxadc: add mt6323 PMIC AUXADC

Roman Vivchar rva333 at protonmail.com
Tue Jun 9 10:29:04 PDT 2026


Hi Conor,

On Tuesday, June 9th, 2026 at 7:01 PM, Conor Dooley <conor at kernel.org> wrote:
> On Tue, Jun 09, 2026 at 04:31:58PM +0300, Roman Vivchar via B4 Relay wrote:
> >  properties:
> >    compatible:
> >      enum:
> > +      - mediatek,mt6323-auxadc
> 
> Commit message needs to explain why a fallback is not suitable.
> pw-bot: changes-requested

Ack.

> 
> >        - mediatek,mt6357-auxadc
> >        - mediatek,mt6358-auxadc
> >        - mediatek,mt6359-auxadc
> > diff --git a/MAINTAINERS b/MAINTAINERS
> > index d1cc0e12fe1f..2551c8cd9e9d 100644
> > --- a/MAINTAINERS
> > +++ b/MAINTAINERS
> > @@ -16256,6 +16256,12 @@ S:	Maintained
> >  F:	Documentation/devicetree/bindings/mmc/mtk-sd.yaml
> >  F:	drivers/mmc/host/mtk-sd.c
> >  
> > +MEDIATEK MT6323 PMIC AUXADC DRIVER
> > +M:	Roman Vivchar <rva333 at protonmail.com>
> > +L:	linux-iio at vger.kernel.org
> > +S:	Maintained
> > +F:	include/dt-bindings/iio/adc/mediatek,mt6323-auxadc.h
> 
> Why is the binding not being included here?
> 

The binding is shared across multiple PMIC ADCs and maintained by
Angelo. Since I'm only familiar with mt6323 and not others like mt6358,
I decided to not include it in the mt6323 entry.

Best regards,
Roman



More information about the linux-arm-kernel mailing list