From owner-p4-projects@FreeBSD.ORG Thu Jul 24 19:59:59 2008 Return-Path: Delivered-To: p4-projects@freebsd.org Received: by hub.freebsd.org (Postfix, from userid 32767) id 65CA91065677; Thu, 24 Jul 2008 19:59:59 +0000 (UTC) Delivered-To: perforce@FreeBSD.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 2B4EE1065674 for ; Thu, 24 Jul 2008 19:59:59 +0000 (UTC) (envelope-from hselasky@FreeBSD.org) Received: from repoman.freebsd.org (repoman.freebsd.org [IPv6:2001:4f8:fff6::29]) by mx1.freebsd.org (Postfix) with ESMTP id 22FDD8FC0A for ; Thu, 24 Jul 2008 19:59:59 +0000 (UTC) (envelope-from hselasky@FreeBSD.org) Received: from repoman.freebsd.org (localhost [127.0.0.1]) by repoman.freebsd.org (8.14.2/8.14.2) with ESMTP id m6OJxx31008870 for ; Thu, 24 Jul 2008 19:59:59 GMT (envelope-from hselasky@FreeBSD.org) Received: (from perforce@localhost) by repoman.freebsd.org (8.14.2/8.14.1/Submit) id m6OJxxr1008868 for perforce@freebsd.org; Thu, 24 Jul 2008 19:59:59 GMT (envelope-from hselasky@FreeBSD.org) Date: Thu, 24 Jul 2008 19:59:59 GMT Message-Id: <200807241959.m6OJxxr1008868@repoman.freebsd.org> X-Authentication-Warning: repoman.freebsd.org: perforce set sender to hselasky@FreeBSD.org using -f From: Hans Petter Selasky To: Perforce Change Reviews Cc: Subject: PERFORCE change 145840 for review X-BeenThere: p4-projects@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: p4 projects tree changes List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 24 Jul 2008 19:59:59 -0000 http://perforce.freebsd.org/chv.cgi?CH=145840 Change 145840 by hselasky@hselasky_laptop001 on 2008/07/24 19:59:25 Revise the need for sync to increase performance. Affected files ... .. //depot/projects/usb/src/sys/dev/usb2/controller/at91dci.c#5 edit .. //depot/projects/usb/src/sys/dev/usb2/controller/uss820dci.c#4 edit Differences ... ==== //depot/projects/usb/src/sys/dev/usb2/controller/at91dci.c#5 (text+ko) ==== @@ -910,7 +910,6 @@ struct at91dci_softc *sc; struct at91dci_td *td; uint32_t x; - uint8_t need_sync; uint8_t ep_no; DPRINTF(8, "addr=%d endpt=%d sumlen=%d speed=%d\n", @@ -953,18 +952,13 @@ if (x != xfer->nframes) { if (xfer->endpoint & UE_DIR_IN) { temp.func = &at91dci_data_tx; - need_sync = 1; } else { temp.func = &at91dci_data_rx; - need_sync = 0; } /* setup "pc" pointer */ temp.pc = xfer->frbuffers + x; - } else { - need_sync = 0; } - while (x != xfer->nframes) { /* DATA0 / DATA1 message */ @@ -1002,20 +996,11 @@ /* always setup a valid "pc" pointer for status and sync */ temp.pc = xfer->frbuffers + 0; - /* check if we need to sync */ - if (need_sync && !xfer->flags_int.isochronous_xfr) { - - /* we need a SYNC point after TX */ - temp.func = &at91dci_data_tx_sync; - temp.len = 0; - temp.short_pkt = 0; - - at91dci_setup_standard_chain_sub(&temp); - } /* check if we should append a status stage */ if (xfer->flags_int.control_xfr && !xfer->flags_int.control_act) { + uint8_t need_sync; /* * Send a DATA1 message and invert the current ==== //depot/projects/usb/src/sys/dev/usb2/controller/uss820dci.c#4 (text+ko) ==== @@ -857,7 +857,6 @@ struct uss820dci_softc *sc; struct uss820dci_td *td; uint32_t x; - uint8_t need_sync; uint8_t ep_no; DPRINTF(8, "addr=%d endpt=%d sumlen=%d speed=%d\n", @@ -900,18 +899,13 @@ if (x != xfer->nframes) { if (xfer->endpoint & UE_DIR_IN) { temp.func = &uss820dci_data_tx; - need_sync = 1; } else { temp.func = &uss820dci_data_rx; - need_sync = 0; } /* setup "pc" pointer */ temp.pc = xfer->frbuffers + x; - } else { - need_sync = 0; } - while (x != xfer->nframes) { /* DATA0 / DATA1 message */ @@ -949,20 +943,11 @@ /* always setup a valid "pc" pointer for status and sync */ temp.pc = xfer->frbuffers + 0; - /* check if we need to sync */ - if (need_sync && !xfer->flags_int.isochronous_xfr) { - - /* we need a SYNC point after TX */ - temp.func = &uss820dci_data_tx_sync; - temp.len = 0; - temp.short_pkt = 0; - - uss820dci_setup_standard_chain_sub(&temp); - } /* check if we should append a status stage */ if (xfer->flags_int.control_xfr && !xfer->flags_int.control_act) { + uint8_t need_sync; /* * Send a DATA1 message and invert the current