summaryrefslogtreecommitdiffstats
path: root/drivers
diff options
context:
space:
mode:
authorDave Jiang <dave.jiang@intel.com>2011-03-02 13:10:45 -0800
committerDan Williams <dan.j.williams@intel.com>2011-07-03 03:55:29 -0700
commitd7628d052242d634dc1e2584c422e690578918a3 (patch)
tree507ddf62beb57b245efc10d8f0e553f7784b06f7 /drivers
parent4d07f7f367f2c2d5547684893e61a7a796c1547f (diff)
downloadop-kernel-dev-d7628d052242d634dc1e2584c422e690578918a3.zip
op-kernel-dev-d7628d052242d634dc1e2584c422e690578918a3.tar.gz
isci: Cleanup warning messages for phy resets
Moving some of the chattiness of warning messages to debug so only the Linux system messages are shown. Signed-off-by: Dave Jiang <dave.jiang@intel.com> Signed-off-by: Dan Williams <dan.j.williams@intel.com>
Diffstat (limited to 'drivers')
-rw-r--r--drivers/scsi/isci/core/scic_sds_controller.c65
-rw-r--r--drivers/scsi/isci/core/scic_sds_phy.c36
-rw-r--r--drivers/scsi/isci/remote_device.c2
3 files changed, 43 insertions, 60 deletions
diff --git a/drivers/scsi/isci/core/scic_sds_controller.c b/drivers/scsi/isci/core/scic_sds_controller.c
index deee7eb..180bb1e 100644
--- a/drivers/scsi/isci/core/scic_sds_controller.c
+++ b/drivers/scsi/isci/core/scic_sds_controller.c
@@ -1669,7 +1669,7 @@ void scic_sds_controller_link_up(
if (link_up)
link_up(scic, sci_port, sci_phy);
else
- dev_warn(scic_to_dev(scic),
+ dev_dbg(scic_to_dev(scic),
"%s: SCIC Controller linkup event from phy %d in "
"unexpected state %d\n",
__func__,
@@ -1694,14 +1694,11 @@ void scic_sds_controller_link_down(
if (link_down)
link_down(scic, sci_port, sci_phy);
else
- dev_warn(scic_to_dev(scic),
+ dev_dbg(scic_to_dev(scic),
"%s: SCIC Controller linkdown event from phy %d in "
"unexpected state %d\n",
__func__,
- sci_phy->phy_index,
- sci_base_state_machine_get_state(
- scic_sds_controller_get_base_state_machine(
- scic)));
+ sci_phy->phy_index, state);
}
/**
@@ -1710,28 +1707,22 @@ void scic_sds_controller_link_down(
*
*/
-void scic_sds_controller_remote_device_started(
- struct scic_sds_controller *this_controller,
- struct scic_sds_remote_device *the_device)
+void scic_sds_controller_remote_device_started(struct scic_sds_controller *scic,
+ struct scic_sds_remote_device *sci_dev)
{
u32 state;
- scic_sds_controller_device_handler_t remote_device_started_handler;
+ scic_sds_controller_device_handler_t started;
- state = this_controller->parent.state_machine.current_state_id;
- remote_device_started_handler = scic_sds_controller_state_handler_table[state].remote_device_started_handler;
+ state = scic->parent.state_machine.current_state_id;
+ started = scic_sds_controller_state_handler_table[state].remote_device_started_handler;
- if (remote_device_started_handler != NULL)
- remote_device_started_handler(this_controller, the_device);
+ if (started)
+ started(scic, sci_dev);
else {
- dev_warn(scic_to_dev(this_controller),
- "%s: SCIC Controller 0x%p remote device started event "
- "from device 0x%p in unexpected state %d\n",
- __func__,
- this_controller,
- the_device,
- sci_base_state_machine_get_state(
- scic_sds_controller_get_base_state_machine(
- this_controller)));
+ dev_dbg(scic_to_dev(scic),
+ "%s: SCIC Controller 0x%p remote device started event "
+ "from device 0x%p in unexpected state %d\n",
+ __func__, scic, sci_dev, state);
}
}
@@ -1761,29 +1752,23 @@ bool scic_sds_controller_has_remote_devices_stopping(
*
*/
-void scic_sds_controller_remote_device_stopped(
- struct scic_sds_controller *this_controller,
- struct scic_sds_remote_device *the_device)
+void scic_sds_controller_remote_device_stopped(struct scic_sds_controller *scic,
+ struct scic_sds_remote_device *sci_dev)
{
u32 state;
- scic_sds_controller_device_handler_t remote_device_stopped_handler;
+ scic_sds_controller_device_handler_t stopped;
- state = this_controller->parent.state_machine.current_state_id;
- remote_device_stopped_handler = scic_sds_controller_state_handler_table[state].remote_device_stopped_handler;
+ state = scic->parent.state_machine.current_state_id;
+ stopped = scic_sds_controller_state_handler_table[state].remote_device_stopped_handler;
- if (remote_device_stopped_handler != NULL)
- remote_device_stopped_handler(this_controller, the_device);
+ if (stopped)
+ stopped(scic, sci_dev);
else {
- dev_warn(scic_to_dev(this_controller),
- "%s: SCIC Controller 0x%p remote device stopped event "
- "from device 0x%p in unexpected state %d\n",
- __func__,
- this_controller,
- the_device,
- sci_base_state_machine_get_state(
- scic_sds_controller_get_base_state_machine(
- this_controller)));
+ dev_dbg(scic_to_dev(scic),
+ "%s: SCIC Controller 0x%p remote device stopped event "
+ "from device 0x%p in unexpected state %d\n",
+ __func__, scic, sci_dev, state);
}
}
diff --git a/drivers/scsi/isci/core/scic_sds_phy.c b/drivers/scsi/isci/core/scic_sds_phy.c
index 6f00ff6..2717458 100644
--- a/drivers/scsi/isci/core/scic_sds_phy.c
+++ b/drivers/scsi/isci/core/scic_sds_phy.c
@@ -808,11 +808,11 @@ static enum sci_status scic_sds_phy_starting_substate_await_ossp_event_handler(
break;
default:
- dev_warn(sciphy_to_dev(this_phy),
- "%s: PHY starting substate machine received "
- "unexpected event_code %x\n",
- __func__,
- event_code);
+ dev_dbg(sciphy_to_dev(this_phy),
+ "%s: PHY starting substate machine received "
+ "unexpected event_code %x\n",
+ __func__,
+ event_code);
result = SCI_FAILURE;
break;
@@ -2000,26 +2000,24 @@ enum sci_status scic_sds_phy_default_start_handler(
/**
*
- * @phy: This is the struct sci_base_phy object which is cast into a struct scic_sds_phy
- * object.
+ * @phy: This is the struct sci_base_phy object which is cast into a
+ * struct scic_sds_phy object.
*
* This is the default method for phy a stop request. It will report a warning
* and exit. enum sci_status SCI_FAILURE_INVALID_STATE
*/
-enum sci_status scic_sds_phy_default_stop_handler(
- struct sci_base_phy *phy)
+enum sci_status scic_sds_phy_default_stop_handler(struct sci_base_phy *base_phy)
{
- struct scic_sds_phy *this_phy;
+ struct scic_sds_phy *sci_phy;
- this_phy = (struct scic_sds_phy *)phy;
+ sci_phy = (struct scic_sds_phy *)base_phy;
- dev_warn(sciphy_to_dev(this_phy),
- "%s: SCIC Phy 0x%p requested to stop from invalid "
- "state %d\n",
- __func__,
- this_phy,
- sci_base_state_machine_get_state(
- &this_phy->parent.state_machine));
+ dev_dbg(sciphy_to_dev(sci_phy),
+ "%s: SCIC Phy 0x%p requested to stop from invalid state %d\n",
+ __func__,
+ sci_phy,
+ sci_base_state_machine_get_state(
+ &sci_phy->parent.state_machine));
return SCI_FAILURE_INVALID_STATE;
}
@@ -2119,7 +2117,7 @@ enum sci_status scic_sds_phy_default_event_handler(
struct scic_sds_phy *this_phy,
u32 event_code)
{
- dev_warn(sciphy_to_dev(this_phy),
+ dev_dbg(sciphy_to_dev(this_phy),
"%s: SCIC Phy 0x%p received unexpected event status %x "
"while in state %d\n",
__func__,
diff --git a/drivers/scsi/isci/remote_device.c b/drivers/scsi/isci/remote_device.c
index dec9033..e684a05 100644
--- a/drivers/scsi/isci/remote_device.c
+++ b/drivers/scsi/isci/remote_device.c
@@ -472,7 +472,7 @@ void isci_remote_device_gone(
struct isci_remote_device *isci_device = isci_dev_from_domain_dev(
domain_dev);
- dev_err(&isci_device->isci_port->isci_host->pdev->dev,
+ dev_dbg(&isci_device->isci_port->isci_host->pdev->dev,
"%s: domain_device = %p, isci_device = %p, isci_port = %p\n",
__func__, domain_dev, isci_device, isci_device->isci_port);
OpenPOWER on IntegriCloud