Fonction Readconfig et check CRC ?

nherreyre

Hello,

En relisant le code de la fonction readConfig, je ne comprend pas comment le crc calculé lors de la lecture de l'EEPROM est vérifié :


bool readConfig (bool clear_on_error) 
{	
	// For whole size of config structure
	for (uint16_t i = 0; i < sizeof(_Config); ++i) {
		// read data
		data = EEPROM.read(i);
			
		// calc CRC
		crc = crc16Update(crc, data);
	}
	
	// CRC Error ?
	if (crc != 0) {    <-----

ne faudrait il pas plutôt tester dans le IF la valeur de la variable "crc" avec celle de "config.crc" ?

je suis peut être passé a coté de quelque chose vu que suis nul en calcul de CRC ;)

a+

Nicolas.

mjeanne

On ne teste pas ici le crc de chaque variable de config, mais le crc de toute la structure.
Mes cours sur le CRC sont loin, mais ce qui est fait ici, c'est un calcul de crc sur chaque valeur de la config (et non sur l'objet "config"), qui contient elle-même une "sauvegarde" du crc. On fait donc un calcul de crc sur l'ensemble "config.* + config.crc", donc le résultat du modulo devrait être 0. Si le crc déjà calculé lors de la sauvegarde n'était pas stocké dans l'objet config, ça ne fonctionnerait pas pareil et on ferait une comparaison config.crc et crc.

Par contre, l'extrait de code à été simplifié ? Parce que je n'y vois pas l'initialisation de crc avant la boucle.

Sinon, pour simplifier la compréhension, le code dans la boucle aurait pu resembler à ça:
ancienCRC = crc;
crc = crc16Update(ancienCRC, data);