Re: [PATCH 17/35] drivers/gpu: Convert remaining uses of pr_warning to pr_warn
Am 17.02.2017 um 08:11 schrieb Joe Perches: To enable eventual removal of pr_warning This makes pr_warn use consistent for drivers/gpu Prior to this patch, there were 15 uses of pr_warning and 20 uses of pr_warn in drivers/gpu Signed-off-by: Joe Perches Acked-by: Christian König . --- drivers/gpu/drm/amd/powerplay/hwmgr/smu7_hwmgr.c | 2 +- drivers/gpu/drm/amd/powerplay/inc/pp_debug.h | 2 +- drivers/gpu/drm/amd/powerplay/smumgr/fiji_smc.c | 4 ++-- drivers/gpu/drm/amd/powerplay/smumgr/iceland_smc.c | 14 +++--- drivers/gpu/drm/amd/powerplay/smumgr/polaris10_smc.c | 4 ++-- drivers/gpu/drm/amd/powerplay/smumgr/tonga_smc.c | 4 ++-- 6 files changed, 15 insertions(+), 15 deletions(-) diff --git a/drivers/gpu/drm/amd/powerplay/hwmgr/smu7_hwmgr.c b/drivers/gpu/drm/amd/powerplay/hwmgr/smu7_hwmgr.c index b1de9e8ccdbc..83266408634e 100644 --- a/drivers/gpu/drm/amd/powerplay/hwmgr/smu7_hwmgr.c +++ b/drivers/gpu/drm/amd/powerplay/hwmgr/smu7_hwmgr.c @@ -1535,7 +1535,7 @@ static int smu7_get_evv_voltages(struct pp_hwmgr *hwmgr) if (vddc >= 2000 || vddc == 0) return -EINVAL; } else { - pr_warning("failed to retrieving EVV voltage!\n"); + pr_warn("failed to retrieving EVV voltage!\n"); continue; } diff --git a/drivers/gpu/drm/amd/powerplay/inc/pp_debug.h b/drivers/gpu/drm/amd/powerplay/inc/pp_debug.h index 072880130cfb..f3f9ebb631a5 100644 --- a/drivers/gpu/drm/amd/powerplay/inc/pp_debug.h +++ b/drivers/gpu/drm/amd/powerplay/inc/pp_debug.h @@ -37,7 +37,7 @@ #define PP_ASSERT_WITH_CODE(cond, msg, code) \ do {\ if (!(cond)) { \ - pr_warning("%s\n", msg); \ + pr_warn("%s\n", msg); \ code; \ } \ } while (0) diff --git a/drivers/gpu/drm/amd/powerplay/smumgr/fiji_smc.c b/drivers/gpu/drm/amd/powerplay/smumgr/fiji_smc.c index 0f7a77b7312e..5450f5ef8e89 100644 --- a/drivers/gpu/drm/amd/powerplay/smumgr/fiji_smc.c +++ b/drivers/gpu/drm/amd/powerplay/smumgr/fiji_smc.c @@ -2131,7 +2131,7 @@ uint32_t fiji_get_offsetof(uint32_t type, uint32_t member) return offsetof(SMU73_Discrete_DpmTable, LowSclkInterruptThreshold); } } - pr_warning("can't get the offset of type %x member %x\n", type, member); + pr_warn("can't get the offset of type %x member %x\n", type, member); return 0; } @@ -2156,7 +2156,7 @@ uint32_t fiji_get_mac_definition(uint32_t value) return SMU73_MAX_LEVELS_MVDD; } - pr_warning("can't get the mac of %x\n", value); + pr_warn("can't get the mac of %x\n", value); return 0; } diff --git a/drivers/gpu/drm/amd/powerplay/smumgr/iceland_smc.c b/drivers/gpu/drm/amd/powerplay/smumgr/iceland_smc.c index ad82161df831..51adf04ab4b3 100644 --- a/drivers/gpu/drm/amd/powerplay/smumgr/iceland_smc.c +++ b/drivers/gpu/drm/amd/powerplay/smumgr/iceland_smc.c @@ -122,7 +122,7 @@ static void iceland_initialize_power_tune_defaults(struct pp_hwmgr *hwmgr) break; default: smu_data->power_tune_defaults = &defaults_iceland; - pr_warning("Unknown V.I. Device ID.\n"); + pr_warn("Unknown V.I. Device ID.\n"); break; } return; @@ -378,7 +378,7 @@ static int iceland_get_std_voltage_value_sidd(struct pp_hwmgr *hwmgr, return -EINVAL); if (NULL == hwmgr->dyn_state.cac_leakage_table) { - pr_warning("CAC Leakage Table does not exist, using vddc.\n"); + pr_warn("CAC Leakage Table does not exist, using vddc.\n"); return 0; } @@ -394,7 +394,7 @@ static int iceland_get_std_voltage_value_sidd(struct pp_hwmgr *hwmgr, *lo = hwmgr->dyn_state.cac_leakage_table->entries[v_index].Vddc * VOLTAGE_SCALE; *hi = (uint16_t)(hwmgr->dyn_state.cac_leakage_table->entries[v_index].Leakage * VOLTAGE_SCALE); } else { - pr_warning("Index from SCLK/VDDC Dependency Table exceeds the CAC Leakage Table index, using maximum index from CAC table.\n"); + pr_warn("Index from SCLK/VDDC Dependency Table exceeds the CAC Leakage Table index, using maximum index from CAC table.\n"); *lo = hwmgr->dyn_state.cac_leakage_table->entries[hwmgr->dyn_state.cac_leakage_table->count - 1].Vddc * VOLTAGE_SCALE; *hi = (uint16_t)(hwmgr->dyn_
Re: [PATCH 17/35] drivers/gpu: Convert remaining uses of pr_warning to pr_warn
On 02/18/2017 01:22 AM, Christian König wrote: > Am 17.02.2017 um 08:11 schrieb Joe Perches: >> To enable eventual removal of pr_warning >> >> This makes pr_warn use consistent for drivers/gpu >> >> Prior to this patch, there were 15 uses of pr_warning and >> 20 uses of pr_warn in drivers/gpu >> >> Signed-off-by: Joe Perches > > Acked-by: Christian König . Reviewed-by: Edward O'Callaghan > >> --- >> drivers/gpu/drm/amd/powerplay/hwmgr/smu7_hwmgr.c | 2 +- >> drivers/gpu/drm/amd/powerplay/inc/pp_debug.h | 2 +- >> drivers/gpu/drm/amd/powerplay/smumgr/fiji_smc.c | 4 ++-- >> drivers/gpu/drm/amd/powerplay/smumgr/iceland_smc.c | 14 >> +++--- >> drivers/gpu/drm/amd/powerplay/smumgr/polaris10_smc.c | 4 ++-- >> drivers/gpu/drm/amd/powerplay/smumgr/tonga_smc.c | 4 ++-- >> 6 files changed, 15 insertions(+), 15 deletions(-) >> >> diff --git a/drivers/gpu/drm/amd/powerplay/hwmgr/smu7_hwmgr.c >> b/drivers/gpu/drm/amd/powerplay/hwmgr/smu7_hwmgr.c >> index b1de9e8ccdbc..83266408634e 100644 >> --- a/drivers/gpu/drm/amd/powerplay/hwmgr/smu7_hwmgr.c >> +++ b/drivers/gpu/drm/amd/powerplay/hwmgr/smu7_hwmgr.c >> @@ -1535,7 +1535,7 @@ static int smu7_get_evv_voltages(struct pp_hwmgr >> *hwmgr) >> if (vddc >= 2000 || vddc == 0) >> return -EINVAL; >> } else { >> -pr_warning("failed to retrieving EVV voltage!\n"); >> +pr_warn("failed to retrieving EVV voltage!\n"); >> continue; >> } >> diff --git a/drivers/gpu/drm/amd/powerplay/inc/pp_debug.h >> b/drivers/gpu/drm/amd/powerplay/inc/pp_debug.h >> index 072880130cfb..f3f9ebb631a5 100644 >> --- a/drivers/gpu/drm/amd/powerplay/inc/pp_debug.h >> +++ b/drivers/gpu/drm/amd/powerplay/inc/pp_debug.h >> @@ -37,7 +37,7 @@ >> #define PP_ASSERT_WITH_CODE(cond, msg, code)\ >> do {\ >> if (!(cond)) {\ >> -pr_warning("%s\n", msg);\ >> +pr_warn("%s\n", msg);\ >> code;\ >> }\ >> } while (0) >> diff --git a/drivers/gpu/drm/amd/powerplay/smumgr/fiji_smc.c >> b/drivers/gpu/drm/amd/powerplay/smumgr/fiji_smc.c >> index 0f7a77b7312e..5450f5ef8e89 100644 >> --- a/drivers/gpu/drm/amd/powerplay/smumgr/fiji_smc.c >> +++ b/drivers/gpu/drm/amd/powerplay/smumgr/fiji_smc.c >> @@ -2131,7 +2131,7 @@ uint32_t fiji_get_offsetof(uint32_t type, >> uint32_t member) >> return offsetof(SMU73_Discrete_DpmTable, >> LowSclkInterruptThreshold); >> } >> } >> -pr_warning("can't get the offset of type %x member %x\n", type, >> member); >> +pr_warn("can't get the offset of type %x member %x\n", type, >> member); >> return 0; >> } >> @@ -2156,7 +2156,7 @@ uint32_t fiji_get_mac_definition(uint32_t value) >> return SMU73_MAX_LEVELS_MVDD; >> } >> -pr_warning("can't get the mac of %x\n", value); >> +pr_warn("can't get the mac of %x\n", value); >> return 0; >> } >> diff --git a/drivers/gpu/drm/amd/powerplay/smumgr/iceland_smc.c >> b/drivers/gpu/drm/amd/powerplay/smumgr/iceland_smc.c >> index ad82161df831..51adf04ab4b3 100644 >> --- a/drivers/gpu/drm/amd/powerplay/smumgr/iceland_smc.c >> +++ b/drivers/gpu/drm/amd/powerplay/smumgr/iceland_smc.c >> @@ -122,7 +122,7 @@ static void >> iceland_initialize_power_tune_defaults(struct pp_hwmgr *hwmgr) >> break; >> default: >> smu_data->power_tune_defaults = &defaults_iceland; >> -pr_warning("Unknown V.I. Device ID.\n"); >> +pr_warn("Unknown V.I. Device ID.\n"); >> break; >> } >> return; >> @@ -378,7 +378,7 @@ static int >> iceland_get_std_voltage_value_sidd(struct pp_hwmgr *hwmgr, >> return -EINVAL); >> if (NULL == hwmgr->dyn_state.cac_leakage_table) { >> -pr_warning("CAC Leakage Table does not exist, using vddc.\n"); >> +pr_warn("CAC Leakage Table does not exist, using vddc.\n"); >> return 0; >> } >> @@ -394,7 +394,7 @@ static int >> iceland_get_std_voltage_value_sidd(struct pp_hwmgr *hwmgr, >> *lo = >> hwmgr->dyn_state.cac_leakage_table->entries[v_index].Vddc * >> VOLTAGE_SCALE; >> *hi = >> (uint16_t)(hwmgr->dyn_state.cac_leakage_table->entries[v_index].Leakage * >> VOLTAGE_SCALE); >> } else { >> -pr_warning("Index from SCLK/VDDC Dependency Table >> exceeds the CAC Leakage Table index, using maximum index from CAC >> table.\n"); >> +pr_warn("Index from SCLK/VDDC Dependency Table >> exceeds the CAC Leakage Table index, using maximum index from CAC >> table.\n"); >> *lo = >> hwmgr->dyn_state.cac_leakage_table->entries[hwmgr->dyn_state.cac_leakage_table->count >> - 1].Vddc * VOLTAGE_SCALE; >> *hi = >> (uint16_t)(hwmgr->dyn_state.cac_leakag
Re: [PATCH 17/35] drivers/gpu: Convert remaining uses of pr_warning to pr_warn
On Sat, Feb 18, 2017 at 12:03 AM, Edward O'Callaghan wrote: > > > On 02/18/2017 01:22 AM, Christian König wrote: >> Am 17.02.2017 um 08:11 schrieb Joe Perches: >>> To enable eventual removal of pr_warning >>> >>> This makes pr_warn use consistent for drivers/gpu >>> >>> Prior to this patch, there were 15 uses of pr_warning and >>> 20 uses of pr_warn in drivers/gpu >>> >>> Signed-off-by: Joe Perches >> >> Acked-by: Christian König . > > Reviewed-by: Edward O'Callaghan Applied. thanks! Alex > >> >>> --- >>> drivers/gpu/drm/amd/powerplay/hwmgr/smu7_hwmgr.c | 2 +- >>> drivers/gpu/drm/amd/powerplay/inc/pp_debug.h | 2 +- >>> drivers/gpu/drm/amd/powerplay/smumgr/fiji_smc.c | 4 ++-- >>> drivers/gpu/drm/amd/powerplay/smumgr/iceland_smc.c | 14 >>> +++--- >>> drivers/gpu/drm/amd/powerplay/smumgr/polaris10_smc.c | 4 ++-- >>> drivers/gpu/drm/amd/powerplay/smumgr/tonga_smc.c | 4 ++-- >>> 6 files changed, 15 insertions(+), 15 deletions(-) >>> >>> diff --git a/drivers/gpu/drm/amd/powerplay/hwmgr/smu7_hwmgr.c >>> b/drivers/gpu/drm/amd/powerplay/hwmgr/smu7_hwmgr.c >>> index b1de9e8ccdbc..83266408634e 100644 >>> --- a/drivers/gpu/drm/amd/powerplay/hwmgr/smu7_hwmgr.c >>> +++ b/drivers/gpu/drm/amd/powerplay/hwmgr/smu7_hwmgr.c >>> @@ -1535,7 +1535,7 @@ static int smu7_get_evv_voltages(struct pp_hwmgr >>> *hwmgr) >>> if (vddc >= 2000 || vddc == 0) >>> return -EINVAL; >>> } else { >>> -pr_warning("failed to retrieving EVV voltage!\n"); >>> +pr_warn("failed to retrieving EVV voltage!\n"); >>> continue; >>> } >>> diff --git a/drivers/gpu/drm/amd/powerplay/inc/pp_debug.h >>> b/drivers/gpu/drm/amd/powerplay/inc/pp_debug.h >>> index 072880130cfb..f3f9ebb631a5 100644 >>> --- a/drivers/gpu/drm/amd/powerplay/inc/pp_debug.h >>> +++ b/drivers/gpu/drm/amd/powerplay/inc/pp_debug.h >>> @@ -37,7 +37,7 @@ >>> #define PP_ASSERT_WITH_CODE(cond, msg, code)\ >>> do {\ >>> if (!(cond)) {\ >>> -pr_warning("%s\n", msg);\ >>> +pr_warn("%s\n", msg);\ >>> code;\ >>> }\ >>> } while (0) >>> diff --git a/drivers/gpu/drm/amd/powerplay/smumgr/fiji_smc.c >>> b/drivers/gpu/drm/amd/powerplay/smumgr/fiji_smc.c >>> index 0f7a77b7312e..5450f5ef8e89 100644 >>> --- a/drivers/gpu/drm/amd/powerplay/smumgr/fiji_smc.c >>> +++ b/drivers/gpu/drm/amd/powerplay/smumgr/fiji_smc.c >>> @@ -2131,7 +2131,7 @@ uint32_t fiji_get_offsetof(uint32_t type, >>> uint32_t member) >>> return offsetof(SMU73_Discrete_DpmTable, >>> LowSclkInterruptThreshold); >>> } >>> } >>> -pr_warning("can't get the offset of type %x member %x\n", type, >>> member); >>> +pr_warn("can't get the offset of type %x member %x\n", type, >>> member); >>> return 0; >>> } >>> @@ -2156,7 +2156,7 @@ uint32_t fiji_get_mac_definition(uint32_t value) >>> return SMU73_MAX_LEVELS_MVDD; >>> } >>> -pr_warning("can't get the mac of %x\n", value); >>> +pr_warn("can't get the mac of %x\n", value); >>> return 0; >>> } >>> diff --git a/drivers/gpu/drm/amd/powerplay/smumgr/iceland_smc.c >>> b/drivers/gpu/drm/amd/powerplay/smumgr/iceland_smc.c >>> index ad82161df831..51adf04ab4b3 100644 >>> --- a/drivers/gpu/drm/amd/powerplay/smumgr/iceland_smc.c >>> +++ b/drivers/gpu/drm/amd/powerplay/smumgr/iceland_smc.c >>> @@ -122,7 +122,7 @@ static void >>> iceland_initialize_power_tune_defaults(struct pp_hwmgr *hwmgr) >>> break; >>> default: >>> smu_data->power_tune_defaults = &defaults_iceland; >>> -pr_warning("Unknown V.I. Device ID.\n"); >>> +pr_warn("Unknown V.I. Device ID.\n"); >>> break; >>> } >>> return; >>> @@ -378,7 +378,7 @@ static int >>> iceland_get_std_voltage_value_sidd(struct pp_hwmgr *hwmgr, >>> return -EINVAL); >>> if (NULL == hwmgr->dyn_state.cac_leakage_table) { >>> -pr_warning("CAC Leakage Table does not exist, using vddc.\n"); >>> +pr_warn("CAC Leakage Table does not exist, using vddc.\n"); >>> return 0; >>> } >>> @@ -394,7 +394,7 @@ static int >>> iceland_get_std_voltage_value_sidd(struct pp_hwmgr *hwmgr, >>> *lo = >>> hwmgr->dyn_state.cac_leakage_table->entries[v_index].Vddc * >>> VOLTAGE_SCALE; >>> *hi = >>> (uint16_t)(hwmgr->dyn_state.cac_leakage_table->entries[v_index].Leakage * >>> VOLTAGE_SCALE); >>> } else { >>> -pr_warning("Index from SCLK/VDDC Dependency Table >>> exceeds the CAC Leakage Table index, using maximum index from CAC >>> table.\n"); >>> +pr_warn("Index from SCLK/VDDC Dependency Table >>> exceeds the CAC Leakage Table index, using maximum index from CAC >>> table.\n"); >>>