Re: [PATCH 1/3] thermal: of-thermal: Add devm version of thermal_zone_of_sensor_register

From: Laxman Dewangan
Date: Wed Mar 09 2016 - 07:23:47 EST


Hi Edurado,
Thanks for review.


On Wednesday 09 March 2016 02:59 AM, Eduardo Valentin wrote:
Hello Laxman,


Minor as follows.

Can you please run ./scripts/checkpatch.pl --strict on this and remove
the warnings, errors, checks?

Taken from existing function and so borrowed the error. Will fix in next revision.



On Fri, Mar 04, 2016 at 07:10:08PM +0530, Laxman Dewangan wrote:
Add resource managed version of thermal_zone_of_sensor_register() and
thermal_zone_of_sensor_unregister().

This helps in reducing the code size in error path, remove of
driver remove callbacks and making proper sequence for deallocations.

Signed-off-by: Laxman Dewangan <ldewangan@xxxxxxxxxx>
---
drivers/thermal/of-thermal.c | 81 ++++++++++++++++++++++++++++++++++++++++++++
include/linux/thermal.h | 18 ++++++++++
Could you also please document these under
Documentation/thermal/sysfs-api.txt?


Sure, I saw that thermal_zone_of_sensor_register/unregister is also not documented.
So I will add the details for this first and then add for newly added interfaces.

Will take care in next series.