Re: usb/149039: commit references a PR
The following reply was made to PR usb/149039; it has been noted by GNATS. From: dfil...@freebsd.org (dfilter service) To: bug-follo...@freebsd.org Cc: Subject: Re: usb/149039: commit references a PR Date: Sat, 27 Nov 2010 19:39:35 + (UTC) Author: thompsa Date: Sat Nov 27 19:39:29 2010 New Revision: 215947 URL: http://svn.freebsd.org/changeset/base/215947 Log: MFC r213803 USB network (UHSO): - Correct network interface flags. PR: usb/149039 Submitted by:Fredrik Lindberg Modified: stable/8/sys/dev/usb/net/uhso.c Directory Properties: stable/8/sys/ (props changed) stable/8/sys/amd64/include/xen/ (props changed) stable/8/sys/cddl/contrib/opensolaris/ (props changed) stable/8/sys/contrib/dev/acpica/ (props changed) stable/8/sys/contrib/pf/ (props changed) stable/8/sys/mips/alchemy/ (props changed) stable/8/sys/mips/atheros/ (props changed) stable/8/sys/mips/cavium/ (props changed) stable/8/sys/mips/cavium/dev/ (props changed) stable/8/sys/mips/rmi/ (props changed) stable/8/sys/mips/rmi/dev/ (props changed) stable/8/sys/mips/sibyte/ (props changed) Modified: stable/8/sys/dev/usb/net/uhso.c == --- stable/8/sys/dev/usb/net/uhso.cSat Nov 27 19:38:40 2010 (r215946) +++ stable/8/sys/dev/usb/net/uhso.cSat Nov 27 19:39:29 2010 (r215947) @@ -1560,7 +1560,7 @@ uhso_attach_ifnet(struct uhso_softc *sc, ifp-if_init = uhso_if_init; ifp-if_start = uhso_if_start; ifp-if_output = uhso_if_output; - ifp-if_flags = 0; + ifp-if_flags = IFF_BROADCAST | IFF_MULTICAST | IFF_NOARP; ifp-if_softc = sc; IFQ_SET_MAXLEN(ifp-if_snd, ifqmaxlen); ifp-if_snd.ifq_drv_maxlen = ifqmaxlen; ___ svn-src-...@freebsd.org mailing list http://lists.freebsd.org/mailman/listinfo/svn-src-all To unsubscribe, send any mail to svn-src-all-unsubscr...@freebsd.org ___ freebsd-usb@freebsd.org mailing list http://lists.freebsd.org/mailman/listinfo/freebsd-usb To unsubscribe, send any mail to freebsd-usb-unsubscr...@freebsd.org
Re: USB controller error logged when resuming after suspend
On Fri, Nov 26, 2010 at 1:49 AM, Bruce Cran br...@cran.org.uk wrote: On Thu, 25 Nov 2010 20:46:42 -0600 Brandon Gooch jamesbrandongo...@gmail.com wrote: It's becoming more well known that the USB stack isn't suspend/resume safe at this point. Have you tried building your USB systems as kernel modules and unloading/loading them via /etc/rs.suspend|resume? I used to have luck doing that here, but recently that has broken as well (running HEAD). I'm not so interested in using suspend/resume as a real feature, but more as a developer to report issues so that in the future we can perhaps have it working for end-users. Apparently there's lots of infrastructure work that still needs to be done before it's going to be reliable unfortunately. Do you mean USB subsystem infrastructure, or something more far-reaching? It seems to me that it's SO close to being functional; in fact, I've taken to not loading any USB drivers at all on my notebook, which seems to be the only reliable way of using suspend/resume -- that's not to say that it's perfect, but Pretty Good(TM). Also, If we could round-up the various sysctl settings and document them in one place([1] or [2]), I imagine many users would have suspend/resume as a workable feature, at least on amd64... -Brandon [1] http://www.freebsd.org/doc/en_US.ISO8859-1/books/handbook/acpi-debug.html [2] http://www.freebsd.org/doc/en/articles/laptop/article.html or ___ freebsd-usb@freebsd.org mailing list http://lists.freebsd.org/mailman/listinfo/freebsd-usb To unsubscribe, send any mail to freebsd-usb-unsubscr...@freebsd.org
Re: usb/149675: commit references a PR
The following reply was made to PR usb/149675; it has been noted by GNATS. From: dfil...@freebsd.org (dfilter service) To: bug-follo...@freebsd.org Cc: Subject: Re: usb/149675: commit references a PR Date: Sun, 28 Nov 2010 07:28:18 + (UTC) Author: thompsa Date: Sun Nov 28 07:28:10 2010 New Revision: 215986 URL: http://svn.freebsd.org/changeset/base/215986 Log: MFC r213872 Fix forwarding of Line Register Status changes to TTY layer. PR: usb/149675 Modified: stable/8/sys/dev/usb/serial/usb_serial.c Directory Properties: stable/8/sys/ (props changed) stable/8/sys/amd64/include/xen/ (props changed) stable/8/sys/cddl/contrib/opensolaris/ (props changed) stable/8/sys/contrib/dev/acpica/ (props changed) stable/8/sys/contrib/pf/ (props changed) Modified: stable/8/sys/dev/usb/serial/usb_serial.c == --- stable/8/sys/dev/usb/serial/usb_serial.c Sun Nov 28 07:23:05 2010 (r215985) +++ stable/8/sys/dev/usb/serial/usb_serial.c Sun Nov 28 07:28:10 2010 (r215986) @@ -942,6 +942,7 @@ ucom_cfg_status_change(struct usb_proc_m uint8_t new_msr; uint8_t new_lsr; uint8_t onoff; + uint8_t lsr_delta; tp = sc-sc_tty; @@ -965,6 +966,7 @@ ucom_cfg_status_change(struct usb_proc_m return; } onoff = ((sc-sc_msr ^ new_msr) SER_DCD); + lsr_delta = (sc-sc_lsr ^ new_lsr); sc-sc_msr = new_msr; sc-sc_lsr = new_lsr; @@ -977,6 +979,30 @@ ucom_cfg_status_change(struct usb_proc_m ttydisc_modem(tp, onoff); } + + if ((lsr_delta ULSR_BI) (sc-sc_lsr ULSR_BI)) { + + DPRINTF(BREAK detected\n); + + ttydisc_rint(tp, 0, TRE_BREAK); + ttydisc_rint_done(tp); + } + + if ((lsr_delta ULSR_FE) (sc-sc_lsr ULSR_FE)) { + + DPRINTF(Frame error detected\n); + + ttydisc_rint(tp, 0, TRE_FRAMING); + ttydisc_rint_done(tp); + } + + if ((lsr_delta ULSR_PE) (sc-sc_lsr ULSR_PE)) { + + DPRINTF(Parity error detected\n); + + ttydisc_rint(tp, 0, TRE_PARITY); + ttydisc_rint_done(tp); + } } void ___ svn-src-...@freebsd.org mailing list http://lists.freebsd.org/mailman/listinfo/svn-src-all To unsubscribe, send any mail to svn-src-all-unsubscr...@freebsd.org ___ freebsd-usb@freebsd.org mailing list http://lists.freebsd.org/mailman/listinfo/freebsd-usb To unsubscribe, send any mail to freebsd-usb-unsubscr...@freebsd.org