[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <F7BAC53E-925E-4FA4-815D-ACB82DF8D240@fb.com>
Date: Tue, 23 Jul 2019 17:13:49 +0000
From: Vijay Khemka <vijaykhemka@...com>
To: Rob Herring <robh+dt@...nel.org>
CC: Mark Rutland <mark.rutland@....com>,
Jiri Kosina <trivial@...nel.org>,
Guenter Roeck <linux@...ck-us.net>,
Herbert Xu <herbert@...dor.apana.org.au>,
Patrick Venture <venture@...gle.com>,
Ard Biesheuvel <ard.biesheuvel@...aro.org>,
Anson Huang <anson.huang@....com>,
Jeremy Gebben <jgebben@...ptlaser.com>,
"devicetree@...r.kernel.org" <devicetree@...r.kernel.org>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
"openbmc @ lists . ozlabs . org" <openbmc@...ts.ozlabs.org>,
Joel Stanley <joel@....id.au>,
"linux-aspeed@...ts.ozlabs.org" <linux-aspeed@...ts.ozlabs.org>,
Sai Dasari <sdasari@...com>
Subject: Re: [PATCH] dt-bindings: Add pxe1610 as a trivial device
On 7/23/19, 7:53 AM, "Rob Herring" <robh+dt@...nel.org> wrote:
On Tue, Jul 23, 2019 at 8:50 AM Rob Herring <robh+dt@...nel.org> wrote:
>
> On Mon, Jul 22, 2019 at 6:46 PM Vijay Khemka <vijaykhemka@...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@...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
> >
Powered by blists - more mailing lists