Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1031008AbbD1VGS (ORCPT ); Tue, 28 Apr 2015 17:06:18 -0400 Received: from mail-gw3-out.broadcom.com ([216.31.210.64]:24390 "EHLO mail-gw3-out.broadcom.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1030851AbbD1VGN (ORCPT ); Tue, 28 Apr 2015 17:06:13 -0400 X-IronPort-AV: E=Sophos;i="5.11,666,1422950400"; d="scan'208";a="63240503" Message-ID: <553FF632.7050706@broadcom.com> Date: Tue, 28 Apr 2015 14:05:54 -0700 From: Jonathan Richardson User-Agent: Mozilla/5.0 (X11; Linux i686; rv:17.0) Gecko/17.0 Thunderbird/17.0 MIME-Version: 1.0 To: Jonathan Richardson CC: Tim Kryger , Dmitry Torokhov , Anatol Pomazau , Arun Ramamurthy , Thierry Reding , Scott Branden , bcm-kernel-feedback-list , , Subject: Re: [PATCH v6 0/4] Fix bugs in kona pwm driver and pwm core References: <1428692333-14400-1-git-send-email-jonathar@broadcom.com> In-Reply-To: <1428692333-14400-1-git-send-email-jonathar@broadcom.com> Content-Type: text/plain; charset="ISO-8859-1" Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2440 Lines: 55 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(-) > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/