From: Greg Kroah-Hartman <gregkh@suse.de>
Date: Tue, 4 May 2010 23:09:26 +0000 (-0700)
Subject: Staging: comedi: adl_pci9111.c: fix sparse warnings
X-Git-Tag: v2.6.35-rc1~441^2^2~216
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=525d1b1395858606103d4663a570cc8725ff2ced;p=profile%2Fivi%2Fkernel-x86-ivi.git

Staging: comedi: adl_pci9111.c: fix sparse warnings

divisor and other problems fixed.

Cc: Ian Abbott <abbotti@mev.co.uk>
Cc: Frank Mori Hess <fmhess@users.sourceforge.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
---

diff --git a/drivers/staging/comedi/drivers/adl_pci9111.c b/drivers/staging/comedi/drivers/adl_pci9111.c
index bbe0e33..36a254c 100644
--- a/drivers/staging/comedi/drivers/adl_pci9111.c
+++ b/drivers/staging/comedi/drivers/adl_pci9111.c
@@ -358,8 +358,8 @@ struct pci9111_private_data {
 
 	int ao_readback;	/*  Last written analog output data */
 
-	int timer_divisor_1;	/*  Divisor values for the 8254 timer pacer */
-	int timer_divisor_2;
+	unsigned int timer_divisor_1;	/*  Divisor values for the 8254 timer pacer */
+	unsigned int timer_divisor_2;
 
 	int is_valid;		/*  Is device valid */
 
@@ -1269,9 +1269,6 @@ found:
 
 	/*  TODO: Warn about non-tested boards. */
 
-	switch (board->device_id) {
-	};
-
 	/*  Read local configuration register base address [PCI_BASE_ADDRESS #1]. */
 
 	lcr_io_base = pci_resource_start(pci_device, 1);
@@ -1381,7 +1378,7 @@ static int pci9111_detach(struct comedi_device *dev)
 {
 	/*  Reset device */
 
-	if (dev->private != 0) {
+	if (dev->private != NULL) {
 		if (dev_private->is_valid)
 			pci9111_reset(dev);
 
@@ -1391,7 +1388,7 @@ static int pci9111_detach(struct comedi_device *dev)
 	if (dev->irq != 0)
 		free_irq(dev->irq, dev);
 
-	if (dev_private != 0 && dev_private->pci_device != 0) {
+	if (dev_private != NULL && dev_private->pci_device != NULL) {
 		if (dev->iobase)
 			comedi_pci_disable(dev_private->pci_device);
 		pci_dev_put(dev_private->pci_device);