Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Date:   Thu, 27 Jan 2022 09:23:33 -0600
From:   Bjorn Helgaas <helgaas@kernel.org>
To:     Aswath Govindraju <a-govindraju@ti.com>
Cc:     Bjorn Helgaas <bhelgaas@google.com>,
        Vignesh Raghavendra <vigneshr@ti.com>,
        Nishanth Menon <nm@ti.com>,
        Kishon Vijay Abraham I <kishon@ti.com>,
        Rob Herring <robh+dt@kernel.org>, linux-pci@vger.kernel.org,
        devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
        Lorenzo Pieralisi <lorenzo.pieralisi@arm.com>
Subject: Re: [PATCH v2] dt-bindings: PCI: ti,j721e: Add device id for J721S2
Message-ID: <20220127152333.GA101708@bhelgaas>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <ab490690-eac2-c714-1359-b4058e2f98ff@ti.com>
Precedence: bulk

[+cc Lorenzo, initial post
https://lore.kernel.org/r/20211130035608.13492-1-a-govindraju@ti.com]

On Tue, Jan 25, 2022 at 12:06:16PM +0530, Aswath Govindraju wrote:
> On 30/11/21 9:26 am, Aswath Govindraju wrote:
> > Document the device id of J721S2 SoC.
> > 
> > Signed-off-by: Aswath Govindraju <a-govindraju@ti.com>
> > ---
> > 
> 
> May I know if this patch can be picked up?

"git log Documentation/devicetree/bindings/pci/ti,j721e-pci-host.yaml"
says most changes to this file have been applied by Lorenzo, so I cc'd
him.

> > changes since v1:
> > - changed (oneOf, items) into enum
> > 
> >  .../devicetree/bindings/pci/ti,j721e-pci-host.yaml   | 12 +++++-------
> >  1 file changed, 5 insertions(+), 7 deletions(-)
> > 
> > diff --git a/Documentation/devicetree/bindings/pci/ti,j721e-pci-host.yaml b/Documentation/devicetree/bindings/pci/ti,j721e-pci-host.yaml
> > index cc900202df29..41df8f1c2d4c 100644
> > --- a/Documentation/devicetree/bindings/pci/ti,j721e-pci-host.yaml
> > +++ b/Documentation/devicetree/bindings/pci/ti,j721e-pci-host.yaml
> > @@ -64,13 +64,11 @@ properties:
> >      const: 0x104c
> >  
> >    device-id:
> > -    oneOf:
> > -      - items:
> > -          - const: 0xb00d
> > -      - items:
> > -          - const: 0xb00f
> > -      - items:
> > -          - const: 0xb010
> > +    enum:
> > +      - 0xb00d
> > +      - 0xb00f
> > +      - 0xb010
> > +      - 0xb013
> >    msi-map: true
> >  
> > 
>