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

[Thierry Reding <thierry.reding@gmail.com>]

[-- Attachment #1: Type: text/plain, Size: 1203 bytes --]

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 though, switches to
>   SPDX identifier of sysfs.c.
> - The patch 3 adds suspend/resume support into sysfs.c.
> - The patch 4 removes suspend/resume support from pwm-rcar.c.
> 
> This is the first trial so that I didn't make patches for other
> PWM drivers to remove suspend/resume support.

Thanks for doing this. It's looking already pretty good. Just a couple
of minor comments in the individual patches.

Thierry

> 
> Yoshihiro Shimoda (4):
>   pwm: Add power management descriptions
>   pwm: sysfs: Switch to SPDX identifier
>   pwm: sysfs: Add suspend/resume support
>   pwm: rcar: Remove suspend/resume support
> 
>  Documentation/pwm.txt  |  7 +++++
>  drivers/pwm/pwm-rcar.c | 39 --------------------------
>  drivers/pwm/sysfs.c    | 75 +++++++++++++++++++++++++++++++++++++++++++-------
>  3 files changed, 72 insertions(+), 49 deletions(-)
> 
> -- 
> 2.7.4
> 

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]