From f3851e73ecdd070bc379677ed7aad958446f26e7 Mon Sep 17 00:00:00 2001 From: Jiri Slaby Date: Tue, 8 May 2007 00:36:16 -0700 Subject: [PATCH] Char: cyclades, init card struct immediately Signed-off-by: Jiri Slaby Signed-off-by: Andrew Morton Signed-off-by: Linus Torvalds --- drivers/char/cyclades.c | 37 +++++++++++++++---------------------- 1 file changed, 15 insertions(+), 22 deletions(-) diff --git a/drivers/char/cyclades.c b/drivers/char/cyclades.c index 85c734b7f548..c7e1433bf8ea 100644 --- a/drivers/char/cyclades.c +++ b/drivers/char/cyclades.c @@ -4834,6 +4834,7 @@ static int __init cy_detect_isa(void) cy_card[j].bus_index = 0; cy_card[j].first_line = cy_next_channel; cy_card[j].num_chips = cy_isa_nchan / 4; + cy_init_card(&cy_card[j], j); nboard++; /* print message */ @@ -4931,6 +4932,7 @@ static int __devinit cy_init_Ze(unsigned long cy_pci_phys0, cy_card[j].first_line = cy_next_channel; cy_card[j].num_chips = -1; cy_card[j].pdev = pdev; + cy_init_card(&cy_card[j], j); pci_set_drvdata(pdev, &cy_card[j]); /* print message */ @@ -5080,6 +5082,7 @@ static int __devinit cy_pci_probe(struct pci_dev *pdev, cy_card[j].first_line = cy_next_channel; cy_card[j].num_chips = cy_pci_nchan / 4; cy_card[j].pdev = pdev; + cy_init_card(&cy_card[j], j); pci_set_drvdata(pdev, &cy_card[j]); /* enable interrupts in the PCI interface */ @@ -5263,6 +5266,7 @@ static int __devinit cy_pci_probe(struct pci_dev *pdev, cy_card[j].first_line = cy_next_channel; cy_card[j].num_chips = -1; cy_card[j].pdev = pdev; + cy_init_card(&cy_card[j], j); pci_set_drvdata(pdev, &cy_card[j]); /* print message */ @@ -5327,6 +5331,7 @@ static void __devexit cy_pci_release(struct pci_dev *pdev) { #ifdef CONFIG_PCI struct cyclades_card *cinfo = pci_get_drvdata(pdev); + unsigned int i; pci_iounmap(pdev, cinfo->base_addr); if (cinfo->ctl_addr) @@ -5340,6 +5345,10 @@ static void __devexit cy_pci_release(struct pci_dev *pdev) pci_release_regions(pdev); cinfo->base_addr = NULL; + for (i = cinfo->first_line; i < cinfo->first_line + cinfo->nports; i++){ + cy_port[i].line = -1; + cy_port[i].magic = -1; + } #endif } @@ -5483,6 +5492,12 @@ static int __init cy_init(void) cy_card[i].base_addr = NULL; } + /* invalidate remaining cy_port structures */ + for (i = 0; i < NR_PORTS; i++) { + cy_port[i].line = -1; + cy_port[i].magic = -1; + } + /* the code below is responsible to find the boards. Each different type of board has its own detection routine. If a board is found, the next cy_card structure available is set by the detection @@ -5498,29 +5513,7 @@ static int __init cy_init(void) cy_nboard = cy_isa_nboard + cy_pci_nboard; - /* invalidate remaining cy_card structures */ - for (i = 0; i < NR_CARDS; i++) { - if (cy_card[i].base_addr == 0) { - cy_card[i].first_line = -1; - cy_card[i].ctl_addr = NULL; - cy_card[i].irq = 0; - cy_card[i].bus_index = 0; - cy_card[i].first_line = 0; - cy_card[i].num_chips = 0; - } - } - /* invalidate remaining cy_port structures */ - for (i = cy_next_channel; i < NR_PORTS; i++) { - cy_port[i].line = -1; - cy_port[i].magic = -1; - } - - /* initialize per-port data structures for each valid board found */ - for (i = 0; i < cy_nboard; i++) - cy_init_card(&cy_card[i], i); - return 0; - } /* cy_init */ static void __exit cy_cleanup_module(void) -- 2.30.2