Re: [PATCH] dt-bindings: mfd: hisilicon: Fix Hi6421 SPMI compatible

[Rob Herring <robh@kernel.org>]

On Mon, Sep 21, 2026 at 11:36:50PM +0000, sashiko-bot@kernel.org wrote:
> Thank you for your contribution! Sashiko AI review found 1 potential issue(s) to consider:
> - [Medium] dt-bindings: mfd: hi6421: invalid gpios property for interrupt line
> --
> 
> commit 4ce4ba18007f2f81249a1f2307d744174beb3e75
> Author: Rob Herring (Arm) <robh@kernel.org>
> 
> dt-bindings: mfd: hisilicon: Fix Hi6421 SPMI compatible
> 
> This commit updates the device tree binding for the Hi6421 SPMI PMIC to use
> the hisilicon,hi6421-spmi compatible string. It also allows the gpios
> property for the GPIO interrupt line used by the HiKey970 PMIC node.
> 
> > diff --git a/Documentation/devicetree/bindings/mfd/hisilicon,hi6421-spmi-pmic.yaml b/Documentation/devicetree/bindings/mfd/hisilicon,hi6421-spmi-pmic.yaml
> > index 6a824351834ee..9275c561e9d49 100644
> > --- a/Documentation/devicetree/bindings/mfd/hisilicon,hi6421-spmi-pmic.yaml
> > +++ b/Documentation/devicetree/bindings/mfd/hisilicon,hi6421-spmi-pmic.yaml
> [ ... ]
> > @@ -32,6 +32,9 @@ properties:
> >    '#interrupt-cells':
> >      const: 2
> >  
> > +  gpios:
> > +    maxItems: 1
> > +
> 
> [Severity: Medium]
> Does this property legitimize a broken device tree description by using gpios
> for an interrupt line?
> 
> Looking at the driver in hi6421v600_irq_probe(), it unconditionally relies on
> platform_get_irq() to fetch its interrupt:
> 
> drivers/misc/hi6421v600-irq.c:hi6421v600_irq_probe() {
> 	...
> 	priv->irq = platform_get_irq(pmic_pdev, 0);
> 	if (priv->irq < 0)
> 		return priv->irq;
> 	...
> }
> 
> Because platform_get_irq() only parses the "interrupts" property, it lacks a
> fallback to parse "gpios". Will this cause the driver to fail to probe when
> using the HiKey970 PMIC node configuration since it cannot find the interrupt?

The staging driver used 'gpios'. That was moved to interrupts, so the 
gpios hunk can be dropped. And then the .dts needs to be updated.

Rob