From: David Brownell <david-b@pacbell.net>
To: linux-usb-devel@lists.sourceforge.net,
Andrew Morton <akpm@linux-foundation.org>
Cc: Rodolfo Giometti <giometti@enneenne.com>,
linux-kernel@vger.kernel.org, Yang-r58472 <LeoLi@freescale.com>,
linux-arm-kernel@lists.arm.linux.org.uk
Subject: Re: [linux-usb-devel] [PATCH] PXA27x UDC driver.
Date: Sat, 30 Jun 2007 07:28:48 -0700 [thread overview]
Message-ID: <200706300728.48411.david-b@pacbell.net> (raw)
In-Reply-To: <20070628142949.02ca9eab.akpm@linux-foundation.org>
On Thursday 28 June 2007, Andrew Morton wrote:
> > +#undef DISABLE_TEST_MODE
>
> enabling DISABLE_TEST_MODE seens to enable test mode. Confused.
Blame it on Intel. ISTR that early pxa2[156]x silicon had
something called "test mode". And a boatload of errata, with
a common thread in workarounds: using the "test mode" helped
many things work better, although it was neither necessary nor
sufficient. If one were to #define DISABLE_TEST_MODE, software
could experiment with other workarounds ... and maybe be able
to get the documented "double buffering" feature to work.
Later silicon stopped documenting "test mode" as such, effectively
making certain workarounds become the standard way to use the chip.
Which doesn't explain why pxa270 code has pxa2[156]x devel hooks,
but explains why pxa2[156]x code wants the hardware "test mode"
to be enabled ... except during developer experiments.
- Dave
next prev parent reply other threads:[~2007-06-30 14:29 UTC|newest]
Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-06-28 10:36 [PATCH] PXA27x UDC driver Rodolfo Giometti
2007-06-28 11:15 ` Li Yang-r58472
2007-06-28 14:12 ` Rodolfo Giometti
2007-06-28 21:29 ` Andrew Morton
2007-06-28 22:44 ` David Brownell
2007-06-30 14:28 ` David Brownell [this message]
2007-07-10 15:49 ` Rodolfo Giometti
2007-07-10 16:16 ` Lothar Wassmann
2007-06-28 21:53 ` David Brownell
2007-06-29 8:58 ` Rodolfo Giometti
2007-06-29 9:33 ` [linux-usb-devel] " Dmitry Krivoschekov
2007-06-30 14:25 ` David Brownell
2007-06-29 9:03 ` [linux-usb-devel] " Dmitry Krivoschekov
2007-06-30 13:46 ` David Brownell
2007-07-02 11:42 ` Dmitry Krivoschekov
2007-07-09 13:51 ` Rodolfo Giometti
2007-07-10 14:50 ` Vernon Sauder
2007-07-10 18:16 ` David Brownell
2007-06-29 17:04 ` Andy Isaacson
2007-07-01 14:55 ` Lennert Buytenhek
2007-07-01 15:00 ` Russell King - ARM Linux
2007-07-01 17:49 ` David Brownell
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=200706300728.48411.david-b@pacbell.net \
--to=david-b@pacbell.net \
--cc=LeoLi@freescale.com \
--cc=akpm@linux-foundation.org \
--cc=giometti@enneenne.com \
--cc=linux-arm-kernel@lists.arm.linux.org.uk \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-usb-devel@lists.sourceforge.net \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox