* [PATCH] dt-bindings: Add pxe1610 as a trivial device
@ 2019-07-23 0:20 Vijay Khemka
2019-07-23 2:18 ` Joel Stanley
2019-07-23 14:50 ` Rob Herring
0 siblings, 2 replies; 11+ messages in thread
From: Vijay Khemka @ 2019-07-23 0:20 UTC (permalink / raw)
To: Rob Herring, Mark Rutland, Jiri Kosina, Guenter Roeck,
Herbert Xu, Patrick Venture, Ard Biesheuvel, Vijay Khemka,
Anson Huang, Jeremy Gebben, devicetree, linux-kernel
Cc: openbmc @ lists . ozlabs . org, joel, linux-aspeed, sdasari
The pxe1610 is a voltage regulator from Infineon. It also supports
other VRs pxe1110 and pxm1310 from Infineon.
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
- infineon,slb9635tt
# Infineon SLB9645 I2C TPM (new protocol, max 400khz)
- infineon,slb9645tt
--
2.17.1
^ permalink raw reply related [flat|nested] 11+ messages in thread* Re: [PATCH] dt-bindings: Add pxe1610 as a trivial device
2019-07-23 0:20 [PATCH] dt-bindings: Add pxe1610 as a trivial device Vijay Khemka
@ 2019-07-23 2:18 ` Joel Stanley
2019-07-23 14:50 ` Rob Herring
1 sibling, 0 replies; 11+ messages in thread
From: Joel Stanley @ 2019-07-23 2:18 UTC (permalink / raw)
To: Vijay Khemka
Cc: Rob Herring, Mark Rutland, Jiri Kosina, Guenter Roeck,
Herbert Xu, Patrick Venture, Ard Biesheuvel, Anson Huang,
Jeremy Gebben, devicetree, Linux Kernel Mailing List,
openbmc @ lists . ozlabs . org, linux-aspeed, Sai Dasari
On Tue, 23 Jul 2019 at 00:46, Vijay Khemka <vijaykhemka@fb.com> wrote:
>
> The pxe1610 is a voltage regulator from Infineon. It also supports
> other VRs pxe1110 and pxm1310 from Infineon.
>
> Signed-off-by: Vijay Khemka <vijaykhemka@fb.com>
Acked-by: Joel Stanley <joel@jms.id.au>
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH] dt-bindings: Add pxe1610 as a trivial device
@ 2019-07-23 2:18 ` Joel Stanley
0 siblings, 0 replies; 11+ messages in thread
From: Joel Stanley @ 2019-07-23 2:18 UTC (permalink / raw)
To: Vijay Khemka
Cc: Rob Herring, Mark Rutland, Jiri Kosina, Guenter Roeck,
Herbert Xu, Patrick Venture, Ard Biesheuvel, Anson Huang,
Jeremy Gebben, devicetree, Linux Kernel Mailing List,
openbmc @ lists . ozlabs . org, linux-aspeed, Sai Dasari
On Tue, 23 Jul 2019 at 00:46, Vijay Khemka <vijaykhemka@fb.com> wrote:
>
> The pxe1610 is a voltage regulator from Infineon. It also supports
> other VRs pxe1110 and pxm1310 from Infineon.
>
> Signed-off-by: Vijay Khemka <vijaykhemka@fb.com>
Acked-by: Joel Stanley <joel@jms.id.au>
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH] dt-bindings: Add pxe1610 as a trivial device
2019-07-23 0:20 [PATCH] dt-bindings: Add pxe1610 as a trivial device Vijay Khemka
@ 2019-07-23 14:50 ` Rob Herring
2019-07-23 14:50 ` Rob Herring
1 sibling, 0 replies; 11+ messages in thread
From: Rob Herring @ 2019-07-23 14:50 UTC (permalink / raw)
To: Vijay Khemka
Cc: Mark Rutland, Jiri Kosina, Guenter Roeck, Herbert Xu,
Patrick Venture, Ard Biesheuvel, Anson Huang, Jeremy Gebben,
devicetree, linux-kernel, openbmc @ lists . ozlabs . org,
Joel Stanley, linux-aspeed, sdasari
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.
>
> 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
>
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH] dt-bindings: Add pxe1610 as a trivial device
@ 2019-07-23 14:50 ` Rob Herring
0 siblings, 0 replies; 11+ messages in thread
From: Rob Herring @ 2019-07-23 14:50 UTC (permalink / raw)
To: Vijay Khemka
Cc: Mark Rutland, Jiri Kosina, Guenter Roeck, Herbert Xu,
Patrick Venture, Ard Biesheuvel, Anson Huang, Jeremy Gebben,
devicetree, linux-kernel, openbmc @ lists . ozlabs . org,
Joel Stanley, linux-aspeed, sdasari
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.
>
> 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
>
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH] dt-bindings: Add pxe1610 as a trivial device
2019-07-23 14:50 ` Rob Herring
@ 2019-07-23 14:52 ` Rob Herring
-1 siblings, 0 replies; 11+ messages in thread
From: Rob Herring @ 2019-07-23 14:52 UTC (permalink / raw)
To: Vijay Khemka
Cc: Mark Rutland, Jiri Kosina, Guenter Roeck, Herbert Xu,
Patrick Venture, Ard Biesheuvel, Anson Huang, Jeremy Gebben,
devicetree, linux-kernel, openbmc @ lists . ozlabs . org,
Joel Stanley, linux-aspeed, sdasari
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?
> >
> > 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
> >
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH] dt-bindings: Add pxe1610 as a trivial device
@ 2019-07-23 14:52 ` Rob Herring
0 siblings, 0 replies; 11+ messages in thread
From: Rob Herring @ 2019-07-23 14:52 UTC (permalink / raw)
To: Vijay Khemka
Cc: Mark Rutland, Jiri Kosina, Guenter Roeck, Herbert Xu,
Patrick Venture, Ard Biesheuvel, Anson Huang, Jeremy Gebben,
devicetree, linux-kernel, openbmc @ lists . ozlabs . org,
Joel Stanley, linux-aspeed, sdasari
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?
> >
> > 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
> >
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH] dt-bindings: Add pxe1610 as a trivial device
2019-07-23 14:52 ` Rob Herring
(?)
@ 2019-07-23 17:13 ` Vijay Khemka
2019-08-02 6:31 ` Joel Stanley
-1 siblings, 1 reply; 11+ messages in thread
From: Vijay Khemka @ 2019-07-23 17:13 UTC (permalink / raw)
To: Rob Herring
Cc: Mark Rutland, Jiri Kosina, Guenter Roeck, Herbert Xu,
Patrick Venture, Ard Biesheuvel, Anson Huang, Jeremy Gebben,
devicetree, linux-kernel, openbmc @ lists . ozlabs . org,
Joel Stanley, linux-aspeed, Sai Dasari
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
> >
^ permalink raw reply [flat|nested] 11+ messages in thread* Re: [PATCH] dt-bindings: Add pxe1610 as a trivial device
2019-07-23 17:13 ` Vijay Khemka
@ 2019-08-02 6:31 ` Joel Stanley
0 siblings, 0 replies; 11+ messages in thread
From: Joel Stanley @ 2019-08-02 6:31 UTC (permalink / raw)
To: Vijay Khemka
Cc: Rob Herring, Mark Rutland, Jiri Kosina, Guenter Roeck,
Herbert Xu, Patrick Venture, Ard Biesheuvel, Anson Huang,
Jeremy Gebben, devicetree, linux-kernel,
openbmc @ lists . ozlabs . org, linux-aspeed, Sai Dasari
Add pxe1610 as a trivial device
On Tue, 23 Jul 2019 at 17:14, Vijay Khemka <vijaykhemka@fb.com> wrote:
>
> 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.
You have these three IDs in the driver:
pxm1310
pxm1310
pxe1610
So all three could be listed in the documentation?
Rob, is this what you wanted Vijay to do?
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH] dt-bindings: Add pxe1610 as a trivial device
@ 2019-08-02 6:31 ` Joel Stanley
0 siblings, 0 replies; 11+ messages in thread
From: Joel Stanley @ 2019-08-02 6:31 UTC (permalink / raw)
To: Vijay Khemka
Cc: Rob Herring, Mark Rutland, Jiri Kosina, Guenter Roeck,
Herbert Xu, Patrick Venture, Ard Biesheuvel, Anson Huang,
Jeremy Gebben, devicetree, linux-kernel,
openbmc @ lists . ozlabs . org, linux-aspeed, Sai Dasari
Add pxe1610 as a trivial device
On Tue, 23 Jul 2019 at 17:14, Vijay Khemka <vijaykhemka@fb.com> wrote:
>
> 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.
You have these three IDs in the driver:
pxm1310
pxm1310
pxe1610
So all three could be listed in the documentation?
Rob, is this what you wanted Vijay to do?
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH] dt-bindings: Add pxe1610 as a trivial device
2019-08-02 6:31 ` Joel Stanley
(?)
@ 2019-08-06 20:35 ` Vijay Khemka
-1 siblings, 0 replies; 11+ messages in thread
From: Vijay Khemka @ 2019-08-06 20:35 UTC (permalink / raw)
To: Joel Stanley
Cc: Rob Herring, Mark Rutland, Jiri Kosina, Guenter Roeck,
Herbert Xu, Patrick Venture, Ard Biesheuvel, Anson Huang,
Jeremy Gebben, devicetree, linux-kernel,
openbmc @ lists . ozlabs . org, linux-aspeed, Sai Dasari
Joel,
I have added all 3 id in the documentation patch and I am not sure if that patch has been applied or not.
Regards
-Vijay
On 8/1/19, 11:31 PM, "Joel Stanley" <joel@jms.id.au> wrote:
Add pxe1610 as a trivial device
On Tue, 23 Jul 2019 at 17:14, Vijay Khemka <vijaykhemka@fb.com> wrote:
>
> 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.
You have these three IDs in the driver:
pxm1310
pxm1310
pxe1610
So all three could be listed in the documentation?
Rob, is this what you wanted Vijay to do?
^ permalink raw reply [flat|nested] 11+ messages in thread
end of thread, other threads:[~2019-08-06 20:36 UTC | newest]
Thread overview: 11+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-07-23 0:20 [PATCH] dt-bindings: Add pxe1610 as a trivial device Vijay Khemka
2019-07-23 2:18 ` Joel Stanley
2019-07-23 2:18 ` Joel Stanley
2019-07-23 14:50 ` Rob Herring
2019-07-23 14:50 ` Rob Herring
2019-07-23 14:52 ` Rob Herring
2019-07-23 14:52 ` Rob Herring
2019-07-23 17:13 ` Vijay Khemka
2019-08-02 6:31 ` Joel Stanley
2019-08-02 6:31 ` Joel Stanley
2019-08-06 20:35 ` Vijay Khemka
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.