From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Return-Path: Date: Wed, 13 Apr 2016 09:25:11 +0100 From: Lee Jones To: Thierry Reding Cc: Boris Brezillon , linux-pwm@vger.kernel.org, Mike Turquette , Stephen Boyd , linux-clk@vger.kernel.org, Mark Brown , Liam Girdwood , Kamil Debski , lm-sensors@lm-sensors.org, Jean Delvare , Guenter Roeck , Dmitry Torokhov , linux-input@vger.kernel.org, Bryan Wu , Richard Purdie , Jacek Anaszewski , linux-leds@vger.kernel.org, Maxime Ripard , Chen-Yu Tsai , linux-sunxi@googlegroups.com, Joachim Eastwood , Thomas Petazzoni , Heiko Stuebner , linux-rockchip@lists.infradead.org, Jingoo Han , linux-fbdev@vger.kernel.org, Jean-Christophe Plagniol-Villard , Tomi Valkeinen , Robert Jarzmik , Alexandre Belloni , Kukjin Kim , Krzysztof Kozlowski , linux-samsung-soc@vger.kernel.org, intel-gfx@lists.freedesktop.org, Daniel Vetter , Jani Nikula , Jonathan Corbet , linux-doc@vger.kernel.org, David Airlie , Daniel Vetter , dri-devel@lists.freedesktop.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, Hartley Sweeten , Ryan Mallon , Alexander Shiyan , Milo Kim Subject: Re: [PATCH v5 03/46] backlight: lm3630a_bl: stop messing with the pwm->period field Message-ID: <20160413082511.GL8094@x1> References: <1459368249-13241-1-git-send-email-boris.brezillon@free-electrons.com> <1459368249-13241-4-git-send-email-boris.brezillon@free-electrons.com> <20160412110809.GI18882@ulmo.ba.sec> <20160412141613.GI8094@x1> <20160412142608.GB25006@ulmo.ba.sec> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 In-Reply-To: <20160412142608.GB25006@ulmo.ba.sec> List-ID: On Tue, 12 Apr 2016, Thierry Reding wrote: > On Tue, Apr 12, 2016 at 03:16:13PM +0100, Lee Jones wrote: > > On Tue, 12 Apr 2016, Thierry Reding wrote: > > > > > On Wed, Mar 30, 2016 at 10:03:26PM +0200, Boris Brezillon wrote: > > > > pwm->period field is not supposed to be changed by PWM users. The only > > > > ones authorized to change it are the PWM core and PWM drivers. > > > > > > > > Signed-off-by: Boris Brezillon > > > > --- > > > > drivers/video/backlight/lm3630a_bl.c | 3 +-- > > > > 1 file changed, 1 insertion(+), 2 deletions(-) > > > > > > Applied, thanks. > > > > Applied? > > You didn't specifically Ack this one, but I presumed that since the > change is essentially the same as for pwm-backlight, and this is another > prerequisite for the remainder of the series it should go in through the > PWM tree as well. If you're taking Backlight patches through a !Backlight tree, you're going to have to send out a pull-request. There are no conflicts currently, so I won't use it right away, but I need it in my arsenal if they do occur. -- Lee Jones Linaro STMicroelectronics Landing Team Lead Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog From mboxrd@z Thu Jan 1 00:00:00 1970 From: Lee Jones Date: Wed, 13 Apr 2016 08:25:11 +0000 Subject: Re: [PATCH v5 03/46] backlight: lm3630a_bl: stop messing with the pwm->period field Message-Id: <20160413082511.GL8094@x1> List-Id: References: <1459368249-13241-1-git-send-email-boris.brezillon@free-electrons.com> <1459368249-13241-4-git-send-email-boris.brezillon@free-electrons.com> <20160412110809.GI18882@ulmo.ba.sec> <20160412141613.GI8094@x1> <20160412142608.GB25006@ulmo.ba.sec> In-Reply-To: <20160412142608.GB25006@ulmo.ba.sec> MIME-Version: 1.0 Content-Type: text/plain; charset="windows-1252" Content-Transfer-Encoding: quoted-printable To: Thierry Reding Cc: Milo Kim , Kamil Debski , Heiko Stuebner , linux-doc@vger.kernel.org, David Airlie , Mike Turquette , linux-fbdev@vger.kernel.org, dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org, linux-sunxi@googlegroups.com, Alexandre Belloni , Daniel Vetter , Robert Jarzmik , linux-clk@vger.kernel.org, linux-leds@vger.kernel.org, Boris Brezillon , Krzysztof Kozlowski , linux-samsung-soc@vger.kernel.org, Alexander Shiyan , Jonathan Corbet , lm-sensors@lm-sensors.org, linux-rockchip@lists.infradead.org, Chen-Yu Tsai , Tomi Valkeinen , linux-input@vger.kernel. On Tue, 12 Apr 2016, Thierry Reding wrote: > On Tue, Apr 12, 2016 at 03:16:13PM +0100, Lee Jones wrote: > > On Tue, 12 Apr 2016, Thierry Reding wrote: > >=20 > > > On Wed, Mar 30, 2016 at 10:03:26PM +0200, Boris Brezillon wrote: > > > > pwm->period field is not supposed to be changed by PWM users. The o= nly > > > > ones authorized to change it are the PWM core and PWM drivers. > > > >=20 > > > > Signed-off-by: Boris Brezillon > > > > --- > > > > drivers/video/backlight/lm3630a_bl.c | 3 +-- > > > > 1 file changed, 1 insertion(+), 2 deletions(-) > > >=20 > > > Applied, thanks. > >=20 > > Applied? >=20 > You didn't specifically Ack this one, but I presumed that since the > change is essentially the same as for pwm-backlight, and this is another > prerequisite for the remainder of the series it should go in through the > PWM tree as well. If you're taking Backlight patches through a !Backlight tree, you're going to have to send out a pull-request. There are no conflicts currently, so I won't use it right away, but I need it in my arsenal if they do occur. --=20 Lee Jones Linaro STMicroelectronics Landing Team Lead Linaro.org =E2=94=82 Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog From mboxrd@z Thu Jan 1 00:00:00 1970 From: Lee Jones Subject: Re: [PATCH v5 03/46] backlight: lm3630a_bl: stop messing with the pwm->period field Date: Wed, 13 Apr 2016 09:25:11 +0100 Message-ID: <20160413082511.GL8094@x1> References: <1459368249-13241-1-git-send-email-boris.brezillon@free-electrons.com> <1459368249-13241-4-git-send-email-boris.brezillon@free-electrons.com> <20160412110809.GI18882@ulmo.ba.sec> <20160412141613.GI8094@x1> <20160412142608.GB25006@ulmo.ba.sec> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Content-Disposition: inline In-Reply-To: <20160412142608.GB25006@ulmo.ba.sec> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" To: Thierry Reding Cc: Milo Kim , Kamil Debski , Heiko Stuebner , linux-doc@vger.kernel.org, David Airlie , Mike Turquette , linux-fbdev@vger.kernel.org, dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org, linux-sunxi@googlegroups.com, Alexandre Belloni , Daniel Vetter , Robert Jarzmik , linux-clk@vger.kernel.org, linux-leds@vger.kernel.org, Boris Brezillon , Krzysztof Kozlowski , linux-samsung-soc@vger.kernel.org, Alexander Shiyan , Jonathan Corbet , lm-sensors@lm-sensors.org, linux-rockchip@lists.infradead.org, Chen-Yu Tsai , Tomi Valkeinen , linux-input@vger.kernel. List-Id: linux-input@vger.kernel.org T24gVHVlLCAxMiBBcHIgMjAxNiwgVGhpZXJyeSBSZWRpbmcgd3JvdGU6Cgo+IE9uIFR1ZSwgQXBy IDEyLCAyMDE2IGF0IDAzOjE2OjEzUE0gKzAxMDAsIExlZSBKb25lcyB3cm90ZToKPiA+IE9uIFR1 ZSwgMTIgQXByIDIwMTYsIFRoaWVycnkgUmVkaW5nIHdyb3RlOgo+ID4gCj4gPiA+IE9uIFdlZCwg TWFyIDMwLCAyMDE2IGF0IDEwOjAzOjI2UE0gKzAyMDAsIEJvcmlzIEJyZXppbGxvbiB3cm90ZToK PiA+ID4gPiBwd20tPnBlcmlvZCBmaWVsZCBpcyBub3Qgc3VwcG9zZWQgdG8gYmUgY2hhbmdlZCBi eSBQV00gdXNlcnMuIFRoZSBvbmx5Cj4gPiA+ID4gb25lcyBhdXRob3JpemVkIHRvIGNoYW5nZSBp dCBhcmUgdGhlIFBXTSBjb3JlIGFuZCBQV00gZHJpdmVycy4KPiA+ID4gPiAKPiA+ID4gPiBTaWdu ZWQtb2ZmLWJ5OiBCb3JpcyBCcmV6aWxsb24gPGJvcmlzLmJyZXppbGxvbkBmcmVlLWVsZWN0cm9u cy5jb20+Cj4gPiA+ID4gLS0tCj4gPiA+ID4gIGRyaXZlcnMvdmlkZW8vYmFja2xpZ2h0L2xtMzYz MGFfYmwuYyB8IDMgKy0tCj4gPiA+ID4gIDEgZmlsZSBjaGFuZ2VkLCAxIGluc2VydGlvbigrKSwg MiBkZWxldGlvbnMoLSkKPiA+ID4gCj4gPiA+IEFwcGxpZWQsIHRoYW5rcy4KPiA+IAo+ID4gQXBw bGllZD8KPiAKPiBZb3UgZGlkbid0IHNwZWNpZmljYWxseSBBY2sgdGhpcyBvbmUsIGJ1dCBJIHBy ZXN1bWVkIHRoYXQgc2luY2UgdGhlCj4gY2hhbmdlIGlzIGVzc2VudGlhbGx5IHRoZSBzYW1lIGFz IGZvciBwd20tYmFja2xpZ2h0LCBhbmQgdGhpcyBpcyBhbm90aGVyCj4gcHJlcmVxdWlzaXRlIGZv ciB0aGUgcmVtYWluZGVyIG9mIHRoZSBzZXJpZXMgaXQgc2hvdWxkIGdvIGluIHRocm91Z2ggdGhl Cj4gUFdNIHRyZWUgYXMgd2VsbC4KCklmIHlvdSdyZSB0YWtpbmcgQmFja2xpZ2h0IHBhdGNoZXMg dGhyb3VnaCBhICFCYWNrbGlnaHQgdHJlZSwgeW91J3JlCmdvaW5nIHRvIGhhdmUgdG8gc2VuZCBv dXQgYSBwdWxsLXJlcXVlc3QuICBUaGVyZSBhcmUgbm8gY29uZmxpY3RzCmN1cnJlbnRseSwgc28g SSB3b24ndCB1c2UgaXQgcmlnaHQgYXdheSwgYnV0IEkgbmVlZCBpdCBpbiBteSBhcnNlbmFsCmlm IHRoZXkgZG8gb2NjdXIuCgotLSAKTGVlIEpvbmVzCkxpbmFybyBTVE1pY3JvZWxlY3Ryb25pY3Mg TGFuZGluZyBUZWFtIExlYWQKTGluYXJvLm9yZyDilIIgT3BlbiBzb3VyY2Ugc29mdHdhcmUgZm9y IEFSTSBTb0NzCkZvbGxvdyBMaW5hcm86IEZhY2Vib29rIHwgVHdpdHRlciB8IEJsb2cKX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KSW50ZWwtZ2Z4IG1haWxp bmcgbGlzdApJbnRlbC1nZnhAbGlzdHMuZnJlZWRlc2t0b3Aub3JnCmh0dHBzOi8vbGlzdHMuZnJl ZWRlc2t0b3Aub3JnL21haWxtYW4vbGlzdGluZm8vaW50ZWwtZ2Z4Cg== From mboxrd@z Thu Jan 1 00:00:00 1970 From: lee.jones@linaro.org (Lee Jones) Date: Wed, 13 Apr 2016 09:25:11 +0100 Subject: [PATCH v5 03/46] backlight: lm3630a_bl: stop messing with the pwm->period field In-Reply-To: <20160412142608.GB25006@ulmo.ba.sec> References: <1459368249-13241-1-git-send-email-boris.brezillon@free-electrons.com> <1459368249-13241-4-git-send-email-boris.brezillon@free-electrons.com> <20160412110809.GI18882@ulmo.ba.sec> <20160412141613.GI8094@x1> <20160412142608.GB25006@ulmo.ba.sec> Message-ID: <20160413082511.GL8094@x1> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Tue, 12 Apr 2016, Thierry Reding wrote: > On Tue, Apr 12, 2016 at 03:16:13PM +0100, Lee Jones wrote: > > On Tue, 12 Apr 2016, Thierry Reding wrote: > > > > > On Wed, Mar 30, 2016 at 10:03:26PM +0200, Boris Brezillon wrote: > > > > pwm->period field is not supposed to be changed by PWM users. The only > > > > ones authorized to change it are the PWM core and PWM drivers. > > > > > > > > Signed-off-by: Boris Brezillon > > > > --- > > > > drivers/video/backlight/lm3630a_bl.c | 3 +-- > > > > 1 file changed, 1 insertion(+), 2 deletions(-) > > > > > > Applied, thanks. > > > > Applied? > > You didn't specifically Ack this one, but I presumed that since the > change is essentially the same as for pwm-backlight, and this is another > prerequisite for the remainder of the series it should go in through the > PWM tree as well. If you're taking Backlight patches through a !Backlight tree, you're going to have to send out a pull-request. There are no conflicts currently, so I won't use it right away, but I need it in my arsenal if they do occur. -- Lee Jones Linaro STMicroelectronics Landing Team Lead Linaro.org ? Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog