Re: [PATCH 4/4] pwm: rcar: Remove suspend/resume support

2019-05-29 Thread Thierry Reding
On Wed, May 29, 2019 at 04:48:02PM +0900, Yoshihiro Shimoda wrote: > According to the Documentation/pwm.txt, all PWM consumers should > implement power management instead of the PWM driver. So, this > patch removes suspend/resume support. > > Signed-off-by: Yoshihiro Shimoda > --- > drivers/pwm/

Re: [PATCH 3/4] pwm: sysfs: Add suspend/resume support

2019-05-29 Thread Thierry Reding
On Wed, May 29, 2019 at 04:48:01PM +0900, Yoshihiro Shimoda wrote: > According to the Documentation/pwm.txt, all PWM consumers should have > power management. Since this sysfs interface is one of consumers so that > this patch adds suspend/resume support. > > Signed-off-by: Yoshihiro Shimoda > --

Re: [PATCH 0/4] pwm: add power management on sysfs and switch to SPDX

2019-05-29 Thread Thierry Reding
On Wed, May 29, 2019 at 04:47:58PM +0900, Yoshihiro Shimoda wrote: > This patch series is affected from the following email: > https://marc.info/?l=linux-renesas-soc&m=155896668906609&w=2 > > - The patch 1 adds descriptions into Documentation/pwm.txt. > - The patch 2 is not related to the topic th

Re: [PATCH v5] pwm: renesas-tpu: Add suspend/resume function

2019-05-27 Thread Thierry Reding
On Mon, May 27, 2019 at 11:22:37AM +0900, Cao Van Dong wrote: > This patch adds suspend/resume function support for Renesas the 16-Bit Timer > Pulse Unit (TPU) driver. This has been tested on the Salvator-XS board > with R-Car M3-N and H3 at renesas-drivers-2019-05-21-v5.2-rc1 tag. > I expect this

Re: [PATCH v3] pwm: Fix deadlock warning when removing PWM device

2019-03-20 Thread Thierry Reding
On Tue, Mar 19, 2019 at 07:40:08PM +0900, Yoshihiro Shimoda wrote: > From: Phong Hoang > > This patch fixes deadlock warning if removing PWM device > when CONFIG_PROVE_LOCKING is enabled. > > This issue can be reproceduced by the following steps on > the R-Car H3 Salvator-X board if the backligh

Re: [PATCH] pwm: Avoid deadlock warning when removing PWM device

2019-03-12 Thread Thierry Reding
On Tue, Mar 12, 2019 at 12:16:34PM +0900, Yoshihiro Shimoda wrote: > From: Phong Hoang > > This patch fixes deadlock warning if removing PWM device > when CONFIG_PROVE_LOCKING is enabled. > > This issue can be reproceduced by the following steps on > the R-Car H3 Salvator-X board if the backligh

Re: [PATCH] pwm: Avoid deadlock warning when removing PWM device

2019-03-12 Thread Thierry Reding
On Tue, Mar 12, 2019 at 12:16:34PM +0900, Yoshihiro Shimoda wrote: > From: Phong Hoang > > This patch fixes deadlock warning if removing PWM device > when CONFIG_PROVE_LOCKING is enabled. > > This issue can be reproceduced by the following steps on > the R-Car H3 Salvator-X board if the backligh

Re: [PATCH] pwm: Avoid deadlock warning when removing PWM device

2019-03-12 Thread Thierry Reding
On Tue, Mar 12, 2019 at 10:54:33AM +0100, Uwe Kleine-König wrote: > On Tue, Mar 12, 2019 at 10:49:59AM +0100, Geert Uytterhoeven wrote: > > Hi Uwe, > > > > On Tue, Mar 12, 2019 at 10:23 AM Uwe Kleine-König > > wrote: > > > On Tue, Mar 12, 2019 at 12:16:34PM +0900, Yoshihiro Shimoda wrote: > > > >

Re: [RFC PATCH v2 6/7] i2c: tegra-bpmp: convert to use new atomic callbacks

2019-03-04 Thread Thierry Reding
, 22 insertions(+), 5 deletions(-) Acked-by: Thierry Reding signature.asc Description: PGP signature

Re: [PATCH v3 2/4] pwm: rcar: Use "atomic" API on rcar_pwm_resume()

2019-03-04 Thread Thierry Reding
On Wed, Jan 09, 2019 at 05:19:06PM +0900, Yoshihiro Shimoda wrote: > To remove legacy API related functions in the future, this patch > uses "atomic" related function instead. No change in behavior. > > Signed-off-by: Yoshihiro Shimoda > Acked-by: Uwe Kleine-König > --- > drivers/pwm/pwm-rcar.c

Re: [PATCH v3 0/4] pwm: rcar: Add support "atomic" API and improve calculation

2019-03-04 Thread Thierry Reding
On Wed, Jan 09, 2019 at 05:19:04PM +0900, Yoshihiro Shimoda wrote: > This patch series adds support for PWM "atomic" API and improve > the clock calculation. > > Changes from v2 [1]: > - Add Acked-by of Uwe in patch [234]/4. > - Modify a condition to support normal polarity only. > - Fix possib

Re: [PATCH] gpio: of: Apply regulator-gpio quirk only to enable-gpios

2019-02-20 Thread Thierry Reding
On Wed, Feb 20, 2019 at 10:01:50AM +0100, Linus Walleij wrote: > On Tue, Feb 19, 2019 at 5:30 PM Marek Vasut wrote: > > > Handling broken DTs only adds to the complexity, but I think this cannot > > be helped, since those DTs can be stored in some ROM. > > I agree, but if the broken DT needs to

Re: [PATCH] gpio: of: Apply regulator-gpio quirk only to enable-gpios

2019-02-19 Thread Thierry Reding
On Tue, Feb 19, 2019 at 05:30:35PM +0100, Marek Vasut wrote: > On 2/19/19 5:08 PM, Thierry Reding wrote: > > On Tue, Feb 19, 2019 at 04:25:59PM +0100, Marek Vasut wrote: > >> On 2/19/19 4:18 PM, Thierry Reding wrote: > >>> On Sat, Feb 16, 2019 at 02:46:27PM +010

Re: [PATCH] gpio: of: Apply regulator-gpio quirk only to enable-gpios

2019-02-19 Thread Thierry Reding
On Tue, Feb 19, 2019 at 04:25:59PM +0100, Marek Vasut wrote: > On 2/19/19 4:18 PM, Thierry Reding wrote: > > On Sat, Feb 16, 2019 at 02:46:27PM +0100, marek.va...@gmail.com wrote: > >> From: Marek Vasut > >> > >> Since commit d6cd33ad7102 ("regulator: gp

Re: [PATCH] gpio: of: Apply regulator-gpio quirk only to enable-gpios

2019-02-19 Thread Thierry Reding
On Sat, Feb 16, 2019 at 02:46:27PM +0100, marek.va...@gmail.com wrote: > From: Marek Vasut > > Since commit d6cd33ad7102 ("regulator: gpio: Convert to use descriptors") > the GPIO regulator had inverted the polarity of the control GPIO. This > problem manifested itself on systems with DT containi

Re: [PATCH] dt-bindings: pwm: rcar: Add r8a774c0 support

2018-12-20 Thread Thierry Reding
On Thu, Dec 13, 2018 at 08:20:15PM +, Fabrizio Castro wrote: > Document RZ/G2E (R8A774C0) SoC bindings. > > Signed-off-by: Fabrizio Castro > --- > Documentation/devicetree/bindings/pwm/renesas,pwm-rcar.txt | 1 + > 1 file changed, 1 insertion(+) Applied, thanks. Thierry signature.asc Des

Re: [PATCH 7/7] drm: Split out drm_probe_helper.h

2018-12-10 Thread Thierry Reding
ont.c | 2 +- > drivers/gpu/drm/xen/xen_drm_front_conn.c | 2 +- > drivers/gpu/drm/xen/xen_drm_front_gem.c | 2 +- > drivers/gpu/drm/xen/xen_drm_front_kms.c | 2 +- > drivers/gpu/drm/zte/zx_drm_drv.c | 2 +- > drivers/gpu/drm/zte/zx_hdmi.c | 2 +- > drivers/gpu/drm/zte/zx_tvenc.c| 2 +- > drivers/gpu/drm/zte/zx_vga.c | 2 +- > drivers/gpu/drm/zte/zx_vou.c | 2 +- > drivers/staging/vboxvideo/vbox_irq.c | 2 +- > drivers/staging/vboxvideo/vbox_mode.c | 2 +- > include/drm/drm_crtc_helper.h | 16 -- > include/drm/drm_probe_helper.h| 50 +++ > 208 files changed, 256 insertions(+), 200 deletions(-) > create mode 100644 include/drm/drm_probe_helper.h Looks good to me: Acked-by: Thierry Reding signature.asc Description: PGP signature

Re: [PATCH 5/6] gpio: gpio-tegra: simplify getting .driver_data

2018-10-22 Thread Thierry Reding
uildbot is happy. > > drivers/gpio/gpio-tegra.c | 6 ++ > 1 file changed, 2 insertions(+), 4 deletions(-) Acked-by: Thierry Reding signature.asc Description: PGP signature

Re: [PATCH] dt-bindings: pwm: renesas: pwm-rcar: document R8A779{7|8}0 bindings

2018-10-16 Thread Thierry Reding
On Mon, Oct 15, 2018 at 01:57:13PM -0500, Rob Herring wrote: > On Fri, Oct 12, 2018 at 01:21:14PM +0200, Thierry Reding wrote: > > On Mon, Oct 01, 2018 at 10:57:39PM +0300, Sergei Shtylyov wrote: > > > Document the R-Car V3{M|H} (R8A779{7|8}0) SoC in the Renesas R-Car PWM > &

Re: [PATCH] dt-bindings: pwm: rcar: Add r8a774a1 support

2018-10-12 Thread Thierry Reding
On Tue, Aug 21, 2018 at 05:03:46PM +0100, Fabrizio Castro wrote: > Document RZ/G2M (R8A774A1) SoC bindings. > > Signed-off-by: Fabrizio Castro > Reviewed-by: Biju Das > --- > Documentation/devicetree/bindings/pwm/renesas,pwm-rcar.txt | 1 + > 1 file changed, 1 insertion(+) Applied, thanks. Th

Re: [PATCH] dt-bindings: pwm: renesas-tpu: Document r8a7744 support

2018-10-12 Thread Thierry Reding
On Thu, Oct 04, 2018 at 05:21:34PM +0100, Biju Das wrote: > Document r8a7744 specific compatible strings. No driver change is > needed as the fallback compatible string "renesas,tpu" activates the > right code in the driver. > > Signed-off-by: Biju Das > Reviewed-by: Chris Paterson > --- > This

Re: [PATCH] dt-bindings: pwm: rcar: Add r8a7744 support

2018-10-12 Thread Thierry Reding
On Thu, Oct 04, 2018 at 05:17:19PM +0100, Biju Das wrote: > Document RZ/G1N (R8A7744) SoC bindings. > > Signed-off-by: Biju Das > Reviewed-by: Chris Paterson > --- > This patch is tested against next-20181004 > --- > Documentation/devicetree/bindings/pwm/renesas,pwm-rcar.txt | 1 + > 1 file cha

Re: [PATCH] dt-bindings: pwm: renesas: pwm-rcar: document R8A779{7|8}0 bindings

2018-10-12 Thread Thierry Reding
On Mon, Oct 01, 2018 at 10:57:39PM +0300, Sergei Shtylyov wrote: > Document the R-Car V3{M|H} (R8A779{7|8}0) SoC in the Renesas R-Car PWM > bindings. R8A77970's hardware is a generic R-Car gen3 PWM, while R8A77980 > has an extra error injection register... > > Signed-off-by: Sergei Shtylyov > >

Re: [PATCH] dt-bindings: pwm: renesas: tpu: document R8A779{7|8}0 bindings

2018-10-12 Thread Thierry Reding
On Sat, Sep 22, 2018 at 10:57:29PM +0300, Sergei Shtylyov wrote: > Document the R-Car V3{M|H} (R8A779{7|8}0) SoC in the Renesas TPU bindings; > the TPU hardware in those is the Renesas standard 4-channel timer pulse > unit. > > Signed-off-by: Sergei Shtylyov > > --- > This patch is against Linus

Re: [PATCH v2] dt-bindings: pwm: renesas: tpu: fix "compatible" prop description

2018-10-12 Thread Thierry Reding
On Sat, Sep 22, 2018 at 10:43:24PM +0300, Sergei Shtylyov wrote: > The "compatible" property description contradicts even the example given: > it only says that there must be a single value while the example has the > fallback value too -- which makes much more sense. Moreover, the generic > proper

Re: [PATCH resend] pwm: renesas-tpu: convert to SPDX identifiers

2018-10-12 Thread Thierry Reding
On Wed, Sep 26, 2018 at 01:45:47AM +, Kuninori Morimoto wrote: > > From: Kuninori Morimoto > > This patch updates license to use SPDX-License-Identifier > instead of verbose license text. > > Signed-off-by: Kuninori Morimoto > Reviewed-by: Simon Horman > --- > Thierry > > 2weeks past, re

Re: [PATCH resend] pwm: rcar: convert to SPDX identifiers

2018-10-12 Thread Thierry Reding
On Wed, Sep 26, 2018 at 01:45:17AM +, Kuninori Morimoto wrote: > > From: Kuninori Morimoto > > This patch updates license to use SPDX-License-Identifier > instead of verbose license text. > > Signed-off-by: Kuninori Morimoto > Reviewed-by: Simon Horman > --- > Thierry > > 2weeks past, re

Re: [PATCH v2] dt-bindings: pwm: renesas: tpu: fix "compatible" prop description

2018-09-24 Thread Thierry Reding
On Sat, Sep 22, 2018 at 10:43:24PM +0300, Sergei Shtylyov wrote: > The "compatible" property description contradicts even the example given: > it only says that there must be a single value while the example has the > fallback value too -- which makes much more sense. Moreover, the generic > proper

Re: [PATCH] dt-bindings: pwm: rcar: Add bindings for R-Car E3 support

2018-08-20 Thread Thierry Reding
On Mon, Jul 30, 2018 at 08:49:51PM +0900, Yoshihiro Shimoda wrote: > This patch adds bindings for R-Car E3. No driver update is needed. > > Signed-off-by: Yoshihiro Shimoda > --- > Documentation/devicetree/bindings/pwm/renesas,pwm-rcar.txt | 1 + > 1 file changed, 1 insertion(+) Applied, thanks

Re: [PATCH 10/61] gpio: simplify getting .drvdata

2018-04-30 Thread Thierry Reding
On Mon, Apr 30, 2018 at 11:03:25AM +0200, Linus Walleij wrote: > On Thu, Apr 19, 2018 at 5:14 PM, Grygorii Strashko > wrote: > > On 04/19/2018 09:05 AM, Wolfram Sang wrote: > >> > >> We should get drvdata from struct device directly. Going via > >> platform_device is an unneeded step back and fort

Re: [PATCH 43/61] pwm: simplify getting .drvdata

2018-04-30 Thread Thierry Reding
On Thu, Apr 19, 2018 at 04:06:13PM +0200, Wolfram Sang wrote: > We should get drvdata from struct device directly. Going via > platform_device is an unneeded step back and forth. > > Signed-off-by: Wolfram Sang > --- > > Build tested only. buildbot is happy. Please apply individually. > > driv

Re: [PATCH] drm/core: Remove drm_dev_unref() and it's uses

2018-04-26 Thread Thierry Reding
cases of drm_dev_unref() with drm_dev_put and remove > the function itself. > > Coccinelle was used for mass-patching. > > Signed-off-by: Vaishali Thakkar Yes, please. Acked-by: Thierry Reding signature.asc Description: PGP signature

Re: [PATCH v2 0/2] pwm: rcar: Add suspend/resume support and cleanup for runtime_pm

2018-03-27 Thread Thierry Reding
On Tue, Mar 13, 2018 at 05:18:16PM +0900, Yoshihiro Shimoda wrote: > This patch set improves power management for Renesas PWM driver. > > Changes from v1: > - Add Reviewed-by Geert-san in patch 1 (Thanks!). > - Check a condition in suspend/resume if requested or not in patch 2. > > Hien Dang (1

Re: [PATCH] dt-bindings: pwm: renesas,pwm-rcar: add bindings for R-Car M3N support

2018-03-27 Thread Thierry Reding
On Fri, Mar 09, 2018 at 08:53:17PM +0900, Yoshihiro Shimoda wrote: > This patch adds bindings for R-Car M3N. No driver update is needed. > > Signed-off-by: Yoshihiro Shimoda > --- > Documentation/devicetree/bindings/pwm/renesas,pwm-rcar.txt | 1 + > 1 file changed, 1 insertion(+) Applied, thank

Re: [PATCH] pwm: rcar: fix a condition to prevent mismatch value setting to duty

2018-03-27 Thread Thierry Reding
On Fri, Mar 09, 2018 at 08:24:21PM +0900, Yoshihiro Shimoda wrote: > From: Ryo Kodama > > This patch fixes an issue that is possible to set mismatch value > to duty for R-Car PWM if we input the following commands: > > # cd /sys/class/pwm// > # echo 0 > export > # cd pwm0 > # echo 30 > perio

Re: [PATCH v4] dt-bindings: pwm: rcar: Document r8a774[35] PWM bindings

2018-03-27 Thread Thierry Reding
On Wed, Dec 20, 2017 at 11:15:43AM +, Fabrizio Castro wrote: > This patch adds compatible strings specific to r8a774[35], no driver > change is needed as the fallback compatible string will activate the > right code. > > Also, this patch replaces the example with a DT snippet used > for adding

Re: [PATCH v3 4/5] dt-bindings: pwm: renesas-tpu: Document r8a774[35] support

2018-03-27 Thread Thierry Reding
On Tue, Dec 19, 2017 at 01:34:59PM +, Fabrizio Castro wrote: > Document r8a774[35] specific compatible strings. No driver change is > needed as the fallback compatible string "renesas,tpu" activates the > right code in the driver. > > Signed-off-by: Fabrizio Castro > Reviewed-by: Biju Das >

Re: [PATCH repost2] pwm: renesas-tpu: remove support for sh7372

2017-08-18 Thread Thierry Reding
On Fri, Aug 18, 2017 at 12:09:01PM +0200, Simon Horman wrote: > Remove support for the SH7372 (SH-Mobile AP4) from the renesas-tpu driver. > > Commit edf4100906044225 ("ARM: shmobile: sh7372 dtsi: Remove Legacy file") > removes this SoC from the kernel in v4.1. > > Acked-by: Geert Uytterhoeven >

Re: [PATCH v4] dt-bindings: pwm: Add R-Car M3-W device tree bindings

2017-07-05 Thread Thierry Reding
On Fri, May 12, 2017 at 10:09:42AM +0200, Simon Horman wrote: > From: Ulrich Hecht > > Add device tree bindings for the PWM controller found on R-Car M3-W SoCs. > > Signed-off-by: Ulrich Hecht > --- > v4 > * Broke out of lager patchset on which it appears to have no dependencies > * Added Revie

Re: [PATCH v2 05/13] drm: panels: Constify device node argument to of_drm_find_panel()

2017-01-03 Thread Thierry Reding
On Sat, Nov 19, 2016 at 05:28:05AM +0200, Laurent Pinchart wrote: > The argument is never modified by the function, make it const. > > Signed-off-by: Laurent Pinchart > --- > drivers/gpu/drm/drm_panel.c | 2 +- > include/drm/drm_panel.h | 4 ++-- > 2 files changed, 3 insertions(+), 3 deletio

Re: [PATCH v2.1 04/13] drm: Add data transmission order bus flag

2017-01-03 Thread Thierry Reding
RM_BUS_FLAG_DATA_MSB_TO_LSB (1<<4) > +/* data is transmitted lsb to msb on the bus */ > +#define DRM_BUS_FLAG_DATA_LSB_TO_MSB (1<<5) Nit: "LSB" and "MSB" because they're abbreviations. If I end up applying this I'll probably do that myself, and I leave

Re: [PATCH v2 06/13] drm: panels: Add LVDS panel driver

2016-11-22 Thread Thierry Reding
On Sat, Nov 19, 2016 at 05:28:06AM +0200, Laurent Pinchart wrote: > This driver supports LVDS panels that don't require device-specific > handling of power supplies or control signals. It implements automatic > backlight handling if the panel is attached to a backlight controller. > > Signed-off-b

Re: [PATCH v2 01/13] devicetree/bindings: display: Document common panel properties

2016-11-22 Thread Thierry Reding
On Sat, Nov 19, 2016 at 05:28:01AM +0200, Laurent Pinchart wrote: > Document properties common to several display panels in a central > location that can be referenced by the panel device tree bindings. > > Signed-off-by: Laurent Pinchart > --- > .../bindings/display/panel/panel-common.txt

Re: [PATCH v2 02/13] devicetree/bindings: display: Add bindings for LVDS panels

2016-11-22 Thread Thierry Reding
On Sat, Nov 19, 2016 at 05:28:02AM +0200, Laurent Pinchart wrote: > LVDS is a physical layer specification defined in ANSI/TIA/EIA-644-A. > Multiple incompatible data link layers have been used over time to > transmit image data to LVDS panels. This binding supports display panels > compatible with

Re: [PATCH v3 5/6] pwm: add R-Car H3 device tree bindings

2016-07-11 Thread Thierry Reding
On Thu, Mar 31, 2016 at 01:39:15PM +0200, Ulrich Hecht wrote: > Signed-off-by: Ulrich Hecht > Acked-by: Geert Uytterhoeven > --- > Documentation/devicetree/bindings/pwm/renesas,pwm-rcar.txt | 1 + > 1 file changed, 1 insertion(+) Applied, thanks. Thierry signature.asc Description: PGP signat

Re: [PATCH v3 1/6] pwm: rcar: Use ARCH_RENESAS

2016-07-11 Thread Thierry Reding
On Thu, Mar 31, 2016 at 01:39:11PM +0200, Ulrich Hecht wrote: > From: Ryo Kodama > > Replace ARCH_RCAR_GEN{1,2} with ARCH_RENESAS in order to support R-Car Gen3. > > Signed-off-by: Ryo Kodama > Signed-off-by: Harunobu Kurokawa > Signed-off-by: Ulrich Hecht > Acked-by: Geert Uytterhoeven > --

Re: [PATCH v2] pwm: improve args checking in pwm_apply_state()

2016-06-23 Thread Thierry Reding
On Wed, Jun 22, 2016 at 01:46:48PM -0700, Brian Norris wrote: > On Wed, Jun 22, 2016 at 10:41:14PM +0200, Boris Brezillon wrote: > > On Wed, 22 Jun 2016 12:16:59 -0700 > > Brian Norris wrote: > > > Notably, you're dropping the 'if (!pwm) { }' safety checks that are part > > > of pwm_disable() and

Re: [PATCH v2] pwm: improve args checking in pwm_apply_state()

2016-06-22 Thread Thierry Reding
On Wed, Jun 22, 2016 at 10:04:22AM +0200, Boris Brezillon wrote: > On Tue, 21 Jun 2016 11:37:31 -0700 > Brian Norris wrote: > > > Hi Geert, > > > > On Tue, Jun 21, 2016 at 04:42:04PM +0200, Geert Uytterhoeven wrote: > > > On Fri, May 27, 2016 at 6:45 PM, Brian Norris > > > wrote: > > > > It

Re: [PATCH] pwm: sysfs: get return value from pwm_apply_state()

2016-06-10 Thread Thierry Reding
On Wed, Jun 08, 2016 at 10:58:23AM +0900, Yoshihiro Shimoda wrote: > From: Ryo Kodama > > This patch adds to check the return value from pwm_apply_state() > used in enable_store(). The error of enable_store() doesn't work > if the return value doesn't received. > > Signed-off-by: Ryo Kodama > S

Re: [PATCH] pwm: rcar: Use ARCH_RENESAS

2016-03-04 Thread Thierry Reding
On Thu, Feb 25, 2016 at 10:03:23AM +0900, Simon Horman wrote: > Make use of ARCH_RENESAS in place of ARCH_SHMOBILE. > > This is part of an ongoing process to migrate from ARCH_SHMOBILE to > ARCH_RENESAS the motivation for which being that RENESAS seems to be a more > appropriate name than SHMOBILE

Re: [PATCH 3/6] pwm: pwm-img: test clock rate to avoid division by 0

2016-03-04 Thread Thierry Reding
On Wed, Mar 02, 2016 at 11:33:34PM +0100, Wolfram Sang wrote: > From: Wolfram Sang > > The clk API may return 0 on clk_get_rate, so we should check the result before > using it as a divisor. > > Signed-off-by: Wolfram Sang > --- > > Should go individually via subsystem tree. > > drivers/pwm/