Commit b4e9e2f0 authored by Sebastian Sanchez's avatar Sebastian Sanchez Committed by Doug Ledford

IB/hfi1: Reclassify type of messages printed for platform config logic

Reclassify messages printed out to /var/log/messages into
warnings and errors to facilitate debugging in the future
for issues related to the platform config logic.
Reviewed-by: default avatarMike Marciniszyn <mike.marciniszyn@intel.com>
Signed-off-by: default avatarSebastian Sanchez <sebastian.sanchez@intel.com>
Signed-off-by: default avatarDennis Dalessandro <dennis.dalessandro@intel.com>
Signed-off-by: default avatarDoug Ledford <dledford@redhat.com>
parent e9c48ebd
...@@ -242,7 +242,7 @@ static int qual_power(struct hfi1_pportdata *ppd) ...@@ -242,7 +242,7 @@ static int qual_power(struct hfi1_pportdata *ppd)
if (ppd->offline_disabled_reason == if (ppd->offline_disabled_reason ==
HFI1_ODR_MASK(OPA_LINKDOWN_REASON_POWER_POLICY)) { HFI1_ODR_MASK(OPA_LINKDOWN_REASON_POWER_POLICY)) {
dd_dev_info( dd_dev_err(
ppd->dd, ppd->dd,
"%s: Port disabled due to system power restrictions\n", "%s: Port disabled due to system power restrictions\n",
__func__); __func__);
...@@ -268,7 +268,7 @@ static int qual_bitrate(struct hfi1_pportdata *ppd) ...@@ -268,7 +268,7 @@ static int qual_bitrate(struct hfi1_pportdata *ppd)
if (ppd->offline_disabled_reason == if (ppd->offline_disabled_reason ==
HFI1_ODR_MASK(OPA_LINKDOWN_REASON_LINKSPEED_POLICY)) { HFI1_ODR_MASK(OPA_LINKDOWN_REASON_LINKSPEED_POLICY)) {
dd_dev_info( dd_dev_err(
ppd->dd, ppd->dd,
"%s: Cable failed bitrate check, disabling port\n", "%s: Cable failed bitrate check, disabling port\n",
__func__); __func__);
...@@ -709,15 +709,15 @@ static void apply_tunings( ...@@ -709,15 +709,15 @@ static void apply_tunings(
ret = load_8051_config(ppd->dd, DC_HOST_COMM_SETTINGS, ret = load_8051_config(ppd->dd, DC_HOST_COMM_SETTINGS,
GENERAL_CONFIG, config_data); GENERAL_CONFIG, config_data);
if (ret != HCMD_SUCCESS) if (ret != HCMD_SUCCESS)
dd_dev_info(ppd->dd, dd_dev_err(ppd->dd,
"%s: Failed set ext device config params\n", "%s: Failed set ext device config params\n",
__func__); __func__);
} }
if (tx_preset_index == OPA_INVALID_INDEX) { if (tx_preset_index == OPA_INVALID_INDEX) {
if (ppd->port_type == PORT_TYPE_QSFP && limiting_active) if (ppd->port_type == PORT_TYPE_QSFP && limiting_active)
dd_dev_info(ppd->dd, "%s: Invalid Tx preset index\n", dd_dev_err(ppd->dd, "%s: Invalid Tx preset index\n",
__func__); __func__);
return; return;
} }
...@@ -900,7 +900,7 @@ static int tune_qsfp(struct hfi1_pportdata *ppd, ...@@ -900,7 +900,7 @@ static int tune_qsfp(struct hfi1_pportdata *ppd,
case 0xD: /* fallthrough */ case 0xD: /* fallthrough */
case 0xF: case 0xF:
default: default:
dd_dev_info(ppd->dd, "%s: Unknown/unsupported cable\n", dd_dev_warn(ppd->dd, "%s: Unknown/unsupported cable\n",
__func__); __func__);
break; break;
} }
...@@ -942,7 +942,7 @@ void tune_serdes(struct hfi1_pportdata *ppd) ...@@ -942,7 +942,7 @@ void tune_serdes(struct hfi1_pportdata *ppd)
case PORT_TYPE_DISCONNECTED: case PORT_TYPE_DISCONNECTED:
ppd->offline_disabled_reason = ppd->offline_disabled_reason =
HFI1_ODR_MASK(OPA_LINKDOWN_REASON_DISCONNECTED); HFI1_ODR_MASK(OPA_LINKDOWN_REASON_DISCONNECTED);
dd_dev_info(dd, "%s: Port disconnected, disabling port\n", dd_dev_warn(dd, "%s: Port disconnected, disabling port\n",
__func__); __func__);
goto bail; goto bail;
case PORT_TYPE_FIXED: case PORT_TYPE_FIXED:
...@@ -1027,7 +1027,7 @@ void tune_serdes(struct hfi1_pportdata *ppd) ...@@ -1027,7 +1027,7 @@ void tune_serdes(struct hfi1_pportdata *ppd)
} }
break; break;
default: default:
dd_dev_info(ppd->dd, "%s: Unknown port type\n", __func__); dd_dev_warn(ppd->dd, "%s: Unknown port type\n", __func__);
ppd->port_type = PORT_TYPE_UNKNOWN; ppd->port_type = PORT_TYPE_UNKNOWN;
tuning_method = OPA_UNKNOWN_TUNING; tuning_method = OPA_UNKNOWN_TUNING;
total_atten = 0; total_atten = 0;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment