Hi, Am 02.06.2010 18:59, schrieb Alex Deucher: > +/* get temperature in millidegrees */ > +u32 evergreen_get_temp(struct radeon_device *rdev) Could it be named evergreen_get_temp_mcelsius_clipped_to_above_zero ?^) Seriously: would it make sense to have: u32 evergreen_get_temp_mkelvin(struct radeon_device *rdev) With all that it implies? Just that it's there?) Greetings, Frieder _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/dri-devel