From mboxrd@z Thu Jan 1 00:00:00 1970 From: Robert Jarzmik Subject: Re: [PATCH v3 01/12] pwm: introduce default period and polarity concepts Date: Mon, 21 Sep 2015 20:20:29 +0200 Message-ID: <877fnjr5w2.fsf@belgarion.home> References: <1442828009-6241-1-git-send-email-boris.brezillon@free-electrons.com> <1442828009-6241-2-git-send-email-boris.brezillon@free-electrons.com> Mime-Version: 1.0 Content-Type: text/plain Return-path: In-Reply-To: <1442828009-6241-2-git-send-email-boris.brezillon@free-electrons.com> (Boris Brezillon's message of "Mon, 21 Sep 2015 11:33:18 +0200") Sender: linux-leds-owner@vger.kernel.org To: Boris Brezillon Cc: Thierry Reding , linux-pwm@vger.kernel.org, Mark Brown , Liam Girdwood , Jingoo Han , Lee Jones , linux-fbdev@vger.kernel.org, Bryan Wu , Richard Purdie , Jacek Anaszewski , linux-leds@vger.kernel.org, Maxime Ripard , linux-sunxi@googlegroups.com, Heiko Stuebner , linux-rockchip@lists.infradead.org, Jean-Christophe Plagniol-Villard , Tomi Valkeinen , Daniel Mack , Haojian Zhuang , linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, Doug Anderson , Uwe =?utf-8?Q?Kleine-K=C3=B6nig?= List-Id: linux-rockchip.vger.kernel.org Boris Brezillon writes: > When requested by a user, the PWM is assigned a default period and polarity > extracted from the DT, the platform data or statically set by the driver. > Those default values are currently stored in the period and polarity > fields of the pwm_device struct, but they will be stored somewhere else > once we have introduced the architecture allowing for hardware state > retrieval. > > The pwm_set_default_polarity and pwm_set_default_period should only be > used by PWM drivers or the PWM core infrastructure to specify the > default period and polarity values. > > PWM users might call the pwm_get_default_period to query the default > period value. There is currently no helper to query the default > polarity, but it might be added later on if there is a need for it. > > This patch also modifies all the places where the default helpers should > be used in place of the standard ones. > > Signed-off-by: Boris Brezillon For pwm-pxa.c : Acked-by: Robert Jarzmik Cheers. -- Robert