STM32 ReadOutProtection

[Phantomix X. (phantomix)]

Hallo

Hat einer von euch es schon mal geschafft, die STM ReadOutProtection vom 
Code aus zu aktivieren?
Ich habe nun schon ein Paar Stunden ins Debugging, Fehlersuche, Google 
investiert und bin zu dem Schluss gekommen, dass der Code aus der STM 
StdPeriphLib 3.4.0 einfach nicht funktioniert.

#include "stm32f10x.h"
int main(void)
{
    FLASH_ReadOutProtection(ENABLE);
    while(1)
    {
    }
}

Das mal als einfaches Beispiel. Ich kann die Funktion durchsteppen und 
das OB->RDP - Register wird korrekt auf 0x00 gesetzt. Jedoch hat dies 
keinen Effekt; selbst nach einem Power-Cycle ist der Flash noch per 
ST-Link-Utility lesbar.

(Der Vollständigkeit halber hier noch der Code aus der StdPeriphLib)

FLASH_Status FLASH_ReadOutProtection(FunctionalState NewState)
{
  FLASH_Status status = FLASH_COMPLETE;
  /* Check the parameters */
  assert_param(IS_FUNCTIONAL_STATE(NewState));
  status = FLASH_WaitForLastOperation(EraseTimeout);
  if(status == FLASH_COMPLETE)
  {
    /* Authorizes the small information block programming */
    FLASH->OPTKEYR = FLASH_KEY1;
    FLASH->OPTKEYR = FLASH_KEY2;
    FLASH->CR |= CR_OPTER_Set;
    FLASH->CR |= CR_STRT_Set;
    /* Wait for last operation to be completed */
    status = FLASH_WaitForLastOperation(EraseTimeout);
    if(status == FLASH_COMPLETE)
    {
      /* if the erase operation is completed, disable the OPTER Bit */
      FLASH->CR &= CR_OPTER_Reset;
      /* Enable the Option Bytes Programming operation */
      FLASH->CR |= CR_OPTPG_Set; 
      if(NewState != DISABLE)
      {
        OB->RDP = 0x00;
      }
      else
      {
        OB->RDP = RDP_Key;  
      }
      /* Wait for last operation to be completed */
      status = FLASH_WaitForLastOperation(EraseTimeout); 
    
      if(status != FLASH_TIMEOUT)
      {
        /* if the program operation is completed, disable the OPTPG Bit */
        FLASH->CR &= CR_OPTPG_Reset;
      }
    }
    else 
    {
      if(status != FLASH_TIMEOUT)
      {
        /* Disable the OPTER Bit */
        FLASH->CR &= CR_OPTER_Reset;
      }
    }
  }
  /* Return the protection operation Status */
  return status;       
}

Hat irgendwer eine Idee?

[Frank A. (_frank)]

Hallo phantomix,

finde ich ein interessantes Thema.
Hast du dir mal die Beispiele zur STD-Lib angesehen?

Wenn ich heute oder morgen mal Zeit habe, probiere ich das mal aus.
Wäre auch für mich ganz interessant.

Gruß Frank

[Phantomix X. (phantomix)]

Hallo!

Die WriteProtection interessiert mich daran eigentlich weniger, obwohl 
sie später evtl auch noch relevant wird.

Ich werd jetzt mal probieren, den ganzen Kram in den RAM zu linken und 
von dort aus zu starten. Wenn das nicht funktioniert, bin ich auch am 
Ende meines Lateins

[Mike R. (thesealion)]

Du musst vorher den Flash freigeben, damit die einstellungen geändert 
werden können.

Bei mir sieht die Routine hierfür so aus, und funktioniert einwandfrei:

/******************************************************************************/
/** @brief  
  * 
  * 
  * @param  None
  * @retval None
  */
/******************************************************************************/
void flashDriver_checkReadout(void) {
   
   if (FLASH_GetReadOutProtectionStatus() != SET) {
      #ifndef DEBUG      
         FLASH_Unlock();

         if (FLASH_ReadOutProtection(ENABLE) != FLASH_COMPLETE) {
            FaultSystem_queueFault(FaultSystem_Flag_Warning, FaultSystem_Flash, FAULT_SYSTEM_FLASH__WRITE, "set protection");
         
            flashInterface_secureFault();
         }
         NVIC_SystemReset();
      #endif   
   }
}

[Phantomix X. (phantomix)]

Mike S. schrieb:
> Du musst vorher den Flash freigeben, damit die einstellungen geändert
> werden können.

Du meinst FLASH_Unlock()

In der Funktion wird die Key-Sequenz geschrieben. Das passiert in der 
FLASH_ReadOutProtection() aber sowieso.


Ich habe deinen Code mal 1:1 ausprobiert und er funktioniert.

Keine Ahnung was da bei mir schief läuft. Ich nehm das jetzt so wie du 
es gemacht hast und portiere es bis es in unsere Software passt. DANKE! 
:-)


Evtl liegts daran, dass ich nicht sofort danach einen Reset ausgelöst 
habe. Den hatte ich per Strom-Aus-Strom-An danach allerdings manuell 
gemacht...

[Thomas W. (diddl)]

Eigentlich erstaunlich, dass es ein Code im STM32 ist. Und nicht ein ST 
Link utility, mit dem man den Schutz setzen kann. So wie bei den AVR 
Fuses.

[Phantomix X. (phantomix)]

Es lag tatsächlich an FLASH_Unlock() und an 
"Wer-Lesen-kann-ist-klar-im-Vorteil":

FLASH_Unlock():

FLASH->KEYR = FLASH_KEY1;
FLASH->KEYR = FLASH_KEY2;

ReadOutProtection(NewState):

FLASH->OPTKEYR = FLASH_KEY1;
FLASH->OPTKEYR = FLASH_KEY2;

[Phantomix X. (phantomix)]

Hallo!

Thomas Winkler schrieb:
> Eigentlich erstaunlich, dass es ein Code im STM32 ist. Und nicht ein ST
> Link utility, mit dem man den Schutz setzen kann. So wie bei den AVR
> Fuses.

Das geht auch. Ist aber bei uns gerade nicht gewünscht, und zwar aus 
zwei Gründen:

- Der Kunde hat bereits ein paar Vorserienmodelle, die wir per Update 
auch dicht machen wollen (bequemer als sie zurückzurufen)

- Der Aufwand in der Produktion veringert sich, wenn der Schutz 
automatisch von der Software gesetzt wird