Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757527AbcJXV0g (ORCPT ); Mon, 24 Oct 2016 17:26:36 -0400 Received: from 18.mo3.mail-out.ovh.net ([87.98.172.162]:50263 "EHLO 18.mo3.mail-out.ovh.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755128AbcJXV0f (ORCPT ); Mon, 24 Oct 2016 17:26:35 -0400 Date: Mon, 24 Oct 2016 23:26:15 +0200 From: Lukasz Majewski To: Boris Brezillon Cc: Thierry Reding , Stefan Agner , linux-pwm@vger.kernel.org, linux-kernel@vger.kernel.org, Fabio Estevam , Fabio Estevam , Lothar Wassmann , Bhuvanchandra DV , kernel@pengutronix.de Subject: Re: [PATCH 0/6] pwm: imx: Provide atomic operation for IMX PWM driver Message-ID: <20161024232615.77d4ea28@jawa> In-Reply-To: <20161024173607.36bc2cb9@bbrezillon> References: <1477259146-19167-1-git-send-email-l.majewski@majess.pl> <20161024173607.36bc2cb9@bbrezillon> X-Mailer: Claws Mail 3.11.1 (GTK+ 2.24.25; x86_64-pc-linux-gnu) MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; boundary="Sig_/JnB9_yaxmO5n.C3EXaQusID"; protocol="application/pgp-signature" X-Ovh-Tracer-Id: 8973703734577119945 X-VR-SPAMSTATE: OK X-VR-SPAMSCORE: -100 X-VR-SPAMCAUSE: gggruggvucftvghtrhhoucdtuddrfeelvddrieeigddtudcutefuodetggdotefrodftvfcurfhrohhfihhlvgemucfqggfjpdevjffgvefmvefgnecuuegrihhlohhuthemuceftddtnecusecvtfgvtghiphhivghnthhsucdlqddutddtmd Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2613 Lines: 86 --Sig_/JnB9_yaxmO5n.C3EXaQusID Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable Hi Boris, > On Sun, 23 Oct 2016 23:45:40 +0200 > Lukasz Majewski wrote: >=20 > > This patch set brings atomic operation to i.MX's PWMv2 driver. > >=20 > > This work has been supported and suggested by Boris Brezillon [1] > > and Stefan Agner, by showing how simple the transition could be :-). > >=20 > > It has been divided into several steps: > > - Separate PWMv1 commits from "generic" and non atomic PWM code. > >=20 > > NOTE: Since I do not have board with PWMv1, I would like to ask > > somebody for testing > >=20 > > - Move some imx_config_v2 code to separate functions > >=20 > > - Provide PWM atomic implementation (the ->apply() driver) in a > > single patch for better readability. > >=20 > > - Remove redundant PWM code (disable, enable, config callbacks) > >=20 > > - Clean up the driver infrastructure > >=20 > > - Provide "polarity_supported" flag to indicate support for > > polarity inversion > >=20 > > This work should be applied on top of following commits: > >=20 > > http://patchwork.ozlabs.org/patch/679706/=20 [2] > > http://patchwork.ozlabs.org/patch/679707/ [3] > > http://patchwork.ozlabs.org/patch/679680/ >=20 > I'm not sure I follow the logic here. Has patch [1] already been > applied? If that's not the case, then you should just drop it and put > your changes on top of mainline. >=20 > [1]http://patchwork.ozlabs.org/patch/679680/ Patches [2] and [3] have been developed initially by Lothar and subsequently picked up by Bhuvanchandra. There is no issue with them. The patch [1] is a bit more tricky. The work has been done by Bhuvanchandra, which adds DTS and core support for polarity inversion. This code works and utilizes the "old" PWM API with enable, disable and config. However, Stefan had some comments about the placement for the polarity setting (in the .config_v2()) and proposed switch to atomic API. To make things easier and cleaner, I decided to put my atomic API rework on top of those patches. In this way I can credit the previous work and avoid rewriting DTS polarity inversion code already developed and validated by Bhuvanchandra. Best regards, =C5=81ukasz Majewski --Sig_/JnB9_yaxmO5n.C3EXaQusID Content-Type: application/pgp-signature Content-Description: OpenPGP digital signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v2 iEYEARECAAYFAlgOfH0ACgkQf9/hG2YwgjH+NACggL2432WkL7MHZ08A+3Z6Nj4w fMkAn0Xqr5Z/IxPLWWVJhTv669Qg8x7Z =2Lbg -----END PGP SIGNATURE----- --Sig_/JnB9_yaxmO5n.C3EXaQusID--