diff options
author | Dave Jiang <dave.jiang@intel.com> | 2011-03-02 13:10:45 -0800 |
---|---|---|
committer | Dan Williams <dan.j.williams@intel.com> | 2011-07-03 03:55:29 -0700 |
commit | d7628d052242d634dc1e2584c422e690578918a3 (patch) | |
tree | 507ddf62beb57b245efc10d8f0e553f7784b06f7 /drivers/scsi/isci/core | |
parent | 4d07f7f367f2c2d5547684893e61a7a796c1547f (diff) | |
download | op-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/scsi/isci/core')
-rw-r--r-- | drivers/scsi/isci/core/scic_sds_controller.c | 65 | ||||
-rw-r--r-- | drivers/scsi/isci/core/scic_sds_phy.c | 36 |
2 files changed, 42 insertions, 59 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__, |