From owner-freebsd-usb@FreeBSD.ORG Fri Jan 13 21:10:13 2012 Return-Path: Delivered-To: freebsd-usb@hub.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 15E5B106566B for ; Fri, 13 Jan 2012 21:10:13 +0000 (UTC) (envelope-from gnats@FreeBSD.org) Received: from freefall.freebsd.org (freefall.freebsd.org [IPv6:2001:4f8:fff6::28]) by mx1.freebsd.org (Postfix) with ESMTP id DEA308FC12 for ; Fri, 13 Jan 2012 21:10:12 +0000 (UTC) Received: from freefall.freebsd.org (localhost [127.0.0.1]) by freefall.freebsd.org (8.14.5/8.14.5) with ESMTP id q0DLACdw021922 for ; Fri, 13 Jan 2012 21:10:12 GMT (envelope-from gnats@freefall.freebsd.org) Received: (from gnats@localhost) by freefall.freebsd.org (8.14.5/8.14.5/Submit) id q0DLACkm021921; Fri, 13 Jan 2012 21:10:12 GMT (envelope-from gnats) Date: Fri, 13 Jan 2012 21:10:12 GMT Message-Id: <201201132110.q0DLACkm021921@freefall.freebsd.org> To: freebsd-usb@FreeBSD.org From: Hans Petter Selasky Cc: Subject: Re: usb/164090: [umodem] [patch] Add sysctl with ucom unit number X-BeenThere: freebsd-usb@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list Reply-To: Hans Petter Selasky List-Id: FreeBSD support for USB List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 13 Jan 2012 21:10:13 -0000 The following reply was made to PR usb/164090; it has been noted by GNATS. From: Hans Petter Selasky To: freebsd-usb@freebsd.org Cc: Mykhaylo Yehorov , freebsd-gnats-submit@freebsd.org Subject: Re: usb/164090: [umodem] [patch] Add sysctl with ucom unit number Date: Fri, 13 Jan 2012 21:54:01 +0100 On Friday 13 January 2012 18:05:29 Mykhaylo Yehorov wrote: > >Number: 164090 > >Category: usb > >Synopsis: [umodem] [patch] Add sysctl with ucom unit number > >Confidential: no > >Severity: non-critical > >Priority: low > >Responsible: freebsd-usb > >State: open > >Quarter: > >Keywords: > >Date-Required: > >Class: change-request > >Submitter-Id: current-users > >Arrival-Date: Fri Jan 13 17:10:08 UTC 2012 > >Closed-Date: > >Last-Modified: > >Originator: Mykhaylo Yehorov > >Release: FreeBSD 9.0-RELEASE > >Organization: > > >Environment: > FreeBSD atom.lan29.local 9.0-RELEASE FreeBSD 9.0-RELEASE #7: Fri Jan 13 > 15:18:21 EET 2012 > mika@atom.lan29.local:/usr/obj/usr/src/sys/MINI_KVA384 i386 > > >Description: > Now there is no easy way to determine which tty device file is used by an > umodem device. With the following patch the umodem driver will create > sysctl node dev.umodem.X.ucom. This sysctl will be helpful in devd > scripts. > > >How-To-Repeat: > > >Fix: > Patch attached with submission follows: > > --- orig/sys/dev/usb/serial/umodem.c 2011-11-11 06:20:22.000000000 +0200 > +++ sys/dev/usb/serial/umodem.c 2012-01-03 13:06:14.371630639 +0200 > @@ -169,6 +169,8 @@ > uint8_t sc_cm_over_data; > uint8_t sc_cm_cap; /* CM capabilities */ > uint8_t sc_acm_cap; /* ACM capabilities */ > + > + int ucom_unit; > }; > > static device_probe_t umodem_probe; > @@ -412,6 +414,13 @@ > } > ucom_set_pnpinfo_usb(&sc->sc_super_ucom, dev); > > + sc->ucom_unit = sc->sc_super_ucom.sc_unit; > + > + SYSCTL_ADD_INT(device_get_sysctl_ctx(dev), > + SYSCTL_CHILDREN(device_get_sysctl_tree(dev)), > + OID_AUTO, "ucom", CTLFLAG_RD, > + &sc->ucom_unit, -1, "Attached ucom unit number"); > + > return (0); > Hi, Could you patch ucom_set_pnpinfo_usb() instead, so that such a sysctl is created for all ucom devices, not only umodem? Can you explain a bit more how you use this information in your scripts? --HPS