[PATCH 2.5.68] Convert elan-104nc to remove check_region().

Bob Miller (rem@osdl.org)
Fri, 2 May 2003 13:42:07 -0700


Moved the request_region() call to replace check_region() and adds
release_region()'s in the error paths that occure before the old
call to request_region(). NOTE: This patch just updates comments.

-- 
Bob Miller					Email: rem@osdl.org
Open Source Development Lab			Phone: 503.626.2455 Ext. 17

diff -Nru a/drivers/mtd/maps/elan-104nc.c b/drivers/mtd/maps/elan-104nc.c --- a/drivers/mtd/maps/elan-104nc.c Fri May 2 09:52:22 2003 +++ b/drivers/mtd/maps/elan-104nc.c Fri May 2 09:52:22 2003 @@ -30,8 +30,8 @@ The single flash device is divided into 3 partition which appear as separate MTD devices. -Linux thinks that the I/O port is used by the PIC and hence check_region() will -always fail. So we don't do it. I just hope it doesn't break anything. +Linux thinks that the I/O port is used by the PIC and hence request_region() +will always fail. So we don't do it. I just hope it doesn't break anything. */ #include <linux/module.h> #include <linux/slab.h> @@ -227,14 +227,14 @@ } iounmap((void *)iomapadr); - release_region(PAGE_IO,PAGE_IO_SIZE); + /* release_region(PAGE_IO,PAGE_IO_SIZE); */ } int __init init_elan_104nc(void) { /* Urg! We use I/O port 0x22 without request_region()ing it */ /* - if (check_region(PAGE_IO,PAGE_IO_SIZE) != 0) { + if (!request_region(PAGE_IO,PAGE_IO_SIZE, "ELAN-104NC flash")) { printk( KERN_ERR"%s: IO ports 0x%x-0x%x in use\n", elan_104nc_map.name, PAGE_IO, PAGE_IO+PAGE_IO_SIZE-1 ); @@ -245,12 +245,11 @@ if (!iomapadr) { printk( KERN_ERR"%s: failed to ioremap memory region\n", elan_104nc_map.name ); + /* + release_region(PAGE_IO,PAGE_IO_SIZE); + */ return -EIO; } - - /* - request_region( PAGE_IO, PAGE_IO_SIZE, "ELAN-104NC flash" ); - */ printk( KERN_INFO"%s: IO:0x%x-0x%x MEM:0x%x-0x%x\n", elan_104nc_map.name,

- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/