From 26bace349ecd08b1d744f7067d24fd1bedb91020 Mon Sep 17 00:00:00 2001 From: Dan Williams Date: Thu, 14 Apr 2011 18:27:49 -0700 Subject: [PATCH] isci: replace sci_sas_link_rate with sas_linkrate Drop duplicated enum definition. Reported-by: Christoph Hellwig Signed-off-by: Dan Williams --- drivers/scsi/isci/core/intel_sas.h | 14 -------------- drivers/scsi/isci/core/scic_phy.h | 3 ++- drivers/scsi/isci/core/scic_remote_device.h | 2 +- drivers/scsi/isci/core/scic_sds_phy.c | 16 ++++++++-------- drivers/scsi/isci/core/scic_sds_phy.h | 3 ++- drivers/scsi/isci/core/scic_sds_port.c | 10 +++++----- drivers/scsi/isci/core/scic_sds_port.h | 2 +- drivers/scsi/isci/core/scic_sds_remote_device.c | 2 +- drivers/scsi/isci/core/scic_sds_remote_device.h | 3 ++- 9 files changed, 22 insertions(+), 33 deletions(-) diff --git a/drivers/scsi/isci/core/intel_sas.h b/drivers/scsi/isci/core/intel_sas.h index eb9686e..d9c9f33 100644 --- a/drivers/scsi/isci/core/intel_sas.h +++ b/drivers/scsi/isci/core/intel_sas.h @@ -180,20 +180,6 @@ struct sas_capabilities { }; /** - * enum _SCI_SAS_LINK_RATE - This enumeration depicts the SAS specification - * defined link speeds. - * - * - */ -enum sci_sas_link_rate { - SCI_SAS_NO_LINK_RATE = 0, - SCI_SATA_SPINUP_HOLD = 0x3, - SCI_SAS_150_GB = 0x8, - SCI_SAS_300_GB = 0x9, - SCI_SAS_600_GB = 0xA -}; - -/** * enum _SCI_SAS_TASK_ATTRIBUTE - This enumeration depicts the SAM/SAS * specification defined task attribute values for a command information * unit. diff --git a/drivers/scsi/isci/core/scic_phy.h b/drivers/scsi/isci/core/scic_phy.h index bf0d3be..784c5b5 100644 --- a/drivers/scsi/isci/core/scic_phy.h +++ b/drivers/scsi/isci/core/scic_phy.h @@ -68,6 +68,7 @@ #include "intel_sata.h" #include "intel_sas.h" +#include struct scic_sds_phy; struct scic_sds_port; @@ -93,7 +94,7 @@ struct scic_phy_properties { * This field specifies the link rate at which the phy is * currently operating. */ - enum sci_sas_link_rate negotiated_link_rate; + enum sas_linkrate negotiated_link_rate; /** * This field indicates the protocols supported by the phy. diff --git a/drivers/scsi/isci/core/scic_remote_device.h b/drivers/scsi/isci/core/scic_remote_device.h index 1401844..62fa7fd 100644 --- a/drivers/scsi/isci/core/scic_remote_device.h +++ b/drivers/scsi/isci/core/scic_remote_device.h @@ -254,7 +254,7 @@ enum sci_status scic_remote_device_reset_complete( * * Return the link rate at which we transfer for the supplied remote device. */ -enum sci_sas_link_rate scic_remote_device_get_connection_rate( +enum sas_linkrate scic_remote_device_get_connection_rate( struct scic_sds_remote_device *remote_device); /** diff --git a/drivers/scsi/isci/core/scic_sds_phy.c b/drivers/scsi/isci/core/scic_sds_phy.c index d5bbc22..40176f0 100644 --- a/drivers/scsi/isci/core/scic_sds_phy.c +++ b/drivers/scsi/isci/core/scic_sds_phy.c @@ -696,7 +696,7 @@ static void scic_sds_phy_start_sata_link_training( */ static void scic_sds_phy_complete_link_training( struct scic_sds_phy *sci_phy, - enum sci_sas_link_rate max_link_rate, + enum sas_linkrate max_link_rate, u32 next_state) { sci_phy->max_negotiated_speed = max_link_rate; @@ -808,21 +808,21 @@ static enum sci_status scic_sds_phy_starting_substate_await_sas_phy_speed_event_ case SCU_EVENT_SAS_15: case SCU_EVENT_SAS_15_SSC: scic_sds_phy_complete_link_training( - this_phy, SCI_SAS_150_GB, SCIC_SDS_PHY_STARTING_SUBSTATE_AWAIT_IAF_UF + this_phy, SAS_LINK_RATE_1_5_GBPS, SCIC_SDS_PHY_STARTING_SUBSTATE_AWAIT_IAF_UF ); break; case SCU_EVENT_SAS_30: case SCU_EVENT_SAS_30_SSC: scic_sds_phy_complete_link_training( - this_phy, SCI_SAS_300_GB, SCIC_SDS_PHY_STARTING_SUBSTATE_AWAIT_IAF_UF + this_phy, SAS_LINK_RATE_3_0_GBPS, SCIC_SDS_PHY_STARTING_SUBSTATE_AWAIT_IAF_UF ); break; case SCU_EVENT_SAS_60: case SCU_EVENT_SAS_60_SSC: scic_sds_phy_complete_link_training( - this_phy, SCI_SAS_600_GB, SCIC_SDS_PHY_STARTING_SUBSTATE_AWAIT_IAF_UF + this_phy, SAS_LINK_RATE_6_0_GBPS, SCIC_SDS_PHY_STARTING_SUBSTATE_AWAIT_IAF_UF ); break; @@ -1082,7 +1082,7 @@ static enum sci_status scic_sds_phy_starting_substate_await_sata_speed_event_han case SCU_EVENT_SATA_15_SSC: scic_sds_phy_complete_link_training( this_phy, - SCI_SAS_150_GB, + SAS_LINK_RATE_1_5_GBPS, SCIC_SDS_PHY_STARTING_SUBSTATE_AWAIT_SIG_FIS_UF ); break; @@ -1091,7 +1091,7 @@ static enum sci_status scic_sds_phy_starting_substate_await_sata_speed_event_han case SCU_EVENT_SATA_30_SSC: scic_sds_phy_complete_link_training( this_phy, - SCI_SAS_300_GB, + SAS_LINK_RATE_3_0_GBPS, SCIC_SDS_PHY_STARTING_SUBSTATE_AWAIT_SIG_FIS_UF ); break; @@ -1100,7 +1100,7 @@ static enum sci_status scic_sds_phy_starting_substate_await_sata_speed_event_han case SCU_EVENT_SATA_60_SSC: scic_sds_phy_complete_link_training( this_phy, - SCI_SAS_600_GB, + SAS_LINK_RATE_6_0_GBPS, SCIC_SDS_PHY_STARTING_SUBSTATE_AWAIT_SIG_FIS_UF ); break; @@ -2439,7 +2439,7 @@ void scic_sds_phy_construct(struct scic_sds_phy *sci_phy, sci_phy->bcn_received_while_port_unassigned = false; sci_phy->protocol = SCIC_SDS_PHY_PROTOCOL_UNKNOWN; sci_phy->link_layer_registers = NULL; - sci_phy->max_negotiated_speed = SCI_SAS_NO_LINK_RATE; + sci_phy->max_negotiated_speed = SAS_LINK_RATE_UNKNOWN; sci_phy->sata_timeout_timer = NULL; /* Clear out the identification buffer data */ diff --git a/drivers/scsi/isci/core/scic_sds_phy.h b/drivers/scsi/isci/core/scic_sds_phy.h index 97c0476..fb99d47 100644 --- a/drivers/scsi/isci/core/scic_sds_phy.h +++ b/drivers/scsi/isci/core/scic_sds_phy.h @@ -60,6 +60,7 @@ #include "intel_sas.h" #include "scu_registers.h" #include "sci_base_state_machine.h" +#include struct scic_sds_port; /** @@ -243,7 +244,7 @@ struct scic_sds_phy { * This field indicates whether the phy supports 1.5 Gb/s, 3.0 Gb/s, * or 6.0 Gb/s operation. */ - enum sci_sas_link_rate max_negotiated_speed; + enum sas_linkrate max_negotiated_speed; /** * This member specifies the protocol being utilized on this phy. This diff --git a/drivers/scsi/isci/core/scic_sds_port.c b/drivers/scsi/isci/core/scic_sds_port.c index ebbde7b..40c1297 100644 --- a/drivers/scsi/isci/core/scic_sds_port.c +++ b/drivers/scsi/isci/core/scic_sds_port.c @@ -684,7 +684,7 @@ void scic_sds_port_deactivate_phy(struct scic_sds_port *sci_port, sci_port->active_phy_mask &= ~(1 << sci_phy->phy_index); - sci_phy->max_negotiated_speed = SCI_SAS_NO_LINK_RATE; + sci_phy->max_negotiated_speed = SAS_LINK_RATE_UNKNOWN; /* Re-assign the phy back to the LP as if it were a narrow port */ writel(sci_phy->phy_index, @@ -977,12 +977,12 @@ static void scic_sds_port_update_viit_entry(struct scic_sds_port *this_port) * This method returns the maximum negotiated speed of the slowest phy in the * port. */ -enum sci_sas_link_rate scic_sds_port_get_max_allowed_speed( +enum sas_linkrate scic_sds_port_get_max_allowed_speed( struct scic_sds_port *this_port) { - u16 index = 0; - enum sci_sas_link_rate max_allowed_speed = SCI_SAS_600_GB; - struct scic_sds_phy *phy = NULL; + u16 index; + enum sas_linkrate max_allowed_speed = SAS_LINK_RATE_6_0_GBPS; + struct scic_sds_phy *phy = NULL; /* * Loop through all of the phys in this port and find the phy with the diff --git a/drivers/scsi/isci/core/scic_sds_port.h b/drivers/scsi/isci/core/scic_sds_port.h index d79c574..a17c64a7 100644 --- a/drivers/scsi/isci/core/scic_sds_port.h +++ b/drivers/scsi/isci/core/scic_sds_port.h @@ -428,7 +428,7 @@ enum sci_status scic_sds_port_complete_io( struct scic_sds_remote_device *the_device, struct scic_sds_request *the_io_request); -enum sci_sas_link_rate scic_sds_port_get_max_allowed_speed( +enum sas_linkrate scic_sds_port_get_max_allowed_speed( struct scic_sds_port *this_port); void scic_sds_port_broadcast_change_received( diff --git a/drivers/scsi/isci/core/scic_sds_remote_device.c b/drivers/scsi/isci/core/scic_sds_remote_device.c index d756216..0976fae 100644 --- a/drivers/scsi/isci/core/scic_sds_remote_device.c +++ b/drivers/scsi/isci/core/scic_sds_remote_device.c @@ -256,7 +256,7 @@ enum sci_status scic_remote_device_reset_complete( } -enum sci_sas_link_rate scic_remote_device_get_connection_rate( +enum sas_linkrate scic_remote_device_get_connection_rate( struct scic_sds_remote_device *sci_dev) { return sci_dev->connection_rate; diff --git a/drivers/scsi/isci/core/scic_sds_remote_device.h b/drivers/scsi/isci/core/scic_sds_remote_device.h index 0e8bb0f..ebb0ac8 100644 --- a/drivers/scsi/isci/core/scic_sds_remote_device.h +++ b/drivers/scsi/isci/core/scic_sds_remote_device.h @@ -59,6 +59,7 @@ #include "intel_sas.h" #include "scu_remote_node_context.h" #include "scic_sds_remote_node_context.h" +#include /** * enum scic_sds_remote_device_states - This enumeration depicts all the states @@ -260,7 +261,7 @@ struct scic_sds_remote_device { * This field is the programmed connection rate for this remote device. It is * used to program the TC with the maximum allowed connection rate. */ - enum sci_sas_link_rate connection_rate; + enum sas_linkrate connection_rate; /** * This field contains the allowed target protocols for this remote device. -- 2.7.4