isci: Removing deprecated functions

Removed all callbacks in the deprecated.c. Core will call the appropriate
functions directly.

Signed-off-by: Dave Jiang <dave.jiang@intel.com>
Signed-off-by: Dan Williams <dan.j.williams@intel.com>

16 files changed, 130 insertions, 1030 deletions

diff --git a/drivers/scsi/isci/Makefile b/drivers/scsi/isci/Makefile
index 34f7af3525f..d402d679a31 100644
--- a/drivers/scsi/isci/Makefile
+++ b/drivers/scsi/isci/Makefile
@@ -8,7 +8,7 @@ EXTRA_CFLAGS += -DDISABLE_ATAPI
 EXTRA_CFLAGS += -Idrivers/scsi/isci/core/ -Idrivers/scsi/isci/
 obj-$(CONFIG_SCSI_ISCI) += isci.o
 isci-objs := init.o phy.o request.o sata.o \
-	     remote_device.o port.o timers.o deprecated.o \
+	     remote_device.o port.o timers.o \
 	     host.o task.o events.o \
 	     core/scic_sds_controller.o  \
 	     core/scic_sds_remote_device.o    \
diff --git a/drivers/scsi/isci/core/scic_sds_controller.c b/drivers/scsi/isci/core/scic_sds_controller.c
index b0f9221d145..d642ff7be6d 100644
--- a/drivers/scsi/isci/core/scic_sds_controller.c
+++ b/drivers/scsi/isci/core/scic_sds_controller.c
@@ -66,7 +66,6 @@
 #include "scic_sds_port.h"
 #include "scic_sds_remote_device.h"
 #include "scic_sds_request.h"
-#include "scic_user_callback.h"
 #include "sci_environment.h"
 #include "sci_util.h"
 #include "scu_completion_codes.h"
@@ -594,7 +593,7 @@ void scic_sds_controller_afe_initialization(struct scic_sds_controller *scic)
 
 	/* Clear DFX Status registers */
 	scu_afe_register_write(scic, afe_dfx_master_control0, 0x0081000f);
-	scic_cb_stall_execution(AFE_REGISTER_WRITE_DELAY);
+	udelay(AFE_REGISTER_WRITE_DELAY);
 
 	/* Configure bias currents to normal */
 	if (is_a0())
@@ -602,7 +601,7 @@ void scic_sds_controller_afe_initialization(struct scic_sds_controller *scic)
 	else
 		scu_afe_register_write(scic, afe_bias_control, 0x00005A00);
 
-	scic_cb_stall_execution(AFE_REGISTER_WRITE_DELAY);
+	udelay(AFE_REGISTER_WRITE_DELAY);
 
 	/* Enable PLL */
 	if (is_b0())
@@ -610,35 +609,35 @@ void scic_sds_controller_afe_initialization(struct scic_sds_controller *scic)
 	else
 		scu_afe_register_write(scic, afe_pll_control0, 0x80040908);
 
-	scic_cb_stall_execution(AFE_REGISTER_WRITE_DELAY);
+	udelay(AFE_REGISTER_WRITE_DELAY);
 
 	/* Wait for the PLL to lock */
 	do {
 		afe_status = scu_afe_register_read(
 			scic, afe_common_block_status);
-		scic_cb_stall_execution(AFE_REGISTER_WRITE_DELAY);
+		udelay(AFE_REGISTER_WRITE_DELAY);
 	} while ((afe_status & 0x00001000) == 0);
 
 	if (is_b0()) {
 		/* Shorten SAS SNW lock time (RxLock timer value from 76 us to 50 us) */
 		scu_afe_register_write(scic, afe_pmsn_master_control0, 0x7bcc96ad);
-		scic_cb_stall_execution(AFE_REGISTER_WRITE_DELAY);
+		udelay(AFE_REGISTER_WRITE_DELAY);
 	}
 
 	for (phy_id = 0; phy_id < SCI_MAX_PHYS; phy_id++) {
 		if (is_b0()) {
 			 /* Configure transmitter SSC parameters */
 			scu_afe_txreg_write(scic, phy_id, afe_tx_ssc_control, 0x00030000);
-			scic_cb_stall_execution(AFE_REGISTER_WRITE_DELAY);
+			udelay(AFE_REGISTER_WRITE_DELAY);
 		} else {
 			/*
 			 * All defaults, except the Receive Word Alignament/Comma Detect
 			 * Enable....(0xe800) */
 			scu_afe_txreg_write(scic, phy_id, afe_xcvr_control0, 0x00004512);
-			scic_cb_stall_execution(AFE_REGISTER_WRITE_DELAY);
+			udelay(AFE_REGISTER_WRITE_DELAY);
 
 			scu_afe_txreg_write(scic, phy_id, afe_xcvr_control1, 0x0050100F);
-			scic_cb_stall_execution(AFE_REGISTER_WRITE_DELAY);
+			udelay(AFE_REGISTER_WRITE_DELAY);
 		}
 
 		/*
@@ -651,26 +650,26 @@ void scic_sds_controller_afe_initialization(struct scic_sds_controller *scic)
 		else {
 			 /* Power down TX and RX (PWRDNTX and PWRDNRX) */
 			scu_afe_txreg_write(scic, phy_id, afe_channel_control, 0x000003d7);
-			scic_cb_stall_execution(AFE_REGISTER_WRITE_DELAY);
+			udelay(AFE_REGISTER_WRITE_DELAY);
 
 			/*
 			 * Power up TX and RX out from power down (PWRDNTX and PWRDNRX)
 			 * & increase TX int & ext bias 20%....(0xe85c) */
 			scu_afe_txreg_write(scic, phy_id, afe_channel_control, 0x000003d4);
 		}
-		scic_cb_stall_execution(AFE_REGISTER_WRITE_DELAY);
+		udelay(AFE_REGISTER_WRITE_DELAY);
 
 		if (is_a0() || is_a2()) {
 			/* Enable TX equalization (0xe824) */
 			scu_afe_txreg_write(scic, phy_id, afe_tx_control, 0x00040000);
-			scic_cb_stall_execution(AFE_REGISTER_WRITE_DELAY);
+			udelay(AFE_REGISTER_WRITE_DELAY);
 		}
 
 		/*
 		 * RDPI=0x0(RX Power On), RXOOBDETPDNC=0x0, TPD=0x0(TX Power On),
 		 * RDD=0x0(RX Detect Enabled) ....(0xe800) */
 		scu_afe_txreg_write(scic, phy_id, afe_xcvr_control0, 0x00004100);
-		scic_cb_stall_execution(AFE_REGISTER_WRITE_DELAY);
+		udelay(AFE_REGISTER_WRITE_DELAY);
 
 		/* Leave DFE/FFE on */
 		if (is_a0())
@@ -679,28 +678,28 @@ void scic_sds_controller_afe_initialization(struct scic_sds_controller *scic)
 			scu_afe_txreg_write(scic, phy_id, afe_rx_ssc_control0, 0x3F11103F);
 		else {
 			scu_afe_txreg_write(scic, phy_id, afe_rx_ssc_control0, 0x3F11103F);
-			scic_cb_stall_execution(AFE_REGISTER_WRITE_DELAY);
+			udelay(AFE_REGISTER_WRITE_DELAY);
 			/* Enable TX equalization (0xe824) */
 			scu_afe_txreg_write(scic, phy_id, afe_tx_control, 0x00040000);
 		}
-		scic_cb_stall_execution(AFE_REGISTER_WRITE_DELAY);
+		udelay(AFE_REGISTER_WRITE_DELAY);
 
 		scu_afe_txreg_write(scic, phy_id, afe_tx_amp_control0, 0x000E7C03);
-		scic_cb_stall_execution(AFE_REGISTER_WRITE_DELAY);
+		udelay(AFE_REGISTER_WRITE_DELAY);
 
 		scu_afe_txreg_write(scic, phy_id, afe_tx_amp_control1, 0x000E7C03);
-		scic_cb_stall_execution(AFE_REGISTER_WRITE_DELAY);
+		udelay(AFE_REGISTER_WRITE_DELAY);
 
 		scu_afe_txreg_write(scic, phy_id, afe_tx_amp_control2, 0x000E7C03);
-		scic_cb_stall_execution(AFE_REGISTER_WRITE_DELAY);
+		udelay(AFE_REGISTER_WRITE_DELAY);
 
 		scu_afe_txreg_write(scic, phy_id, afe_tx_amp_control3, 0x000E7C03);
-		scic_cb_stall_execution(AFE_REGISTER_WRITE_DELAY);
+		udelay(AFE_REGISTER_WRITE_DELAY);
 	}
 
 	/* Transfer control to the PEs */
 	scu_afe_register_write(scic, afe_dfx_master_control0, 0x00010f00);
-	scic_cb_stall_execution(AFE_REGISTER_WRITE_DELAY);
+	udelay(AFE_REGISTER_WRITE_DELAY);
 }
 
 /*
@@ -2446,7 +2445,7 @@ void scic_sds_controller_reset_hardware(
 	SMU_SMUSRCR_WRITE(scic, 0xFFFFFFFF);
 
 	/* Delay for 1ms to before clearing the CQP and UFQPR. */
-	scic_cb_stall_execution(1000);
+	udelay(1000);
 
 	/* The write to the CQGR clears the CQP */
 	SMU_CQGR_WRITE(scic, 0x00000000);
@@ -2840,7 +2839,7 @@ static enum sci_status scic_sds_controller_reset_state_initialize_handler(
 
 		while (terminate_loop-- && (result != SCI_SUCCESS)) {
 			/* Loop until the hardware reports success */
-			scic_cb_stall_execution(SCU_CONTEXT_RAM_INIT_STALL_TIME);
+			udelay(SCU_CONTEXT_RAM_INIT_STALL_TIME);
 			status = SMU_SMUCSR_READ(this_controller);
 
 			if ((status & SCU_RAM_INIT_COMPLETED) == SCU_RAM_INIT_COMPLETED) {
diff --git a/drivers/scsi/isci/core/scic_sds_phy.c b/drivers/scsi/isci/core/scic_sds_phy.c
index 92a5d294809..d4a5e38aef6 100644
--- a/drivers/scsi/isci/core/scic_sds_phy.c
+++ b/drivers/scsi/isci/core/scic_sds_phy.c
@@ -63,7 +63,6 @@
 #include "scic_sds_phy_registers.h"
 #include "scic_sds_port.h"
 #include "scic_sds_remote_node_context.h"
-#include "scic_user_callback.h"
 #include "sci_environment.h"
 #include "sci_util.h"
 #include "scu_event_codes.h"
diff --git a/drivers/scsi/isci/core/scic_sds_port.c b/drivers/scsi/isci/core/scic_sds_port.c
index f31e6dc4252..d374c7ac0b7 100644
--- a/drivers/scsi/isci/core/scic_sds_port.c
+++ b/drivers/scsi/isci/core/scic_sds_port.c
@@ -66,7 +66,6 @@
 #include "scic_sds_remote_device.h"
 #include "scic_sds_remote_node_context.h"
 #include "scic_sds_request.h"
-#include "scic_user_callback.h"
 #include "sci_environment.h"
 
 
diff --git a/drivers/scsi/isci/core/scic_sds_remote_device.c b/drivers/scsi/isci/core/scic_sds_remote_device.c
index 3a8d5636475..a7cb4bc39a1 100644
--- a/drivers/scsi/isci/core/scic_sds_remote_device.c
+++ b/drivers/scsi/isci/core/scic_sds_remote_device.c
@@ -64,7 +64,6 @@
 #include "scic_sds_remote_device.h"
 #include "scic_sds_remote_node_context.h"
 #include "scic_sds_request.h"
-#include "scic_user_callback.h"
 #include "sci_environment.h"
 #include "sci_util.h"
 #include "scu_event_codes.h"
diff --git a/drivers/scsi/isci/core/scic_sds_request.c b/drivers/scsi/isci/core/scic_sds_request.c
index 909361f680f..7b9ce1e878a 100644
--- a/drivers/scsi/isci/core/scic_sds_request.c
+++ b/drivers/scsi/isci/core/scic_sds_request.c
@@ -70,7 +70,6 @@
 #include "scic_sds_smp_request.h"
 #include "scic_sds_stp_request.h"
 #include "scic_sds_unsolicited_frame_control.h"
-#include "scic_user_callback.h"
 #include "sci_environment.h"
 #include "sci_util.h"
 #include "scu_completion_codes.h"
@@ -413,25 +412,26 @@ static void scic_sds_ssp_io_request_assign_buffers(
  *
  */
 static void scic_sds_io_request_build_ssp_command_iu(
-	struct scic_sds_request *this_request)
+	struct scic_sds_request *sds_request)
 {
 	struct sci_ssp_command_iu *command_frame;
-	void *os_handle;
 	u32 cdb_length;
 	u32 *cdb_buffer;
+	struct isci_request *isci_request =
+		(struct isci_request *)sci_object_get_association(sds_request);
 
 	command_frame =
-		(struct sci_ssp_command_iu *)this_request->command_buffer;
-
-	os_handle = scic_sds_request_get_user_request(this_request);
+		(struct sci_ssp_command_iu *)sds_request->command_buffer;
 
 	command_frame->lun_upper = 0;
-	command_frame->lun_lower = scic_cb_ssp_io_request_get_lun(os_handle);
+	command_frame->lun_lower =
+		isci_request_ssp_io_request_get_lun(isci_request);
 
 	((u32 *)command_frame)[2] = 0;
 
-	cdb_length = scic_cb_ssp_io_request_get_cdb_length(os_handle);
-	cdb_buffer = (u32 *)scic_cb_ssp_io_request_get_cdb_address(os_handle);
+	cdb_length = isci_request_ssp_io_request_get_cdb_length(isci_request);
+	cdb_buffer = (u32 *)isci_request_ssp_io_request_get_cdb_address(
+					isci_request);
 
 	if (cdb_length > 16) {
 		command_frame->additional_cdb_length = cdb_length - 16;
@@ -446,9 +446,9 @@ static void scic_sds_io_request_build_ssp_command_iu(
 
 	command_frame->enable_first_burst = 0;
 	command_frame->task_priority =
-		scic_cb_ssp_io_request_get_command_priority(os_handle);
+		isci_request_ssp_io_request_get_command_priority(isci_request);
 	command_frame->task_attribute =
-		scic_cb_ssp_io_request_get_task_attribute(os_handle);
+		isci_request_ssp_io_request_get_task_attribute(isci_request);
 }
 
 
@@ -458,25 +458,26 @@ static void scic_sds_io_request_build_ssp_command_iu(
  *
  */
 static void scic_sds_task_request_build_ssp_task_iu(
-	struct scic_sds_request *this_request)
+	struct scic_sds_request *sds_request)
 {
 	struct sci_ssp_task_iu *command_frame;
-	void *os_handle;
+	struct isci_request *isci_request =
+		(struct isci_request *)sci_object_get_association(sds_request);
 
 	command_frame =
-		(struct sci_ssp_task_iu *)this_request->command_buffer;
-
-	os_handle = scic_sds_request_get_user_request(this_request);
+		(struct sci_ssp_task_iu *)sds_request->command_buffer;
 
 	command_frame->lun_upper = 0;
-	command_frame->lun_lower = scic_cb_ssp_task_request_get_lun(os_handle);
+	command_frame->lun_lower = isci_request_ssp_io_request_get_lun(
+					isci_request);
 
 	((u32 *)command_frame)[2] = 0;
 
 	command_frame->task_function =
-		scic_cb_ssp_task_request_get_function(os_handle);
+		isci_task_ssp_request_get_function(isci_request);
 	command_frame->task_tag =
-		scic_cb_ssp_task_request_get_io_tag_to_manage(os_handle);
+		isci_task_ssp_request_get_io_tag_to_manage(
+				isci_request);
 }
 
 
@@ -899,18 +900,15 @@ enum sci_status scic_task_request_construct(
 enum sci_status scic_io_request_construct_basic_ssp(
 	struct scic_sds_request *sci_req)
 {
-	void *os_handle;
+	struct isci_request *isci_request =
+		(struct isci_request *)sci_object_get_association(sci_req);
 
 	sci_req->protocol = SCIC_SSP_PROTOCOL;
 
-	os_handle = scic_sds_request_get_user_request(sci_req);
-
 	scu_ssp_io_request_construct_task_context(
 		sci_req,
-		scic_cb_io_request_get_data_direction(os_handle),
-		scic_cb_io_request_get_transfer_length(os_handle)
-		);
-
+		isci_request_io_request_get_data_direction(isci_request),
+		isci_request_io_request_get_transfer_length(isci_request));
 
 	scic_sds_io_request_build_ssp_command_iu(sci_req);
 
@@ -941,7 +939,8 @@ enum sci_status scic_task_request_construct_ssp(
 }
 
 
-enum sci_status scic_io_request_construct_basic_sata(struct scic_sds_request *sci_req)
+enum sci_status scic_io_request_construct_basic_sata(
+		struct scic_sds_request *sci_req)
 {
 	enum sci_status status;
 	struct scic_sds_stp_request *stp_req;
@@ -949,15 +948,18 @@ enum sci_status scic_io_request_construct_basic_sata(struct scic_sds_request *sc
 	u32 len;
 	enum dma_data_direction dir;
 	bool copy = false;
+	struct isci_request *isci_request =
+		(struct isci_request *)sci_object_get_association(sci_req);
+	struct sas_task *task = isci_request_access_task(isci_request);
 
 	stp_req = container_of(sci_req, typeof(*stp_req), parent);
 
 	sci_req->protocol = SCIC_STP_PROTOCOL;
 
-	len = scic_cb_io_request_get_transfer_length(sci_req->user_request);
-	dir = scic_cb_io_request_get_data_direction(sci_req->user_request);
-	proto = scic_cb_request_get_sat_protocol(sci_req->user_request);
-	copy = scic_cb_io_request_do_copy_rx_frames(stp_req->parent.user_request);
+	len = isci_request_io_request_get_transfer_length(isci_request);
+	dir = isci_request_io_request_get_data_direction(isci_request);
+	proto = isci_sata_get_sat_protocol(isci_request);
+	copy = (task->data_dir == DMA_NONE) ? false : true;
 
 	status = scic_io_request_construct_sata(sci_req, proto, len, dir, copy);
 
@@ -975,7 +977,11 @@ enum sci_status scic_task_request_construct_sata(
 	struct scic_sds_request *sci_req)
 {
 	enum sci_status status;
-	u8 sat_protocol = scic_cb_request_get_sat_protocol(sci_req->user_request);
+	u8 sat_protocol;
+	struct isci_request *isci_request =
+		(struct isci_request *)sci_object_get_association(sci_req);
+
+	sat_protocol = isci_sata_get_sat_protocol(isci_request);
 
 	switch (sat_protocol) {
 	case SAT_PROTOCOL_ATA_HARD_RESET:
@@ -1172,27 +1178,28 @@ enum sci_status scic_sds_io_request_frame_handler(
  *    the response data.
  *
  */
-void scic_sds_io_request_copy_response(
-	struct scic_sds_request *this_request)
+void scic_sds_io_request_copy_response(struct scic_sds_request *sds_request)
 {
 	void *response_buffer;
 	u32 user_response_length;
 	u32 core_response_length;
 	struct sci_ssp_response_iu *ssp_response;
+	struct isci_request *isci_request =
+		(struct isci_request *)sci_object_get_association(sds_request);
 
-	ssp_response = (struct sci_ssp_response_iu *)this_request->response_buffer;
+	ssp_response =
+		(struct sci_ssp_response_iu *)sds_request->response_buffer;
 
-	response_buffer = scic_cb_ssp_task_request_get_response_data_address(
-		this_request->user_request
-		);
+	response_buffer =
+		isci_task_ssp_request_get_response_data_address(
+				isci_request);
 
-	user_response_length = scic_cb_ssp_task_request_get_response_data_length(
-		this_request->user_request
-		);
+	user_response_length =
+		isci_task_ssp_request_get_response_data_length(
+				isci_request);
 
 	core_response_length = sci_ssp_get_response_data_length(
-		ssp_response->response_data_length
-		);
+					ssp_response->response_data_length);
 
 	user_response_length = min(user_response_length, core_response_length);
 
diff --git a/drivers/scsi/isci/core/scic_sds_smp_remote_device.c b/drivers/scsi/isci/core/scic_sds_smp_remote_device.c
index e90d46edd06..93e6ab83b01 100644
--- a/drivers/scsi/isci/core/scic_sds_smp_remote_device.c
+++ b/drivers/scsi/isci/core/scic_sds_smp_remote_device.c
@@ -58,7 +58,6 @@
 #include "scic_sds_port.h"
 #include "scic_sds_remote_device.h"
 #include "scic_sds_request.h"
-#include "scic_user_callback.h"
 #include "sci_environment.h"
 #include "sci_util.h"
 #include "scu_event_codes.h"
diff --git a/drivers/scsi/isci/core/scic_sds_stp_remote_device.c b/drivers/scsi/isci/core/scic_sds_stp_remote_device.c
index ce2cb7b4c7c..1d8d9013068 100644
--- a/drivers/scsi/isci/core/scic_sds_stp_remote_device.c
+++ b/drivers/scsi/isci/core/scic_sds_stp_remote_device.c
@@ -62,7 +62,6 @@
 #include "scic_sds_port.h"
 #include "scic_sds_remote_device.h"
 #include "scic_sds_request.h"
-#include "scic_user_callback.h"
 #include "sci_environment.h"
 #include "sci_util.h"
 #include "scu_event_codes.h"
@@ -217,52 +216,56 @@ static enum sci_status scic_sds_stp_remote_device_ready_substate_start_request_h
  * @device:
  * @request:
  *
- * If this is a softreset we may want to have a different substate. enum sci_status
+ * If this is a softreset we may want to have a different substate.
+ * enum sci_status
  */
 static enum sci_status scic_sds_stp_remote_device_ready_idle_substate_start_io_handler(
-	struct sci_base_remote_device *device,
-	struct sci_base_request *request)
+	struct sci_base_remote_device *base_device,
+	struct sci_base_request *base_request)
 {
 	enum sci_status status;
-	struct scic_sds_remote_device *this_device = (struct scic_sds_remote_device *)device;
-	struct scic_sds_request *io_request  = (struct scic_sds_request *)request;
+	struct scic_sds_remote_device *device =
+		(struct scic_sds_remote_device *)&base_device->parent;
+	struct scic_sds_request *sds_request  =
+		(struct scic_sds_request *)&base_request->parent;
+	struct isci_request *isci_request =
+		(struct isci_request *)sci_object_get_association(sds_request);
 
 
 	/* Will the port allow the io request to start? */
-	status = this_device->owning_port->state_handlers->start_io_handler(
-		this_device->owning_port,
-		this_device,
-		io_request
-		);
+	status = device->owning_port->state_handlers->start_io_handler(
+			device->owning_port,
+			device,
+			sds_request);
 
 	if (status == SCI_SUCCESS) {
 		status =
-			scic_sds_remote_node_context_start_io(this_device->rnc, io_request);
+			scic_sds_remote_node_context_start_io(device->rnc,
+							      sds_request);
 
-		if (status == SCI_SUCCESS) {
-			status = io_request->state_handlers->parent.start_handler(request);
-		}
+		if (status == SCI_SUCCESS)
+			status =
+				sds_request->state_handlers->
+					parent.start_handler(base_request);
 
 		if (status == SCI_SUCCESS) {
-			if (
-				scic_cb_request_get_sat_protocol(io_request->user_request)
-				== SAT_PROTOCOL_FPDMA
-				) {
+			if (isci_sata_get_sat_protocol(isci_request) ==
+					SAT_PROTOCOL_FPDMA)
 				sci_base_state_machine_change_state(
-					&this_device->ready_substate_machine,
-					SCIC_SDS_STP_REMOTE_DEVICE_READY_SUBSTATE_NCQ
-					);
-			} else {
-				this_device->working_request = io_request;
+					&device->ready_substate_machine,
+					SCIC_SDS_STP_REMOTE_DEVICE_READY_SUBSTATE_NCQ);
+			else {
+				device->working_request = sds_request;
 
 				sci_base_state_machine_change_state(
-					&this_device->ready_substate_machine,
-					SCIC_SDS_STP_REMOTE_DEVICE_READY_SUBSTATE_CMD
-					);
+					&device->ready_substate_machine,
+					SCIC_SDS_STP_REMOTE_DEVICE_READY_SUBSTATE_CMD);
 			}
 		}
 
-		scic_sds_remote_device_start_request(this_device, io_request, status);
+		scic_sds_remote_device_start_request(device,
+						     sds_request,
+						     status);
 	}
 
 	return status;
@@ -304,35 +307,38 @@ static enum sci_status scic_sds_stp_remote_device_ready_idle_substate_event_hand
  * ***************************************************************************** */
 
 static enum sci_status scic_sds_stp_remote_device_ready_ncq_substate_start_io_handler(
-	struct sci_base_remote_device *device,
-	struct sci_base_request *request)
+	struct sci_base_remote_device *base_device,
+	struct sci_base_request *base_request)
 {
 	enum sci_status status;
-	struct scic_sds_remote_device *this_device = (struct scic_sds_remote_device *)device;
-	struct scic_sds_request *io_request  = (struct scic_sds_request *)request;
-
-	if (
-		scic_cb_request_get_sat_protocol(io_request->user_request)
-		== SAT_PROTOCOL_FPDMA
-		) {
-		status = this_device->owning_port->state_handlers->start_io_handler(
-			this_device->owning_port,
-			this_device,
-			io_request
-			);
+	struct scic_sds_remote_device *device =
+		(struct scic_sds_remote_device *)&base_device->parent;
+	struct scic_sds_request *sds_request  =
+		(struct scic_sds_request *)&base_request->parent;
+	struct isci_request *isci_request =
+		(struct isci_request *)sci_object_get_association(sds_request);
+
+	if (isci_sata_get_sat_protocol(isci_request) == SAT_PROTOCOL_FPDMA) {
+		status = device->owning_port->state_handlers->start_io_handler(
+				device->owning_port,
+				device,
+				sds_request);
 
 		if (status == SCI_SUCCESS) {
-			status = scic_sds_remote_node_context_start_io(this_device->rnc, io_request);
+			status = scic_sds_remote_node_context_start_io(
+					device->rnc,
+					sds_request);
 
-			if (status == SCI_SUCCESS) {
-				status = io_request->state_handlers->parent.start_handler(request);
-			}
+			if (status == SCI_SUCCESS)
+				status = sds_request->state_handlers->
+					parent.start_handler(base_request);
 
-			scic_sds_remote_device_start_request(this_device, io_request, status);
+			scic_sds_remote_device_start_request(device,
+							     sds_request,
+							     status);
 		}
-	} else {
+	} else
 		status = SCI_FAILURE_INVALID_STATE;
-	}
 
 	return status;
 }
diff --git a/drivers/scsi/isci/core/scic_sds_stp_request.c b/drivers/scsi/isci/core/scic_sds_stp_request.c
index 0a07207c841..0f17a28dd19 100644
--- a/drivers/scsi/isci/core/scic_sds_stp_request.c
+++ b/drivers/scsi/isci/core/scic_sds_stp_request.c
@@ -67,7 +67,6 @@
 #include "scic_sds_stp_pio_request.h"
 #include "scic_sds_stp_request.h"
 #include "scic_sds_unsolicited_frame_control.h"
-#include "scic_user_callback.h"
 #include "sci_environment.h"
 #include "sci_util.h"
 #include "scu_completion_codes.h"
diff --git a/drivers/scsi/isci/core/scic_sds_unsolicited_frame_control.c b/drivers/scsi/isci/core/scic_sds_unsolicited_frame_control.c
index 7274812af50..66be58b002d 100644
--- a/drivers/scsi/isci/core/scic_sds_unsolicited_frame_control.c
+++ b/drivers/scsi/isci/core/scic_sds_unsolicited_frame_control.c
@@ -64,7 +64,6 @@
 #include "scic_sds_unsolicited_frame_control.h"
 #include "scu_registers.h"
 #include "scic_sds_controller.h"
-#include "scic_user_callback.h"
 #include "sci_util.h"
 #include "sci_environment.h"
 
diff --git a/drivers/scsi/isci/core/scic_user_callback.h b/drivers/scsi/isci/core/scic_user_callback.h
deleted file mode 100644
index c097711c5f2..00000000000
--- a/drivers/scsi/isci/core/scic_user_callback.h
+++ /dev/null
@@ -1,421 +0,0 @@
-/*
- * This file is provided under a dual BSD/GPLv2 license.  When using or
- * redistributing this file, you may do so under either license.
- *
- * GPL LICENSE SUMMARY
- *
- * Copyright(c) 2008 - 2011 Intel Corporation. All rights reserved.
- *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of version 2 of the GNU General Public License as
- * published by the Free Software Foundation.
- *
- * This program is distributed in the hope that it will be useful, but
- * WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
- * General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 51 Franklin St - Fifth Floor, Boston, MA 02110-1301 USA.
- * The full GNU General Public License is included in this distribution
- * in the file called LICENSE.GPL.
- *
- * BSD LICENSE
- *
- * Copyright(c) 2008 - 2011 Intel Corporation. All rights reserved.
- * All rights reserved.
- *
- * Redistribution and use in source and binary forms, with or without
- * modification, are permitted provided that the following conditions
- * are met:
- *
- *   * Redistributions of source code must retain the above copyright
- *     notice, this list of conditions and the following disclaimer.
- *   * Redistributions in binary form must reproduce the above copyright
- *     notice, this list of conditions and the following disclaimer in
- *     the documentation and/or other materials provided with the
- *     distribution.
- *   * Neither the name of Intel Corporation nor the names of its
- *     contributors may be used to endorse or promote products derived
- *     from this software without specific prior written permission.
- *
- * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
- * "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
- * LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
- * A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
- * OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
- * SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
- * LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
- * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
- * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
- * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
- * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
- */
-
-#ifndef _SCIC_USER_CALLBACK_H_
-#define _SCIC_USER_CALLBACK_H_
-
-/**
- * This file contains all of the interface methods/macros that must be
- *    implemented by an SCI Core user.
- *
- *
- */
-
-
-#include "sci_status.h"
-#include "scic_io_request.h"
-
-struct scic_sds_request;
-struct scic_sds_phy;
-struct scic_sds_port;
-struct scic_sds_remote_device;
-struct scic_sds_controller;
-
-/**
- * scic_cb_stall_execution() - This method is called when the core requires the
- *    OS driver to stall execution.  This method is utilized during
- *    initialization or non-performance paths only.
- * @microseconds: This parameter specifies the number of microseconds for which
- *    to stall.  The operating system driver is allowed to round this value up
- *    where necessary.
- *
- * none.
- */
-void scic_cb_stall_execution(
-	u32 microseconds);
-
-#ifndef SCI_GET_PHYSICAL_ADDRESS_OPTIMIZATION_ENABLED
-/**
- * scic_cb_io_request_get_physical_address() - This callback method asks the
- *    user to provide the physical address for the supplied virtual address
- *    when building an io request object.
- * @controller: This parameter is the core controller object handle.
- * @io_request: This parameter is the io request object handle for which the
- *    physical address is being requested.
- * @virtual_address: This paramter is the virtual address which is to be
- *    returned as a physical address.
- * @physical_address: The physical address for the supplied virtual address.
- *
- * None.
- */
-void scic_cb_io_request_get_physical_address(
-	struct scic_sds_controller *controller,
-	struct scic_sds_request *io_request,
-	void *virtual_address,
-	dma_addr_t *physical_address);
-#endif /* SCI_GET_PHYSICAL_ADDRESS_OPTIMIZATION_ENABLED */
-
-/**
- * scic_cb_io_request_get_transfer_length() - This callback method asks the
- *    user to provide the number of bytes to be transfered as part of this
- *    request.
- * @scic_user_io_request: This parameter points to the user's IO request
- *    object.  It is a cookie that allows the user to provide the necessary
- *    information for this callback.
- *
- * This method returns the number of payload data bytes to be transfered for
- * this IO request.
- */
-u32 scic_cb_io_request_get_transfer_length(
-	void *scic_user_io_request);
-
-/**
- * scic_cb_io_request_get_data_direction() - This callback method asks the user
- *    to provide the data direction for this request.
- * @scic_user_io_request: This parameter points to the user's IO request
- *    object.  It is a cookie that allows the user to provide the necessary
- *    information for this callback.
- *
- */
-enum dma_data_direction scic_cb_io_request_get_data_direction(void *req);
-
-#ifndef SCI_SGL_OPTIMIZATION_ENABLED
-/**
- * scic_cb_io_request_get_next_sge() - This callback method asks the user to
- *    provide the address to where the next Scatter-Gather Element is located.
- *    Details regarding usage: - Regarding the first SGE: the user should
- *    initialize an index, or a pointer, prior to construction of the request
- *    that will reference the very first scatter-gather element.  This is
- *    important since this method is called for every scatter-gather element,
- *    including the first element. - Regarding the last SGE: the user should
- *    return NULL from this method when this method is called and the SGL has
- *    exhausted all elements.
- * @scic_user_io_request: This parameter points to the user's IO request
- *    object.  It is a cookie that allows the user to provide the necessary
- *    information for this callback.
- * @current_sge_address: This parameter specifies the address for the current
- *    SGE (i.e. the one that has just processed).
- * @next_sge: An address specifying the location for the next scatter gather
- *    element to be processed.
- *
- * None
- */
-void scic_cb_io_request_get_next_sge(
-	void *scic_user_io_request,
-	void *current_sge_address,
-	void **next_sge);
-#endif /* SCI_SGL_OPTIMIZATION_ENABLED */
-
-/**
- * scic_cb_sge_get_address_field() - This callback method asks the user to
- *    provide the contents of the "address" field in the Scatter-Gather Element.
- * @scic_user_io_request: This parameter points to the user's IO request
- *    object.  It is a cookie that allows the user to provide the necessary
- *    information for this callback.
- * @sge_address: This parameter specifies the address for the SGE from which to
- *    retrieve the address field.
- *
- * A physical address specifying the contents of the SGE's address field.
- */
-dma_addr_t scic_cb_sge_get_address_field(
-	void *scic_user_io_request,
-	void *sge_address);
-
-/**
- * scic_cb_sge_get_length_field() - This callback method asks the user to
- *    provide the contents of the "length" field in the Scatter-Gather Element.
- * @scic_user_io_request: This parameter points to the user's IO request
- *    object.  It is a cookie that allows the user to provide the necessary
- *    information for this callback.
- * @sge_address: This parameter specifies the address for the SGE from which to
- *    retrieve the address field.
- *
- * This method returns the length field specified inside the SGE referenced by
- * the sge_address parameter.
- */
-u32 scic_cb_sge_get_length_field(
-	void *scic_user_io_request,
-	void *sge_address);
-
-/**
- * scic_cb_ssp_io_request_get_cdb_address() - This callback method asks the
- *    user to provide the address for the command descriptor block (CDB)
- *    associated with this IO request.
- * @scic_user_io_request: This parameter points to the user's IO request
- *    object.  It is a cookie that allows the user to provide the necessary
- *    information for this callback.
- *
- * This method returns the virtual address of the CDB.
- */
-void *scic_cb_ssp_io_request_get_cdb_address(
-	void *scic_user_io_request);
-
-/**
- * scic_cb_ssp_io_request_get_cdb_length() - This callback method asks the user
- *    to provide the length of the command descriptor block (CDB) associated
- *    with this IO request.
- * @scic_user_io_request: This parameter points to the user's IO request
- *    object.  It is a cookie that allows the user to provide the necessary
- *    information for this callback.
- *
- * This method returns the length of the CDB.
- */
-u32 scic_cb_ssp_io_request_get_cdb_length(
-	void *scic_user_io_request);
-
-/**
- * scic_cb_ssp_io_request_get_lun() - This callback method asks the user to
- *    provide the Logical Unit (LUN) associated with this IO request.
- * @scic_user_io_request: This parameter points to the user's IO request
- *    object.  It is a cookie that allows the user to provide the necessary
- *    information for this callback.
- *
- * The contents of the value returned from this callback are defined by the
- * protocol standard (e.g. T10 SAS specification).  Please refer to the
- * transport command information unit description in the associated standard.
- * This method returns the LUN associated with this request. This should be u64?
- */
-u32 scic_cb_ssp_io_request_get_lun(
-	void *scic_user_io_request);
-
-/**
- * scic_cb_ssp_io_request_get_task_attribute() - This callback method asks the
- *    user to provide the task attribute associated with this IO request.
- * @scic_user_io_request: This parameter points to the user's IO request
- *    object.  It is a cookie that allows the user to provide the necessary
- *    information for this callback.
- *
- * The contents of the value returned from this callback are defined by the
- * protocol standard (e.g. T10 SAS specification).  Please refer to the
- * transport command information unit description in the associated standard.
- * This method returns the task attribute associated with this IO request.
- */
-u32 scic_cb_ssp_io_request_get_task_attribute(
-	void *scic_user_io_request);
-
-/**
- * scic_cb_ssp_io_request_get_command_priority() - This callback method asks
- *    the user to provide the command priority associated with this IO request.
- * @scic_user_io_request: This parameter points to the user's IO request
- *    object.  It is a cookie that allows the user to provide the necessary
- *    information for this callback.
- *
- * The contents of the value returned from this callback are defined by the
- * protocol standard (e.g. T10 SAS specification).  Please refer to the
- * transport command information unit description in the associated standard.
- * This method returns the command priority associated with this IO request.
- */
-u32 scic_cb_ssp_io_request_get_command_priority(
-	void *scic_user_io_request);
-
-/**
- * scic_cb_io_request_do_copy_rx_frames() - This callback method asks the user
- *    if the received RX frame data is to be copied to the SGL or should be
- *    stored by the SCI core to be retrieved later with the
- *    scic_io_request_get_rx_frame().
- * @scic_user_io_request: This parameter points to the user's IO request
- *    object.  It is a cookie that allows the user to provide the necessary
- *    information for this callback.
- *
- * This method returns true if the SCI core should copy the received frame data
- * to the SGL location or false if the SCI user wants to retrieve the frame
- * data at a later time.
- */
-bool scic_cb_io_request_do_copy_rx_frames(
-	void *scic_user_io_request);
-
-/**
- * scic_cb_request_get_sat_protocol() - This callback method asks the user to
- *    return the SAT protocol definition for this IO request.  This method is
- *    only called by the SCI core if the request type constructed is SATA.
- * @scic_user_io_request: This parameter points to the user's IO request
- *    object.  It is a cookie that allows the user to provide the necessary
- *    information for this callback.
- *
- * This method returns one of the sat.h defined protocols for the given io
- * request.
- */
-u8 scic_cb_request_get_sat_protocol(
-	void *scic_user_io_request);
-
-
-/**
- * scic_cb_ssp_task_request_get_lun() - This method returns the Logical Unit to
- *    be utilized for this task management request.
- * @scic_user_task_request: This parameter points to the user's task request
- *    object.  It is a cookie that allows the user to provide the necessary
- *    information for this callback.
- *
- * The contents of the value returned from this callback are defined by the
- * protocol standard (e.g. T10 SAS specification).  Please refer to the
- * transport task information unit description in the associated standard. This
- * method returns the LUN associated with this request. This should be u64?
- */
-u32 scic_cb_ssp_task_request_get_lun(
-	void *scic_user_task_request);
-
-/**
- * scic_cb_ssp_task_request_get_function() - This method returns the task
- *    management function to be utilized for this task request.
- * @scic_user_task_request: This parameter points to the user's task request
- *    object.  It is a cookie that allows the user to provide the necessary
- *    information for this callback.
- *
- * The contents of the value returned from this callback are defined by the
- * protocol standard (e.g. T10 SAS specification).  Please refer to the
- * transport task information unit description in the associated standard. This
- * method returns an unsigned byte representing the task management function to
- * be performed.
- */
-u8 scic_cb_ssp_task_request_get_function(
-	void *scic_user_task_request);
-
-/**
- * scic_cb_ssp_task_request_get_io_tag_to_manage() - This method returns the
- *    task management IO tag to be managed. Depending upon the task management
- *    function the value returned from this method may be ignored.
- * @scic_user_task_request: This parameter points to the user's task request
- *    object.  It is a cookie that allows the user to provide the necessary
- *    information for this callback.
- *
- * This method returns an unsigned 16-bit word depicting the IO tag to be
- * managed.
- */
-u16 scic_cb_ssp_task_request_get_io_tag_to_manage(
-	void *scic_user_task_request);
-
-/**
- * scic_cb_ssp_task_request_get_response_data_address() - This callback method
- *    asks the user to provide the virtual address of the response data buffer
- *    for the supplied IO request.
- * @scic_user_task_request: This parameter points to the user's task request
- *    object.  It is a cookie that allows the user to provide the necessary
- *    information for this callback.
- *
- * This method returns the virtual address for the response data buffer
- * associated with this IO request.
- */
-void *scic_cb_ssp_task_request_get_response_data_address(
-	void *scic_user_task_request);
-
-/**
- * scic_cb_ssp_task_request_get_response_data_length() - This callback method
- *    asks the user to provide the length of the response data buffer for the
- *    supplied IO request.
- * @scic_user_task_request: This parameter points to the user's task request
- *    object.  It is a cookie that allows the user to provide the necessary
- *    information for this callback.
- *
- * This method returns the length of the response buffer data associated with
- * this IO request.
- */
-u32 scic_cb_ssp_task_request_get_response_data_length(
-	void *scic_user_task_request);
-
-/**
- * scic_cb_pci_get_bar() - In this method the user must return the base address
- *    register (BAR) value for the supplied base address register number.
- * @controller: The controller for which to retrieve the bar number.
- * @bar_number: This parameter depicts the BAR index/number to be read.
- *
- * Return a pointer value indicating the contents of the BAR. NULL indicates an
- * invalid BAR index/number was specified. All other values indicate a valid
- * VIRTUAL address from the BAR.
- */
-void *scic_cb_pci_get_bar(
-	struct scic_sds_controller *controller,
-	u16 bar_number);
-
-/**
- * scic_cb_get_virtual_address() - This callback method asks the user to
- *    provide the virtual address for the supplied physical address.
- * @controller: This parameter is the core controller object handle.
- * @physical_address: This parameter is the physical address which is to be
- *    returned as a virtual address.
- *
- * The method returns the virtual address for the supplied physical address.
- */
-void *scic_cb_get_virtual_address(
-	struct scic_sds_controller *controller,
-	dma_addr_t physical_address);
-
-#if !defined(DISABLE_ATAPI)
-/**
- * scic_cb_stp_packet_io_request_get_cdb_address() - This user callback gets
- *    from stp packet io's user request the CDB address.
- * @scic_user_io_request:
- *
- * The cdb adress.
- */
-void *scic_cb_stp_packet_io_request_get_cdb_address(
-	void *scic_user_io_request);
-
-/**
- * scic_cb_stp_packet_io_request_get_cdb_length() - This user callback gets
- *    from stp packet io's user request the CDB length.
- * @scic_user_io_request:
- *
- * The cdb length.
- */
-u32 scic_cb_stp_packet_io_request_get_cdb_length(
-	void *scic_user_io_request);
-#else /* !defined(DISABLE_ATAPI) */
-#define scic_cb_stp_packet_io_request_get_cdb_address(scic_user_io_request) NULL
-#define scic_cb_stp_packet_io_request_get_cdb_length(scic_user_io_request) 0
-#endif /* !defined(DISABLE_ATAPI) */
-
-
-#endif  /* _SCIC_USER_CALLBACK_H_ */
-
diff --git a/drivers/scsi/isci/deprecated.c b/drivers/scsi/isci/deprecated.c
deleted file mode 100644
index 0ee6679b810..00000000000
--- a/drivers/scsi/isci/deprecated.c
+++ /dev/null
@@ -1,485 +0,0 @@
-/*
- * This file is provided under a dual BSD/GPLv2 license.  When using or
- * redistributing this file, you may do so under either license.
- *
- * GPL LICENSE SUMMARY
- *
- * Copyright(c) 2008 - 2011 Intel Corporation. All rights reserved.
- *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of version 2 of the GNU General Public License as
- * published by the Free Software Foundation.
- *
- * This program is distributed in the hope that it will be useful, but
- * WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
- * General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 51 Franklin St - Fifth Floor, Boston, MA 02110-1301 USA.
- * The full GNU General Public License is included in this distribution
- * in the file called LICENSE.GPL.
- *
- * BSD LICENSE
- *
- * Copyright(c) 2008 - 2011 Intel Corporation. All rights reserved.
- * All rights reserved.
- *
- * Redistribution and use in source and binary forms, with or without
- * modification, are permitted provided that the following conditions
- * are met:
- *
- *   * Redistributions of source code must retain the above copyright
- *     notice, this list of conditions and the following disclaimer.
- *   * Redistributions in binary form must reproduce the above copyright
- *     notice, this list of conditions and the following disclaimer in
- *     the documentation and/or other materials provided with the
- *     distribution.
- *   * Neither the name of Intel Corporation nor the names of its
- *     contributors may be used to endorse or promote products derived
- *     from this software without specific prior written permission.
- *
- * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
- * "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
- * LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
- * A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
- * OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
- * SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
- * LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
- * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
- * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
- * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
- * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
- */
-
-
-/**
- * This file contains isci module object implementation.
- *
- *
- */
-
-#include "isci.h"
-#include "request.h"
-#include "sata.h"
-#include "task.h"
-
-
-/**
- * scic_cb_stall_execution() - This method is called when the core requires the
- *    OS driver to stall execution.  This method is utilized during
- *    initialization or non-performance paths only.
- * @microseconds: This parameter specifies the number of microseconds for which
- *    to stall.  The operating system driver is allowed to round this value up
- *    where necessary.
- *
- * none.
- */
-void scic_cb_stall_execution(
-	u32 microseconds)
-{
-	udelay(microseconds);
-}
-
-
-/**
- * scic_cb_io_request_get_physical_address() - This callback method asks the
- *    user to provide the physical address for the supplied virtual address
- *    when building an io request object.
- * @controller: This parameter is the core controller object handle.
- * @io_request: This parameter is the io request object handle for which the
- *    physical address is being requested.
- *
- *
- */
-void scic_cb_io_request_get_physical_address(
-	struct scic_sds_controller *controller,
-	struct scic_sds_request *io_request,
-	void *virtual_address,
-	dma_addr_t *physical_address)
-{
-	struct isci_request *request =
-		(struct isci_request *)sci_object_get_association(io_request);
-
-	char *requested_address = (char *)virtual_address;
-	char *base_address = (char *)request;
-
-	BUG_ON(requested_address < base_address);
-	BUG_ON((requested_address - base_address) >=
-			request->request_alloc_size);
-
-	*physical_address = request->request_daddr +
-		(requested_address - base_address);
-}
-
-/**
- * scic_cb_io_request_get_transfer_length() - This callback method asks the
- *    user to provide the number of bytes to be transfered as part of this
- *    request.
- * @scic_user_io_request: This parameter points to the user's IO request
- *    object.  It is a cookie that allows the user to provide the necessary
- *    information for this callback.
- *
- * This method returns the number of payload data bytes to be transfered for
- * this IO request.
- */
-u32 scic_cb_io_request_get_transfer_length(
-	void *scic_user_io_request)
-{
-	return isci_request_io_request_get_transfer_length(
-		       scic_user_io_request
-		       );
-}
-
-
-/**
- * scic_cb_io_request_get_data_direction() - This callback method asks the user
- *    to provide the data direction for this request.
- * @scic_user_io_request: This parameter points to the user's IO request
- *    object.  It is a cookie that allows the user to provide the necessary
- *    information for this callback.
- */
-enum dma_data_direction scic_cb_io_request_get_data_direction(void *req)
-{
-	return isci_request_io_request_get_data_direction(req);
-}
-
-
-/**
- * scic_cb_io_request_get_next_sge() - This callback method asks the user to
- *    provide the address to where the next Scatter-Gather Element is located.
- * @scic_user_io_request: This parameter points to the user's IO request
- *    object.  It is a cookie that allows the user to provide the necessary
- *    information for this callback.
- * @current_sge_address: This parameter specifies the address for the current
- *    SGE (i.e. the one that has just processed).
- *
- * An address specifying the location for the next scatter gather element to be
- * processed.
- */
-void scic_cb_io_request_get_next_sge(
-	void *scic_user_io_request,
-	void *current_sge_address,
-	void **next_sge)
-{
-	*next_sge = isci_request_io_request_get_next_sge(
-		scic_user_io_request,
-		current_sge_address
-		);
-}
-
-/**
- * scic_cb_sge_get_address_field() - This callback method asks the user to
- *    provide the contents of the "address" field in the Scatter-Gather Element.
- * @scic_user_io_request: This parameter points to the user's IO request
- *    object.  It is a cookie that allows the user to provide the necessary
- *    information for this callback.
- * @sge_address: This parameter specifies the address for the SGE from which to
- *    retrieve the address field.
- *
- * A physical address specifying the contents of the SGE's address field.
- */
-dma_addr_t scic_cb_sge_get_address_field(
-	void *scic_user_io_request,
-	void *sge_address)
-{
-	return isci_request_sge_get_address_field(
-		       scic_user_io_request,
-		       sge_address
-		       );
-}
-
-/**
- * scic_cb_sge_get_length_field() - This callback method asks the user to
- *    provide the contents of the "length" field in the Scatter-Gather Element.
- * @scic_user_io_request: This parameter points to the user's IO request
- *    object.  It is a cookie that allows the user to provide the necessary
- *    information for this callback.
- * @sge_address: This parameter specifies the address for the SGE from which to
- *    retrieve the address field.
- *
- * This method returns the length field specified inside the SGE referenced by
- * the sge_address parameter.
- */
-u32 scic_cb_sge_get_length_field(
-	void *scic_user_io_request,
-	void *sge_address)
-{
-	return isci_request_sge_get_length_field(
-		       scic_user_io_request,
-		       sge_address
-		       );
-}
-
-/**
- * scic_cb_ssp_io_request_get_cdb_address() - This callback method asks the
- *    user to provide the address for the command descriptor block (CDB)
- *    associated with this IO request.
- * @scic_user_io_request: This parameter points to the user's IO request
- *    object.  It is a cookie that allows the user to provide the necessary
- *    information for this callback.
- *
- * This method returns the virtual address of the CDB.
- */
-void *scic_cb_ssp_io_request_get_cdb_address(
-	void *scic_user_io_request)
-{
-	return isci_request_ssp_io_request_get_cdb_address(
-		       scic_user_io_request
-		       );
-}
-
-/**
- * scic_cb_ssp_io_request_get_cdb_length() - This callback method asks the user
- *    to provide the length of the command descriptor block (CDB) associated
- *    with this IO request.
- * @scic_user_io_request: This parameter points to the user's IO request
- *    object.  It is a cookie that allows the user to provide the necessary
- *    information for this callback.
- *
- * This method returns the length of the CDB.
- */
-u32 scic_cb_ssp_io_request_get_cdb_length(
-	void *scic_user_io_request)
-{
-	return isci_request_ssp_io_request_get_cdb_length(
-		       scic_user_io_request
-		       );
-}
-
-/**
- * scic_cb_ssp_io_request_get_lun() - This callback method asks the user to
- *    provide the Logical Unit (LUN) associated with this IO request.
- * @scic_user_io_request: This parameter points to the user's IO request
- *    object.  It is a cookie that allows the user to provide the necessary
- *    information for this callback.
- *
- * This method returns the LUN associated with this request. This should be u64?
- */
-u32 scic_cb_ssp_io_request_get_lun(
-	void *scic_user_io_request)
-{
-	return isci_request_ssp_io_request_get_lun(scic_user_io_request);
-}
-
-/**
- * scic_cb_ssp_io_request_get_task_attribute() - This callback method asks the
- *    user to provide the task attribute associated with this IO request.
- * @scic_user_io_request: This parameter points to the user's IO request
- *    object.  It is a cookie that allows the user to provide the necessary
- *    information for this callback.
- *
- * This method returns the task attribute associated with this IO request.
- */
-u32 scic_cb_ssp_io_request_get_task_attribute(
-	void *scic_user_io_request)
-{
-	return isci_request_ssp_io_request_get_task_attribute(
-		       scic_user_io_request
-		       );
-}
-
-/**
- * scic_cb_ssp_io_request_get_command_priority() - This callback method asks
- *    the user to provide the command priority associated with this IO request.
- * @scic_user_io_request: This parameter points to the user's IO request
- *    object.  It is a cookie that allows the user to provide the necessary
- *    information for this callback.
- *
- * This method returns the command priority associated with this IO request.
- */
-u32 scic_cb_ssp_io_request_get_command_priority(
-	void *scic_user_io_request)
-{
-	return isci_request_ssp_io_request_get_command_priority(
-		       scic_user_io_request
-		       );
-}
-
-/**
- * scic_cb_ssp_task_request_get_lun() - This method returns the Logical Unit to
- *    be utilized for this task management request.
- * @scic_user_task_request: This parameter points to the user's task request
- *    object.  It is a cookie that allows the user to provide the necessary
- *    information for this callback.
- *
- * This method returns the LUN associated with this request. This should be u64?
- */
-u32 scic_cb_ssp_task_request_get_lun(
-	void *scic_user_task_request)
-{
-	return isci_task_ssp_request_get_lun(
-		       (struct isci_request *)scic_user_task_request
-		       );
-}
-
-/**
- * scic_cb_ssp_task_request_get_function() - This method returns the task
- *    management function to be utilized for this task request.
- * @scic_user_task_request: This parameter points to the user's task request
- *    object.  It is a cookie that allows the user to provide the necessary
- *    information for this callback.
- *
- * This method returns an unsigned byte representing the task management
- * function to be performed.
- */
-u8 scic_cb_ssp_task_request_get_function(
-	void *scic_user_task_request)
-{
-	return isci_task_ssp_request_get_function(
-		       (struct isci_request *)scic_user_task_request
-		       );
-}
-
-/**
- * scic_cb_ssp_task_request_get_io_tag_to_manage() - This method returns the
- *    task management IO tag to be managed. Depending upon the task management
- *    function the value returned from this method may be ignored.
- * @scic_user_task_request: This parameter points to the user's task request
- *    object.  It is a cookie that allows the user to provide the necessary
- *    information for this callback.
- *
- * This method returns an unsigned 16-bit word depicting the IO tag to be
- * managed.
- */
-u16 scic_cb_ssp_task_request_get_io_tag_to_manage(
-	void *scic_user_task_request)
-{
-	return isci_task_ssp_request_get_io_tag_to_manage(
-		       (struct isci_request *)scic_user_task_request
-		       );
-}
-
-/**
- * scic_cb_ssp_task_request_get_response_data_address() - This callback method
- *    asks the user to provide the virtual address of the response data buffer
- *    for the supplied IO request.
- * @scic_user_task_request: This parameter points to the user's task request
- *    object.  It is a cookie that allows the user to provide the necessary
- *    information for this callback.
- *
- * This method returns the virtual address for the response data buffer
- * associated with this IO request.
- */
-void *scic_cb_ssp_task_request_get_response_data_address(
-	void *scic_user_task_request)
-{
-	return isci_task_ssp_request_get_response_data_address(
-		       (struct isci_request *)scic_user_task_request
-		       );
-}
-
-/**
- * scic_cb_ssp_task_request_get_response_data_length() - This callback method
- *    asks the user to provide the length of the response data buffer for the
- *    supplied IO request.
- * @scic_user_task_request: This parameter points to the user's task request
- *    object.  It is a cookie that allows the user to provide the necessary
- *    information for this callback.
- *
- * This method returns the length of the response buffer data associated with
- * this IO request.
- */
-u32 scic_cb_ssp_task_request_get_response_data_length(
-	void *scic_user_task_request)
-{
-	return isci_task_ssp_request_get_response_data_length(
-		       (struct isci_request *)scic_user_task_request
-		       );
-}
-
-#if !defined(DISABLE_ATAPI)
-/**
- * scic_cb_stp_packet_io_request_get_cdb_address() - This user callback asks
- *    the user to provide stp packet io's the CDB address.
- * @scic_user_io_request:
- *
- * The packet IO's cdb adress.
- */
-void *scic_cb_stp_packet_io_request_get_cdb_address(
-	void *scic_user_io_request)
-{
-	return isci_request_stp_packet_io_request_get_cdb_address(
-		       scic_user_io_request
-		       );
-}
-
-
-/**
- * scic_cb_stp_packet_io_request_get_cdb_length() - This user callback asks the
- *    user to provide stp packet io's the CDB length.
- * @scic_user_io_request:
- *
- * The packet IO's cdb length.
- */
-u32 scic_cb_stp_packet_io_request_get_cdb_length(
-	void *scic_user_io_request)
-{
-	return isci_request_stp_packet_io_request_get_cdb_length(
-		       scic_user_io_request
-		       );
-}
-#endif /* #if !defined(DISABLE_ATAPI) */
-
-
-/**
- * scic_cb_io_request_do_copy_rx_frames() - This callback method asks the user
- *    if the received RX frame data is to be copied to the SGL or should be
- *    stored by the SCI core to be retrieved later with the
- *    scic_io_request_get_rx_frame().
- * @scic_user_io_request: This parameter points to the user's IO request
- *    object.  It is a cookie that allows the user to provide the necessary
- *    information for this callback.
- *
- * This method returns true if the SCI core should copy the received frame data
- * to the SGL location or false if the SCI user wants to retrieve the frame
- * data at a later time.
- */
-bool scic_cb_io_request_do_copy_rx_frames(
-	void *scic_user_io_request)
-{
-	struct sas_task *task
-		= isci_request_access_task(
-		(struct isci_request *)scic_user_io_request
-		);
-
-	return (task->data_dir == DMA_NONE) ? false : true;
-}
-
-/**
- * scic_cb_get_virtual_address() - This callback method asks the user to
- *    provide the virtual address for the supplied physical address.
- * @controller: This parameter is the core controller object handle.
- * @physical_address: This parameter is the physical address which is to be
- *    returned as a virtual address.
- *
- * The method returns the virtual address for the supplied physical address.
- */
-void *scic_cb_get_virtual_address(
-	struct scic_sds_controller *controller,
-	dma_addr_t physical_address)
-{
-	void *virt_addr = (void *)phys_to_virt(physical_address);
-
-	return virt_addr;
-}
-
-/**
- * scic_cb_request_get_sat_protocol() - This callback method asks the user to
- *    return the SAT protocol definition for this IO request.  This method is
- *    only called by the SCI core if the request type constructed is SATA.
- * @scic_user_io_request: This parameter points to the user's IO request
- *    object.  It is a cookie that allows the user to provide the necessary
- *    information for this callback.
- *
- * This method returns one of the sat.h defined protocols for the given io
- * request.
- */
-u8 scic_cb_request_get_sat_protocol(
-	void *scic_user_io_request)
-{
-	return isci_sata_get_sat_protocol(
-		       (struct isci_request *)scic_user_io_request
-		       );
-}
diff --git a/drivers/scsi/isci/host.h b/drivers/scsi/isci/host.h
index 26768c5bbe0..06154a625ae 100644
--- a/drivers/scsi/isci/host.h
+++ b/drivers/scsi/isci/host.h
@@ -61,7 +61,6 @@
 /*#include "task.h"*/
 #include "timers.h"
 #include "remote_device.h"
-#include "scic_user_callback.h"
 
 #define DRV_NAME "isci"
 #define SCI_PCI_BAR_COUNT 2
diff --git a/drivers/scsi/isci/isci.h b/drivers/scsi/isci/isci.h
index 9ec91f8e5fd..9b9aa50954c 100644
--- a/drivers/scsi/isci/isci.h
+++ b/drivers/scsi/isci/isci.h
@@ -86,6 +86,8 @@
 #include "sci_status.h"
 #include "request.h"
 #include "events.h"
+#include "task.h"
+#include "sata.h"
 
 extern struct kmem_cache *isci_kmem_cache;
 extern struct isci_firmware *isci_firmware;
diff --git a/drivers/scsi/isci/remote_device.h b/drivers/scsi/isci/remote_device.h
index a208f81785e..48e37cfef6f 100644
--- a/drivers/scsi/isci/remote_device.h
+++ b/drivers/scsi/isci/remote_device.h
@@ -55,7 +55,6 @@
 
 #if !defined(_ISCI_REMOTE_DEVICE_H_)
 #define _ISCI_REMOTE_DEVICE_H_
-#include "scic_user_callback.h"
 
 struct isci_host;
 struct scic_sds_remote_device;
diff --git a/drivers/scsi/isci/sata.c b/drivers/scsi/isci/sata.c
index 19b0eea93ca..6fbf15944b2 100644
--- a/drivers/scsi/isci/sata.c
+++ b/drivers/scsi/isci/sata.c
@@ -324,7 +324,7 @@ int isci_task_send_lu_reset_sata(
 
 	/* Leave SRST high for a bit. */
 	#define ISCI_SRST_ASSERT_DELAY 100 /* usecs */
-	scic_cb_stall_execution(ISCI_SRST_ASSERT_DELAY);
+	udelay(ISCI_SRST_ASSERT_DELAY);
 
 	/* Deassert SRST. */
 	isci_task_build_tmf(&tmf, isci_device, isci_tmf_sata_srst_low,