* Re: Thermal: CPU Package temperature thermal
[not found] <20130711211543.0E83166098F@gitolite.kernel.org>
@ 2013-07-11 23:08 ` Dave Jones
2013-07-11 23:26 ` Srinivas Pandruvada
0 siblings, 1 reply; 2+ messages in thread
From: Dave Jones @ 2013-07-11 23:08 UTC (permalink / raw)
To: Linux Kernel Mailing List; +Cc: rui.zhang, srinivas.pandruvada
> +static int sys_get_curr_temp(struct thermal_zone_device *tzd, unsigned long *temp)
> +static int sys_get_trip_temp(struct thermal_zone_device *tzd,
> +int sys_set_trip_temp(struct thermal_zone_device *tzd, int trip,
> +static int sys_get_trip_type(struct thermal_zone_device *thermal,
I saw these and thought "huh, syscalls in a module?"
Perhaps a better prefix can be used here for these function names ?
Dave
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: Thermal: CPU Package temperature thermal
2013-07-11 23:08 ` Thermal: CPU Package temperature thermal Dave Jones
@ 2013-07-11 23:26 ` Srinivas Pandruvada
0 siblings, 0 replies; 2+ messages in thread
From: Srinivas Pandruvada @ 2013-07-11 23:26 UTC (permalink / raw)
To: Dave Jones, Linux Kernel Mailing List, rui.zhang
On 07/11/2013 04:08 PM, Dave Jones wrote:
> > +static int sys_get_curr_temp(struct thermal_zone_device *tzd, unsigned long *temp)
> > +static int sys_get_trip_temp(struct thermal_zone_device *tzd,
> > +int sys_set_trip_temp(struct thermal_zone_device *tzd, int trip,
> > +static int sys_get_trip_type(struct thermal_zone_device *thermal,
>
> I saw these and thought "huh, syscalls in a module?"
>
> Perhaps a better prefix can be used here for these function names ?
>
> Dave
>
>
Certainly. Here "sys" stands for "sys-fs".
Thanks,
Srinivas
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2013-07-11 23:19 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <20130711211543.0E83166098F@gitolite.kernel.org>
2013-07-11 23:08 ` Thermal: CPU Package temperature thermal Dave Jones
2013-07-11 23:26 ` Srinivas Pandruvada
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox