Bug in Cypress' fw.c ?
- From: "Bill Davy" <Bill@xxxxxxxxxxxxxxx>
- Date: Wed, 29 Aug 2007 12:03:54 +0100
I am having terrible trouble explaining (what I think is) a bug in Cypress'
software to their technical support. I fear others may have struggled too.
Perhaps there are other bugs and we are all wasting time finding them. At
least, I've not found anywhere where they are discussed and newly downloaded
code is old.
Anyway, for what it is worth, I believe:
if(GotSUD) // Wait for SUDAV
{
SetupCommand(); // Implement setup command
GotSUD = FALSE; // Clear SUDAV flag
}
is wrong as if there is an interrupt after the "EP0CS |= bmHSNAK;" at the
end of SetupCommand() and the PC is quick, it can send the next USB command
resulting in GotSUD being set (true), which is promptly lost. The fix is
trivial:
if(GotSUD) // Wait for SUDAV
{
GotSUD = FALSE; // Clear SUDAV flag
SetupCommand(); // Implement setup command
}
I grant you it is not a big problem, but why is it so hard for them to
recognise it, and why is there no process for publishing such things?
I have found other possible errors in the sample code.
Rgds,
Bill
.
- Prev by Date: Re: finding sample rate
- Next by Date: [OT] Freescale English?
- Previous by thread: VMA vs LMA?
- Next by thread: [OT] Freescale English?
- Index(es):
Relevant Pages
|
