The patch titled Char: stallion, use dynamic dev has been added to the -mm tree. Its filename is char-stallion-use-dynamic-dev.patch See http://www.zip.com.au/~akpm/linux/patches/stuff/added-to-mm.txt to find out what to do about this ------------------------------------------------------ Subject: Char: stallion, use dynamic dev From: Jiri Slaby <jirislaby@xxxxxxxxx> Use dynamic tty device registering depending on board's port count. (i -> retval change is relevant, since gcc complains about signedness of i) Signed-off-by: Jiri Slaby <jirislaby@xxxxxxxxx> Cc: Alan Cox <alan@xxxxxxxxxxxxxxxxxxx> Signed-off-by: Andrew Morton <akpm@xxxxxxxx> --- drivers/char/stallion.c | 33 +++++++++++++++++++++++++++------ 1 files changed, 27 insertions(+), 6 deletions(-) diff -puN drivers/char/stallion.c~char-stallion-use-dynamic-dev drivers/char/stallion.c --- a/drivers/char/stallion.c~char-stallion-use-dynamic-dev +++ a/drivers/char/stallion.c @@ -2350,7 +2350,7 @@ static int __devinit stl_pciprobe(struct const struct pci_device_id *ent) { struct stlbrd *brdp; - unsigned int brdtype = ent->driver_data; + unsigned int i, brdtype = ent->driver_data; int brdnr, retval = -ENODEV; if ((pdev->class >> 8) == PCI_CLASS_STORAGE_IDE) @@ -2411,6 +2411,10 @@ static int __devinit stl_pciprobe(struct pci_set_drvdata(pdev, brdp); + for (i = 0; i < brdp->nrports; i++) + tty_register_device(stl_serial, + brdp->brdnr * STL_MAXPORTS + i, &pdev->dev); + return 0; err_null: stl_brds[brdp->brdnr] = NULL; @@ -2423,6 +2427,7 @@ err: static void __devexit stl_pciremove(struct pci_dev *pdev) { struct stlbrd *brdp = pci_get_drvdata(pdev); + unsigned int i; free_irq(brdp->irq, brdp); @@ -2432,6 +2437,10 @@ static void __devexit stl_pciremove(stru if (brdp->iosize2 > 0) release_region(brdp->ioaddr2, brdp->iosize2); + for (i = 0; i < brdp->nrports; i++) + tty_unregister_device(stl_serial, + brdp->brdnr * STL_MAXPORTS + i); + stl_brds[brdp->brdnr] = NULL; kfree(brdp); } @@ -4695,7 +4704,7 @@ static int __init stallion_module_init(v { struct stlbrd *brdp; struct stlconf conf; - unsigned int i; + unsigned int i, j; int retval; printk(KERN_INFO "%s: version %s\n", stl_drvtitle, stl_drvversion); @@ -4722,6 +4731,9 @@ static int __init stallion_module_init(v if (stl_brdinit(brdp)) kfree(brdp); else { + for (j = 0; j < brdp->nrports; j++) + tty_register_device(stl_serial, + brdp->brdnr * STL_MAXPORTS + j, NULL); stl_brds[brdp->brdnr] = brdp; stl_nrbrds = i + 1; } @@ -4763,7 +4775,7 @@ static int __init stallion_module_init(v stl_serial->type = TTY_DRIVER_TYPE_SERIAL; stl_serial->subtype = SERIAL_TYPE_NORMAL; stl_serial->init_termios = stl_deftermios; - stl_serial->flags = TTY_DRIVER_REAL_RAW; + stl_serial->flags = TTY_DRIVER_REAL_RAW | TTY_DRIVER_DYNAMIC_DEV; tty_set_operations(stl_serial, &stl_ops); retval = tty_register_driver(stl_serial); @@ -4789,7 +4801,9 @@ err: static void __exit stallion_module_exit(void) { - int i; + struct stlbrd *brdp; + unsigned int i, j; + int retval; pr_debug("cleanup_module()\n"); @@ -4802,14 +4816,21 @@ static void __exit stallion_module_exit( * a hangup on every open port - to try to flush out any processes * hanging onto ports. */ + for (i = 0; i < stl_nrbrds; i++) { + if ((brdp = stl_brds[i]) == NULL || (brdp->state & STL_PROBED)) + continue; + for (j = 0; j < brdp->nrports; j++) + tty_unregister_device(stl_serial, + brdp->brdnr * STL_MAXPORTS + j); + } tty_unregister_driver(stl_serial); put_tty_driver(stl_serial); for (i = 0; i < 4; i++) class_device_destroy(stallion_class, MKDEV(STL_SIOMEMMAJOR, i)); - if ((i = unregister_chrdev(STL_SIOMEMMAJOR, "staliomem"))) + if ((retval = unregister_chrdev(STL_SIOMEMMAJOR, "staliomem"))) printk("STALLION: failed to un-register serial memory device, " - "errno=%d\n", -i); + "errno=%d\n", -retval); class_destroy(stallion_class); pci_unregister_driver(&stl_pcidriver); _ Patches currently in -mm which might be from jirislaby@xxxxxxxxx are mxser-correct-tty-driver-name.patch pci-mxser-pci-refcounts.patch mxser-make-an-experimental-clone.patch char-mxser_new-correct-include-file.patch char-mxser_new-upgrade-to-191.patch char-mxser_new-rework-to-allow-dynamic-structs.patch char-mxser_new-use-__devinit-macros.patch char-mxser_new-pci_request_region-for-pci-regions.patch char-mxser_new-check-request_region-retvals.patch char-mxser_new-kill-unneeded-memsets.patch char-mxser_new-revert-spin_lock-changes.patch char-mxser_new-remove-request-for-testers-line.patch char-mxser_new-debug-printk-dependent-on-debug.patch char-mxser_new-alter-license-terms.patch char-mxser_new-code-upside-down.patch char-mxser_new-cmspar-is-defined.patch char-remove-unneded-termbits-redefinitions-mxser_new.patch char-mxser_new-eliminate-tty-ldisc-deref.patch char-mxser_new-testbit-for-bit-testing.patch char-mxser_new-correct-fail-paths.patch char-mxser_new-dont-check-tty_unregister-retval.patch char-mxser_new-compress-isa-finding.patch char-mxser_new-register-tty-devices-on-the-fly.patch char-mxser_new-compact-structures-round2.patch char-mxser_new-reverse-if-else-paths-patch.patch char-mxser_new-comments-cleanup.patch char-mxser_new-correct-intr-handler-proto.patch char-mxser_new-delete-ttys-and-termios.patch char-mxser_new-pci-probing.patch char-mxser_new-clean-macros.patch maintainers-add-me-to-isicom-mxser.patch mxser_new-correct-tty-driver-name.patch char-stallion-use-pr_debug-macro.patch char-stallion-remove-unneeded-casts.patch char-stallion-kill-typedefs.patch char-stallion-move-init-deinit.patch char-stallion-uninline-functions.patch char-stallion-mark-functions-as-init.patch char-stallion-remove-many-prototypes.patch char-isicom-expand-function.patch char-isicom-rename-init-function.patch char-isicom-remove-isa-code.patch char-isicom-remove-unneeded-memset.patch char-isicom-move-to-tty_register_device.patch char-isicom-use-pci_request_region.patch char-isicom-check-kmalloc-retval.patch char-isicom-use-completion.patch char-isicom-simplify-timer.patch char-isicom-remove-cvs-stuff.patch char-isicom-fix-tty-index-check.patch char-sx-convert-to-pci-probing.patch char-sx-use-kcalloc.patch char-sx-mark-functions-as-devinit.patch char-sx-use-eisa-probing.patch char-sx-ifdef-isa-code.patch char-sx-lock-boards-struct.patch char-sx-remove-duplicite-code.patch char-sx-whitespace-cleanup.patch char-sx-simplify-timer-logic.patch char-sx-fix-return-in-module-init.patch char-sx-use-pci_iomap.patch char-sx-request-regions.patch char-stallion-convert-to-pci-probing.patch char-stallion-prints-cleanup.patch char-stallion-implement-fail-paths.patch char-stallion-correct-__init-macros.patch char-stallion-functions-cleanup.patch char-stallion-fix-fail-paths.patch char-stallion-brd-struct-locking.patch char-stallion-remove-syntactic-sugar.patch char-stallion-variables-cleanup.patch char-stallion-use-dynamic-dev.patch - To unsubscribe from this list: send the line "unsubscribe mm-commits" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html