Code von IAR zu CCS portieren

[Master-Jimmy (Gast)]

Ich würde gerne eine bestehende Firmware portieren.

Im bestehenden Code hat es einen Header mit einer Reihe von "Defines" im 
folgenden Stil:

#define O_LED_1 (P3OUT_bit.P3OUT_0)

Diese Ausgänge werden dann im Weiteren folgendermassen benutzt:

if(is_shutter_closed()) {
  O_LED_1 = 1;
else
  O_LED_1 = 0;

Das Problem ist nun, dass dieser bitweise Zugriff auf Port-Pins 
IAR-spezifisch ist. Ich könnte also nun entweder den ganzen Code 
durchgehen und umschreiben:

#define LED1_ON  (P3OUT |= BIT0)
#define LED1_OFF (P3OUT &= ~(BIT0))

if(is_shutter_closed()) {
  LED1_ON
else
  LED1_OFF

Oder aber ich "rüste" den bitweisen Zugriff nach. Dafür wollte ich etwas 
in der folgenden Art und Weise definieren:

volatile union {
  struct {
    unsigned char P1OUT_0 : 1;
    unsigned char P1OUT_1 : 1;
    unsigned char P1OUT_2 : 1;
    unsigned char P1OUT_3 : 1;
    unsigned char P1OUT_4 : 1;
    unsigned char P1OUT_5 : 1;
    unsigned char P1OUT_6 : 1;
    unsigned char P1OUT_7 : 1;
  };
  unsigned char P1OUT;
} P1OUT_bit;

Dummerweise erschaffe ich so aber ein neues Bitfeld namens "P1OUT" in 
Zeile 12. Wie kann ich das mit dem bestehenden P1OUT verknüpfen?

[Jim M. (turboj)]

Vorsicht: Der Bitfield Zugriff ist read-modify-write und damit nicht 
mehr atomar. Das knallt wenn es P3 Zugriffe im Interrupt gibt.

Daher ist die Macro Methode vorzuziehen.

[900ss (900ss)]

So sollte es gehen.
Beitrag "Re: Direkter Portzugriff beim Arduino"

Hier nochmal das gleiche von Peter D.
Beitrag "Re: Einfachere Schreibweise gesucht"

Das Problem atomar oder nicht hat auch schon vorher existiert, je nach 
Architektur.
Aber sollte man beachten.

[Master-Jimmy (Gast)]

Vielen Dank für den Hinweis bzgl. "atomar". Muss ich mir noch anschauen, 
ob das in meinem Fall nun ein Problem ist oder nicht.

Ich konnte mittlerweile testweise eine LED auf einem MSP-EXP430FR5739 
blinken lassen. Das ist evtl. für den nächsten ganz informativ, wenn er 
auf diesen Thread stossen sollte:

#include <msp430.h>

#define FOREVER (1)

typedef union {
  struct {
    volatile unsigned char _0 : 1;
    volatile unsigned char _1 : 1;
    volatile unsigned char _2 : 1;
    volatile unsigned char _3 : 1;
    volatile unsigned char _4 : 1;
    volatile unsigned char _5 : 1;
    volatile unsigned char _6 : 1;
    volatile unsigned char _7 : 1;
  } pin;
  volatile unsigned char PORT;
} port_t;

volatile port_t *port_j = (volatile port_t*)(0x0320 + 0x02);

#define O_LED_1 (port_j->pin._0)

void main(void) {
  WDTCTL = WDTPW + WDTHOLD;
  PM5CTL0 &= ~LOCKLPM5;

  PJOUT = 0;
  PJDIR |= BIT0;

  while(FOREVER) {
    volatile unsigned short i = 40000;

    if(PJOUT & BIT0) {
      O_LED_1 = 0;
    } else {
      O_LED_1 = 1;
    }

    do {
      --i;
    } while(i != 0);
  }
}

[Master-Jimmy (Gast)]

Master-Jimmy schrieb:
> volatile port_t *port_j = (volatile port_t*)(0x0320 + 0x02);

Die "Magic Numbers" entnimmt man dem Datenblatt - im Falle des 
MSP430FR5739 ist dies konkret die Tab. 6-24:
https://www.ti.com/lit/gpn/msp430fr5739