diff options
author | Philip Elcan <pelcan@nvidia.com> | 2018-08-29 15:46:12 -0400 |
---|---|---|
committer | mobile promotions <svcmobile_promotions@nvidia.com> | 2018-09-07 00:33:50 -0400 |
commit | 7f8226887c28267d3c2351692d4429ead1e17695 (patch) | |
tree | aa09c45e496b058044c9d9a94c94d9922c572285 /drivers/gpu/nvgpu/volt/volt_dev.c | |
parent | 0e58ebaae13dd59b6aba5297f898e7c89fcd2742 (diff) |
gpu: nvgpu: cleanup return types for MISRA 10.3
This is a big cleanup of return types across a number of modules in the
nvgpu driver. Many functions were returning u32 but using negative
return codes. This is a MISRA 10.3 violation by assigning signed values
to a u32.
JIRA NVGPU-647
Change-Id: I59ee66706321f5b5b1a07ed8c24b81583e9ba28c
Signed-off-by: Philip Elcan <pelcan@nvidia.com>
Reviewed-on: https://git-master.nvidia.com/r/1810743
Reviewed-by: mobile promotions <svcmobile_promotions@nvidia.com>
Tested-by: mobile promotions <svcmobile_promotions@nvidia.com>
Diffstat (limited to 'drivers/gpu/nvgpu/volt/volt_dev.c')
-rw-r--r-- | drivers/gpu/nvgpu/volt/volt_dev.c | 30 |
1 files changed, 15 insertions, 15 deletions
diff --git a/drivers/gpu/nvgpu/volt/volt_dev.c b/drivers/gpu/nvgpu/volt/volt_dev.c index 728b0eaf..b6e69a37 100644 --- a/drivers/gpu/nvgpu/volt/volt_dev.c +++ b/drivers/gpu/nvgpu/volt/volt_dev.c | |||
@@ -38,10 +38,10 @@ | |||
38 | #define VOLT_DEV_PWM_VOLTAGE_STEPS_INVALID 0 | 38 | #define VOLT_DEV_PWM_VOLTAGE_STEPS_INVALID 0 |
39 | #define VOLT_DEV_PWM_VOLTAGE_STEPS_DEFAULT 1 | 39 | #define VOLT_DEV_PWM_VOLTAGE_STEPS_DEFAULT 1 |
40 | 40 | ||
41 | static u32 volt_device_pmu_data_init_super(struct gk20a *g, | 41 | static int volt_device_pmu_data_init_super(struct gk20a *g, |
42 | struct boardobj *pboard_obj, struct nv_pmu_boardobj *ppmudata) | 42 | struct boardobj *pboard_obj, struct nv_pmu_boardobj *ppmudata) |
43 | { | 43 | { |
44 | u32 status; | 44 | int status; |
45 | struct voltage_device *pdev; | 45 | struct voltage_device *pdev; |
46 | struct nv_pmu_volt_volt_device_boardobj_set *pset; | 46 | struct nv_pmu_volt_volt_device_boardobj_set *pset; |
47 | 47 | ||
@@ -61,10 +61,10 @@ static u32 volt_device_pmu_data_init_super(struct gk20a *g, | |||
61 | return status; | 61 | return status; |
62 | } | 62 | } |
63 | 63 | ||
64 | static u32 volt_device_pmu_data_init_pwm(struct gk20a *g, | 64 | static int volt_device_pmu_data_init_pwm(struct gk20a *g, |
65 | struct boardobj *pboard_obj, struct nv_pmu_boardobj *ppmudata) | 65 | struct boardobj *pboard_obj, struct nv_pmu_boardobj *ppmudata) |
66 | { | 66 | { |
67 | u32 status = 0; | 67 | int status = 0; |
68 | struct voltage_device_pwm *pdev; | 68 | struct voltage_device_pwm *pdev; |
69 | struct nv_pmu_volt_volt_device_pwm_boardobj_set *pset; | 69 | struct nv_pmu_volt_volt_device_pwm_boardobj_set *pset; |
70 | 70 | ||
@@ -84,12 +84,12 @@ static u32 volt_device_pmu_data_init_pwm(struct gk20a *g, | |||
84 | return status; | 84 | return status; |
85 | } | 85 | } |
86 | 86 | ||
87 | static u32 construct_volt_device(struct gk20a *g, | 87 | static int construct_volt_device(struct gk20a *g, |
88 | struct boardobj **ppboardobj, u16 size, void *pargs) | 88 | struct boardobj **ppboardobj, u16 size, void *pargs) |
89 | { | 89 | { |
90 | struct voltage_device *ptmp_dev = (struct voltage_device *)pargs; | 90 | struct voltage_device *ptmp_dev = (struct voltage_device *)pargs; |
91 | struct voltage_device *pvolt_dev = NULL; | 91 | struct voltage_device *pvolt_dev = NULL; |
92 | u32 status = 0; | 92 | int status = 0; |
93 | 93 | ||
94 | status = boardobj_construct_super(g, ppboardobj, size, pargs); | 94 | status = boardobj_construct_super(g, ppboardobj, size, pargs); |
95 | if (status) { | 95 | if (status) { |
@@ -113,7 +113,7 @@ static u32 construct_volt_device(struct gk20a *g, | |||
113 | return status; | 113 | return status; |
114 | } | 114 | } |
115 | 115 | ||
116 | static u32 construct_pwm_volt_device(struct gk20a *g, | 116 | static int construct_pwm_volt_device(struct gk20a *g, |
117 | struct boardobj **ppboardobj, | 117 | struct boardobj **ppboardobj, |
118 | u16 size, void *pargs) | 118 | u16 size, void *pargs) |
119 | { | 119 | { |
@@ -121,7 +121,7 @@ static u32 construct_pwm_volt_device(struct gk20a *g, | |||
121 | struct voltage_device_pwm *ptmp_dev = | 121 | struct voltage_device_pwm *ptmp_dev = |
122 | (struct voltage_device_pwm *)pargs; | 122 | (struct voltage_device_pwm *)pargs; |
123 | struct voltage_device_pwm *pdev = NULL; | 123 | struct voltage_device_pwm *pdev = NULL; |
124 | u32 status = 0; | 124 | int status = 0; |
125 | 125 | ||
126 | status = construct_volt_device(g, ppboardobj, size, pargs); | 126 | status = construct_volt_device(g, ppboardobj, size, pargs); |
127 | if (status) { | 127 | if (status) { |
@@ -186,7 +186,7 @@ static struct voltage_device *volt_volt_device_construct(struct gk20a *g, | |||
186 | struct boardobj *pboard_obj = NULL; | 186 | struct boardobj *pboard_obj = NULL; |
187 | 187 | ||
188 | if (BOARDOBJ_GET_TYPE(pargs) == CTRL_VOLT_DEVICE_TYPE_PWM) { | 188 | if (BOARDOBJ_GET_TYPE(pargs) == CTRL_VOLT_DEVICE_TYPE_PWM) { |
189 | u32 status = construct_pwm_volt_device(g, &pboard_obj, | 189 | int status = construct_pwm_volt_device(g, &pboard_obj, |
190 | sizeof(struct voltage_device_pwm), pargs); | 190 | sizeof(struct voltage_device_pwm), pargs); |
191 | if (status) { | 191 | if (status) { |
192 | nvgpu_err(g, | 192 | nvgpu_err(g, |
@@ -199,12 +199,12 @@ static struct voltage_device *volt_volt_device_construct(struct gk20a *g, | |||
199 | return (struct voltage_device *)pboard_obj; | 199 | return (struct voltage_device *)pboard_obj; |
200 | } | 200 | } |
201 | 201 | ||
202 | static u32 volt_get_voltage_device_table_1x_psv(struct gk20a *g, | 202 | static int volt_get_voltage_device_table_1x_psv(struct gk20a *g, |
203 | struct vbios_voltage_device_table_1x_entry *p_bios_entry, | 203 | struct vbios_voltage_device_table_1x_entry *p_bios_entry, |
204 | struct voltage_device_metadata *p_Volt_Device_Meta_Data, | 204 | struct voltage_device_metadata *p_Volt_Device_Meta_Data, |
205 | u8 entry_Idx) | 205 | u8 entry_Idx) |
206 | { | 206 | { |
207 | u32 status = 0; | 207 | int status = 0; |
208 | u32 entry_cnt = 0; | 208 | u32 entry_cnt = 0; |
209 | struct voltage_device *pvolt_dev = NULL; | 209 | struct voltage_device *pvolt_dev = NULL; |
210 | struct voltage_device_pwm *pvolt_dev_pwm = NULL; | 210 | struct voltage_device_pwm *pvolt_dev_pwm = NULL; |
@@ -413,7 +413,7 @@ done: | |||
413 | return status; | 413 | return status; |
414 | } | 414 | } |
415 | 415 | ||
416 | static u32 _volt_device_devgrp_pmudata_instget(struct gk20a *g, | 416 | static int _volt_device_devgrp_pmudata_instget(struct gk20a *g, |
417 | struct nv_pmu_boardobjgrp *pmuboardobjgrp, | 417 | struct nv_pmu_boardobjgrp *pmuboardobjgrp, |
418 | struct nv_pmu_boardobj **ppboardobjpmudata, u8 idx) | 418 | struct nv_pmu_boardobj **ppboardobjpmudata, u8 idx) |
419 | { | 419 | { |
@@ -435,7 +435,7 @@ static u32 _volt_device_devgrp_pmudata_instget(struct gk20a *g, | |||
435 | return 0; | 435 | return 0; |
436 | } | 436 | } |
437 | 437 | ||
438 | static u32 _volt_device_devgrp_pmustatus_instget(struct gk20a *g, | 438 | static int _volt_device_devgrp_pmustatus_instget(struct gk20a *g, |
439 | void *pboardobjgrppmu, | 439 | void *pboardobjgrppmu, |
440 | struct nv_pmu_boardobj_query **ppboardobjpmustatus, u8 idx) | 440 | struct nv_pmu_boardobj_query **ppboardobjpmustatus, u8 idx) |
441 | { | 441 | { |
@@ -518,9 +518,9 @@ done: | |||
518 | return status; | 518 | return status; |
519 | } | 519 | } |
520 | 520 | ||
521 | u32 volt_dev_pmu_setup(struct gk20a *g) | 521 | int volt_dev_pmu_setup(struct gk20a *g) |
522 | { | 522 | { |
523 | u32 status; | 523 | int status; |
524 | struct boardobjgrp *pboardobjgrp = NULL; | 524 | struct boardobjgrp *pboardobjgrp = NULL; |
525 | 525 | ||
526 | nvgpu_log_info(g, " "); | 526 | nvgpu_log_info(g, " "); |