Colin King
2017-Jun-27 08:08 UTC
[Nouveau] [PATCH] drm/nouveau/therm: fix spelling mistake on array thresolds
From: Colin Ian King <colin.king at canonical.com> Array thresolds should be named thresholds, rename it. Also make it static static const char * const Signed-off-by: Colin Ian King <colin.king at canonical.com> --- drivers/gpu/drm/nouveau/nvkm/subdev/therm/temp.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/gpu/drm/nouveau/nvkm/subdev/therm/temp.c b/drivers/gpu/drm/nouveau/nvkm/subdev/therm/temp.c index e93b2410c38b..6449771b9dc6 100644 --- a/drivers/gpu/drm/nouveau/nvkm/subdev/therm/temp.c +++ b/drivers/gpu/drm/nouveau/nvkm/subdev/therm/temp.c @@ -83,7 +83,7 @@ nvkm_therm_sensor_event(struct nvkm_therm *therm, enum nvkm_therm_thrs thrs, { struct nvkm_subdev *subdev = &therm->subdev; bool active; - const char *thresolds[] = { + static const char * const thresholds[] = { "fanboost", "downclock", "critical", "shutdown" }; int temperature = therm->func->temp_get(therm); @@ -94,10 +94,10 @@ nvkm_therm_sensor_event(struct nvkm_therm *therm, enum nvkm_therm_thrs thrs, if (dir == NVKM_THERM_THRS_FALLING) nvkm_info(subdev, "temperature (%i C) went below the '%s' threshold\n", - temperature, thresolds[thrs]); + temperature, thresholds[thrs]); else nvkm_info(subdev, "temperature (%i C) hit the '%s' threshold\n", - temperature, thresolds[thrs]); + temperature, thresholds[thrs]); active = (dir == NVKM_THERM_THRS_RISING); switch (thrs) { -- 2.11.0
Martin Peres
2017-Jun-27 10:07 UTC
[Nouveau] [PATCH] drm/nouveau/therm: fix spelling mistake on array thresolds
On 27/06/17 11:08, Colin King wrote:> From: Colin Ian King <colin.king at canonical.com> > > Array thresolds should be named thresholds, rename it. Also make it static > static const char * const > > Signed-off-by: Colin Ian King <colin.king at canonical.com>Thanks for doing this, Reviewed-by: Martin Peres <martin.peres at free.fr>> --- > drivers/gpu/drm/nouveau/nvkm/subdev/therm/temp.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/drivers/gpu/drm/nouveau/nvkm/subdev/therm/temp.c b/drivers/gpu/drm/nouveau/nvkm/subdev/therm/temp.c > index e93b2410c38b..6449771b9dc6 100644 > --- a/drivers/gpu/drm/nouveau/nvkm/subdev/therm/temp.c > +++ b/drivers/gpu/drm/nouveau/nvkm/subdev/therm/temp.c > @@ -83,7 +83,7 @@ nvkm_therm_sensor_event(struct nvkm_therm *therm, enum nvkm_therm_thrs thrs, > { > struct nvkm_subdev *subdev = &therm->subdev; > bool active; > - const char *thresolds[] = { > + static const char * const thresholds[] = { > "fanboost", "downclock", "critical", "shutdown" > }; > int temperature = therm->func->temp_get(therm); > @@ -94,10 +94,10 @@ nvkm_therm_sensor_event(struct nvkm_therm *therm, enum nvkm_therm_thrs thrs, > if (dir == NVKM_THERM_THRS_FALLING) > nvkm_info(subdev, > "temperature (%i C) went below the '%s' threshold\n", > - temperature, thresolds[thrs]); > + temperature, thresholds[thrs]); > else > nvkm_info(subdev, "temperature (%i C) hit the '%s' threshold\n", > - temperature, thresolds[thrs]); > + temperature, thresholds[thrs]); > > active = (dir == NVKM_THERM_THRS_RISING); > switch (thrs) { >
Possibly Parallel Threads
- [PATCH 1/3] drm/nouveau/therm: turn on a fan only when crossing threshold in positive direction
- noveau: emergency shutdown handling is overcomplex and broken
- [PATCH v2 1/2] drm/nouveau: return temperatures in temp_get() via parameter
- [PATCH v2 1/2] drm/nouveau: return temperatures in temp_get() via parameter
- [PATCH 03/32] therm: Split return code and value in nvkm_get_temp