Discussion :

[Rich31]

Bonjour,

Tout d'abord, petite précision, je développe avec Eclipse dans une VM avec Linux.

Je viens d'utiliser un utilitaire pour transformer une dbc en un .h. Dans ce .h je me retrouve avec des structures que j'ai quelques difficultés à manipuler:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
 
/**
 * Name:        HPCM_Status_2
 * Comment:     <<empty>>
 */
static struct DBC_HPC_BB_V3_MSG_HPCM_Status_2 {
	const uint32_t ID;
	const uint8_t IDE;
	const uint8_t DLC;
	 const struct {
		signal_positioned ExternalTemperature;
	} getSig;
	 struct __attribute__((packed)) DBC_HPC_BB_V3_MSG_HPCM_Status_2_getSigVal {
		uint16_t ExternalTemperature	: 16;
	} *getSigVal;
} DBC_HPC_BB_V3_MSG_HPCM_Status_2 __attribute__((unused)) = {
	.ID = 0x18FEF500,
	.IDE = CAN_ID_EXT,
	.DLC = 8,
	.getSig  = {
		.ExternalTemperature = {
			.position = 24,
			.attributes = &DBC_HPC_BB_V3_SIG_ExternalTemperature
		}
	},
	.getSigVal = 0
};

Je souhaiterai changer la valeur de ExternalTemperature dans différentes fonctions mais tous mes essais echouent. Voici une de mes fonction:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
 
void CANDATATYPES_InitHPCMStatus2Struct (struct DBC_HPC_BB_V3_MSG_HPCM_Status_2_getSigVal* HPCMStatus2Struct)
{
	HPCMStatus2Struct->ExternalTemperature	= 5;
}

Et voici le dernier essai que j'ai fait:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
CANDATATYPES_InitHPCMStatus1Struct (&DBC_HPC_BB_V3_MSG_HPCM_Status_2.getSigVal);

Lorsque j’exécute la fonction CANDATATYPES_InitHPCMStatus2Struct, HPCMStatus2Struct->ExternalTemperature prends bien la valeur 5, mais au retour, c'est DBC_HPC_BB_V3_MSG_HPCM_Status_2.getSigVal qui a la valeur 5 et non pas DBC_HPC_BB_V3_MSG_HPCM_Status_2.getSigVal->ExternalTemperature.

Je m'arrache les cheveux depuis plusieurs heures maintenant, je ne sais pas par quel bout le prendre et comment le tourner pour y arriver.

Merci d'avance pour votre aide.

[exe2bin]

Salut , as-tu essayé :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
 
void CANDATATYPES_InitHPCMStatus2Struct (struct DBC_HPC_BB_V3_MSG_HPCM_Status_2_getSigVal* HPCMStatus2Struct)
{
	HPCMStatus2Struct->getSigVal.ExternalTemperature	= 5;
}

?

[Rich31]

Ca ne peu pas passer car si je comprends bien (j'ai fait l'essai quand même, Eclipse me le refuse a l’écriture et ca plante a la compil), dans la déclaration de la structure de base DBC_HPC_BB_V3_MSG_HPCM_Status_2, *getSigVal est un pointeur de la structure DBC_HPC_BB_V3_MSG_HPCM_Status_2_getSigValm mais je ne sais pas comment y accéder.

Ce que j'ai du mal a comprendre aussi, c'est le .getSigVal = 0!!

Dites moi si je me trompe, mais la déclaration de la structure et son initialisation est bien faite par la partie en gras ci dessous:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
static struct DBC_HPC_BB_V3_MSG_HPCM_Status_2 {
	const uint32_t ID;
	const uint8_t IDE;
	const uint8_t DLC;
	 const struct {
		signal_positioned ExternalTemperature;
	} getSig;
	 struct __attribute__((packed)) DBC_HPC_BB_V3_MSG_HPCM_Status_2_getSigVal {
		uint16_t ExternalTemperature	: 16;
	} *getSigVal;
} DBC_HPC_BB_V3_MSG_HPCM_Status_2 __attribute__((unused)) = {
	.ID = 0x18FEF500,
	.IDE = CAN_ID_EXT,
	.DLC = 8,
	.getSig  = {
		.ExternalTemperature = {
			.position = 24,
			.attributes = &DBC_HPC_BB_V3_SIG_ExternalTemperature
		}
	},
	.getSigVal = 0
};

[dalfab]

Bonjour,

Ligne 15 getSigVal est indiqué comme pointeur. Il ne faut donc surtout pas de & dans l'appel
CANDATATYPES_InitHPCMStatus1Struct (&DBC_HPC_BB_V3_MSG_HPCM_Status_2.getSigVal);.
Le compilateur doit indiquer des warnings sur cette ligne, il suffit de les lire.

[Rich31]

En effet, il fait un warning mais lorsque j’enlève le &, je n’ai plus de warning mais il plante carrément à l’execution de la commande qui écrit la température.

[dalfab]

C'est normal qu'il plante, c'est un pointeur initialisé à 0 (ligne 21.)
Un pointeur ça doit pointer quelque part, si tu n'as pas besoin de pointeur ôte l'étoile ligne 15 et remet le &.

[Rich31]

C'est normal qu'il plante, c'est un pointeur initialisé à 0 (ligne 21.)
Un pointeur ça doit pointer quelque part, si tu n'as pas besoin de pointeur ôte l'étoile ligne 15 et remet le &.

Je vais essayé en rentrant, merci.
À la base je ne voulais pas trop toucher au fichier qui contient les déclarations car il est généré par un petit programme à partir d’une dbc, et je vais devoir le modifier à chaque fois que je le régénèrerais. Mais si c’est la seule solution...

Je te tiens au courant quand j’ai fait le test.

[Rich31]

C'est normal qu'il plante, c'est un pointeur initialisé à 0 (ligne 21.)
Un pointeur ça doit pointer quelque part, si tu n'as pas besoin de pointeur ôte l'étoile ligne 15 et remet le &.

Alors, je viens de faire le test en modifiant la declaration comme suit:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
 
static struct DBC_HPC_BB_V3_MSG_HPCM_Status_1 {
	const uint32_t ID;
	const uint8_t IDE;
	const uint8_t DLC;
	 const struct {
		signal_positioned ParkBrakeStatus;
	} getSig;
	 struct __attribute__((packed)) DBC_HPC_BB_V3_MSG_HPCM_Status_1_getSigVal {
		uint8_t ParkBrakeStatus	: 2;
	} getSigVal;
} DBC_HPC_BB_V3_MSG_HPCM_Status_1 __attribute__((unused)) = {
	.ID = 0x18FEF100,
	.IDE = CAN_ID_EXT,
	.DLC = 1,
	.getSig  = {
		.ParkBrakeStatus = {
			.position = 2,
			.attributes = &DBC_HPC_BB_V3_SIG_ParkBrakeStatus
		}
	},
	.getSigVal = {.ParkBrakeStatus = 0}

et dans ce cas ça fonctionne. Merci!!

Par contre ceci m'a obligé a modifier mon .h généré, et je dois donc faire cette modification pour une 20ene de structures qui sont plus lourdes a modifier. Certaines ont jusqu’à 10 paramètres. J'aimerais trouver la solution sans être obligé de modifier la declaration, mais je ne sais pas comment m'y prendre...