]> Pileus Git - ~andy/linux/blobdiff - drivers/staging/sep/sep_driver.c
staging: sep: Remove un-needed debug prints
[~andy/linux] / drivers / staging / sep / sep_driver.c
index ff9df36d5461cc674e3fa828abe1be20c55bc446..a1b0bfe5e0035e0d5702532a45982ba1e883e4e7 100644 (file)
@@ -53,6 +53,7 @@
 #include <asm/cacheflush.h>
 #include <linux/sched.h>
 #include <linux/delay.h>
+#include <linux/jiffies.h>
 #include <linux/rar_register.h>
 
 #include "../memrar/memrar.h"
@@ -90,15 +91,15 @@ static int sep_load_firmware(struct sep_device *sep)
        char *res_name = "resident.image.bin";
        char *extapp_name = "extapp.image.bin";
        int error ;
-       unsigned int work1, work2, work3;
+       unsigned long work1, work2, work3;
 
-       /* set addresses and load resident */
+       /* Set addresses and load resident */
        sep->resident_bus = sep->rar_bus;
        sep->resident_addr = sep->rar_addr;
 
        error = request_firmware(&fw, res_name, &sep->pdev->dev);
        if (error) {
-               dev_warn(&sep->pdev->dev, "cant request resident fw\n");
+               dev_warn(&sep->pdev->dev, "can't request resident fw\n");
                return error;
        }
 
@@ -106,27 +107,23 @@ static int sep_load_firmware(struct sep_device *sep)
        sep->resident_size = fw->size;
        release_firmware(fw);
 
-       dev_dbg(&sep->pdev->dev, "resident virtual is %p\n",
-               sep->resident_addr);
        dev_dbg(&sep->pdev->dev, "resident bus is %lx\n",
                (unsigned long)sep->resident_bus);
-       dev_dbg(&sep->pdev->dev, "resident size is %08x\n",
-               sep->resident_size);
 
-       /* set addresses for dcache (no loading needed) */
-       work1 = (unsigned int)sep->resident_bus;
-       work2 = (unsigned int)sep->resident_size;
+       /* Set addresses for dcache (no loading needed) */
+       work1 = (unsigned long)sep->resident_bus;
+       work2 = (unsigned long)sep->resident_size;
        work3 = (work1 + work2 + (1024 * 4)) & 0xfffff000;
        sep->dcache_bus = (dma_addr_t)work3;
 
-       work1 = (unsigned int)sep->resident_addr;
-       work2 = (unsigned int)sep->resident_size;
+       work1 = (unsigned long)sep->resident_addr;
+       work2 = (unsigned long)sep->resident_size;
        work3 = (work1 + work2 + (1024 * 4)) & 0xfffff000;
        sep->dcache_addr = (void *)work3;
 
        sep->dcache_size = 1024 * 128;
 
-       /* set addresses and load cache */
+       /* Set addresses and load cache */
        sep->cache_bus = sep->dcache_bus + sep->dcache_size;
        sep->cache_addr = sep->dcache_addr + sep->dcache_size;
 
@@ -140,14 +137,10 @@ static int sep_load_firmware(struct sep_device *sep)
        sep->cache_size = fw->size;
        release_firmware(fw);
 
-       dev_dbg(&sep->pdev->dev, "cache virtual is %p\n",
-               sep->cache_addr);
        dev_dbg(&sep->pdev->dev, "cache bus is %08lx\n",
                (unsigned long)sep->cache_bus);
-       dev_dbg(&sep->pdev->dev, "cache size is %08x\n",
-               sep->cache_size);
 
-       /* set addresses and load extapp */
+       /* Set addresses and load extapp */
        sep->extapp_bus = sep->cache_bus + (1024 * 370);
        sep->extapp_addr = sep->cache_addr + (1024 * 370);
 
@@ -161,12 +154,8 @@ static int sep_load_firmware(struct sep_device *sep)
        sep->extapp_size = fw->size;
        release_firmware(fw);
 
-       dev_dbg(&sep->pdev->dev, "extapp virtual is %p\n",
-               sep->extapp_addr);
        dev_dbg(&sep->pdev->dev, "extapp bus is %08llx\n",
                (unsigned long long)sep->extapp_bus);
-       dev_dbg(&sep->pdev->dev, "extapp size is %08x\n",
-               sep->extapp_size);
 
        return error;
 }
@@ -177,7 +166,7 @@ MODULE_FIRMWARE("sep/extapp.image.bin");
 
 /**
  *     sep_dump_message - dump the message that is pending
- *     @sep: sep device
+ *     @sep: SEP device
  */
 static void sep_dump_message(struct sep_device *sep)
 {
@@ -205,7 +194,7 @@ static int sep_map_and_alloc_shared_area(struct sep_device *sep)
                return -ENOMEM;
        }
        dev_dbg(&sep->pdev->dev,
-               "sep: shared_addr %x bytes @%p (bus %llx)\n",
+               "shared_addr %zx bytes @%p (bus %llx)\n",
                                sep->shared_size, sep->shared_addr,
                                (unsigned long long)sep->shared_bus);
        return 0;
@@ -217,7 +206,6 @@ static int sep_map_and_alloc_shared_area(struct sep_device *sep)
  */
 static void sep_unmap_and_free_shared_area(struct sep_device *sep)
 {
-       dev_dbg(&sep->pdev->dev, "shared area unmap and free\n");
        dma_free_coherent(&sep->pdev->dev, sep->shared_size,
                                sep->shared_addr, sep->shared_bus);
 }
@@ -249,36 +237,29 @@ static int sep_singleton_open(struct inode *inode_ptr, struct file *file_ptr)
        struct sep_device *sep;
 
        /*
-        * Get the sep device structure and use it for the
+        * Get the SEP device structure and use it for the
         * private_data field in filp for other methods
         */
        sep = sep_dev;
 
        file_ptr->private_data = sep;
 
-       dev_dbg(&sep->pdev->dev, "Singleton open for pid %d\n",
-               current->pid);
-
-       dev_dbg(&sep->pdev->dev, "calling test and set for singleton 0\n");
        if (test_and_set_bit(0, &sep->singleton_access_flag)) {
                error = -EBUSY;
                goto end_function;
        }
 
-       dev_dbg(&sep->pdev->dev,
-               "sep_singleton_open end\n");
 end_function:
-
        return error;
 }
 
 /**
  *     sep_open - device open method
- *     @inode: inode of sep device
- *     @filp: file handle to sep device
+ *     @inode: inode of SEP device
+ *     @filp: file handle to SEP device
  *
  *     Open method for the SEP device. Called when userspace opens
- *     the SEP device node. 
+ *     the SEP device node.
  *
  *     Returns zero on success otherwise an error code.
  */
@@ -287,7 +268,7 @@ static int sep_open(struct inode *inode, struct file *filp)
        struct sep_device *sep;
 
        /*
-        * Get the sep device structure and use it for the
+        * Get the SEP device structure and use it for the
         * private_data field in filp for other methods
         */
        sep = sep_dev;
@@ -312,16 +293,14 @@ static int sep_singleton_release(struct inode *inode, struct file *filp)
 {
        struct sep_device *sep = filp->private_data;
 
-       dev_dbg(&sep->pdev->dev, "Singleton release for pid %d\n",
-                                                       current->pid);
        clear_bit(0, &sep->singleton_access_flag);
        return 0;
 }
 
 /**
  *     sep_request_daemonopen - request daemon open method
- *     @inode: inode of sep device
- *     @filp: file handle to sep device
+ *     @inode: inode of SEP device
+ *     @filp: file handle to SEP device
  *
  *     Open method for the SEP request daemon. Called when
  *     request daemon in userspace opens the SEP device node.
@@ -339,7 +318,6 @@ static int sep_request_daemon_open(struct inode *inode, struct file *filp)
                current->pid);
 
        /* There is supposed to be only one request daemon */
-       dev_dbg(&sep->pdev->dev, "calling test and set for req_dmon open 0\n");
        if (test_and_set_bit(0, &sep->request_daemon_open))
                error = -EBUSY;
        return error;
@@ -356,10 +334,10 @@ static int sep_request_daemon_release(struct inode *inode, struct file *filp)
 {
        struct sep_device *sep = filp->private_data;
 
-       dev_dbg(&sep->pdev->dev, "Reques daemon release for pid %d\n",
+       dev_dbg(&sep->pdev->dev, "Request daemon release for pid %d\n",
                current->pid);
 
-       /* clear the request_daemon_open flag */
+       /* Clear the request_daemon_open flag */
        clear_bit(0, &sep->request_daemon_open);
        return 0;
 }
@@ -375,20 +353,15 @@ static int sep_req_daemon_send_reply_command_handler(struct sep_device *sep)
 {
        unsigned long lck_flags;
 
-       dev_dbg(&sep->pdev->dev,
-               "sep_req_daemon_send_reply_command_handler start\n");
-
        sep_dump_message(sep);
 
-       /* counters are lockable region */
+       /* Counters are lockable region */
        spin_lock_irqsave(&sep->snd_rply_lck, lck_flags);
        sep->send_ct++;
        sep->reply_ct++;
 
-       /* send the interrupt to SEP */
-       sep_write_reg(sep, HW_HOST_HOST_SEP_GPR2_REG_ADDR,
-               sep->send_ct);
-
+       /* Send the interrupt to SEP */
+       sep_write_reg(sep, HW_HOST_HOST_SEP_GPR2_REG_ADDR, sep->send_ct);
        sep->send_ct++;
 
        spin_unlock_irqrestore(&sep->snd_rply_lck, lck_flags);
@@ -397,9 +370,6 @@ static int sep_req_daemon_send_reply_command_handler(struct sep_device *sep)
                "sep_req_daemon_send_reply send_ct %lx reply_ct %lx\n",
                sep->send_ct, sep->reply_ct);
 
-       dev_dbg(&sep->pdev->dev,
-               "sep_req_daemon_send_reply_command_handler end\n");
-
        return 0;
 }
 
@@ -408,21 +378,19 @@ static int sep_req_daemon_send_reply_command_handler(struct sep_device *sep)
  *     sep_free_dma_table_data_handler - free DMA table
  *     @sep: pointere to struct sep_device
  *
- *     Handles the request to  free dma table for synchronic actions
+ *     Handles the request to  free DMA table for synchronic actions
  */
 static int sep_free_dma_table_data_handler(struct sep_device *sep)
 {
        int count;
        int dcb_counter;
-       /* pointer to the current dma_resource struct */
+       /* Pointer to the current dma_resource struct */
        struct sep_dma_resource *dma;
 
-       dev_dbg(&sep->pdev->dev, "sep_free_dma_table_data_handler start\n");
-
        for (dcb_counter = 0; dcb_counter < sep->nr_dcb_creat; dcb_counter++) {
                dma = &sep->dma_res_arr[dcb_counter];
 
-               /* unmap and free input map array */
+               /* Unmap and free input map array */
                if (dma->in_map_array) {
                        for (count = 0; count < dma->in_num_pages; count++) {
                                dma_unmap_page(&sep->pdev->dev,
@@ -433,7 +401,7 @@ static int sep_free_dma_table_data_handler(struct sep_device *sep)
                        kfree(dma->in_map_array);
                }
 
-               /* unmap output map array, DON'T free it yet */
+               /* Unmap output map array, DON'T free it yet */
                if (dma->out_map_array) {
                        for (count = 0; count < dma->out_num_pages; count++) {
                                dma_unmap_page(&sep->pdev->dev,
@@ -444,7 +412,7 @@ static int sep_free_dma_table_data_handler(struct sep_device *sep)
                        kfree(dma->out_map_array);
                }
 
-               /* free page cache for output */
+               /* Free page cache for output */
                if (dma->in_page_array) {
                        for (count = 0; count < dma->in_num_pages; count++) {
                                flush_dcache_page(dma->in_page_array[count]);
@@ -463,13 +431,13 @@ static int sep_free_dma_table_data_handler(struct sep_device *sep)
                        kfree(dma->out_page_array);
                }
 
-               /* reset all the values */
-               dma->in_page_array = 0;
-               dma->out_page_array = 0;
+               /* Reset all the values */
+               dma->in_page_array = NULL;
+               dma->out_page_array = NULL;
                dma->in_num_pages = 0;
                dma->out_num_pages = 0;
-               dma->in_map_array = 0;
-               dma->out_map_array = 0;
+               dma->in_map_array = NULL;
+               dma->out_map_array = NULL;
                dma->in_map_num_entries = 0;
                dma->out_map_num_entries = 0;
        }
@@ -477,7 +445,6 @@ static int sep_free_dma_table_data_handler(struct sep_device *sep)
        sep->nr_dcb_creat = 0;
        sep->num_lli_tables_created = 0;
 
-       dev_dbg(&sep->pdev->dev, "sep_free_dma_table_data_handler end\n");
        return 0;
 }
 
@@ -496,19 +463,14 @@ static int sep_request_daemon_mmap(struct file  *filp,
        dma_addr_t bus_address;
        int error = 0;
 
-       dev_dbg(&sep->pdev->dev, "daemon mmap start\n");
-
        if ((vma->vm_end - vma->vm_start) > SEP_DRIVER_MMMAP_AREA_SIZE) {
                error = -EINVAL;
                goto end_function;
        }
 
-       /* get physical address */
+       /* Get physical address */
        bus_address = sep->shared_bus;
 
-       dev_dbg(&sep->pdev->dev, "bus_address is %08lx\n",
-                                       (unsigned long)bus_address);
-
        if (remap_pfn_range(vma, vma->vm_start, bus_address >> PAGE_SHIFT,
                vma->vm_end - vma->vm_start, vma->vm_page_prot)) {
 
@@ -518,13 +480,12 @@ static int sep_request_daemon_mmap(struct file  *filp,
        }
 
 end_function:
-       dev_dbg(&sep->pdev->dev, "daemon mmap end\n");
        return error;
 }
 
 /**
  *     sep_request_daemon_poll - poll implementation
- *     @sep: struct sep_device * for current sep device
+ *     @sep: struct sep_device * for current SEP device
  *     @filp: struct file * for open file
  *     @wait: poll_table * for poll
  *
@@ -539,15 +500,13 @@ static unsigned int sep_request_daemon_poll(struct file *filp,
        unsigned long lck_flags;
        struct sep_device *sep = filp->private_data;
 
-       dev_dbg(&sep->pdev->dev, "daemon poll: start\n");
-
        poll_wait(filp, &sep->event_request_daemon, wait);
 
        dev_dbg(&sep->pdev->dev, "daemon poll: send_ct is %lx reply ct is %lx\n",
                                                sep->send_ct, sep->reply_ct);
 
        spin_lock_irqsave(&sep->snd_rply_lck, lck_flags);
-       /* check if the data is ready */
+       /* Check if the data is ready */
        if (sep->send_ct == sep->reply_ct) {
                spin_unlock_irqrestore(&sep->snd_rply_lck, lck_flags);
 
@@ -555,13 +514,13 @@ static unsigned int sep_request_daemon_poll(struct file *filp,
                dev_dbg(&sep->pdev->dev,
                        "daemon poll: data check (GPR2) is %x\n", retval2);
 
-               /* check if PRINT request */
+               /* Check if PRINT request */
                if ((retval2 >> 30) & 0x1) {
                        dev_dbg(&sep->pdev->dev, "daemon poll: PRINTF request in\n");
                        mask |= POLLIN;
                        goto end_function;
                }
-               /* check if NVS request */
+               /* Check if NVS request */
                if (retval2 >> 31) {
                        dev_dbg(&sep->pdev->dev, "daemon poll: NVS request in\n");
                        mask |= POLLPRI | POLLWRNORM;
@@ -573,7 +532,6 @@ static unsigned int sep_request_daemon_poll(struct file *filp,
                mask = 0;
        }
 end_function:
-       dev_dbg(&sep->pdev->dev, "daemon poll: exit\n");
        return mask;
 }
 
@@ -591,12 +549,11 @@ static int sep_release(struct inode *inode, struct file *filp)
        dev_dbg(&sep->pdev->dev, "Release for pid %d\n", current->pid);
 
        mutex_lock(&sep->sep_mutex);
-       /* is this the process that has a transaction open?
+       /* Is this the process that has a transaction open?
         * If so, lets reset pid_doing_transaction to 0 and
         * clear the in use flags, and then wake up sep_event
         * so that other processes can do transactions
         */
-       dev_dbg(&sep->pdev->dev, "waking up event and mmap_event\n");
        if (sep->pid_doing_transaction == current->pid) {
                clear_bit(SEP_MMAP_LOCK_BIT, &sep->in_use_flags);
                clear_bit(SEP_SEND_MSG_LOCK_BIT, &sep->in_use_flags);
@@ -614,7 +571,7 @@ static int sep_release(struct inode *inode, struct file *filp)
  *     @filp: pointer to struct file
  *     @vma: pointer to vm_area_struct
  *
- *     Called on an mmap of our space via the normal sep device
+ *     Called on an mmap of our space via the normal SEP device
  */
 static int sep_mmap(struct file *filp, struct vm_area_struct *vma)
 {
@@ -622,8 +579,6 @@ static int sep_mmap(struct file *filp, struct vm_area_struct *vma)
        struct sep_device *sep = filp->private_data;
        unsigned long error = 0;
 
-       dev_dbg(&sep->pdev->dev, "mmap start\n");
-
        /* Set the transaction busy (own the device) */
        wait_event_interruptible(sep->event,
                test_and_set_bit(SEP_MMAP_LOCK_BIT,
@@ -636,8 +591,8 @@ static int sep_mmap(struct file *filp, struct vm_area_struct *vma)
        /*
         * The pid_doing_transaction indicates that this process
         * now owns the facilities to performa a transaction with
-        * the sep. While this process is performing a transaction,
-        * no other process who has the sep device open can perform
+        * the SEP. While this process is performing a transaction,
+        * no other process who has the SEP device open can perform
         * any transactions. This method allows more than one process
         * to have the device open at any given time, which provides
         * finer granularity for device utilization by multiple
@@ -647,12 +602,12 @@ static int sep_mmap(struct file *filp, struct vm_area_struct *vma)
        sep->pid_doing_transaction = current->pid;
        mutex_unlock(&sep->sep_mutex);
 
-       /* zero the pools and the number of data pool alocation pointers */
+       /* Zero the pools and the number of data pool alocation pointers */
        sep->data_pool_bytes_allocated = 0;
        sep->num_of_data_allocations = 0;
 
        /*
-        * check that the size of the mapped range is as the size of the message
+        * Check that the size of the mapped range is as the size of the message
         * shared area
         */
        if ((vma->vm_end - vma->vm_start) > SEP_DRIVER_MMMAP_AREA_SIZE) {
@@ -662,31 +617,26 @@ static int sep_mmap(struct file *filp, struct vm_area_struct *vma)
 
        dev_dbg(&sep->pdev->dev, "shared_addr is %p\n", sep->shared_addr);
 
-       /* get bus address */
+       /* Get bus address */
        bus_addr = sep->shared_bus;
 
-       dev_dbg(&sep->pdev->dev,
-               "bus_address is %lx\n", (unsigned long)bus_addr);
-
        if (remap_pfn_range(vma, vma->vm_start, bus_addr >> PAGE_SHIFT,
                vma->vm_end - vma->vm_start, vma->vm_page_prot)) {
                dev_warn(&sep->pdev->dev, "remap_page_range failed\n");
                error = -EAGAIN;
                goto end_function_with_error;
        }
-       dev_dbg(&sep->pdev->dev, "mmap end\n");
        goto end_function;
 
 end_function_with_error:
-       /* clear the bit */
+       /* Clear the bit */
        clear_bit(SEP_MMAP_LOCK_BIT, &sep->in_use_flags);
        mutex_lock(&sep->sep_mutex);
        sep->pid_doing_transaction = 0;
        mutex_unlock(&sep->sep_mutex);
 
-       /* raise event for stuck contextes */
+       /* Raise event for stuck contextes */
 
-       dev_warn(&sep->pdev->dev, "mmap error - waking up event\n");
        wake_up(&sep->event);
 
 end_function:
@@ -710,8 +660,6 @@ static unsigned int sep_poll(struct file *filp, poll_table *wait)
 
        struct sep_device *sep = filp->private_data;
 
-       dev_dbg(&sep->pdev->dev, "poll: start\n");
-
        /* Am I the process that owns the transaction? */
        mutex_lock(&sep->sep_mutex);
        if (current->pid != sep->pid_doing_transaction) {
@@ -722,14 +670,13 @@ static unsigned int sep_poll(struct file *filp, poll_table *wait)
        }
        mutex_unlock(&sep->sep_mutex);
 
-       /* check if send command or send_reply were activated previously */
+       /* Check if send command or send_reply were activated previously */
        if (!test_bit(SEP_SEND_MSG_LOCK_BIT, &sep->in_use_flags)) {
-               dev_warn(&sep->pdev->dev, "poll; lock bit set\n");
                mask = POLLERR;
                goto end_function;
        }
 
-       /* add the event to the polling wait table */
+       /* Add the event to the polling wait table */
        dev_dbg(&sep->pdev->dev, "poll: calling wait sep_event\n");
 
        poll_wait(filp, &sep->event, wait);
@@ -737,7 +684,7 @@ static unsigned int sep_poll(struct file *filp, poll_table *wait)
        dev_dbg(&sep->pdev->dev, "poll: send_ct is %lx reply ct is %lx\n",
                sep->send_ct, sep->reply_ct);
 
-       /* check if error occured during poll */
+       /* Check if error occured during poll */
        retval2 = sep_read_reg(sep, HW_HOST_SEP_HOST_GPR3_REG_ADDR);
        if (retval2 != 0x0) {
                dev_warn(&sep->pdev->dev, "poll; poll error %x\n", retval2);
@@ -753,25 +700,24 @@ static unsigned int sep_poll(struct file *filp, poll_table *wait)
                dev_dbg(&sep->pdev->dev, "poll: data ready check (GPR2)  %x\n",
                        retval);
 
-               /* check if printf request  */
+               /* Check if printf request  */
                if ((retval >> 30) & 0x1) {
-                       dev_dbg(&sep->pdev->dev, "poll: sep printf request\n");
+                       dev_dbg(&sep->pdev->dev, "poll: SEP printf request\n");
                        wake_up(&sep->event_request_daemon);
                        goto end_function;
                }
 
-               /* check if the this is sep reply or request */
+               /* Check if the this is SEP reply or request */
                if (retval >> 31) {
-                       dev_dbg(&sep->pdev->dev, "poll: sep request\n");
+                       dev_dbg(&sep->pdev->dev, "poll: SEP request\n");
                        wake_up(&sep->event_request_daemon);
                } else {
                        dev_dbg(&sep->pdev->dev, "poll: normal return\n");
-                       /* in case it is again by send_reply_comand */
+                       /* In case it is again by send_reply_comand */
                        clear_bit(SEP_SEND_MSG_LOCK_BIT, &sep->in_use_flags);
                        sep_dump_message(sep);
                        dev_dbg(&sep->pdev->dev,
-                               "poll; sep reply POLLIN | POLLRDNORM\n");
-
+                               "poll; SEP reply POLLIN | POLLRDNORM\n");
                        mask |= POLLIN | POLLRDNORM;
                }
        } else {
@@ -782,7 +728,6 @@ static unsigned int sep_poll(struct file *filp, poll_table *wait)
        }
 
 end_function:
-       dev_dbg(&sep->pdev->dev, "poll: end\n");
        return mask;
 }
 
@@ -803,19 +748,17 @@ static u32 *sep_time_address(struct sep_device *sep)
  *     @sep: the SEP we are setting the time for
  *
  *     Calculates time and sets it at the predefined address.
- *     Called with the sep mutex held.
+ *     Called with the SEP mutex held.
  */
 static unsigned long sep_set_time(struct sep_device *sep)
 {
        struct timeval time;
-       u32 *time_addr; /* address of time as seen by the kernel */
-
+       u32 *time_addr; /* Address of time as seen by the kernel */
 
-       dev_dbg(&sep->pdev->dev, "sep:sep_set_time start\n");
 
        do_gettimeofday(&time);
 
-       /* set value in the SYSTEM MEMORY offset */
+       /* Set value in the SYSTEM MEMORY offset */
        time_addr = sep_time_address(sep);
 
        time_addr[0] = SEP_TIME_VAL_TOKEN;
@@ -830,21 +773,19 @@ static unsigned long sep_set_time(struct sep_device *sep)
 
 /**
  *     sep_set_caller_id_handler - insert caller id entry
- *     @sep: sep device
+ *     @sep: SEP device
  *     @arg: pointer to struct caller_id_struct
  *
  *     Inserts the data into the caller id table. Note that this function
  *     falls under the ioctl lock
  */
-static int sep_set_caller_id_handler(struct sep_device *sep, u32 arg)
+static int sep_set_caller_id_handler(struct sep_device *sep, unsigned long arg)
 {
        void __user *hash;
        int   error = 0;
        int   i;
        struct caller_id_struct command_args;
 
-       dev_dbg(&sep->pdev->dev, "sep_set_caller_id_handler start\n");
-
        for (i = 0; i < SEP_CALLER_ID_TABLE_NUM_ENTRIES; i++) {
                if (sep->caller_id_table[i].pid == 0)
                        break;
@@ -858,7 +799,7 @@ static int sep_set_caller_id_handler(struct sep_device *sep, u32 arg)
                goto end_function;
        }
 
-       /* copy the data */
+       /* Copy the data */
        if (copy_from_user(&command_args, (void __user *)arg,
                sizeof(command_args))) {
                error = -EFAULT;
@@ -888,25 +829,22 @@ static int sep_set_caller_id_handler(struct sep_device *sep, u32 arg)
                hash, command_args.callerIdSizeInBytes))
                error = -EFAULT;
 end_function:
-       dev_dbg(&sep->pdev->dev, "sep_set_caller_id_handler end\n");
        return error;
 }
 
 /**
  *     sep_set_current_caller_id - set the caller id
- *     @sep: pointer to struct_sep
+ *     @sep: pointer to struct_sep_device
  *
- *     Set the caller ID (if it exists) to the sep. Note that this
+ *     Set the caller ID (if it exists) to the SEP. Note that this
  *     function falls under the ioctl lock
  */
 static int sep_set_current_caller_id(struct sep_device *sep)
 {
        int i;
+       u32 *hash_buf_ptr;
 
-       dev_dbg(&sep->pdev->dev, "sep_set_current_caller_id start\n");
-       dev_dbg(&sep->pdev->dev, "current process is %d\n", current->pid);
-
-       /* zero the previous value */
+       /* Zero the previous value */
        memset(sep->shared_addr + SEP_CALLER_ID_OFFSET_BYTES,
                                        0, SEP_CALLER_ID_HASH_SIZE_IN_BYTES);
 
@@ -920,13 +858,19 @@ static int sep_set_current_caller_id(struct sep_device *sep)
                        break;
                }
        }
-       dev_dbg(&sep->pdev->dev, "sep_set_current_caller_id end\n");
+       /* Ensure data is in little endian */
+       hash_buf_ptr = (u32 *)sep->shared_addr +
+               SEP_CALLER_ID_OFFSET_BYTES;
+
+       for (i = 0; i < SEP_CALLER_ID_HASH_SIZE_IN_WORDS; i++)
+               hash_buf_ptr[i] = cpu_to_le32(hash_buf_ptr[i]);
+
        return 0;
 }
 
 /**
  *     sep_send_command_handler - kick off a command
- *     @sep: sep being signalled
+ *     @sep: SEP being signalled
  *
  *     This function raises interrupt to SEP that signals that is has a new
  *     command from the host
@@ -938,21 +882,17 @@ static int sep_send_command_handler(struct sep_device *sep)
        unsigned long lck_flags;
        int error = 0;
 
-       dev_dbg(&sep->pdev->dev, "sep_send_command_handler start\n");
-
        if (test_and_set_bit(SEP_SEND_MSG_LOCK_BIT, &sep->in_use_flags)) {
                error = -EPROTO;
                goto end_function;
        }
        sep_set_time(sep);
 
-       /* only Medfield has caller id */
-       if (sep->mrst == 0)
-               sep_set_current_caller_id(sep);
+       sep_set_current_caller_id(sep);
 
        sep_dump_message(sep);
 
-       /* update counter */
+       /* Update counter */
        spin_lock_irqsave(&sep->snd_rply_lck, lck_flags);
        sep->send_ct++;
        spin_unlock_irqrestore(&sep->snd_rply_lck, lck_flags);
@@ -961,17 +901,16 @@ static int sep_send_command_handler(struct sep_device *sep)
                "sep_send_command_handler send_ct %lx reply_ct %lx\n",
                                                sep->send_ct, sep->reply_ct);
 
-       /* send interrupt to SEP */
+       /* Send interrupt to SEP */
        sep_write_reg(sep, HW_HOST_HOST_SEP_GPR0_REG_ADDR, 0x2);
 
 end_function:
-       dev_dbg(&sep->pdev->dev, "sep_send_command_handler end\n");
        return error;
 }
 
 /**
  *     sep_allocate_data_pool_memory_handler -allocate pool memory
- *     @sep: pointer to struct_sep
+ *     @sep: pointer to struct sep_device
  *     @arg: pointer to struct alloc_struct
  *
  *     This function handles the allocate data pool memory request
@@ -989,9 +928,6 @@ static int sep_allocate_data_pool_memory_handler(struct sep_device *sep,
        /* Holds the allocated buffer address in the system memory pool */
        u32 *token_addr;
 
-       dev_dbg(&sep->pdev->dev,
-               "sep_allocate_data_pool_memory_handler start\n");
-
        if (copy_from_user(&command_args, (void __user *)arg,
                                        sizeof(struct alloc_struct))) {
                error = -EFAULT;
@@ -1006,52 +942,35 @@ static int sep_allocate_data_pool_memory_handler(struct sep_device *sep,
        }
 
        dev_dbg(&sep->pdev->dev,
-               "bytes_allocated: %x\n", (int)sep->data_pool_bytes_allocated);
+               "data pool bytes_allocated: %x\n", (int)sep->data_pool_bytes_allocated);
        dev_dbg(&sep->pdev->dev,
                "offset: %x\n", SEP_DRIVER_DATA_POOL_AREA_OFFSET_IN_BYTES);
        /* Set the virtual and bus address */
        command_args.offset = SEP_DRIVER_DATA_POOL_AREA_OFFSET_IN_BYTES +
                sep->data_pool_bytes_allocated;
 
-       dev_dbg(&sep->pdev->dev,
-               "command_args.offset: %x\n", command_args.offset);
-
-       /* Place in the shared area that is known by the sep */
+       /* Place in the shared area that is known by the SEP */
        token_addr = (u32 *)(sep->shared_addr +
                SEP_DRIVER_DATA_POOL_ALLOCATION_OFFSET_IN_BYTES +
                (sep->num_of_data_allocations)*2*sizeof(u32));
 
-       dev_dbg(&sep->pdev->dev, "allocation offset: %x\n",
-               SEP_DRIVER_DATA_POOL_ALLOCATION_OFFSET_IN_BYTES);
-       dev_dbg(&sep->pdev->dev, "data pool token addr is %p\n", token_addr);
-
        token_addr[0] = SEP_DATA_POOL_POINTERS_VAL_TOKEN;
        token_addr[1] = (u32)sep->shared_bus +
                SEP_DRIVER_DATA_POOL_AREA_OFFSET_IN_BYTES +
                sep->data_pool_bytes_allocated;
 
-       dev_dbg(&sep->pdev->dev, "data pool token [0] %x\n", token_addr[0]);
-       dev_dbg(&sep->pdev->dev, "data pool token [1] %x\n", token_addr[1]);
-
        /* Write the memory back to the user space */
        error = copy_to_user((void *)arg, (void *)&command_args,
                sizeof(struct alloc_struct));
        if (error) {
                error = -EFAULT;
-               dev_warn(&sep->pdev->dev,
-                       "allocate data pool copy to user error\n");
                goto end_function;
        }
 
-       /* update the allocation */
+       /* Update the allocation */
        sep->data_pool_bytes_allocated += command_args.num_bytes;
        sep->num_of_data_allocations += 1;
 
-       dev_dbg(&sep->pdev->dev, "data_allocations %d\n",
-               sep->num_of_data_allocations);
-       dev_dbg(&sep->pdev->dev, "bytes allocated  %d\n",
-               (int)sep->data_pool_bytes_allocated);
-
 end_function:
        dev_dbg(&sep->pdev->dev, "sep_allocate_data_pool_memory_handler end\n");
        return error;
@@ -1072,25 +991,21 @@ end_function:
  *     the kernel (when ioctl is not used)
  */
 static int sep_lock_kernel_pages(struct sep_device *sep,
-       u32 kernel_virt_addr,
+       unsigned long kernel_virt_addr,
        u32 data_size,
        struct sep_lli_entry **lli_array_ptr,
        int in_out_flag)
 
 {
        int error = 0;
-       /* array of lli */
+       /* Array of lli */
        struct sep_lli_entry *lli_array;
-       /* map array */
+       /* Map array */
        struct sep_dma_map *map_array;
 
-       dev_dbg(&sep->pdev->dev,
-               "sep_lock_kernel_pages start\n");
-
-       dev_dbg(&sep->pdev->dev,
-               "kernel_virt_addr is %08x\n", kernel_virt_addr);
-       dev_dbg(&sep->pdev->dev,
-               "data_size is %x\n", data_size);
+       dev_dbg(&sep->pdev->dev, "lock kernel pages kernel_virt_addr is %08lx\n",
+                               (unsigned long)kernel_virt_addr);
+       dev_dbg(&sep->pdev->dev, "data_size is %x\n", data_size);
 
        lli_array = kmalloc(sizeof(struct sep_lli_entry), GFP_ATOMIC);
        if (!lli_array) {
@@ -1110,7 +1025,7 @@ static int sep_lock_kernel_pages(struct sep_device *sep,
 
 
        /*
-        * set the start address of the first page - app data may start not at
+        * Set the start address of the first page - app data may start not at
         * the beginning of the page
         */
        lli_array[0].bus_address = (u32)map_array[0].dma_addr;
@@ -1121,17 +1036,17 @@ static int sep_lock_kernel_pages(struct sep_device *sep,
                (unsigned long)lli_array[0].bus_address,
                lli_array[0].block_size);
 
-       /* set the output parameters */
+       /* Set the output parameters */
        if (in_out_flag == SEP_DRIVER_IN_FLAG) {
                *lli_array_ptr = lli_array;
                sep->dma_res_arr[sep->nr_dcb_creat].in_num_pages = 1;
-               sep->dma_res_arr[sep->nr_dcb_creat].in_page_array = 0;
+               sep->dma_res_arr[sep->nr_dcb_creat].in_page_array = NULL;
                sep->dma_res_arr[sep->nr_dcb_creat].in_map_array = map_array;
                sep->dma_res_arr[sep->nr_dcb_creat].in_map_num_entries = 1;
        } else {
                *lli_array_ptr = lli_array;
                sep->dma_res_arr[sep->nr_dcb_creat].out_num_pages = 1;
-               sep->dma_res_arr[sep->nr_dcb_creat].out_page_array = 0;
+               sep->dma_res_arr[sep->nr_dcb_creat].out_page_array = NULL;
                sep->dma_res_arr[sep->nr_dcb_creat].out_map_array = map_array;
                sep->dma_res_arr[sep->nr_dcb_creat].out_map_num_entries = 1;
        }
@@ -1141,7 +1056,6 @@ end_function_with_error:
        kfree(lli_array);
 
 end_function:
-       dev_dbg(&sep->pdev->dev, "sep_lock_kernel_pages end\n");
        return error;
 }
 
@@ -1168,38 +1082,33 @@ static int sep_lock_user_pages(struct sep_device *sep,
        int error = 0;
        u32 count;
        int result;
-       /* the the page of the end address of the user space buffer */
+       /* The the page of the end address of the user space buffer */
        u32 end_page;
-       /* the page of the start address of the user space buffer */
+       /* The page of the start address of the user space buffer */
        u32 start_page;
-       /* the range in pages */
+       /* The range in pages */
        u32 num_pages;
-       /* array of pointers to page */
+       /* Array of pointers to page */
        struct page **page_array;
-       /* array of lli */
+       /* Array of lli */
        struct sep_lli_entry *lli_array;
-       /* map array */
+       /* Map array */
        struct sep_dma_map *map_array;
-       /* direction of the DMA mapping for locked pages */
+       /* Direction of the DMA mapping for locked pages */
        enum dma_data_direction dir;
 
-       dev_dbg(&sep->pdev->dev,
-               "sep_lock_user_pages start\n");
-
-       /* set start and end pages  and num pages */
+       /* Set start and end pages  and num pages */
        end_page = (app_virt_addr + data_size - 1) >> PAGE_SHIFT;
        start_page = app_virt_addr >> PAGE_SHIFT;
        num_pages = end_page - start_page + 1;
 
-       dev_dbg(&sep->pdev->dev, "app_virt_addr is %x\n", app_virt_addr);
+       dev_dbg(&sep->pdev->dev, "lock user pages app_virt_addr is %x\n", app_virt_addr);
        dev_dbg(&sep->pdev->dev, "data_size is %x\n", data_size);
        dev_dbg(&sep->pdev->dev, "start_page is %x\n", start_page);
        dev_dbg(&sep->pdev->dev, "end_page is %x\n", end_page);
        dev_dbg(&sep->pdev->dev, "num_pages is %x\n", num_pages);
 
-       dev_dbg(&sep->pdev->dev, "starting page_array malloc\n");
-
-       /* allocate array of pages structure pointers */
+       /* Allocate array of pages structure pointers */
        page_array = kmalloc(sizeof(struct page *) * num_pages, GFP_ATOMIC);
        if (!page_array) {
                error = -ENOMEM;
@@ -1221,18 +1130,16 @@ static int sep_lock_user_pages(struct sep_device *sep,
                goto end_function_with_error2;
        }
 
-       dev_dbg(&sep->pdev->dev, "starting get_user_pages\n");
-
-       /* convert the application virtual address into a set of physical */
+       /* Convert the application virtual address into a set of physical */
        down_read(&current->mm->mmap_sem);
        result = get_user_pages(current, current->mm, app_virt_addr,
                num_pages,
                ((in_out_flag == SEP_DRIVER_IN_FLAG) ? 0 : 1),
-               0, page_array, 0);
+               0, page_array, NULL);
 
        up_read(&current->mm->mmap_sem);
 
-       /* check the number of pages locked - if not all then exit with error */
+       /* Check the number of pages locked - if not all then exit with error */
        if (result != num_pages) {
                dev_warn(&sep->pdev->dev,
                        "not all pages locked by get_user_pages\n");
@@ -1242,26 +1149,25 @@ static int sep_lock_user_pages(struct sep_device *sep,
 
        dev_dbg(&sep->pdev->dev, "get_user_pages succeeded\n");
 
-       /* set direction */
+       /* Set direction */
        if (in_out_flag == SEP_DRIVER_IN_FLAG)
                dir = DMA_TO_DEVICE;
        else
                dir = DMA_FROM_DEVICE;
 
        /*
-        * fill the array using page array data and
-        * map the pages - this action
-        * will also flush the cache as needed
+        * Fill the array using page array data and
+        * map the pages - this action will also flush the cache as needed
         */
        for (count = 0; count < num_pages; count++) {
-               /* fill the map array */
+               /* Fill the map array */
                map_array[count].dma_addr =
                        dma_map_page(&sep->pdev->dev, page_array[count],
                        0, PAGE_SIZE, /*dir*/DMA_BIDIRECTIONAL);
 
                map_array[count].size = PAGE_SIZE;
 
-               /* fill the lli array entry */
+               /* Fill the lli array entry */
                lli_array[count].bus_address = (u32)map_array[count].dma_addr;
                lli_array[count].block_size = PAGE_SIZE;
 
@@ -1270,11 +1176,11 @@ static int sep_lock_user_pages(struct sep_device *sep,
                        count, lli_array[count].block_size);
        }
 
-       /* check the offset for the first page */
+       /* Check the offset for the first page */
        lli_array[0].bus_address =
                lli_array[0].bus_address + (app_virt_addr & (~PAGE_MASK));
 
-       /* check that not all the data is in the first page only */
+       /* Check that not all the data is in the first page only */
        if ((PAGE_SIZE - (app_virt_addr & (~PAGE_MASK))) >= data_size)
                lli_array[0].block_size = data_size;
        else
@@ -1286,7 +1192,7 @@ static int sep_lock_user_pages(struct sep_device *sep,
                (unsigned long)lli_array[count].bus_address,
                lli_array[count].block_size);
 
-       /* check the size of the last page */
+       /* Check the size of the last page */
        if (num_pages > 1) {
                lli_array[num_pages - 1].block_size =
                        (app_virt_addr + data_size) & (~PAGE_MASK);
@@ -1299,7 +1205,7 @@ static int sep_lock_user_pages(struct sep_device *sep,
                        lli_array[count].block_size);
        }
 
-       /* set output params acording to the in_out flag */
+       /* Set output params acording to the in_out flag */
        if (in_out_flag == SEP_DRIVER_IN_FLAG) {
                *lli_array_ptr = lli_array;
                sep->dma_res_arr[sep->nr_dcb_creat].in_num_pages = num_pages;
@@ -1319,18 +1225,17 @@ static int sep_lock_user_pages(struct sep_device *sep,
        goto end_function;
 
 end_function_with_error3:
-       /* free lli array */
+       /* Free lli array */
        kfree(lli_array);
 
 end_function_with_error2:
        kfree(map_array);
 
 end_function_with_error1:
-       /* free page array */
+       /* Free page array */
        kfree(page_array);
 
 end_function:
-       dev_dbg(&sep->pdev->dev, "sep_lock_user_pages end\n");
        return error;
 }
 
@@ -1352,15 +1257,15 @@ static u32 sep_calculate_lli_table_max_size(struct sep_device *sep,
        u32 *last_table_flag)
 {
        u32 counter;
-       /* table data size */
+       /* Table data size */
        u32 table_data_size = 0;
-       /* data size for the next table */
+       /* Data size for the next table */
        u32 next_table_data_size;
 
        *last_table_flag = 0;
 
        /*
-        * calculate the data in the out lli table till we fill the whole
+        * Calculate the data in the out lli table till we fill the whole
         * table or till the data has ended
         */
        for (counter = 0;
@@ -1369,20 +1274,19 @@ static u32 sep_calculate_lli_table_max_size(struct sep_device *sep,
                table_data_size += lli_in_array_ptr[counter].block_size;
 
        /*
-        * check if we reached the last entry,
+        * Check if we reached the last entry,
         * meaning this ia the last table to build,
         * and no need to check the block alignment
         */
        if (counter == num_array_entries) {
-               /* set the last table flag */
+               /* Set the last table flag */
                *last_table_flag = 1;
                goto end_function;
        }
 
        /*
-        * calculate the data size of the next table.
-        * Stop if no entries left or
-        * if data size is more the DMA restriction
+        * Calculate the data size of the next table.
+        * Stop if no entries left or if data size is more the DMA restriction
         */
        next_table_data_size = 0;
        for (; counter < num_array_entries; counter++) {
@@ -1392,7 +1296,7 @@ static u32 sep_calculate_lli_table_max_size(struct sep_device *sep,
        }
 
        /*
-        * check if the next table data size is less then DMA rstriction.
+        * Check if the next table data size is less then DMA rstriction.
         * if it is - recalculate the current table size, so that the next
         * table data size will be adaquete for DMA
         */
@@ -1402,8 +1306,6 @@ static u32 sep_calculate_lli_table_max_size(struct sep_device *sep,
                table_data_size -= (SEP_DRIVER_MIN_DATA_SIZE_PER_TABLE -
                        next_table_data_size);
 
-       dev_dbg(&sep->pdev->dev, "table data size is %x\n",
-                                                       table_data_size);
 end_function:
        return table_data_size;
 }
@@ -1427,23 +1329,21 @@ static void sep_build_lli_table(struct sep_device *sep,
        u32 *num_table_entries_ptr,
        u32 table_data_size)
 {
-       /* current table data size */
+       /* Current table data size */
        u32 curr_table_data_size;
-       /* counter of lli array entry */
+       /* Counter of lli array entry */
        u32 array_counter;
 
-       dev_dbg(&sep->pdev->dev, "sep_build_lli_table start\n");
-
-       /* init currrent table data size and lli array entry counter */
+       /* Init currrent table data size and lli array entry counter */
        curr_table_data_size = 0;
        array_counter = 0;
        *num_table_entries_ptr = 1;
 
-       dev_dbg(&sep->pdev->dev, "table_data_size is %x\n", table_data_size);
+       dev_dbg(&sep->pdev->dev, "build lli table table_data_size is %x\n", table_data_size);
 
-       /* fill the table till table size reaches the needed amount */
+       /* Fill the table till table size reaches the needed amount */
        while (curr_table_data_size < table_data_size) {
-               /* update the number of entries in table */
+               /* Update the number of entries in table */
                (*num_table_entries_ptr)++;
 
                lli_table_ptr->bus_address =
@@ -1461,24 +1361,24 @@ static void sep_build_lli_table(struct sep_device *sep,
                dev_dbg(&sep->pdev->dev, "lli_table_ptr->block_size is %x\n",
                        lli_table_ptr->block_size);
 
-               /* check for overflow of the table data */
+               /* Check for overflow of the table data */
                if (curr_table_data_size > table_data_size) {
                        dev_dbg(&sep->pdev->dev,
                                "curr_table_data_size too large\n");
 
-                       /* update the size of block in the table */
+                       /* Update the size of block in the table */
                        lli_table_ptr->block_size -=
                        cpu_to_le32((curr_table_data_size - table_data_size));
 
-                       /* update the physical address in the lli array */
+                       /* Update the physical address in the lli array */
                        lli_array_ptr[array_counter].bus_address +=
                        cpu_to_le32(lli_table_ptr->block_size);
 
-                       /* update the block size left in the lli array */
+                       /* Update the block size left in the lli array */
                        lli_array_ptr[array_counter].block_size =
                                (curr_table_data_size - table_data_size);
                } else
-                       /* advance to the next entry in the lli_array */
+                       /* Advance to the next entry in the lli_array */
                        array_counter++;
 
                dev_dbg(&sep->pdev->dev,
@@ -1488,27 +1388,17 @@ static void sep_build_lli_table(struct sep_device *sep,
                        "lli_table_ptr->block_size is %x\n",
                        lli_table_ptr->block_size);
 
-               /* move to the next entry in table */
+               /* Move to the next entry in table */
                lli_table_ptr++;
        }
 
-       /* set the info entry to default */
+       /* Set the info entry to default */
        lli_table_ptr->bus_address = 0xffffffff;
        lli_table_ptr->block_size = 0;
 
-       dev_dbg(&sep->pdev->dev, "lli_table_ptr is %p\n", lli_table_ptr);
-       dev_dbg(&sep->pdev->dev, "lli_table_ptr->bus_address is %08lx\n",
-                               (unsigned long)lli_table_ptr->bus_address);
-       dev_dbg(&sep->pdev->dev, "lli_table_ptr->block_size is %x\n",
-                                               lli_table_ptr->block_size);
-
-       /* set the output parameter */
+       /* Set the output parameter */
        *num_processed_entries_ptr += array_counter;
 
-       dev_dbg(&sep->pdev->dev, "num_processed_entries_ptr is %x\n",
-               *num_processed_entries_ptr);
-
-       dev_dbg(&sep->pdev->dev, "sep_build_lli_table end\n");
 }
 
 /**
@@ -1545,8 +1435,8 @@ static dma_addr_t sep_shared_area_virt_to_bus(struct sep_device *sep,
 static void *sep_shared_area_bus_to_virt(struct sep_device *sep,
        dma_addr_t bus_address)
 {
-       dev_dbg(&sep->pdev->dev, "shared bus to virt b=%x v=%x\n",
-               (u32)bus_address, (u32)(sep->shared_addr +
+       dev_dbg(&sep->pdev->dev, "shared bus to virt b=%lx v=%lx\n",
+               (unsigned long)bus_address, (unsigned long)(sep->shared_addr +
                        (size_t)(bus_address - sep->shared_bus)));
 
        return sep->shared_addr + (size_t)(bus_address - sep->shared_bus);
@@ -1571,14 +1461,14 @@ static void sep_debug_print_lli_tables(struct sep_device *sep,
 
        dev_dbg(&sep->pdev->dev, "sep_debug_print_lli_tables start\n");
 
-       while ((unsigned long) lli_table_ptr != 0xffffffff) {
+       while ((unsigned long) lli_table_ptr->bus_address != 0xffffffff) {
                dev_dbg(&sep->pdev->dev,
                        "lli table %08lx, table_data_size is %lu\n",
                        table_count, table_data_size);
                dev_dbg(&sep->pdev->dev, "num_table_entries is %lu\n",
                                                        num_table_entries);
 
-               /* print entries of the table (without info entry) */
+               /* Print entries of the table (without info entry) */
                for (entries_count = 0; entries_count < num_table_entries;
                        entries_count++, lli_table_ptr++) {
 
@@ -1591,7 +1481,7 @@ static void sep_debug_print_lli_tables(struct sep_device *sep,
                                (unsigned long)lli_table_ptr->bus_address,
                                lli_table_ptr->block_size);
                }
-               /* point to the info entry */
+               /* Point to the info entry */
                lli_table_ptr--;
 
                dev_dbg(&sep->pdev->dev,
@@ -1605,18 +1495,16 @@ static void sep_debug_print_lli_tables(struct sep_device *sep,
 
                table_data_size = lli_table_ptr->block_size & 0xffffff;
                num_table_entries = (lli_table_ptr->block_size >> 24) & 0xff;
-               lli_table_ptr = (struct sep_lli_entry *)
-                       (lli_table_ptr->bus_address);
 
                dev_dbg(&sep->pdev->dev,
                        "phys table_data_size is %lu num_table_entries is"
-                       " %lu lli_table_ptr is%lu\n", table_data_size,
-                       num_table_entries, (unsigned long)lli_table_ptr);
+                       " %lu bus_address is%lu\n", table_data_size,
+                       num_table_entries, (unsigned long)lli_table_ptr->bus_address);
 
-               if ((unsigned long)lli_table_ptr != 0xffffffff)
+               if ((unsigned long)lli_table_ptr->bus_address != 0xffffffff)
                        lli_table_ptr = (struct sep_lli_entry *)
                                sep_shared_bus_to_virt(sep,
-                               (unsigned long)lli_table_ptr);
+                               (unsigned long)lli_table_ptr->bus_address);
 
                table_count++;
        }
@@ -1640,9 +1528,7 @@ static void sep_prepare_empty_lli_table(struct sep_device *sep,
 {
        struct sep_lli_entry *lli_table_ptr;
 
-       dev_dbg(&sep->pdev->dev, "sep_prepare_empty_lli_table start\n");
-
-       /* find the area for new table */
+       /* Find the area for new table */
        lli_table_ptr =
                (struct sep_lli_entry *)(sep->shared_addr +
                SYNCHRONIC_DMA_TABLES_AREA_OFFSET_BYTES +
@@ -1656,22 +1542,19 @@ static void sep_prepare_empty_lli_table(struct sep_device *sep,
        lli_table_ptr->bus_address = 0xFFFFFFFF;
        lli_table_ptr->block_size = 0;
 
-       /* set the output parameter value */
+       /* Set the output parameter value */
        *lli_table_addr_ptr = sep->shared_bus +
                SYNCHRONIC_DMA_TABLES_AREA_OFFSET_BYTES +
                sep->num_lli_tables_created *
                sizeof(struct sep_lli_entry) *
                SEP_DRIVER_ENTRIES_PER_TABLE_IN_SEP;
 
-       /* set the num of entries and table data size for empty table */
+       /* Set the num of entries and table data size for empty table */
        *num_entries_ptr = 2;
        *table_data_size_ptr = 0;
 
-       /* update the number of created tables */
+       /* Update the number of created tables */
        sep->num_lli_tables_created++;
-
-       dev_dbg(&sep->pdev->dev, "sep_prepare_empty_lli_table start\n");
-
 }
 
 /**
@@ -1686,7 +1569,7 @@ static void sep_prepare_empty_lli_table(struct sep_device *sep,
  *
  *     This function prepares only input DMA table for synhronic symmetric
  *     operations (HASH)
- *     Note that all bus addresses that are passed to the sep
+ *     Note that all bus addresses that are passed to the SEP
  *     are in 32 bit format; the SEP is a 32 bit device
  */
 static int sep_prepare_input_dma_table(struct sep_device *sep,
@@ -1699,54 +1582,53 @@ static int sep_prepare_input_dma_table(struct sep_device *sep,
        bool is_kva)
 {
        int error = 0;
-       /* pointer to the info entry of the table - the last entry */
+       /* Pointer to the info entry of the table - the last entry */
        struct sep_lli_entry *info_entry_ptr;
-       /* array of pointers to page */
+       /* Array of pointers to page */
        struct sep_lli_entry *lli_array_ptr;
-       /* points to the first entry to be processed in the lli_in_array */
+       /* Points to the first entry to be processed in the lli_in_array */
        u32 current_entry = 0;
-       /* num entries in the virtual buffer */
+       /* Num entries in the virtual buffer */
        u32 sep_lli_entries = 0;
-       /* lli table pointer */
+       /* Lli table pointer */
        struct sep_lli_entry *in_lli_table_ptr;
-       /* the total data in one table */
+       /* The total data in one table */
        u32 table_data_size = 0;
-       /* flag for last table */
+       /* Flag for last table */
        u32 last_table_flag = 0;
-       /* number of entries in lli table */
+       /* Number of entries in lli table */
        u32 num_entries_in_table = 0;
-       /* next table address */
-       u32 lli_table_alloc_addr = 0;
+       /* Next table address */
+       void *lli_table_alloc_addr = 0;
 
-       dev_dbg(&sep->pdev->dev, "sep_prepare_input_dma_table start\n");
-       dev_dbg(&sep->pdev->dev, "data_size is %x\n", data_size);
+       dev_dbg(&sep->pdev->dev, "prepare intput dma table data_size is %x\n", data_size);
        dev_dbg(&sep->pdev->dev, "block_size is %x\n", block_size);
 
-       /* initialize the pages pointers */
-       sep->dma_res_arr[sep->nr_dcb_creat].in_page_array = 0;
+       /* Initialize the pages pointers */
+       sep->dma_res_arr[sep->nr_dcb_creat].in_page_array = NULL;
        sep->dma_res_arr[sep->nr_dcb_creat].in_num_pages = 0;
 
-       /* set the kernel address for first table to be allocated */
-       lli_table_alloc_addr = (u32)(sep->shared_addr +
+       /* Set the kernel address for first table to be allocated */
+       lli_table_alloc_addr = (void *)(sep->shared_addr +
                SYNCHRONIC_DMA_TABLES_AREA_OFFSET_BYTES +
                sep->num_lli_tables_created * sizeof(struct sep_lli_entry) *
                SEP_DRIVER_ENTRIES_PER_TABLE_IN_SEP);
 
        if (data_size == 0) {
-               /* special case  - create meptu table - 2 entries, zero data */
+               /* Special case  - create meptu table - 2 entries, zero data */
                sep_prepare_empty_lli_table(sep, lli_table_ptr,
                                num_entries_ptr, table_data_size_ptr);
                goto update_dcb_counter;
        }
 
-       /* check if the pages are in Kernel Virtual Address layout */
+       /* Check if the pages are in Kernel Virtual Address layout */
        if (is_kva == true)
-               /* lock the pages in the kernel */
+               /* Lock the pages in the kernel */
                error = sep_lock_kernel_pages(sep, app_virt_addr,
                        data_size, &lli_array_ptr, SEP_DRIVER_IN_FLAG);
        else
                /*
-                * lock the pages of the user buffer
+                * Lock the pages of the user buffer
                 * and translate them to pages
                 */
                error = sep_lock_user_pages(sep, app_virt_addr,
@@ -1759,14 +1641,14 @@ static int sep_prepare_input_dma_table(struct sep_device *sep,
                sep->dma_res_arr[sep->nr_dcb_creat].in_num_pages);
 
        current_entry = 0;
-       info_entry_ptr = 0;
+       info_entry_ptr = NULL;
 
        sep_lli_entries = sep->dma_res_arr[sep->nr_dcb_creat].in_num_pages;
 
-       /* loop till all the entries in in array are not processed */
+       /* Loop till all the entries in in array are not processed */
        while (current_entry < sep_lli_entries) {
 
-               /* set the new input and output tables */
+               /* Set the new input and output tables */
                in_lli_table_ptr =
                        (struct sep_lli_entry *)lli_table_alloc_addr;
 
@@ -1774,7 +1656,7 @@ static int sep_prepare_input_dma_table(struct sep_device *sep,
                        SEP_DRIVER_ENTRIES_PER_TABLE_IN_SEP;
 
                if (lli_table_alloc_addr >
-                       ((u32)sep->shared_addr +
+                       ((void *)sep->shared_addr +
                        SYNCHRONIC_DMA_TABLES_AREA_OFFSET_BYTES +
                        SYNCHRONIC_DMA_TABLES_AREA_SIZE_BYTES)) {
 
@@ -1783,17 +1665,17 @@ static int sep_prepare_input_dma_table(struct sep_device *sep,
 
                }
 
-               /* update the number of created tables */
+               /* Update the number of created tables */
                sep->num_lli_tables_created++;
 
-               /* calculate the maximum size of data for input table */
+               /* Calculate the maximum size of data for input table */
                table_data_size = sep_calculate_lli_table_max_size(sep,
                        &lli_array_ptr[current_entry],
                        (sep_lli_entries - current_entry),
                        &last_table_flag);
 
                /*
-                * if this is not the last table -
+                * If this is not the last table -
                 * then allign it to the block size
                 */
                if (!last_table_flag)
@@ -1803,14 +1685,14 @@ static int sep_prepare_input_dma_table(struct sep_device *sep,
                dev_dbg(&sep->pdev->dev, "output table_data_size is %x\n",
                                                        table_data_size);
 
-               /* construct input lli table */
+               /* Construct input lli table */
                sep_build_lli_table(sep, &lli_array_ptr[current_entry],
                        in_lli_table_ptr,
                        &current_entry, &num_entries_in_table, table_data_size);
 
-               if (info_entry_ptr == 0) {
+               if (info_entry_ptr == NULL) {
 
-                       /* set the output parameters to physical addresses */
+                       /* Set the output parameters to physical addresses */
                        *lli_table_ptr = sep_shared_area_virt_to_bus(sep,
                                in_lli_table_ptr);
                        *num_entries_ptr = num_entries_in_table;
@@ -1821,43 +1703,42 @@ static int sep_prepare_input_dma_table(struct sep_device *sep,
                                (unsigned long)*lli_table_ptr);
 
                } else {
-                       /* update the info entry of the previous in table */
+                       /* Update the info entry of the previous in table */
                        info_entry_ptr->bus_address =
                                sep_shared_area_virt_to_bus(sep,
                                                        in_lli_table_ptr);
-                       info_entry_ptr->block_size = 
+                       info_entry_ptr->block_size =
                                ((num_entries_in_table) << 24) |
                                (table_data_size);
                }
-               /* save the pointer to the info entry of the current tables */
+               /* Save the pointer to the info entry of the current tables */
                info_entry_ptr = in_lli_table_ptr + num_entries_in_table - 1;
        }
-       /* print input tables */
+       /* Print input tables */
        sep_debug_print_lli_tables(sep, (struct sep_lli_entry *)
                sep_shared_area_bus_to_virt(sep, *lli_table_ptr),
                *num_entries_ptr, *table_data_size_ptr);
-       /* the array of the pages */
+       /* The array of the pages */
        kfree(lli_array_ptr);
 
 update_dcb_counter:
-       /* update dcb counter */
+       /* Update DCB counter */
        sep->nr_dcb_creat++;
        goto end_function;
 
 end_function_error:
-       /* free all the allocated resources */
+       /* Free all the allocated resources */
        kfree(sep->dma_res_arr[sep->nr_dcb_creat].in_map_array);
        kfree(lli_array_ptr);
        kfree(sep->dma_res_arr[sep->nr_dcb_creat].in_page_array);
 
 end_function:
-       dev_dbg(&sep->pdev->dev, "sep_prepare_input_dma_table end\n");
        return error;
 
 }
 /**
  *     sep_construct_dma_tables_from_lli - prepare AES/DES mappings
- *     @sep: pointer to struct_sep
+ *     @sep: pointer to struct sep_device
  *     @lli_in_array:
  *     @sep_in_lli_entries:
  *     @lli_out_array:
@@ -1869,10 +1750,10 @@ end_function:
  *     @out_num_entries_ptr
  *     @table_data_size_ptr
  *
- *     This function creates the input and output dma tables for
+ *     This function creates the input and output DMA tables for
  *     symmetric operations (AES/DES) according to the block
  *     size from LLI arays
- *     Note that all bus addresses that are passed to the sep
+ *     Note that all bus addresses that are passed to the SEP
  *     are in 32 bit format; the SEP is a 32 bit device
  */
 static int sep_construct_dma_tables_from_lli(
@@ -1888,59 +1769,57 @@ static int sep_construct_dma_tables_from_lli(
        u32     *out_num_entries_ptr,
        u32     *table_data_size_ptr)
 {
-       /* points to the area where next lli table can be allocated */
-       u32 lli_table_alloc_addr = 0;
-       /* input lli table */
-       struct sep_lli_entry *in_lli_table_ptr = 0;
-       /* output lli table */
-       struct sep_lli_entry *out_lli_table_ptr = 0;
-       /* pointer to the info entry of the table - the last entry */
-       struct sep_lli_entry *info_in_entry_ptr = 0;
-       /* pointer to the info entry of the table - the last entry */
-       struct sep_lli_entry *info_out_entry_ptr = 0;
-       /* points to the first entry to be processed in the lli_in_array */
+       /* Points to the area where next lli table can be allocated */
+       void *lli_table_alloc_addr = 0;
+       /* Input lli table */
+       struct sep_lli_entry *in_lli_table_ptr = NULL;
+       /* Output lli table */
+       struct sep_lli_entry *out_lli_table_ptr = NULL;
+       /* Pointer to the info entry of the table - the last entry */
+       struct sep_lli_entry *info_in_entry_ptr = NULL;
+       /* Pointer to the info entry of the table - the last entry */
+       struct sep_lli_entry *info_out_entry_ptr = NULL;
+       /* Points to the first entry to be processed in the lli_in_array */
        u32 current_in_entry = 0;
-       /* points to the first entry to be processed in the lli_out_array */
+       /* Points to the first entry to be processed in the lli_out_array */
        u32 current_out_entry = 0;
-       /* max size of the input table */
+       /* Max size of the input table */
        u32 in_table_data_size = 0;
-       /* max size of the output table */
+       /* Max size of the output table */
        u32 out_table_data_size = 0;
-       /* flag te signifies if this is the last tables build */
+       /* Flag te signifies if this is the last tables build */
        u32 last_table_flag = 0;
-       /* the data size that should be in table */
+       /* The data size that should be in table */
        u32 table_data_size = 0;
-       /* number of etnries in the input table */
+       /* Number of etnries in the input table */
        u32 num_entries_in_table = 0;
-       /* number of etnries in the output table */
+       /* Number of etnries in the output table */
        u32 num_entries_out_table = 0;
 
-       dev_dbg(&sep->pdev->dev, "sep_construct_dma_tables_from_lli start\n");
-
-       /* initiate to point after the message area */
-       lli_table_alloc_addr = (u32)(sep->shared_addr +
+       /* Initiate to point after the message area */
+       lli_table_alloc_addr = (void *)(sep->shared_addr +
                SYNCHRONIC_DMA_TABLES_AREA_OFFSET_BYTES +
                (sep->num_lli_tables_created *
                (sizeof(struct sep_lli_entry) *
                SEP_DRIVER_ENTRIES_PER_TABLE_IN_SEP)));
 
-       /* loop till all the entries in in array are not processed */
+       /* Loop till all the entries in in array are not processed */
        while (current_in_entry < sep_in_lli_entries) {
-               /* set the new input and output tables */
+               /* Set the new input and output tables */
                in_lli_table_ptr =
                        (struct sep_lli_entry *)lli_table_alloc_addr;
 
                lli_table_alloc_addr += sizeof(struct sep_lli_entry) *
                        SEP_DRIVER_ENTRIES_PER_TABLE_IN_SEP;
 
-               /* set the first output tables */
+               /* Set the first output tables */
                out_lli_table_ptr =
                        (struct sep_lli_entry *)lli_table_alloc_addr;
 
-               /* check if the DMA table area limit was overrun */
+               /* Check if the DMA table area limit was overrun */
                if ((lli_table_alloc_addr + sizeof(struct sep_lli_entry) *
                        SEP_DRIVER_ENTRIES_PER_TABLE_IN_SEP) >
-                       ((u32)sep->shared_addr +
+                       ((void *)sep->shared_addr +
                        SYNCHRONIC_DMA_TABLES_AREA_OFFSET_BYTES +
                        SYNCHRONIC_DMA_TABLES_AREA_SIZE_BYTES)) {
 
@@ -1948,20 +1827,20 @@ static int sep_construct_dma_tables_from_lli(
                        return -ENOMEM;
                }
 
-               /* update the number of the lli tables created */
+               /* Update the number of the lli tables created */
                sep->num_lli_tables_created += 2;
 
                lli_table_alloc_addr += sizeof(struct sep_lli_entry) *
                        SEP_DRIVER_ENTRIES_PER_TABLE_IN_SEP;
 
-               /* calculate the maximum size of data for input table */
+               /* Calculate the maximum size of data for input table */
                in_table_data_size =
                        sep_calculate_lli_table_max_size(sep,
                        &lli_in_array[current_in_entry],
                        (sep_in_lli_entries - current_in_entry),
                        &last_table_flag);
 
-               /* calculate the maximum size of data for output table */
+               /* Calculate the maximum size of data for output table */
                out_table_data_size =
                        sep_calculate_lli_table_max_size(sep,
                        &lli_out_array[current_out_entry],
@@ -1969,18 +1848,18 @@ static int sep_construct_dma_tables_from_lli(
                        &last_table_flag);
 
                dev_dbg(&sep->pdev->dev,
-                       "in_table_data_size is %x\n",
+                       "construct tables from lli in_table_data_size is %x\n",
                        in_table_data_size);
 
                dev_dbg(&sep->pdev->dev,
-                       "out_table_data_size is %x\n",
+                       "construct tables from lli out_table_data_size is %x\n",
                        out_table_data_size);
 
                table_data_size = in_table_data_size;
 
                if (!last_table_flag) {
                        /*
-                        * if this is not the last table,
+                        * If this is not the last table,
                         * then must check where the data is smallest
                         * and then align it to the block size
                         */
@@ -1988,33 +1867,30 @@ static int sep_construct_dma_tables_from_lli(
                                table_data_size = out_table_data_size;
 
                        /*
-                        * now calculate the table size so that
+                        * Now calculate the table size so that
                         * it will be module block size
                         */
                        table_data_size = (table_data_size / block_size) *
                                block_size;
                }
 
-               dev_dbg(&sep->pdev->dev, "table_data_size is %x\n",
-                                                       table_data_size);
-
-               /* construct input lli table */
+               /* Construct input lli table */
                sep_build_lli_table(sep, &lli_in_array[current_in_entry],
                        in_lli_table_ptr,
                        &current_in_entry,
                        &num_entries_in_table,
                        table_data_size);
 
-               /* construct output lli table */
+               /* Construct output lli table */
                sep_build_lli_table(sep, &lli_out_array[current_out_entry],
                        out_lli_table_ptr,
                        &current_out_entry,
                        &num_entries_out_table,
                        table_data_size);
 
-               /* if info entry is null - this is the first table built */
-               if (info_in_entry_ptr == 0) {
-                       /* set the output parameters to physical addresses */
+               /* If info entry is null - this is the first table built */
+               if (info_in_entry_ptr == NULL) {
+                       /* Set the output parameters to physical addresses */
                        *lli_table_in_ptr =
                        sep_shared_area_virt_to_bus(sep, in_lli_table_ptr);
 
@@ -2034,7 +1910,7 @@ static int sep_construct_dma_tables_from_lli(
                        "output lli_table_out_ptr is %08lx\n",
                                (unsigned long)*lli_table_out_ptr);
                } else {
-                       /* update the info entry of the previous in table */
+                       /* Update the info entry of the previous in table */
                        info_in_entry_ptr->bus_address =
                                sep_shared_area_virt_to_bus(sep,
                                in_lli_table_ptr);
@@ -2043,7 +1919,7 @@ static int sep_construct_dma_tables_from_lli(
                                ((num_entries_in_table) << 24) |
                                (table_data_size);
 
-                       /* update the info entry of the previous in table */
+                       /* Update the info entry of the previous in table */
                        info_out_entry_ptr->bus_address =
                                sep_shared_area_virt_to_bus(sep,
                                out_lli_table_ptr);
@@ -2063,7 +1939,7 @@ static int sep_construct_dma_tables_from_lli(
                                info_out_entry_ptr->block_size);
                }
 
-               /* save the pointer to the info entry of the current tables */
+               /* Save the pointer to the info entry of the current tables */
                info_in_entry_ptr = in_lli_table_ptr +
                        num_entries_in_table - 1;
                info_out_entry_ptr = out_lli_table_ptr +
@@ -2080,21 +1956,20 @@ static int sep_construct_dma_tables_from_lli(
                        (unsigned long)info_out_entry_ptr);
        }
 
-       /* print input tables */
+       /* Print input tables */
        sep_debug_print_lli_tables(sep,
        (struct sep_lli_entry *)
        sep_shared_area_bus_to_virt(sep, *lli_table_in_ptr),
        *in_num_entries_ptr,
        *table_data_size_ptr);
 
-       /* print output tables */
+       /* Print output tables */
        sep_debug_print_lli_tables(sep,
        (struct sep_lli_entry *)
        sep_shared_area_bus_to_virt(sep, *lli_table_out_ptr),
        *out_num_entries_ptr,
        *table_data_size_ptr);
 
-       dev_dbg(&sep->pdev->dev, "sep_construct_dma_tables_from_lli end\n");
        return 0;
 }
 
@@ -2114,7 +1989,7 @@ static int sep_construct_dma_tables_from_lli(
  *     This function builds input and output DMA tables for synhronic
  *     symmetric operations (AES, DES, HASH). It also checks that each table
  *     is of the modular block size
- *     Note that all bus addresses that are passed to the sep
+ *     Note that all bus addresses that are passed to the SEP
  *     are in 32 bit format; the SEP is a 32 bit device
  */
 static int sep_prepare_input_output_dma_table(struct sep_device *sep,
@@ -2131,15 +2006,13 @@ static int sep_prepare_input_output_dma_table(struct sep_device *sep,
 
 {
        int error = 0;
-       /* array of pointers of page */
+       /* Array of pointers of page */
        struct sep_lli_entry *lli_in_array;
-       /* array of pointers of page */
+       /* Array of pointers of page */
        struct sep_lli_entry *lli_out_array;
 
-       dev_dbg(&sep->pdev->dev, "sep_prepare_input_output_dma_table start\n");
-
        if (data_size == 0) {
-               /* prepare empty table for input and output */
+               /* Prepare empty table for input and output */
                sep_prepare_empty_lli_table(sep, lli_table_in_ptr,
                        in_num_entries_ptr, table_data_size_ptr);
 
@@ -2149,11 +2022,11 @@ static int sep_prepare_input_output_dma_table(struct sep_device *sep,
                goto update_dcb_counter;
        }
 
-       /* initialize the pages pointers */
-       sep->dma_res_arr[sep->nr_dcb_creat].in_page_array = 0;
-       sep->dma_res_arr[sep->nr_dcb_creat].out_page_array = 0;
+       /* Initialize the pages pointers */
+       sep->dma_res_arr[sep->nr_dcb_creat].in_page_array = NULL;
+       sep->dma_res_arr[sep->nr_dcb_creat].out_page_array = NULL;
 
-       /* lock the pages of the buffer and translate them to pages */
+       /* Lock the pages of the buffer and translate them to pages */
        if (is_kva == true) {
                error = sep_lock_kernel_pages(sep, app_virt_in_addr,
                        data_size, &lli_in_array, SEP_DRIVER_IN_FLAG);
@@ -2178,7 +2051,7 @@ static int sep_prepare_input_output_dma_table(struct sep_device *sep,
                error = sep_lock_user_pages(sep, app_virt_in_addr,
                                data_size, &lli_in_array, SEP_DRIVER_IN_FLAG);
                if (error) {
-                       dev_warn(&sep->pdev->dev, 
+                       dev_warn(&sep->pdev->dev,
                                "sep_lock_user_pages for input virtual buffer failed\n");
                        goto end_function;
                }
@@ -2193,14 +2066,14 @@ static int sep_prepare_input_output_dma_table(struct sep_device *sep,
                }
        }
 
-       dev_dbg(&sep->pdev->dev, "sep_in_num_pages is %x\n",
+       dev_dbg(&sep->pdev->dev, "prep input output dma table sep_in_num_pages is %x\n",
                sep->dma_res_arr[sep->nr_dcb_creat].in_num_pages);
        dev_dbg(&sep->pdev->dev, "sep_out_num_pages is %x\n",
                sep->dma_res_arr[sep->nr_dcb_creat].out_num_pages);
        dev_dbg(&sep->pdev->dev, "SEP_DRIVER_ENTRIES_PER_TABLE_IN_SEP is %x\n",
                SEP_DRIVER_ENTRIES_PER_TABLE_IN_SEP);
 
-       /* call the fucntion that creates table from the lli arrays */
+       /* Call the fucntion that creates table from the lli arrays */
        error = sep_construct_dma_tables_from_lli(sep, lli_in_array,
                sep->dma_res_arr[sep->nr_dcb_creat].in_num_pages,
                lli_out_array,
@@ -2218,15 +2091,8 @@ static int sep_prepare_input_output_dma_table(struct sep_device *sep,
        kfree(lli_in_array);
 
 update_dcb_counter:
-       /* update dcb counter */
+       /* Update DCB counter */
        sep->nr_dcb_creat++;
-       /* fall through - free the lli entry arrays */
-       dev_dbg(&sep->pdev->dev, "in_num_entries_ptr is %08x\n",
-                                               *in_num_entries_ptr);
-       dev_dbg(&sep->pdev->dev, "out_num_entries_ptr is %08x\n",
-                                               *out_num_entries_ptr);
-       dev_dbg(&sep->pdev->dev, "table_data_size_ptr is %08x\n",
-                                               *table_data_size_ptr);
 
        goto end_function;
 
@@ -2242,8 +2108,6 @@ end_function_free_lli_in:
        kfree(lli_in_array);
 
 end_function:
-       dev_dbg(&sep->pdev->dev,
-               "sep_prepare_input_output_dma_table end result = %d\n", error);
 
        return error;
 
@@ -2259,52 +2123,50 @@ end_function:
  *     @isapplet: bool; to indicate external app
  *     @is_kva: bool; kernel buffer; only used for kernel crypto module
  *
- *     This function prepares the linked dma tables and puts the
- *     address for the linked list of tables inta a dcb (data control
- *     block) the address of which is known by the sep hardware
- *     Note that all bus addresses that are passed to the sep
+ *     This function prepares the linked DMA tables and puts the
+ *     address for the linked list of tables inta a DCB (data control
+ *     block) the address of which is known by the SEP hardware
+ *     Note that all bus addresses that are passed to the SEP
  *     are in 32 bit format; the SEP is a 32 bit device
  */
 static int sep_prepare_input_output_dma_table_in_dcb(struct sep_device *sep,
-       u32  app_in_address,
-       u32  app_out_address,
-       u32              data_in_size,
-       u32              block_size,
-       u32              tail_block_size,
-       bool            isapplet,
-       bool            is_kva)
+       unsigned long  app_in_address,
+       unsigned long  app_out_address,
+       u32  data_in_size,
+       u32  block_size,
+       u32  tail_block_size,
+       bool isapplet,
+       bool    is_kva)
 {
        int error = 0;
-       /* size of tail */
+       /* Size of tail */
        u32 tail_size = 0;
-       /* address of the created dcb table */
-       struct sep_dcblock *dcb_table_ptr = 0;
-       /* the physical address of the first input DMA table */
+       /* Address of the created DCB table */
+       struct sep_dcblock *dcb_table_ptr = NULL;
+       /* The physical address of the first input DMA table */
        dma_addr_t in_first_mlli_address = 0;
-       /* number of entries in the first input DMA table */
+       /* Number of entries in the first input DMA table */
        u32  in_first_num_entries = 0;
-       /* the physical address of the first output DMA table */
+       /* The physical address of the first output DMA table */
        dma_addr_t  out_first_mlli_address = 0;
-       /* number of entries in the first output DMA table */
+       /* Number of entries in the first output DMA table */
        u32  out_first_num_entries = 0;
-       /* data in the first input/output table */
+       /* Data in the first input/output table */
        u32  first_data_size = 0;
 
-       dev_dbg(&sep->pdev->dev, "prepare_input_output_dma_table_in_dcb start\n");
-
        if (sep->nr_dcb_creat == SEP_MAX_NUM_SYNC_DMA_OPS) {
-               /* No more DCBS to allocate */
-               dev_warn(&sep->pdev->dev, "no more dcb's available\n");
+               /* No more DCBs to allocate */
+               dev_warn(&sep->pdev->dev, "no more DCBs available\n");
                error = -ENOSPC;
                goto end_function;
        }
 
-       /* allocate new DCB */
+       /* Allocate new DCB */
        dcb_table_ptr = (struct sep_dcblock *)(sep->shared_addr +
                SEP_DRIVER_SYSTEM_DCB_MEMORY_OFFSET_IN_BYTES +
                (sep->nr_dcb_creat * sizeof(struct sep_dcblock)));
 
-       /* set the default values in the dcb */
+       /* Set the default values in the DCB */
        dcb_table_ptr->input_mlli_address = 0;
        dcb_table_ptr->input_mlli_num_entries = 0;
        dcb_table_ptr->input_mlli_data_size = 0;
@@ -2324,7 +2186,7 @@ static int sep_prepare_input_output_dma_table_in_dcb(struct sep_device *sep,
                        }
                        if (tail_block_size)
                                /*
-                                * case the tail size should be
+                                * Case the tail size should be
                                 * bigger than the real block size
                                 */
                                tail_size = tail_block_size +
@@ -2332,7 +2194,7 @@ static int sep_prepare_input_output_dma_table_in_dcb(struct sep_device *sep,
                                                tail_block_size) % block_size);
                }
 
-               /* check if there is enough data for dma operation */
+               /* Check if there is enough data for DMA operation */
                if (data_in_size < SEP_DRIVER_MIN_DATA_SIZE_PER_TABLE) {
                        if (is_kva == true) {
                                memcpy(dcb_table_ptr->tail_data,
@@ -2348,7 +2210,7 @@ static int sep_prepare_input_output_dma_table_in_dcb(struct sep_device *sep,
 
                        dcb_table_ptr->tail_data_size = data_in_size;
 
-                       /* set the output user-space address for mem2mem op */
+                       /* Set the output user-space address for mem2mem op */
                        if (app_out_address)
                                dcb_table_ptr->out_vr_tail_pt =
                                                        (u32)app_out_address;
@@ -2367,7 +2229,7 @@ static int sep_prepare_input_output_dma_table_in_dcb(struct sep_device *sep,
                                        (void *)(app_in_address + data_in_size -
                                        tail_size), tail_size);
                        } else {
-                               /* we have tail data - copy it to dcb */
+                               /* We have tail data - copy it to DCB */
                                if (copy_from_user(dcb_table_ptr->tail_data,
                                        (void *)(app_in_address +
                                        data_in_size - tail_size), tail_size)) {
@@ -2381,7 +2243,7 @@ static int sep_prepare_input_output_dma_table_in_dcb(struct sep_device *sep,
                                 * according to tail data size
                                 */
                                dcb_table_ptr->out_vr_tail_pt =
-                                       app_out_address + data_in_size
+                                       (u32)app_out_address + data_in_size
                                        - tail_size;
 
                        /* Save the real tail data size */
@@ -2393,9 +2255,9 @@ static int sep_prepare_input_output_dma_table_in_dcb(struct sep_device *sep,
                        data_in_size = (data_in_size - tail_size);
                }
        }
-       /* check if we need to build only input table or input/output */
+       /* Check if we need to build only input table or input/output */
        if (app_out_address) {
-               /* prepare input/output tables */
+               /* Prepare input/output tables */
                error = sep_prepare_input_output_dma_table(sep,
                        app_in_address,
                        app_out_address,
@@ -2408,7 +2270,7 @@ static int sep_prepare_input_output_dma_table_in_dcb(struct sep_device *sep,
                        &first_data_size,
                        is_kva);
        } else {
-               /* prepare input tables */
+               /* Prepare input tables */
                error = sep_prepare_input_dma_table(sep,
                        app_in_address,
                        data_in_size,
@@ -2420,11 +2282,11 @@ static int sep_prepare_input_output_dma_table_in_dcb(struct sep_device *sep,
        }
 
        if (error) {
-               dev_warn(&sep->pdev->dev, "prepare dma table call failed from prepare dcb call\n");
+               dev_warn(&sep->pdev->dev, "prepare DMA table call failed from prepare DCB call\n");
                goto end_function;
        }
 
-       /* set the dcb values */
+       /* Set the DCB values */
        dcb_table_ptr->input_mlli_address = in_first_mlli_address;
        dcb_table_ptr->input_mlli_num_entries = in_first_num_entries;
        dcb_table_ptr->input_mlli_data_size = first_data_size;
@@ -2433,15 +2295,13 @@ static int sep_prepare_input_output_dma_table_in_dcb(struct sep_device *sep,
        dcb_table_ptr->output_mlli_data_size = first_data_size;
 
 end_function:
-       dev_dbg(&sep->pdev->dev,
-               "sep_prepare_input_output_dma_table_in_dcb end\n");
        return error;
 
 }
 
 
 /**
- *     sep_create_sync_dma_tables_handler - create sync dma tables
+ *     sep_create_sync_dma_tables_handler - create sync DMA tables
  *     @sep: pointer to struct sep_device
  *     @arg: pointer to struct bld_syn_tab_struct
  *
@@ -2454,19 +2314,16 @@ static int sep_create_sync_dma_tables_handler(struct sep_device *sep,
 {
        int error = 0;
 
-       /* command arguments */
+       /* Command arguments */
        struct bld_syn_tab_struct command_args;
 
-       dev_dbg(&sep->pdev->dev,
-               "sep_create_sync_dma_tables_handler start\n");
-
        if (copy_from_user(&command_args, (void __user *)arg,
                                        sizeof(struct bld_syn_tab_struct))) {
                error = -EFAULT;
                goto end_function;
        }
 
-       dev_dbg(&sep->pdev->dev, "app_in_address is %08llx\n",
+       dev_dbg(&sep->pdev->dev, "create dma table handler app_in_address is %08llx\n",
                                                command_args.app_in_address);
        dev_dbg(&sep->pdev->dev, "app_out_address is %08llx\n",
                                                command_args.app_out_address);
@@ -2475,15 +2332,15 @@ static int sep_create_sync_dma_tables_handler(struct sep_device *sep,
        dev_dbg(&sep->pdev->dev, "block_size is %u\n",
                                                command_args.block_size);
 
-       /* validate user parameters */
+       /* Validate user parameters */
        if (!command_args.app_in_address) {
                error = -EINVAL;
                goto end_function;
        }
 
        error = sep_prepare_input_output_dma_table_in_dcb(sep,
-               command_args.app_in_address,
-               command_args.app_out_address,
+               (unsigned long)command_args.app_in_address,
+               (unsigned long)command_args.app_out_address,
                command_args.data_in_size,
                command_args.block_size,
                0x0,
@@ -2491,7 +2348,6 @@ static int sep_create_sync_dma_tables_handler(struct sep_device *sep,
                false);
 
 end_function:
-       dev_dbg(&sep->pdev->dev, "sep_create_sync_dma_tables_handler end\n");
        return error;
 }
 
@@ -2501,7 +2357,7 @@ end_function:
  *     @isapplet: indicates external application (used for kernel access)
  *     @is_kva: indicates kernel addresses (only used for kernel crypto)
  *
- *     This function frees the dma tables and dcb block
+ *     This function frees the DMA tables and DCB
  */
 static int sep_free_dma_tables_and_dcb(struct sep_device *sep, bool isapplet,
        bool is_kva)
@@ -2510,81 +2366,63 @@ static int sep_free_dma_tables_and_dcb(struct sep_device *sep, bool isapplet,
        int error = 0;
        int error_temp = 0;
        struct sep_dcblock *dcb_table_ptr;
-
-       dev_dbg(&sep->pdev->dev, "sep_free_dma_tables_and_dcb start\n");
+       unsigned long pt_hold;
+       void *tail_pt;
 
        if (isapplet == true) {
-               /* set pointer to first dcb table */
+               /* Set pointer to first DCB table */
                dcb_table_ptr = (struct sep_dcblock *)
                        (sep->shared_addr +
                        SEP_DRIVER_SYSTEM_DCB_MEMORY_OFFSET_IN_BYTES);
 
-               /* go over each dcb and see if tail pointer must be updated */
+               /* Go over each DCB and see if tail pointer must be updated */
                for (i = 0; i < sep->nr_dcb_creat; i++, dcb_table_ptr++) {
                        if (dcb_table_ptr->out_vr_tail_pt) {
+                               pt_hold = (unsigned long)dcb_table_ptr->out_vr_tail_pt;
+                               tail_pt = (void *)pt_hold;
                                if (is_kva == true) {
-                                       memcpy((void *)dcb_table_ptr->out_vr_tail_pt,
+                                       memcpy(tail_pt,
                                                dcb_table_ptr->tail_data,
                                                dcb_table_ptr->tail_data_size);
                                } else {
                                        error_temp = copy_to_user(
-                                               (void *)dcb_table_ptr->out_vr_tail_pt,
+                                               tail_pt,
                                                dcb_table_ptr->tail_data,
                                                dcb_table_ptr->tail_data_size);
                                }
                                if (error_temp) {
-                                       /* release the dma resource */
+                                       /* Release the DMA resource */
                                        error = -EFAULT;
                                        break;
                                }
                        }
                }
        }
-       /* free the output pages, if any */
+       /* Free the output pages, if any */
        sep_free_dma_table_data_handler(sep);
 
-       dev_dbg(&sep->pdev->dev, "sep_free_dma_tables_and_dcb end\n");
        return error;
 }
 
 /**
  *     sep_get_static_pool_addr_handler - get static pool address
  *     @sep: pointer to struct sep_device
- *     @arg: parameters from user space application
  *
  *     This function sets the bus and virtual addresses of the static pool
- *     and returns the virtual address
  */
-static int sep_get_static_pool_addr_handler(struct sep_device *sep,
-       unsigned long arg)
+static int sep_get_static_pool_addr_handler(struct sep_device *sep)
 {
-       struct stat_pool_addr_struct command_args;
-       u32 *static_pool_addr = 0;
-       unsigned long addr_hold;
-
-       dev_dbg(&sep->pdev->dev, "sep_get_static_pool_addr_handler start\n");
+       u32 *static_pool_addr = NULL;
 
        static_pool_addr = (u32 *)(sep->shared_addr +
                SEP_DRIVER_SYSTEM_RAR_MEMORY_OFFSET_IN_BYTES);
 
        static_pool_addr[0] = SEP_STATIC_POOL_VAL_TOKEN;
-       static_pool_addr[1] = sep->shared_bus +
+       static_pool_addr[1] = (u32)sep->shared_bus +
                SEP_DRIVER_STATIC_AREA_OFFSET_IN_BYTES;
 
-       addr_hold = (unsigned long)
-               (sep->shared_addr + SEP_DRIVER_STATIC_AREA_OFFSET_IN_BYTES);
-       command_args.static_virt_address = (aligned_u64)addr_hold;
-
-       dev_dbg(&sep->pdev->dev, "static pool: physical %x virtual %x\n",
-               (u32)static_pool_addr[1],
-               (u32)command_args.static_virt_address);
-
-       /* send the parameters to user application */
-       if (copy_to_user((void __user *) arg, &command_args,
-               sizeof(struct stat_pool_addr_struct)))
-               return -EFAULT;
-
-       dev_dbg(&sep->pdev->dev, "sep_get_static_pool_addr_handler end\n");
+       dev_dbg(&sep->pdev->dev, "static pool segment: physical %x\n",
+               (u32)static_pool_addr[1]);
 
        return 0;
 }
@@ -2598,18 +2436,15 @@ static int sep_start_handler(struct sep_device *sep)
        unsigned long reg_val;
        unsigned long error = 0;
 
-       dev_dbg(&sep->pdev->dev, "sep_start_handler start\n");
-
-       /* wait in polling for message from SEP */
-       do
+       /* Wait in polling for message from SEP */
+       do {
                reg_val = sep_read_reg(sep, HW_HOST_SEP_HOST_GPR3_REG_ADDR);
-       while (!reg_val);
+       while (!reg_val);
 
-       /* check the value */
+       /* Check the value */
        if (reg_val == 0x1)
-               /* fatal error - read error status from GPRO */
+               /* Fatal error - read error status from GPRO */
                error = sep_read_reg(sep, HW_HOST_SEP_HOST_GPR0_REG_ADDR);
-       dev_dbg(&sep->pdev->dev, "sep_start_handler end\n");
        return error;
 }
 
@@ -2619,7 +2454,7 @@ static int sep_start_handler(struct sep_device *sep)
  *     @length: buffer size
  *
  *     This function performs a checksum for messages that are sent
- *     to the sep
+ *     to the SEP.
  */
 static u32 sep_check_sum_calc(u8 *data, u32 length)
 {
@@ -2651,7 +2486,7 @@ static u32 sep_check_sum_calc(u8 *data, u32 length)
  *     Handles the request for SEP initialization
  *     Note that this will go away for Medfield once the SCU
  *     SEP initialization is complete
- *     Also note that the message to the sep has components
+ *     Also note that the message to the SEP has components
  *     from user space as well as components written by the driver
  *     This is becuase the portions of the message that pertain to
  *     physical addresses must be set by the driver after the message
@@ -2668,9 +2503,7 @@ static int sep_init_handler(struct sep_device *sep, unsigned long arg)
        unsigned long addr_hold;
        struct init_struct command_args;
 
-       dev_dbg(&sep->pdev->dev, "sep_init_handler start\n");
-
-       /* make sure that we have not initialized already */
+       /* Make sure that we have not initialized already */
        reg_val = sep_read_reg(sep, HW_HOST_SEP_HOST_GPR3_REG_ADDR);
 
        if (reg_val != 0x2) {
@@ -2679,13 +2512,13 @@ static int sep_init_handler(struct sep_device *sep, unsigned long arg)
                goto end_function;
        }
 
-       /* only root can initialize */
+       /* Only root can initialize */
        if (!capable(CAP_SYS_ADMIN)) {
                error = -EACCES;
                goto end_function;
        }
 
-       /* copy in the parameters */
+       /* Copy in the parameters */
        error = copy_from_user(&command_args, (void __user *)arg,
                sizeof(struct init_struct));
 
@@ -2694,14 +2527,14 @@ static int sep_init_handler(struct sep_device *sep, unsigned long arg)
                goto end_function;
        }
 
-       /* validate parameters */
+       /* Validate parameters */
        if (!command_args.message_addr || !command_args.sep_sram_addr ||
                command_args.message_size_in_words > 14) {
                error = -EINVAL;
                goto end_function;
        }
 
-       /* copy in the sep init message */
+       /* Copy in the SEP init message */
        addr_hold = (unsigned long)command_args.message_addr;
        error = copy_from_user(message_buff,
                (void __user *)addr_hold,
@@ -2712,16 +2545,16 @@ static int sep_init_handler(struct sep_device *sep, unsigned long arg)
                goto end_function;
        }
 
-       /* load resident, cache, and extapp firmware */
+       /* Load resident, cache, and extapp firmware */
        error = sep_load_firmware(sep);
 
        if (error) {
                dev_warn(&sep->pdev->dev,
-                       "init; copy sep init message failed %x\n", error);
+                       "init; copy SEP init message failed %x\n", error);
                goto end_function;
        }
 
-       /* compute the base address */
+       /* Compute the base address */
        new_base_addr = sep->shared_bus;
 
        if (sep->resident_bus < new_base_addr)
@@ -2733,7 +2566,7 @@ static int sep_init_handler(struct sep_device *sep, unsigned long arg)
        if (sep->dcache_bus < new_base_addr)
                new_base_addr = sep->dcache_bus;
 
-       /* put physical addresses in sep message */
+       /* Put physical addresses in SEP message */
        message_buff[3] = (u32)new_base_addr;
        message_buff[4] = (u32)sep->shared_bus;
        message_buff[6] = (u32)sep->resident_bus;
@@ -2745,16 +2578,16 @@ static int sep_init_handler(struct sep_device *sep, unsigned long arg)
                sep_check_sum_calc((u8 *)message_buff,
                command_args.message_size_in_words*sizeof(u32));
 
-       /* debug print of message */
+       /* Debug print of message */
        for (counter = 0; counter < command_args.message_size_in_words;
                                                                counter++)
-               dev_dbg(&sep->pdev->dev, "init; sep message word %d is %x\n",
+               dev_dbg(&sep->pdev->dev, "init; SEP message word %d is %x\n",
                        counter, message_buff[counter]);
 
-       /* tell the sep the sram address */
+       /* Tell the SEP the sram address */
        sep_write_reg(sep, HW_SRAM_ADDR_REG_ADDR, command_args.sep_sram_addr);
 
-       /* push the message to the sep */
+       /* Push the message to the SEP */
        for (counter = 0; counter < command_args.message_size_in_words;
                                                                counter++) {
                sep_write_reg(sep, HW_SRAM_DATA_REG_ADDR,
@@ -2762,15 +2595,14 @@ static int sep_init_handler(struct sep_device *sep, unsigned long arg)
                sep_wait_sram_write(sep);
        }
 
-       /* signal sep that message is ready and to init */
+       /* Signal SEP that message is ready and to init */
        sep_write_reg(sep, HW_HOST_HOST_SEP_GPR0_REG_ADDR, 0x1);
 
-       /* wait for acknowledge */
-       dev_dbg(&sep->pdev->dev, "init; waiting for msg response\n");
+       /* Wait for acknowledge */
 
-       do
+       do {
                reg_val = sep_read_reg(sep, HW_HOST_SEP_HOST_GPR3_REG_ADDR);
-       while (!(reg_val & 0xFFFFFFFD));
+       while (!(reg_val & 0xFFFFFFFD));
 
        if (reg_val == 0x1) {
                dev_warn(&sep->pdev->dev, "init; device int failed\n");
@@ -2780,20 +2612,16 @@ static int sep_init_handler(struct sep_device *sep, unsigned long arg)
                dev_warn(&sep->pdev->dev, "init; error is %x\n", error);
                goto end_function;
        }
-       dev_dbg(&sep->pdev->dev, "init; end CC INIT, reg_val is %x\n", reg_val);
-
-       /* signal sep to zero the GPR3 */
+       /* Signal SEP to zero the GPR3 */
        sep_write_reg(sep, HW_HOST_HOST_SEP_GPR0_REG_ADDR, 0x10);
 
-       /* wait for response */
-       dev_dbg(&sep->pdev->dev, "init; waiting for zero set response\n");
+       /* Wait for response */
 
-       do
+       do {
                reg_val = sep_read_reg(sep, HW_HOST_SEP_HOST_GPR3_REG_ADDR);
-       while (reg_val != 0);
+       while (reg_val != 0);
 
 end_function:
-       dev_dbg(&sep->pdev->dev, "init is done\n");
        return error;
 }
 
@@ -2805,37 +2633,32 @@ end_function:
  */
 static int sep_end_transaction_handler(struct sep_device *sep)
 {
-       dev_dbg(&sep->pdev->dev, "sep_end_transaction_handler start\n");
-
-       /* clear the data pool pointers Token */
+       /* Clear the data pool pointers Token */
        memset((void *)(sep->shared_addr +
                SEP_DRIVER_DATA_POOL_ALLOCATION_OFFSET_IN_BYTES),
                0, sep->num_of_data_allocations*2*sizeof(u32));
 
-       /* check that all the dma resources were freed */
+       /* Check that all the DMA resources were freed */
        sep_free_dma_table_data_handler(sep);
 
        clear_bit(SEP_MMAP_LOCK_BIT, &sep->in_use_flags);
 
        /*
-        * we are now through with the transaction. Let's
+        * We are now through with the transaction. Let's
         * allow other processes who have the device open
         * to perform transactions
         */
        mutex_lock(&sep->sep_mutex);
        sep->pid_doing_transaction = 0;
        mutex_unlock(&sep->sep_mutex);
-       /* raise event for stuck contextes */
+       /* Raise event for stuck contextes */
        wake_up(&sep->event);
 
-       dev_dbg(&sep->pdev->dev, "waking up event\n");
-       dev_dbg(&sep->pdev->dev, "sep_end_transaction_handler end\n");
-
        return 0;
 }
 
 /**
- *     sep_prepare_dcb_handler - prepare a control block 
+ *     sep_prepare_dcb_handler - prepare a control block
  *     @sep: pointer to struct sep_device
  *     @arg: pointer to user parameters
  *
@@ -2844,13 +2667,10 @@ static int sep_end_transaction_handler(struct sep_device *sep)
  */
 static int sep_prepare_dcb_handler(struct sep_device *sep, unsigned long arg)
 {
-       /* error */
        int error;
-       /* command arguments */
+       /* Command arguments */
        struct build_dcb_struct command_args;
 
-       dev_dbg(&sep->pdev->dev, "sep_prepare_dcb_handler start\n");
-
        /* Get the command arguments */
        if (copy_from_user(&command_args, (void __user *)arg,
                                        sizeof(struct build_dcb_struct))) {
@@ -2858,7 +2678,7 @@ static int sep_prepare_dcb_handler(struct sep_device *sep, unsigned long arg)
                goto end_function;
        }
 
-       dev_dbg(&sep->pdev->dev, "app_in_address is %08llx\n",
+       dev_dbg(&sep->pdev->dev, "prep dcb handler app_in_address is %08llx\n",
                                                command_args.app_in_address);
        dev_dbg(&sep->pdev->dev, "app_out_address is %08llx\n",
                                                command_args.app_out_address);
@@ -2870,12 +2690,12 @@ static int sep_prepare_dcb_handler(struct sep_device *sep, unsigned long arg)
                                                command_args.tail_block_size);
 
        error = sep_prepare_input_output_dma_table_in_dcb(sep,
-               command_args.app_in_address, command_args.app_out_address,
+               (unsigned long)command_args.app_in_address,
+               (unsigned long)command_args.app_out_address,
                command_args.data_in_size, command_args.block_size,
                command_args.tail_block_size, true, false);
 
 end_function:
-       dev_dbg(&sep->pdev->dev, "sep_prepare_dcb_handler end\n");
        return error;
 
 }
@@ -2891,17 +2711,15 @@ static int sep_free_dcb_handler(struct sep_device *sep)
 {
        int error ;
 
-       dev_dbg(&sep->pdev->dev, "sep_prepare_dcb_handler start\n");
-       dev_dbg(&sep->pdev->dev, "num of DCBs %x\n", sep->nr_dcb_creat);
+       dev_dbg(&sep->pdev->dev, "free dcbs num of DCBs %x\n", sep->nr_dcb_creat);
 
        error = sep_free_dma_tables_and_dcb(sep, false, false);
 
-       dev_dbg(&sep->pdev->dev, "sep_free_dcb_handler end\n");
        return error;
 }
 
 /**
- *     sep_rar_prepare_output_msg_handler - prepare an output message 
+ *     sep_rar_prepare_output_msg_handler - prepare an output message
  *     @sep: pointer to struct sep_device
  *     @arg: pointer to user parameters
  *
@@ -2912,30 +2730,27 @@ static int sep_rar_prepare_output_msg_handler(struct sep_device *sep,
        unsigned long arg)
 {
        int error = 0;
-       /* command args */
+       /* Command args */
        struct rar_hndl_to_bus_struct command_args;
        struct RAR_buffer rar_buf;
-       /* bus address */
+       /* Bus address */
        dma_addr_t  rar_bus = 0;
-       /* holds the RAR address in the system memory offset */
+       /* Holds the RAR address in the system memory offset */
        u32 *rar_addr;
 
-       dev_dbg(&sep->pdev->dev, "sep_rar_prepare_output_msg_handler start\n");
-
-       /* copy the data */
+       /* Copy the data */
        if (copy_from_user(&command_args, (void __user *)arg,
                                                sizeof(command_args))) {
                error = -EFAULT;
                goto end_function;
        }
 
-       /* call to translation function only if user handle is not NULL */
+       /* Call to translation function only if user handle is not NULL */
        if (command_args.rar_handle) {
                memset(&rar_buf, 0, sizeof(rar_buf));
                rar_buf.info.handle = (u32)command_args.rar_handle;
 
                if (rar_handle_to_bus(&rar_buf, 1) != 1) {
-                       dev_dbg(&sep->pdev->dev, "rar_handle_to_bus failure\n");
                        error = -EFAULT;
                        goto end_function;
                }
@@ -2943,16 +2758,15 @@ static int sep_rar_prepare_output_msg_handler(struct sep_device *sep,
        }
        dev_dbg(&sep->pdev->dev, "rar msg; rar_addr_bus = %x\n", (u32)rar_bus);
 
-       /* set value in the SYSTEM MEMORY offset */
+       /* Set value in the SYSTEM MEMORY offset */
        rar_addr = (u32 *)(sep->shared_addr +
                SEP_DRIVER_SYSTEM_RAR_MEMORY_OFFSET_IN_BYTES);
 
-       /* copy the physical address to the System Area for the sep */
+       /* Copy the physical address to the System Area for the SEP */
        rar_addr[0] = SEP_RAR_VAL_TOKEN;
        rar_addr[1] = rar_bus;
 
 end_function:
-       dev_dbg(&sep->pdev->dev, "sep_rar_prepare_output_msg_handler start\n");
        return error;
 }
 
@@ -2961,25 +2775,21 @@ end_function:
  *     @sep: pointer to struct sep_device
  *     @arg: pointer to user parameters
  *
- *     This function tells the sep where the extapp is located
+ *     This function tells the SEP where the extapp is located
  */
 static int sep_realloc_ext_cache_handler(struct sep_device *sep,
        unsigned long arg)
 {
-       /* holds the new ext cache address in the system memory offset */
+       /* Holds the new ext cache address in the system memory offset */
        u32 *system_addr;
 
-       /* set value in the SYSTEM MEMORY offset */
+       /* Set value in the SYSTEM MEMORY offset */
        system_addr = (u32 *)(sep->shared_addr +
                SEP_DRIVER_SYSTEM_EXT_CACHE_ADDR_OFFSET_IN_BYTES);
 
-       /* copy the physical address to the System Area for the sep */
+       /* Copy the physical address to the System Area for the SEP */
        system_addr[0] = SEP_EXT_CACHE_ADDR_VAL_TOKEN;
-       dev_dbg(&sep->pdev->dev, "ext cache init; system addr 0 is %x\n",
-                                                       system_addr[0]);
        system_addr[1] = sep->extapp_bus;
-       dev_dbg(&sep->pdev->dev, "ext cache init; system addr 1 is %x\n",
-                                                       system_addr[1]);
 
        return 0;
 }
@@ -2997,39 +2807,13 @@ static long sep_ioctl(struct file *filp, unsigned int cmd, unsigned long arg)
        int error = 0;
        struct sep_device *sep = filp->private_data;
 
-       dev_dbg(&sep->pdev->dev, "ioctl start\n");
-
-       dev_dbg(&sep->pdev->dev, "cmd is %x\n", cmd);
-       dev_dbg(&sep->pdev->dev,
-               "SEP_IOCSENDSEPCOMMAND is %x\n", SEP_IOCSENDSEPCOMMAND);
-       dev_dbg(&sep->pdev->dev,
-               "SEP_IOCALLOCDATAPOLL is %x\n", SEP_IOCALLOCDATAPOLL);
-       dev_dbg(&sep->pdev->dev,
-               "SEP_IOCCREATESYMDMATABLE is %x\n", SEP_IOCCREATESYMDMATABLE);
-       dev_dbg(&sep->pdev->dev,
-               "SEP_IOCFREEDMATABLEDATA is %x\n", SEP_IOCFREEDMATABLEDATA);
-       dev_dbg(&sep->pdev->dev,
-               "SEP_IOCSEPSTART is %x\n", SEP_IOCSEPSTART);
-       dev_dbg(&sep->pdev->dev,
-               "SEP_IOCSEPINIT is %x\n", SEP_IOCSEPINIT);
-       dev_dbg(&sep->pdev->dev,
-               "SEP_IOCGETSTATICPOOLADDR is %x\n", SEP_IOCGETSTATICPOOLADDR);
-       dev_dbg(&sep->pdev->dev,
-               "SEP_IOCENDTRANSACTION is %x\n", SEP_IOCENDTRANSACTION);
-       dev_dbg(&sep->pdev->dev,
-               "SEP_IOCREALLOCEXTCACHE is %x\n", SEP_IOCREALLOCEXTCACHE);
-       dev_dbg(&sep->pdev->dev,
-               "SEP_IOCRARPREPAREMESSAGE is %x\n", SEP_IOCRARPREPAREMESSAGE);
-       dev_dbg(&sep->pdev->dev,
-               "SEP_IOCPREPAREDCB is %x\n", SEP_IOCPREPAREDCB);
-       dev_dbg(&sep->pdev->dev,
-               "SEP_IOCFREEDCB is %x\n", SEP_IOCFREEDCB);
+       dev_dbg(&sep->pdev->dev, "ioctl cmd is %x\n", cmd);
 
-       /* make sure we own this device */
+       /* Make sure we own this device */
        mutex_lock(&sep->sep_mutex);
        if ((current->pid != sep->pid_doing_transaction) &&
                                (sep->pid_doing_transaction != 0)) {
-               dev_dbg(&sep->pdev->dev, "ioctl pid is not owner\n");
+               dev_warn(&sep->pdev->dev, "ioctl pid is not owner\n");
                mutex_unlock(&sep->sep_mutex);
                error = -EACCES;
                goto end_function;
@@ -3037,60 +2821,58 @@ static long sep_ioctl(struct file *filp, unsigned int cmd, unsigned long arg)
 
        mutex_unlock(&sep->sep_mutex);
 
-       /* check that the command is for sep device */
+       /* Check that the command is for SEP device */
        if (_IOC_TYPE(cmd) != SEP_IOC_MAGIC_NUMBER) {
                error = -ENOTTY;
                goto end_function;
        }
 
-       /* lock to prevent the daemon to interfere with operation */
+       /* Lock to prevent the daemon to interfere with operation */
        mutex_lock(&sep->ioctl_mutex);
 
        switch (cmd) {
        case SEP_IOCSENDSEPCOMMAND:
-               /* send command to SEP */
+               /* Send command to SEP */
                error = sep_send_command_handler(sep);
                break;
        case SEP_IOCALLOCDATAPOLL:
-               /* allocate data pool */
+               /* Allocate data pool */
                error = sep_allocate_data_pool_memory_handler(sep, arg);
                break;
        case SEP_IOCCREATESYMDMATABLE:
-               /* create dma table for synhronic operation */
+               /* Create DMA table for synhronic operation */
                error = sep_create_sync_dma_tables_handler(sep, arg);
                break;
        case SEP_IOCFREEDMATABLEDATA:
-               /* free the pages */
+               /* Free the pages */
                error = sep_free_dma_table_data_handler(sep);
                break;
        case SEP_IOCSEPSTART:
-               /* start command to sep */
-               if (sep->pdev->revision == 0) /* only for old chip */
+               /* Start command to SEP */
+               if (sep->pdev->revision == 0) /* Only for old chip */
                        error = sep_start_handler(sep);
                else
-                       error = -EPERM; /* not permitted on new chip */
+                       error = -EPERM; /* Not permitted on new chip */
                break;
        case SEP_IOCSEPINIT:
-               /* init command to sep */
-               if (sep->pdev->revision == 0) /* only for old chip */
+               /* Init command to SEP */
+               if (sep->pdev->revision == 0) /* Only for old chip */
                        error = sep_init_handler(sep, arg);
                else
-                       error = -EPERM; /* not permitted on new chip */
+                       error = -EPERM; /* Not permitted on new chip */
                break;
        case SEP_IOCGETSTATICPOOLADDR:
-               /* get the physical and virtual addresses of the static pool */
-               error = sep_get_static_pool_addr_handler(sep, arg);
+               /* Inform the SEP the bus address of the static pool */
+               error = sep_get_static_pool_addr_handler(sep);
                break;
        case SEP_IOCENDTRANSACTION:
                error = sep_end_transaction_handler(sep);
                break;
        case SEP_IOCREALLOCEXTCACHE:
-               if (sep->mrst)
-                       error = -ENODEV;
-               if (sep->pdev->revision == 0) /* only for old chip */
+               if (sep->pdev->revision == 0) /* Only for old chip */
                        error = sep_realloc_ext_cache_handler(sep, arg);
                else
-                       error = -EPERM; /* not permitted on new chip */
+                       error = -EPERM; /* Not permitted on new chip */
                break;
        case SEP_IOCRARPREPAREMESSAGE:
                error = sep_rar_prepare_output_msg_handler(sep, arg);
@@ -3123,24 +2905,22 @@ end_function:
  */
 static long sep_singleton_ioctl(struct file  *filp, u32 cmd, unsigned long arg)
 {
-       /* error */
        long error = 0;
        struct sep_device *sep = filp->private_data;
 
-       dev_dbg(&sep->pdev->dev, "singleton_ioctl start\n");
-       dev_dbg(&sep->pdev->dev, "cmd is %x\n", cmd);
+       dev_dbg(&sep->pdev->dev, "singleton ioctl cmd is %x\n", cmd);
 
-       /* check that the command is for sep device */
+       /* Check that the command is for the SEP device */
        if (_IOC_TYPE(cmd) != SEP_IOC_MAGIC_NUMBER) {
                error =  -ENOTTY;
                goto end_function;
        }
 
-       /* make sure we own this device */
+       /* Make sure we own this device */
        mutex_lock(&sep->sep_mutex);
        if ((current->pid != sep->pid_doing_transaction) &&
                                (sep->pid_doing_transaction != 0)) {
-               dev_dbg(&sep->pdev->dev, "singleton ioctl pid is not owner\n");
+               dev_warn(&sep->pdev->dev, "singleton ioctl pid is not owner\n");
                mutex_unlock(&sep->sep_mutex);
                error = -EACCES;
                goto end_function;
@@ -3160,7 +2940,6 @@ static long sep_singleton_ioctl(struct file  *filp, u32 cmd, unsigned long arg)
        }
 
 end_function:
-       dev_dbg(&sep->pdev->dev, "singleton ioctl end\n");
        return error;
 }
 
@@ -3179,45 +2958,43 @@ static long sep_request_daemon_ioctl(struct file *filp, u32 cmd,
        long error;
        struct sep_device *sep = filp->private_data;
 
-       dev_dbg(&sep->pdev->dev, "daemon ioctl: start\n");
        dev_dbg(&sep->pdev->dev, "daemon ioctl: cmd is %x\n", cmd);
 
-       /* check that the command is for sep device */
+       /* Check that the command is for SEP device */
        if (_IOC_TYPE(cmd) != SEP_IOC_MAGIC_NUMBER) {
                error = -ENOTTY;
                goto end_function;
        }
 
-       /* only one process can access ioctl at any given time */
+       /* Only one process can access ioctl at any given time */
        mutex_lock(&sep->ioctl_mutex);
 
        switch (cmd) {
        case SEP_IOCSENDSEPRPLYCOMMAND:
-               /* send reply command to SEP */
+               /* Send reply command to SEP */
                error = sep_req_daemon_send_reply_command_handler(sep);
                break;
        case SEP_IOCENDTRANSACTION:
                /*
-                * end req daemon transaction, do nothing
+                * End req daemon transaction, do nothing
                 * will be removed upon update in middleware
                 * API library
                 */
                error = 0;
                break;
        default:
-               dev_dbg(&sep->pdev->dev, "daemon ioctl: no such IOCTL\n");
+               dev_warn(&sep->pdev->dev, "daemon ioctl: no such IOCTL\n");
                error = -ENOTTY;
        }
        mutex_unlock(&sep->ioctl_mutex);
 
 end_function:
-       dev_dbg(&sep->pdev->dev, "daemon ioctl: end\n");
        return error;
 
 }
 
 /**
- *     sep_inthandler - Interrupt Handler
+ *     sep_inthandler - interrupt handler
  *     @irq: interrupt
  *     @dev_id: device id
  */
@@ -3228,12 +3005,11 @@ static irqreturn_t sep_inthandler(int irq, void *dev_id)
        u32 reg_val, reg_val2 = 0;
        struct sep_device *sep = dev_id;
 
-       /* read the IRR register to check if this is SEP interrupt */
+       /* Read the IRR register to check if this is SEP interrupt */
        reg_val = sep_read_reg(sep, HW_HOST_IRR_REG_ADDR);
-       dev_dbg(&sep->pdev->dev, "SEP Interrupt - reg is %08x\n", reg_val);
 
        if (reg_val & (0x1 << 13)) {
-               /* lock and update the counter of reply messages */
+               /* Lock and update the counter of reply messages */
                spin_lock_irqsave(&sep->snd_rply_lck, lck_flags);
                sep->reply_ct++;
                spin_unlock_irqrestore(&sep->snd_rply_lck, lck_flags);
@@ -3241,7 +3017,7 @@ static irqreturn_t sep_inthandler(int irq, void *dev_id)
                dev_dbg(&sep->pdev->dev, "sep int: send_ct %lx reply_ct %lx\n",
                                        sep->send_ct, sep->reply_ct);
 
-               /* is this printf or daemon request? */
+               /* Is this printf or daemon request? */
                reg_val2 = sep_read_reg(sep, HW_HOST_SEP_HOST_GPR2_REG_ADDR);
                dev_dbg(&sep->pdev->dev,
                        "SEP Interrupt - reg2 is %08x\n", reg_val2);
@@ -3253,11 +3029,11 @@ static irqreturn_t sep_inthandler(int irq, void *dev_id)
                        dev_dbg(&sep->pdev->dev, "int: daemon request\n");
                        wake_up(&sep->event_request_daemon);
                } else {
-                       dev_dbg(&sep->pdev->dev, "int: sep reply\n");
+                       dev_dbg(&sep->pdev->dev, "int: SEP reply\n");
                        wake_up(&sep->event);
                }
        } else {
-               dev_dbg(&sep->pdev->dev, "int: not sep interrupt\n");
+               dev_dbg(&sep->pdev->dev, "int: not SEP interrupt\n");
                int_error = IRQ_NONE;
        }
        if (int_error == IRQ_HANDLED)
@@ -3267,54 +3043,126 @@ static irqreturn_t sep_inthandler(int irq, void *dev_id)
 }
 
 /**
- *     sep_callback - RAR callback
- *     @sep_context_pointer: pointer to struct sep_device
+ *     sep_reconfig_shared_area - reconfigure shared area
+ *     @sep: pointer to struct sep_device
  *
- *     Function that is called by rar_register when it is ready with
- *     a region (only for Moorestown)
+ *     Reconfig the shared area between HOST and SEP - needed in case
+ *     the DX_CC_Init function was called before OS loading.
  */
-static int sep_callback(unsigned long sep_context_pointer)
+static int sep_reconfig_shared_area(struct sep_device *sep)
 {
-       int error;
-       struct sep_device *sep = (struct sep_device *)sep_context_pointer;
-       dma_addr_t rar_end_address;
+       int ret_val;
 
-       dev_dbg(&sep->pdev->dev, "callback start\n");
+       /* use to limit waiting for SEP */
+       unsigned long end_time;
 
-       error = rar_get_address(RAR_TYPE_IMAGE, &sep->rar_bus,
-                                                       &rar_end_address);
+       /* Send the new SHARED MESSAGE AREA to the SEP */
+       dev_dbg(&sep->pdev->dev, "reconfig shared; sending %08llx to sep\n",
+                               (unsigned long long)sep->shared_bus);
 
-       if (error) {
-               dev_warn(&sep->pdev->dev, "mrst cant get rar region\n");
-               goto end_function;
-       }
+       sep_write_reg(sep, HW_HOST_HOST_SEP_GPR1_REG_ADDR, sep->shared_bus);
 
-       sep->rar_size = (size_t)(rar_end_address - sep->rar_bus + 1);
+       /* Poll for SEP response */
+       ret_val = sep_read_reg(sep, HW_HOST_SEP_HOST_GPR1_REG_ADDR);
 
-       if (!request_mem_region(sep->rar_bus, sep->rar_size,
-                                                       "sep_sec_driver")) {
-               dev_warn(&sep->pdev->dev,
-                               "request mem region for mrst failed\n");
-               error = -1;
-               goto end_function;
+       end_time = jiffies + (WAIT_TIME * HZ);
+
+       while ((time_before(jiffies, end_time)) && (ret_val != 0xffffffff) &&
+               (ret_val != sep->shared_bus))
+               ret_val = sep_read_reg(sep, HW_HOST_SEP_HOST_GPR1_REG_ADDR);
+
+       /* Check the return value (register) */
+       if (ret_val != sep->shared_bus) {
+               dev_warn(&sep->pdev->dev, "could not reconfig shared area\n");
+               dev_warn(&sep->pdev->dev, "result was %x\n", ret_val);
+               ret_val = -ENOMEM;
+       } else
+               ret_val = 0;
+
+       dev_dbg(&sep->pdev->dev, "reconfig shared area end\n");
+       return ret_val;
+}
+
+/* File operation for singleton SEP operations */
+static const struct file_operations singleton_file_operations = {
+       .owner = THIS_MODULE,
+       .unlocked_ioctl = sep_singleton_ioctl,
+       .poll = sep_poll,
+       .open = sep_singleton_open,
+       .release = sep_singleton_release,
+       .mmap = sep_mmap,
+};
+
+/* File operation for daemon operations */
+static const struct file_operations daemon_file_operations = {
+       .owner = THIS_MODULE,
+       .unlocked_ioctl = sep_request_daemon_ioctl,
+       .poll = sep_request_daemon_poll,
+       .open = sep_request_daemon_open,
+       .release = sep_request_daemon_release,
+       .mmap = sep_request_daemon_mmap,
+};
+
+/* The files operations structure of the driver */
+static const struct file_operations sep_file_operations = {
+       .owner = THIS_MODULE,
+       .unlocked_ioctl = sep_ioctl,
+       .poll = sep_poll,
+       .open = sep_open,
+       .release = sep_release,
+       .mmap = sep_mmap,
+};
+
+/**
+ *     sep_register_driver_with_fs - register misc devices
+ *     @sep: pointer to struct sep_device
+ *
+ *     This function registers the driver with the file system
+ */
+static int sep_register_driver_with_fs(struct sep_device *sep)
+{
+       int ret_val;
+
+       sep->miscdev_sep.minor = MISC_DYNAMIC_MINOR;
+       sep->miscdev_sep.name = SEP_DEV_NAME;
+       sep->miscdev_sep.fops = &sep_file_operations;
+
+       sep->miscdev_singleton.minor = MISC_DYNAMIC_MINOR;
+       sep->miscdev_singleton.name = SEP_DEV_SINGLETON;
+       sep->miscdev_singleton.fops = &singleton_file_operations;
+
+       sep->miscdev_daemon.minor = MISC_DYNAMIC_MINOR;
+       sep->miscdev_daemon.name = SEP_DEV_DAEMON;
+       sep->miscdev_daemon.fops = &daemon_file_operations;
+
+       ret_val = misc_register(&sep->miscdev_sep);
+       if (ret_val) {
+               dev_warn(&sep->pdev->dev, "misc reg fails for SEP %x\n",
+                       ret_val);
+               return ret_val;
        }
 
-       sep->rar_addr = ioremap_nocache(sep->rar_bus, sep->rar_size);
-       if (!sep->rar_addr) {
-               dev_warn(&sep->pdev->dev,
-                               "ioremap nocache for mrst rar failed\n");
-               error = -ENOMEM;
-               goto end_function;
+       ret_val = misc_register(&sep->miscdev_singleton);
+       if (ret_val) {
+               dev_warn(&sep->pdev->dev, "misc reg fails for sing %x\n",
+                       ret_val);
+               misc_deregister(&sep->miscdev_sep);
+               return ret_val;
        }
-       dev_dbg(&sep->pdev->dev, "rar start is %p, phy is %llx, size is %x\n",
-                       sep->rar_addr, (unsigned long long)sep->rar_bus,
-                       sep->rar_size);
 
-end_function:
-       dev_dbg(&sep->pdev->dev, "callback end\n");
-       return error;
+       ret_val = misc_register(&sep->miscdev_daemon);
+       if (ret_val) {
+               dev_warn(&sep->pdev->dev, "misc reg fails for dmn %x\n",
+                       ret_val);
+               misc_deregister(&sep->miscdev_sep);
+               misc_deregister(&sep->miscdev_singleton);
+
+               return ret_val;
+       }
+       return ret_val;
 }
 
+
 /**
  *     sep_probe - probe a matching PCI device
  *     @pdev: pci_device
@@ -3329,64 +3177,67 @@ static int __devinit sep_probe(struct pci_dev *pdev,
        int error = 0;
        struct sep_device *sep;
 
-       pr_debug("Sep pci probe starting\n");
        if (sep_dev != NULL) {
                dev_warn(&pdev->dev, "only one SEP supported.\n");
                return -EBUSY;
        }
 
-       /* enable the device */
+       /* Enable the device */
        error = pci_enable_device(pdev);
        if (error) {
                dev_warn(&pdev->dev, "error enabling pci device\n");
                goto end_function;
        }
 
-       /* allocate the sep_device structure for this device */
+       /* Allocate the sep_device structure for this device */
        sep_dev = kzalloc(sizeof(struct sep_device), GFP_ATOMIC);
        if (sep_dev == NULL) {
                dev_warn(&pdev->dev,
                        "can't kmalloc the sep_device structure\n");
-               return -ENOMEM;
+               error = -ENOMEM;
+               goto end_function_disable_device;
        }
 
        /*
-        * we're going to use another variable for actually
+        * We're going to use another variable for actually
         * working with the device; this way, if we have
         * multiple devices in the future, it would be easier
         * to make appropriate changes
         */
        sep = sep_dev;
 
-       sep->pdev = pdev;
+       sep->pdev = pci_dev_get(pdev);
 
-       if (pdev->device == MRST_PCI_DEVICE_ID)
-               sep->mrst = 1;
+       init_waitqueue_head(&sep->event);
+       init_waitqueue_head(&sep->event_request_daemon);
+       spin_lock_init(&sep->snd_rply_lck);
+       mutex_init(&sep->sep_mutex);
+       mutex_init(&sep->ioctl_mutex);
 
-       dev_dbg(&sep->pdev->dev, "PCI obtained, device being prepared\n");
+       dev_dbg(&sep->pdev->dev, "sep probe: PCI obtained, device being prepared\n");
        dev_dbg(&sep->pdev->dev, "revision is %d\n", sep->pdev->revision);
 
-       /* set up our register area */
+       /* Set up our register area */
        sep->reg_physical_addr = pci_resource_start(sep->pdev, 0);
        if (!sep->reg_physical_addr) {
                dev_warn(&sep->pdev->dev, "Error getting register start\n");
-               pci_dev_put(sep->pdev);
-               return -ENODEV;
+               error = -ENODEV;
+               goto end_function_free_sep_dev;
        }
 
        sep->reg_physical_end = pci_resource_end(sep->pdev, 0);
        if (!sep->reg_physical_end) {
                dev_warn(&sep->pdev->dev, "Error getting register end\n");
-               pci_dev_put(sep->pdev);
-               return -ENODEV;
+               error = -ENODEV;
+               goto end_function_free_sep_dev;
        }
 
        sep->reg_addr = ioremap_nocache(sep->reg_physical_addr,
                (size_t)(sep->reg_physical_end - sep->reg_physical_addr + 1));
        if (!sep->reg_addr) {
                dev_warn(&sep->pdev->dev, "Error getting register virtual\n");
-               pci_dev_put(sep->pdev);
-               return -ENODEV;
+               error = -ENODEV;
+               goto end_function_free_sep_dev;
        }
 
        dev_dbg(&sep->pdev->dev,
@@ -3395,7 +3246,7 @@ static int __devinit sep_probe(struct pci_dev *pdev,
                (unsigned long long)sep->reg_physical_end,
                sep->reg_addr);
 
-       /* allocate the shared area */
+       /* Allocate the shared area */
        sep->shared_size = SEP_DRIVER_MESSAGE_SHARED_AREA_SIZE_IN_BYTES +
                SYNCHRONIC_DMA_TABLES_AREA_SIZE_BYTES +
                SEP_DRIVER_DATA_POOL_SHARED_AREA_SIZE_IN_BYTES +
@@ -3404,202 +3255,115 @@ static int __devinit sep_probe(struct pci_dev *pdev,
 
        if (sep_map_and_alloc_shared_area(sep)) {
                error = -ENOMEM;
-               /* allocation failed */
+               /* Allocation failed */
                goto end_function_error;
        }
 
-       /* the next section depends on type of unit */
-       if (sep->mrst) {
-               error = register_rar(RAR_TYPE_IMAGE, &sep_callback,
-                       (unsigned long)sep);
-               if (error) {
-                       dev_dbg(&sep->pdev->dev,
-                               "error register_rar\n");
-                       goto end_function_deallocate_sep_shared_area;
-               }
-       } else {
-               sep->rar_size = FAKE_RAR_SIZE;
-               sep->rar_addr = dma_alloc_coherent(NULL,
-                       sep->rar_size, &sep->rar_bus, GFP_KERNEL);
-               if (sep->rar_addr == NULL) {
-                       dev_warn(&sep->pdev->dev, "cant allocate mfld rar\n");
-                       error = -ENOMEM;
-                       goto end_function_deallocate_sep_shared_area;
-               }
-
-               dev_dbg(&sep->pdev->dev, "rar start is %p, phy is %llx,"
-                       " size is %x\n", sep->rar_addr,
-                       (unsigned long long)sep->rar_bus,
-                       sep->rar_size);
+       sep->rar_size = FAKE_RAR_SIZE;
+       sep->rar_addr = dma_alloc_coherent(&sep->pdev->dev,
+               sep->rar_size, &sep->rar_bus, GFP_KERNEL);
+       if (sep->rar_addr == NULL) {
+               dev_warn(&sep->pdev->dev, "can't allocate mfld rar\n");
+               error = -ENOMEM;
+               goto end_function_deallocate_sep_shared_area;
        }
 
-       dev_dbg(&sep->pdev->dev, "about to write IMR and ICR REG_ADDR\n");
+       dev_dbg(&sep->pdev->dev, "rar start is %p, phy is %llx,"
+               " size is %zx\n", sep->rar_addr,
+               (unsigned long long)sep->rar_bus,
+               sep->rar_size);
 
-       /* clear ICR register */
+       /* Clear ICR register */
        sep_write_reg(sep, HW_HOST_ICR_REG_ADDR, 0xFFFFFFFF);
 
-       /* set the IMR register - open only GPR 2 */
+       /* Set the IMR register - open only GPR 2 */
        sep_write_reg(sep, HW_HOST_IMR_REG_ADDR, (~(0x1 << 13)));
 
-       dev_dbg(&sep->pdev->dev, "about to call request_irq\n");
-       /* get the interrupt line */
+       /* Read send/receive counters from SEP */
+       sep->reply_ct = sep_read_reg(sep, HW_HOST_SEP_HOST_GPR2_REG_ADDR);
+       sep->reply_ct &= 0x3FFFFFFF;
+       sep->send_ct = sep->reply_ct;
+
+       /* Get the interrupt line */
        error = request_irq(pdev->irq, sep_inthandler, IRQF_SHARED,
                "sep_driver", sep);
 
-       if (!error)
-               goto end_function;
+       if (error)
+               goto end_function_dealloc_rar;
 
+       /* The new chip requires ashared area reconfigure */
+       if (sep->pdev->revision == 4) { /* Only for new chip */
+               error = sep_reconfig_shared_area(sep);
+               if (error)
+                       goto end_function_free_irq;
+       }
+       /* Finally magic up the device nodes */
+       /* Register driver with the fs */
+       error = sep_register_driver_with_fs(sep);
+       if (error == 0)
+               /* Success */
+               return 0;
+
+end_function_free_irq:
+       free_irq(pdev->irq, sep);
+
+end_function_dealloc_rar:
        if (sep->rar_addr)
                dma_free_coherent(&sep->pdev->dev, sep->rar_size,
                        sep->rar_addr, sep->rar_bus);
        goto end_function;
 
 end_function_deallocate_sep_shared_area:
-       /* de-allocate shared area */
+       /* De-allocate shared area */
        sep_unmap_and_free_shared_area(sep);
 
 end_function_error:
        iounmap(sep->reg_addr);
+
+end_function_free_sep_dev:
+       pci_dev_put(sep_dev->pdev);
        kfree(sep_dev);
        sep_dev = NULL;
 
+end_function_disable_device:
+       pci_disable_device(pdev);
+
 end_function:
        return error;
 }
 
+static void sep_remove(struct pci_dev *pdev)
+{
+       struct sep_device *sep = sep_dev;
+
+       /* Unregister from fs */
+       misc_deregister(&sep->miscdev_sep);
+       misc_deregister(&sep->miscdev_singleton);
+       misc_deregister(&sep->miscdev_daemon);
+
+       /* Free the irq */
+       free_irq(sep->pdev->irq, sep);
+
+       /* Free the shared area  */
+       sep_unmap_and_free_shared_area(sep_dev);
+       iounmap((void *) sep_dev->reg_addr);
+}
+
 static DEFINE_PCI_DEVICE_TABLE(sep_pci_id_tbl) = {
-       {PCI_DEVICE(PCI_VENDOR_ID_INTEL, MRST_PCI_DEVICE_ID)},
        {PCI_DEVICE(PCI_VENDOR_ID_INTEL, MFLD_PCI_DEVICE_ID)},
        {0}
 };
 
 MODULE_DEVICE_TABLE(pci, sep_pci_id_tbl);
 
-/* field for registering driver to PCI device */
+/* Field for registering driver to PCI device */
 static struct pci_driver sep_pci_driver = {
        .name = "sep_sec_driver",
        .id_table = sep_pci_id_tbl,
-       .probe = sep_probe
-       /* FIXME: remove handler */
+       .probe = sep_probe,
+       .remove = sep_remove
 };
 
-/* file operation for singleton sep operations */
-static const struct file_operations singleton_file_operations = {
-       .owner = THIS_MODULE,
-       .unlocked_ioctl = sep_singleton_ioctl,
-       .poll = sep_poll,
-       .open = sep_singleton_open,
-       .release = sep_singleton_release,
-       .mmap = sep_mmap,
-};
-
-/* file operation for daemon operations */
-static const struct file_operations daemon_file_operations = {
-       .owner = THIS_MODULE,
-       .unlocked_ioctl = sep_request_daemon_ioctl,
-       .poll = sep_request_daemon_poll,
-       .open = sep_request_daemon_open,
-       .release = sep_request_daemon_release,
-       .mmap = sep_request_daemon_mmap,
-};
-
-/* the files operations structure of the driver */
-static const struct file_operations sep_file_operations = {
-       .owner = THIS_MODULE,
-       .unlocked_ioctl = sep_ioctl,
-       .poll = sep_poll,
-       .open = sep_open,
-       .release = sep_release,
-       .mmap = sep_mmap,
-};
-
-/**
- *     sep_reconfig_shared_area - reconfigure shared area
- *     @sep: pointer to struct sep_device
- *
- *     Reconfig the shared area between HOST and SEP - needed in case
- *     the DX_CC_Init function was called before OS loading.
- */
-static int sep_reconfig_shared_area(struct sep_device *sep)
-{
-       int ret_val;
-
-       dev_dbg(&sep->pdev->dev, "reconfig shared area start\n");
-
-       /* send the new SHARED MESSAGE AREA to the SEP */
-       dev_dbg(&sep->pdev->dev, "sending %08llx to sep\n",
-                               (unsigned long long)sep->shared_bus);
-
-       sep_write_reg(sep, HW_HOST_HOST_SEP_GPR1_REG_ADDR, sep->shared_bus);
-
-       /* poll for SEP response */
-       ret_val = sep_read_reg(sep, HW_HOST_SEP_HOST_GPR1_REG_ADDR);
-
-       while (ret_val != 0xffffffff && ret_val != sep->shared_bus)
-               ret_val = sep_read_reg(sep, HW_HOST_SEP_HOST_GPR1_REG_ADDR);
-
-       /* check the return value (register) */
-       if (ret_val != sep->shared_bus) {
-               dev_warn(&sep->pdev->dev, "could not reconfig shared area\n");
-               dev_warn(&sep->pdev->dev, "result was %x\n", ret_val);
-               ret_val = -ENOMEM;
-       } else
-               ret_val = 0;
-
-       dev_dbg(&sep->pdev->dev, "reconfig shared area end\n");
-       return ret_val;
-}
-
-/**
- *     sep_register_driver_to_fs - register misc devices
- *     @sep: pointer to struct sep_device
- *
- *     This function registers the driver to the file system
- */
-static int sep_register_driver_to_fs(struct sep_device *sep)
-{
-       int ret_val;
-
-       sep->miscdev_sep.minor = MISC_DYNAMIC_MINOR;
-       sep->miscdev_sep.name = SEP_DEV_NAME;
-       sep->miscdev_sep.fops = &sep_file_operations;
-
-       sep->miscdev_singleton.minor = MISC_DYNAMIC_MINOR;
-       sep->miscdev_singleton.name = SEP_DEV_SINGLETON;
-       sep->miscdev_singleton.fops = &singleton_file_operations;
-
-       sep->miscdev_daemon.minor = MISC_DYNAMIC_MINOR;
-       sep->miscdev_daemon.name = SEP_DEV_DAEMON;
-       sep->miscdev_daemon.fops = &daemon_file_operations;
-
-       ret_val = misc_register(&sep->miscdev_sep);
-       if (ret_val) {
-               dev_warn(&sep->pdev->dev, "misc reg fails for sep %x\n",
-                       ret_val);
-               return ret_val;
-       }
-
-       ret_val = misc_register(&sep->miscdev_singleton);
-       if (ret_val) {
-               dev_warn(&sep->pdev->dev, "misc reg fails for sing %x\n",
-                       ret_val);
-               misc_deregister(&sep->miscdev_sep);
-               return ret_val;
-       }
-
-       if (!sep->mrst) {
-               ret_val = misc_register(&sep->miscdev_daemon);
-               if (ret_val) {
-                       dev_warn(&sep->pdev->dev, "misc reg fails for dmn %x\n",
-                               ret_val);
-                       misc_deregister(&sep->miscdev_sep);
-                       misc_deregister(&sep->miscdev_singleton);
-
-                       return ret_val;
-               }
-       }
-       return ret_val;
-}
 
 /**
  *     sep_init - init function
@@ -3608,85 +3372,19 @@ static int sep_register_driver_to_fs(struct sep_device *sep)
  */
 static int __init sep_init(void)
 {
-       int ret_val = 0;
-       struct sep_device *sep = NULL;
-
-       pr_debug("Sep driver: Init start\n");
-
-       ret_val = pci_register_driver(&sep_pci_driver);
-       if (ret_val) {
-               pr_debug("sep_driver:sep_driver_to_device failed, ret_val is %d\n",
-                                                               ret_val);
-               goto end_function;
-       }
-
-       sep = sep_dev;
-
-       init_waitqueue_head(&sep->event);
-       init_waitqueue_head(&sep->event_request_daemon);
-       spin_lock_init(&sep->snd_rply_lck);
-       mutex_init(&sep->sep_mutex);
-       mutex_init(&sep->ioctl_mutex);
-
-       /* new chip requires share area reconfigure */
-       if (sep->pdev->revision == 4) { /* only for new chip */
-               ret_val = sep_reconfig_shared_area(sep);
-               if (ret_val)
-                       goto end_function_unregister_pci;
-       }
-
-       /* register driver to fs */
-       ret_val = sep_register_driver_to_fs(sep);
-       if (ret_val) {
-               dev_warn(&sep->pdev->dev, "error registering device to file\n");
-               goto end_function_unregister_pci;
-       }
-       goto end_function;
-
-end_function_unregister_pci:
-       pci_unregister_driver(&sep_pci_driver);
-
-end_function:
-       dev_dbg(&sep->pdev->dev, "Init end\n");
-       return ret_val;
+       return pci_register_driver(&sep_pci_driver);
 }
 
 
 /**
- *     sep_exit - called to unload driver 
+ *     sep_exit - called to unload driver
  *
  *     Drop the misc devices then remove and unmap the various resources
  *     that are not released by the driver remove method.
  */
 static void __exit sep_exit(void)
 {
-       struct sep_device *sep;
-
-       sep = sep_dev;
-       pr_debug("Exit start\n");
-
-       /* unregister from fs */
-       misc_deregister(&sep->miscdev_sep);
-       misc_deregister(&sep->miscdev_singleton);
-       misc_deregister(&sep->miscdev_daemon);
-
-       /* free the irq */
-       free_irq(sep->pdev->irq, sep);
-
-       /* unregister the driver */
        pci_unregister_driver(&sep_pci_driver);
-
-       /* free shared area  */
-       if (sep_dev) {
-               sep_unmap_and_free_shared_area(sep_dev);
-               dev_dbg(&sep->pdev->dev,
-                       "free pages SEP SHARED AREA\n");
-               iounmap((void *) sep_dev->reg_addr);
-               dev_dbg(&sep->pdev->dev,
-                       "iounmap\n");
-       }
-       pr_debug("release_mem_region\n");
-       pr_debug("Exit end\n");
 }