Re: [PATCHv2 0/3] thermal: lookup temperature
Hello Pavel, On 22-04-2013 07:06, Pavel Machek wrote: Hi! Here is V2 of temperature lookup helper function. This has been split into two API as suggested on V1. The usage of it is exemplified on patch 03. Eduardo Valentin (3): thermal: introduce thermal_zone_get_zone_by_name helper function Is the function name maybe a bit long? Perhaps using tz_ prefix instead of "thermal_zone_" would help? or thermal_get_by_name? tz_get_by_name? Pavel This patch follows the naming convention on this file. If we are going to use different prefixes, then I suggest sending a patch that changes the functions that are exported by this file. Besides the names you suggested do not reflect exactly the purpose of the introduced API. Thanks for your suggestion. All best, Thanks. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCHv2 0/3] thermal: lookup temperature
Hi! > Here is V2 of temperature lookup helper function. This has been > split into two API as suggested on V1. > > The usage of it is exemplified on patch 03. > > > Eduardo Valentin (3): > thermal: introduce thermal_zone_get_zone_by_name helper function Is the function name maybe a bit long? Perhaps using tz_ prefix instead of "thermal_zone_" would help? or thermal_get_by_name? tz_get_by_name? Pavel -- (english) http://www.livejournal.com/~pavelmachek (cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[PATCHv2 0/3] thermal: lookup temperature
Hello Rui, Here is V2 of temperature lookup helper function. This has been split into two API as suggested on V1. The usage of it is exemplified on patch 03. Eduardo Valentin (3): thermal: introduce thermal_zone_get_zone_by_name helper function thermal: expose thermal_zone_get_temp API staging: ti-soc-thermal: remove external heat while extrapolating hotspot drivers/staging/ti-soc-thermal/ti-thermal-common.c | 30 +++ drivers/thermal/thermal_sys.c | 54 ++- include/linux/thermal.h|2 + 3 files changed, 73 insertions(+), 13 deletions(-) -- 1.7.7.1.488.ge8e1c -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/