Skip to content

Fix bugs in kona pwm driver and pwm core

Jonathan Richardson linux-pwm lkml 12 versions – Applied

Status from this message: Applied, thanks.

Get series with b4
b4 am [email protected]

Versions newest first · linked by author and title

  1. v10 Latest1 patch 1 replyApplied
  2. v92 patches 5 repliesApplied
  3. v85 patches 9 repliesApplied
Earlier versions · 9
VersionPostedPatchesRepliesStatus
v75 patches3 repliesSuperseded
v64 patches4 repliesSuperseded
v5 posting 2/22 patches5 repliesSuperseded
v4 posting 2/22 patches2 repliesSuperseded
v32 patches4 repliesSuperseded
v5 posting 1/22 patches1 replySuperseded
v4 posting 1/23 patches2 repliesSuperseded
v22 patches16 repliesSuperseded
v12 patches3 repliesSuperseded

Changes in v10 from the cover letter

- pwm_set_polarity() enabled check is protected using the same mutex as pwm_enable() in pwm core. Implemented as suggested by Thierry.

Patches in v10 Applied

#Subject and filesSizeRepliesApplied
1/1pwm: core: Set enable state properly on failed call to enabledrivers/pwm/core.c include/linux/pwm.h+28 -71 reply

Reviews trailers per patch

The patch has a Reviewed-by or Acked-by.

PatchJonathan RichardsonDmitry Torokhov
1/1T-b R-b

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.

Read the v10 thread

Cover letter [PATCH v10 0/1] Fix bugs in kona pwm driver and pwm core

Fixes a bug in the pwm core where the enabled state was incorrect on failed calls to enable.

Changes from v9:
- pwm_set_polarity() enabled check is protected using the same mutex as pwm_enable() in pwm core. Implemented as suggested by Thierry.

Changes from v8:
- Accepted patches not included. Patch to introduce debug statements to config in kona pwm driver not included - will be addressed later.
- Added mutex to pwm core enable function to prevent potential concurrency issue, as suggested by Thierry.
- Fixed commit message for kona pwm changes. Also changed wording in comments from enable to trigger.

Changes from v7:
- Polarity changes take effect immediately instead of being deferred until enable is called.

Changes from v6:
- Move new debugging info for duty cycle and period in config function into its own commit.
- Add kona_pwmc_prepare_for_settings() function to remove duplicated code.

Changes from v5:
- Address Dmitry's comment on code cleanup of pwm_enable() in pwm core.
- Including all patches from different contributors in this patchset. Some were left out in v4.

Changes from v4:
- Rebased to Tim Kryger's patch that adds support in pwm core to add driver with inversed polarity.
- Removed patch 2 that resolved difference between hardware default polarity and pwm framework default polarity. The default polarity is set properly now when the pwm driver is registered with the pwm framework.

Jonathan Richardson (1):
pwm: core: Set enable state properly on failed call to enable

 drivers/pwm/core.c  | 33 ++++++++++++++++++++++++++-------
 include/linux/pwm.h |  2 ++
 2 files changed, 28 insertions(+), 7 deletions(-)
Signature · 1 line
1.9.1