Messages in this thread | | | Date | Tue, 28 Apr 2015 14:05:54 -0700 | From | Jonathan Richardson <> | Subject | Re: [PATCH v6 0/4] Fix bugs in kona pwm driver and pwm core |
| |
Pinging maintainer and others. Could you please let me know when you can provide feedback on this patchset.
Tim, were you ok with these changes now or do you still have concerns about changing the driver? I didn't hear back from after our last conversation. If there are still concerns we could possibly make a new cygnus pwm driver, though it would be mostly the same as the existing kona driver. Or add a DT version to the existing driver that has the new modified behaviour - though I think this would be messy.
Thanks.
On 15-04-10 11:58 AM, Jonathan Richardson wrote: > This patchset fixes a number of bugs in the Broadcom Kona pwm driver. It also > fixes a bug in the pwm core where the enabled state was incorrect on failed > calls to enable. Also, a new function was added to the pwm core to add pwm chips > with inversed polarity for chips that have a different default polarity than the > core. The prevents incorrect polarity being reported. > > These changes are required for the Kona pwm driver to work on Cygnus and the > same pwm IP is being used. > > 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. > > Arun Ramamurthy (1): > drivers: pwm: bcm-kona: Dont set polarity in probe > > Jonathan Richardson (2): > pwm: kona: Fix incorrect config, disable, and polarity procedures > pwm: core: Set enable state properly on failed call to enable > > Tim Kryger (1): > drivers: pwm: core: Add pwmchip_add_inversed > > drivers/pwm/core.c | 52 +++++++++++++++++++++------ > drivers/pwm/pwm-bcm-kona.c | 84 +++++++++++++++++++++++++++++++++----------- > include/linux/pwm.h | 6 ++++ > 3 files changed, 110 insertions(+), 32 deletions(-) >
| |