[PATCH] drm/radeon/kms: add support for internal thermal sensors (v3)

Frieder Ferlemann frieder.ferlemann at web.de
Wed Jun 2 14:46:51 PDT 2010


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


More information about the dri-devel mailing list