Staging: sep: fix 2 warnings
authorRandy Dunlap <randy.dunlap@oracle.com>
Tue, 17 Nov 2009 18:17:42 +0000 (10:17 -0800)
committerGreg Kroah-Hartman <gregkh@suse.de>
Fri, 11 Dec 2009 20:23:19 +0000 (12:23 -0800)
Fix printk format warning:
drivers/staging/sep/sep_driver.c:276: warning: format '%08llx' expects type 'long long unsigned int', but argument 2 has type 'dma_addr_t'

and variable may be used uninitialized (correct):
drivers/staging/sep/sep_driver.c:1774: warning: 'error' may be used uninitialized in this function

Signed-off-by: Randy Dunlap <randy.dunlap@oracle.com>
Acked-by: Alan Cox <alan@linux.intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
drivers/staging/sep/sep_driver.c

index f890a16..67c7d2c 100644 (file)
@@ -273,7 +273,8 @@ static dma_addr_t sep_shared_virt_to_bus(struct sep_device *sep,
                                                void *virt_address)
 {
        dma_addr_t pa = sep->shared_bus + (virt_address - sep->shared_addr);
-       edbg("sep: virt to bus b %08llx v %p\n", pa, virt_address);
+       edbg("sep: virt to bus b %08llx v %p\n",
+               (unsigned long long)pa, virt_address);
        return pa;
 }
 
@@ -1788,6 +1789,7 @@ static int sep_create_flow_dma_tables_handler(struct sep_device *sep,
        first_table_data.physical_address = 0xffffffff;
 
        /* find the free structure for flow data */
+       error = -EINVAL;
        flow_context_ptr = sep_find_flow_context(sep, SEP_FREE_FLOW_ID);
        if (flow_context_ptr == NULL)
                goto end_function;