2020-11-18 17:48:15

by Clemens Gruber

[permalink] [raw]
Subject: [PATCH] dt-bindings: pwm: pca9685: Add staggered-outputs property

The pca9685 driver supports a new staggered-outputs property for reduced
current surges and EMI. This adds documentation for the new DT property.

Signed-off-by: Clemens Gruber <[email protected]>
---
Documentation/devicetree/bindings/pwm/nxp,pca9685-pwm.txt | 2 ++
1 file changed, 2 insertions(+)

diff --git a/Documentation/devicetree/bindings/pwm/nxp,pca9685-pwm.txt b/Documentation/devicetree/bindings/pwm/nxp,pca9685-pwm.txt
index f21b55c95738..911bbb65984e 100644
--- a/Documentation/devicetree/bindings/pwm/nxp,pca9685-pwm.txt
+++ b/Documentation/devicetree/bindings/pwm/nxp,pca9685-pwm.txt
@@ -12,6 +12,8 @@ Optional properties:
- invert (bool): boolean to enable inverted logic
- open-drain (bool): boolean to configure outputs with open-drain structure;
if omitted use totem-pole structure
+ - staggered-outputs (bool): boolean to enable staggered output ON times to
+ minimize current surges and EMI

Example:

--
2.29.2


2020-11-20 08:26:18

by Uwe Kleine-König

[permalink] [raw]
Subject: Re: [PATCH] dt-bindings: pwm: pca9685: Add staggered-outputs property

On Wed, Nov 18, 2020 at 06:45:15PM +0100, Clemens Gruber wrote:
> The pca9685 driver supports a new staggered-outputs property for reduced
> current surges and EMI. This adds documentation for the new DT property.

Maybe point out the commit that added this support here?

Best regards
Uwe

--
Pengutronix e.K. | Uwe Kleine-K?nig |
Industrial Linux Solutions | https://www.pengutronix.de/ |


Attachments:
(No filename) (457.00 B)
signature.asc (499.00 B)
Download all attachments

2020-11-20 12:40:54

by Clemens Gruber

[permalink] [raw]
Subject: Re: [PATCH] dt-bindings: pwm: pca9685: Add staggered-outputs property

Hi,

On Fri, Nov 20, 2020 at 09:21:50AM +0100, Uwe Kleine-K?nig wrote:
> On Wed, Nov 18, 2020 at 06:45:15PM +0100, Clemens Gruber wrote:
> > The pca9685 driver supports a new staggered-outputs property for reduced
> > current surges and EMI. This adds documentation for the new DT property.
>
> Maybe point out the commit that added this support here?

The pca9685 driver patch was not yet accepted and there will be a v2.

Should I wait until the support is in Linus' tree and only then send a
dt-bindings documentation patch, mentioning the commit?

Best regards,
Clemens

2020-11-20 12:52:04

by Uwe Kleine-König

[permalink] [raw]
Subject: Re: [PATCH] dt-bindings: pwm: pca9685: Add staggered-outputs property

On Fri, Nov 20, 2020 at 01:36:54PM +0100, Clemens Gruber wrote:
> Hi,
>
> On Fri, Nov 20, 2020 at 09:21:50AM +0100, Uwe Kleine-K?nig wrote:
> > On Wed, Nov 18, 2020 at 06:45:15PM +0100, Clemens Gruber wrote:
> > > The pca9685 driver supports a new staggered-outputs property for reduced
> > > current surges and EMI. This adds documentation for the new DT property.
> >
> > Maybe point out the commit that added this support here?
>
> The pca9685 driver patch was not yet accepted and there will be a v2.
>
> Should I wait until the support is in Linus' tree and only then send a
> dt-bindings documentation patch, mentioning the commit?

Then I'd say add the dt-bindings patch to the series adding this
support.

Best regards
Uwe

--
Pengutronix e.K. | Uwe Kleine-K?nig |
Industrial Linux Solutions | https://www.pengutronix.de/ |


Attachments:
(No filename) (914.00 B)
signature.asc (499.00 B)
Download all attachments