Can't get the pwm code working

Hi there,

this is giving me headaches quite a while, I'm trying to get the breathing LED (http://osx-launchpad.blogspot.com/2010/11/breathing-led-effect-with-launchpad.html) running with two leds. I'm aiming to get a simple left to right (red to green led) fading effect.

I tried a couple of things now (e.g. Timer A + WDT) but nothing seems to work, I found the post:

http://e2e.ti.com/support/microcontrollers/msp43016-bit_ultra-low_power_mcus/f/166/t/60128.aspx#215108

which looks promising to me, I changed the code to my needs but still can't get it working :-(

the red led is flashing continuously (probably at 20Hz) and the green led is fading out then stays full on for 10 seconds or so...

do I have a timing/performance problem or what's going wrong here?

my code:

#include <msp430g2231.h>
#include <signal.h> //add for interrupt

const unsigned char curve[] = {
  1,     1,     1,     1,     1,     1,     1,     1,
  1,     1,     1,     1,     1,     1,     1,     1,
  1,     1,     1,     2,     2,     2,     2,     2,
  2,     2,     3,     3,     3,     3,     3,     3,
  4,     4,     4,     4,     4,     5,     5,     5,
  5,     6,     6,     6,     6,     7,     7,     7,
  8,     8,     8,     8,     9,     9,     9,    10,
  10,    10,    11,    11,    11,    12,    12,    13,
  13,    13,    14,    14,    15,    15,    15,    16,
  16,    17,    17,    18,    18,    18,    19,    19,
  20,    20,    21,    21,    22,    22,    23,    23,
  24,    24,    25,    25,    26,    26,    27,    27,
  28,    29,    29,    30,    30,    31,    31,    32,
  33,    33,    34,    34,    35,    36,    36,    37,
  38,    38,    39,    39,    40,    41,    41,    42,
  43,    43,    44,    45,    46,    46,    47,    48,
  48,    49,    50,    50,    51,    52,    53,    53,
  54,    55,    56,    56,    57,    58,    59,    59,
  60,    61,    62,    62,    63,    64,    65,    66,
  66,    67,    68,    69,    70,    70,    71,    72,
  73,    74,    75,    75,    76,    77,    78,    79,
  80,    80,    81,    82,    83,    84,    85,    86,
  87,    87,    88,    89,    90,    91,    92,    93,
  94,    95,    95,    96,    97,    98,    99,   100,
  101,   102,   103,   104,   105,   106,   106,   107,
  108,   109,   110,   111,   112,   113,   114,   115,
  116,   117,   118,   119,   120,   121,   122,   122,
  123,   124,   125,   126,   127,   128,   129,   130,
  131,   132,   133,   134,   135,   136,   137,   138,
  139,   140,   141,   142,   143,   144,   145,   146,
  147,   148,   149,   150,   151,   152,   153,   154,
  155,   156
};
unsigned int period = 625;
unsigned char redflg = 0; //flag to toggle LED
unsigned char greenflg=0;
unsigned int red_on = 1;  //duty cycle time (counts)
unsigned int green_on = 156;  //duty cycle time (counts)
unsigned int red_index =1;
unsigned int green_index =500;

void main(void)
{
  WDTCTL = WDTPW + WDTHOLD;                        // Stop watchdog timer
 
  //setup MCLK
  DCOCTL= 0;
  BCSCTL1= CALBC1_1MHZ;
  DCOCTL= CALDCO_1MHZ; //set Clock to 1 MHz
  BCSCTL2 &= ~(DIVS_3);  //unset Clock divider
  //BCSCTL2 |= DIVS_3; //set Clock Divider MCLK/8 = 125kHz
 
  TACTL = TASSEL_2 | MC_2 | ID_3;                         // TA clock = SMCLK, mode control = continuous and set Timer divider /8 --> MCLK/8 = 125kHz
  TACCTL0 = OUTMOD_4 + CCIE;                       // output mode = toggle, interrupt enabled The output is toggled when the timer counts to the TACCRx value. The output period is double the timer period.
  TACCTL1 = OUTMOD_1 + CCIE;                       // output mode = set, interrupt enabled The output is set when the timer counts to the TACCRx value. It remains set until a reset of the timer, or until another output mode is selected and affects the output.
  P1SEL =  BIT0 + BIT6;                            //  P1.0 to TA0.0, P1.6 to TA0.1
  P1DIR =  BIT0 + BIT6;                            // P1.0 and P1.6 to output direction
 
  //setup Timer A
  TACCR0 = 1;
  TACCR1 = 1;
 
  __bis_SR_register(CPUOFF | GIE); //go to sleep with interrupts enabled
}//end main


// TimerA CCR0 interrupt service routine higher priority than CCR1
#pragma vector=TIMERA0_VECTOR
__interrupt void timer_A0(void)
{
  if (redflg == 0)
  {
    TACCR0 = TAR + red_on;  //actual Timer Counter (TAR) + red_on time the LED output stays high
    redflg = 1;
  }
  else
  {
    TACCR0 = TAR + (period - red_on); // Period 625 - duty cyle if TAR + (period - red_on) is higher than TACCR0 mx (OxFFFFh) TACCR0 rolls over automatically.
    
    if (red_index < 500) {
      red_on = curve[red_index++ >> 1];
    } else if (red_index < 1000) {
      red_on = curve[(999 - red_index++) >> 1];
    } else {
      red_index = 0;
      red_on=1;
    }
    redflg = 0;
  }
}
// TimerA CCR1 interrupt service routine
#pragma vector=TIMERA1_VECTOR
__interrupt void timer_A1(void)
{
  if (greenflg == 0)
  {
    TACCTL1 = OUTMOD_5 + CCIE;                     // output mode = reset, interrupt enabled The output is reset when the timer counts to the TACCRx value. It remains reset until another output mode is selected and affects the output.
    TACCR1 = TAR + green_on;
    greenflg = 1;
  }
  else
  {
    TACCTL1 = OUTMOD_1 + CCIE;                     // output mode = set, interrupt enabled
    TACCR1 = TAR + (period - green_on);
    
    if (green_index < 500) {
      green_on = curve[green_index++ >> 1];
    } else if (green_index < 1000) {
      green_on = curve[(999 - green_index++) >> 1];
    } else {
      green_index = 0;
      green_on=1;
    }
    greenflg = 0;
  }
}