Re: [PATCH 02/10] dt-bindings: PCI: qcom: Fix msi-interrupt conditional

[Dmitry Baryshkov]

On 07/07/2022 16:34, Dmitry Baryshkov wrote:
> On 29/06/2022 17:09, Johan Hovold wrote:
> > Fix the msi-interrupt conditional which always evaluated to false due to
> > a misspelled property name ("compatibles" in plural).
> >
> > Signed-off-by: Johan Hovold <johan+linaro@xxxxxxxxxx>
>
> Fixes: 6700a9b00f0a ("dt-bindings: PCI: qcom: Do not require resets on 
> msm8996 platforms")

Ugh, I was thinking about a different chunk (which also has this issue), 
so please ignore the Fixes.

> Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@xxxxxxxxxx>
>
>
> > ---
> >   Documentation/devicetree/bindings/pci/qcom,pcie.yaml | 2 +-
> >   1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/Documentation/devicetree/bindings/pci/qcom,pcie.yaml 
> > b/Documentation/devicetree/bindings/pci/qcom,pcie.yaml
> > index ed9f9462a758..a1b4fc70e162 100644
> > --- a/Documentation/devicetree/bindings/pci/qcom,pcie.yaml
> > +++ b/Documentation/devicetree/bindings/pci/qcom,pcie.yaml
> > @@ -628,7 +628,7 @@ allOf:
> >       # On older chipsets it's always 1 msi interrupt
> >     - if:
> >         properties:
> > -        compatibles:
> > +        compatible:
> >             contains:
> >               enum:
> >                 - qcom,pcie-msm8996


--
With best wishes
Dmitry