Error in flash.c ?

lil-vince · ‎2009-02-20

Posted on February 21, 2009 at 05:28

Error in flash.c ?

lil-vince · ‎2011-05-17

Posted on May 17, 2011 at 13:03

Hi all,

I think there is an error in function:

void FLASH_ClearFlag(u16 FLASH_FLAG)

This function clear the BSY Flag in the Flash Status Register whereas this Flag is readonly...

And in Flash program example:

Code:

/* Clear All pending flags */ FLASH_ClearFlag(FLASH_FLAG_BSY | FLASH_FLAG_EOP | FLASH_FLAG_PGERR | FLASH_FLAG_WRPRTERR);

[ This message was edited by: lil-vince on 18-02-2009 12:07 ]

lil-vince · ‎2011-05-17

Posted on May 17, 2011 at 13:03

Hi again,

Another thing:

In a lot of functions like FLASH_ProgramWord there is:

Code:

/* Wait for last operation to be completed */ status = FLASH_WaitForLastOperation(TimeoutValue); if(status != FLASH_BUSY) { /* Change a bit in the Flash control register */ }

and as this is the FLASH_WaitForLastOperation function:

Code:

FLASH_Status FLASH_WaitForLastOperation(u32 Timeout) { FLASH_Status status = FLASH_COMPLETE; /* Check for the Flash Status */ status = FLASH_GetStatus(); /* Wait for a Flash operation to complete or a TIMEOUT to occur */ while((status == FLASH_BUSY) && (Timeout != 0x00)) { delay(); status = FLASH_GetStatus(); Timeout--; } if(Timeout == 0x00 ) { status = FLASH_TIMEOUT; } /* Return the operation status */ return status; }

For me itÂ´s impossible that status = FLASH_BUSY when returning from FLASH_WaitForLastOperation function (I donÂ´t say that the busy bit canÂ´t be set, but status canÂ´t be equal to FLASH_BUSY)...

Am I wrong or is it a test ''to make this beautiful''? 😉

And after this remarks,I have 2 simple questions:

-Can the busy bit (from the Flash status Register) be set for another reason than a programming or erasing operation? so, once itÂ´s no more set, it canÂ´t be set again without a programming or erasing operation?

-Is it possible that the erase or programming operation fail?

I ask that as there is a ''Timeout control'' in the FLASH_WaitForLastOperation function and how I write my program without ST librairies, I ask me if I will make this control or not...

Thanks

[ This message was edited by: lil-vince on 18-02-2009 16:24 ]

🙂

[ This message was edited by: lil-vince on 18-02-2009 16:34 ]

lil-vince · ‎2011-05-17

Posted on May 17, 2011 at 13:03

Up 🙂

I would like especially an answer about the timeout: It is necessary (is there a real risk of ''no-end'' flash operation)?

Thanks,

Vince