Re: [PATCH] dt-bindings: Add pxe1610 as a trivial device

[Vijay Khemka <vijaykhemka@fb.com>]

On 7/23/19, 7:53 AM, "Rob Herring" <robh+dt@kernel.org> wrote:

    On Tue, Jul 23, 2019 at 8:50 AM Rob Herring <robh+dt@kernel.org> wrote:
    >
    > On Mon, Jul 22, 2019 at 6:46 PM Vijay Khemka <vijaykhemka@fb.com> wrote:
    > >
    > > The pxe1610 is a voltage regulator from Infineon. It also supports
    > > other VRs pxe1110 and pxm1310 from Infineon.
    
    What happened to the other compatibles? S/w doesn't need to know the
    differences?
As far as driver is concerned, it doesn't need to know differences.
    
    > >
    > > Signed-off-by: Vijay Khemka <vijaykhemka@fb.com>
    > > ---
    > >  Documentation/devicetree/bindings/trivial-devices.yaml | 2 ++
    > >  1 file changed, 2 insertions(+)
    > >
    > > diff --git a/Documentation/devicetree/bindings/trivial-devices.yaml b/Documentation/devicetree/bindings/trivial-devices.yaml
    > > index 2e742d399e87..1be648828a31 100644
    > > --- a/Documentation/devicetree/bindings/trivial-devices.yaml
    > > +++ b/Documentation/devicetree/bindings/trivial-devices.yaml
    > > @@ -99,6 +99,8 @@ properties:
    > >              # Infineon IR38064 Voltage Regulator
    > >            - infineon,ir38064
    > >              # Infineon SLB9635 (Soft-) I2C TPM (old protocol, max 100khz)
    > > +          - infineon,pxe1610
    > > +            # Infineon PXE1610, PXE1110 and PXM1310 Voltage Regulators
    >
    > The comment goes above the entry.
    >
    > >            - infineon,slb9635tt
    > >              # Infineon SLB9645 I2C TPM (new protocol, max 400khz)
    > >            - infineon,slb9645tt
    > > --
    > > 2.17.1
    > >