usb/140242: dev/usb/controller/ehci_ix4xx.c shortcomings

Hans Petter Selasky hselasky at c2i.net
Tue Nov 3 11:20:04 UTC 2009


The following reply was made to PR usb/140242; it has been noted by GNATS.

From: Hans Petter Selasky <hselasky at c2i.net>
To: freebsd-usb at freebsd.org
Cc: Sebastian Huber <sebastian.huber at embedded-brains.de>,
 freebsd-gnats-submit at freebsd.org
Subject: Re: usb/140242: dev/usb/controller/ehci_ix4xx.c shortcomings
Date: Tue, 3 Nov 2009 11:59:33 +0100

 On Tuesday 03 November 2009 10:52:52 Sebastian Huber wrote:
 > >Number:         140242
 > >Category:       usb
 > >Synopsis:       dev/usb/controller/ehci_ix4xx.c shortcomings
 > >Confidential:   no
 > >Severity:       non-critical
 > >Priority:       low
 > >Responsible:    freebsd-usb
 > >State:          open
 > >Quarter:
 > >Keywords:
 > >Date-Required:
 > >Class:          sw-bug
 > >Submitter-Id:   current-users
 > >Arrival-Date:   Tue Nov 03 10:00:12 UTC 2009
 > >Closed-Date:
 > >Last-Modified:
 > >Originator:     Sebastian Huber
 > >Release:        9-current
 > >Organization:
 >
 > embedded brains GmbH
 >
 > >Environment:
 > >Description:
 >
 > In ehci_ixp_attach() setting of sc->sc_bus.usbrev is superfluous since this
 > is also set in ehci_init().
 >
 > Calling ehci_reset() without a valid sc->sc_offs value makes no sense.
 >
 > >How-To-Repeat:
 > >
 > >Fix:
 >
 > Index: ehci_ixp4xx.c
 > ===================================================================
 > --- ehci_ixp4xx.c       (revision 198848)
 > +++ ehci_ixp4xx.c       (working copy)
 > @@ -157,8 +157,6 @@
 >                 return (ENOMEM);
 >         }
 >
 > -       sc->sc_bus.usbrev = USB_REV_2_0;
 > -
 >         /* NB: hints fix the memory location and irq */
 >
 >         rid = 0;
 > @@ -230,7 +228,6 @@
 >
 >                      | EHCI_SCFLG_BIGEMMIO
 >                      | EHCI_SCFLG_NORESTERM
 >
 >                      ;
 > -       (void) ehci_reset(sc);
 >
 >         err = ehci_init(sc);
 >         if (!err) {
 >
 > >Release-Note:
 > >Audit-Trail:
 > >Unformatted:
 >
 
 Committed to USB P4:
 
 http://p4web.freebsd.org/chv.cgi?CH=170109
 
 --HPS
 


More information about the freebsd-usb mailing list