Re: [PATCH 0/6] thermal: Align devm_thermal_zone_{device,of_sensor}_register

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Dec 12, 2018 at 02:49:21AM +0100, Marek Vasut wrote:
> This patchset attempts to align devm_thermal_zone_{device,of_sensor}_register()
> functions and allow passing tz params to devm_thermal_zone_of_sensor_register()
> to allow settings no_hwmon params without digging into the tz structures.
> 
> The approach is as follows:
> 1) Add {devm_,}thermal_zone_of_sensor_register_params(), which behaves just
>    like the old variant without _params() suffix, except it takes the struct
>    thermal_zone_params as an argument. The old _params()-less version of the
>    function is converted to use the new function and pass params = NULL.
>    The old _params()-less versions of the functions are retained to avoid
>    breaking the ABI.
> 2) Add the no_hwmon handling into thermal_zone_of_sensor_register_params(),
>    that's patch 3.
> 3) Convert drivers from devm_thermal_zone_of_sensor_register() with structure
>    diggings to devm_thermal_zone_of_sensor_register_params() without any
>    extra structure digging, that's the rest of the patches.
> 
> Marek Vasut (6):
>   thermal: split thermal_zone_of_sensor_register{,_param}()
>   thermal: split devm_thermal_zone_of_sensor_register{,_param}()
>   thermal: Register hwmon in thermal_zone_of_sensor_register_param()
>   thermal: stm32: Convert to
>     devm_thermal_zone_of_sensor_register_params()
>   thermal: rcar_thermal: Convert to
>     devm_thermal_zone_of_sensor_register_params()
>   thermal: rcar_gen3_thermal: Register hwmon sysfs interface
> 

Fix the errors reported by the bot and also please run checkpatch.pl on
your series.

>  drivers/thermal/of-thermal.c        | 104 +++++++++++++++++++++++++---
>  drivers/thermal/rcar_gen3_thermal.c |   8 ++-
>  drivers/thermal/rcar_thermal.c      |  33 ++++-----
>  drivers/thermal/st/stm_thermal.c    |  20 ++----
>  include/linux/thermal.h             |  25 +++++++
>  5 files changed, 144 insertions(+), 46 deletions(-)
> 
> Cc: Daniel Lezcano <daniel.lezcano@xxxxxxxxxx>
> Cc: Eduardo Valentin <edubezval@xxxxxxxxx>
> Cc: Wolfram Sang <wsa+renesas@xxxxxxxxxxxxxxxxxxxx>
> Cc: Zhang Rui <rui.zhang@xxxxxxxxx>
> Cc: linux-renesas-soc@xxxxxxxxxxxxxxx
> To: linux-pm@xxxxxxxxxxxxxxx
> 
> -- 
> 2.18.0
> 



[Index of Archives]     [Linux Samsung SOC]     [Linux Wireless]     [Linux Kernel]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]

  Powered by Linux