MSP430FR2433: Port 1 Interrupt problem: JMP __TI_ISR_TRAP

Hi Lucheni,

Could you run step by step and see where the code run to? I see you add to much delay in your ISR. Could you try to decrease the delay_cycles ?

> #elif GPIO_PORT_S1 == GPIO_PORT_P2

This causes the compiler to generate either the PORT1_VECTOR or the PORT2_VECTOR, but (as coded) you need both of them. 

I recommend you remove all the GPIO_PORT_S1 checks, so you get two ISRs.

By viewing other posts, I tried to use DuPont cable shortcut 5v and GND to let the board fully discharge. (launchpad unconnected with the computer)

Every time debugging after discharge the bord, it could run step by step to: __bis_SR_register(LPM3_bits + GIE);

When I press the three buttons connected to PORT 2, it still works correctly.

Just when I try to shortcut Gnd to P1.5(just like pressing a button connect to P1.5) the code run away to: JMP __TI_ISR_TRAP.

The delay_cycles in code ISR is used for key debounce, cause there is no debounce capacitor on launchpad

I tried to change the example code(version without driverlib)

#include <msp430.h>

int main(void)
{
    WDTCTL = WDTPW | WDTHOLD;               // Stop watchdog timer

    // Configure GPIO
    P1OUT &= ~BIT0;                         // Clear P1.0 output latch for a defined power-on state
    P1DIR |= BIT0;                          // Set P1.0 to output direction
    P2OUT &= ~(BIT3+BIT7);
    P2DIR |= BIT3+BIT7;

    P1OUT |= BIT3;                          // Configure P1.3 as pulled-up
    P1REN |= BIT3;                          // P1.3 pull-up register enable
    P1IES |= BIT3;                          // P1.3 Hi/Low edge
    P1IE |= BIT3;                           // P1.3 interrupt enabled
    P2OUT |= BIT3+BIT7;                     // Configure P2.3&P2.7 as pulled-up
    P2REN |= BIT3+BIT7;                     // P2.3&P2.7 pull-up register enable
    P2IES |= BIT3+BIT7;                     // P2.3&P2.7 Hi/Low edge
    P2IE |= BIT3+BIT7;                      // P2.3&P2.7 interrupt enabled

    // Disable the GPIO power-on default high-impedance mode
    // to activate previously configured port settings
    PM5CTL0 &= ~LOCKLPM5;

    P1IFG &= ~BIT3;                         // P1.3 IFG cleared
    P2IFG &= ~(BIT3+BIT7);

    while(1)
    {
        __bis_SR_register(LPM3_bits | GIE); // Enter LPM3 w/interrupt
        __no_operation();                   // For debug
        P1OUT ^= BIT0;                      // P1.0 = toggle
    }
}

// Port 1 interrupt service routine
#pragma vector=PORT1_VECTOR
__interrupt void Port_1(void)
{
    P1IFG &= ~BIT3;                         // Clear P1.3 IFG
    __bic_SR_register_on_exit(LPM3_bits);   // Exit LPM3
}

// Port 2 interrupt service routine
#pragma vector=PORT2_VECTOR
__interrupt void Port_2(void)
{
    P2IFG &= ~(BIT3+BIT7);                         // Clear P1.3 IFG
    __bic_SR_register_on_exit(LPM3_bits);   // Exit LPM3
}

This code seems run well now.

Then I tried to change the example code(version with driverlib)

#include "driverlib.h"
#include "Board.h"

void main (void)
{
    //Stop watchdog timer
    WDT_A_hold(WDT_A_BASE);

    //Set LED to output direction
    GPIO_setAsOutputPin(GPIO_PORT_P1,GPIO_PIN0 + GPIO_PIN1);

    //Set LED pins HI
    GPIO_setOutputHighOnPin(GPIO_PORT_P1,GPIO_PIN0 + GPIO_PIN1);

    //Enable S1,S2 internal resistance as pull-Up resistance
    GPIO_setAsInputPinWithPullUpResistor(GPIO_PORT_P1,GPIO_PIN5);
    GPIO_setAsInputPinWithPullUpResistor(GPIO_PORT_P2,GPIO_PIN2 + GPIO_PIN3 + GPIO_PIN7);

    //S1,S2 interrupt enabled
    GPIO_enableInterrupt(GPIO_PORT_P1,GPIO_PIN5);
    GPIO_enableInterrupt(GPIO_PORT_P2,GPIO_PIN2 + GPIO_PIN3 + GPIO_PIN7);

    //S1 Hi/Lo edge
    GPIO_selectInterruptEdge(GPIO_PORT_P1,GPIO_PIN5,GPIO_HIGH_TO_LOW_TRANSITION);
    GPIO_selectInterruptEdge(GPIO_PORT_P2,GPIO_PIN2 + GPIO_PIN3 + GPIO_PIN7,GPIO_HIGH_TO_LOW_TRANSITION);


    //S1 IFG cleared
    GPIO_clearInterrupt(GPIO_PORT_P1,GPIO_PIN5);
    GPIO_clearInterrupt(GPIO_PORT_P2,GPIO_PIN2 + GPIO_PIN3 + GPIO_PIN7);

    PMM_unlockLPM5();

    //Enter LPM3 w/interrupt
    __bis_SR_register(LPM3_bits + GIE);

    //For debugger
    __no_operation();
}

//******************************************************************************
//
//This is the PORT1_VECTOR interrupt vector service routine
//
//******************************************************************************
#pragma vector=PORT1_VECTOR
void P1_ISR (void)
{
    __delay_cycles(10000);
    GPIO_toggleOutputOnPin(GPIO_PORT_P1,GPIO_PIN0 + GPIO_PIN1);
    GPIO_clearInterrupt(GPIO_PORT_P1,GPIO_PIN5);
}
//******************************************************************************
//
//This is the PORT2_VECTOR interrupt vector service routine
//
//******************************************************************************
#pragma vector=PORT2_VECTOR
void P2_ISR (void)
{
    __delay_cycles(10000);
    if (GPIO_getInterruptStatus(GPIO_PORT_P2 , GPIO_PIN2))
    {
        GPIO_toggleOutputOnPin(GPIO_PORT_P1,GPIO_PIN0 + GPIO_PIN1);
        if (GPIO_INPUT_PIN_HIGH == GPIO_getInputPinValue(GPIO_PORT_P2 , GPIO_PIN2))
        {
            GPIO_selectInterruptEdge(GPIO_PORT_P2,GPIO_PIN2,GPIO_HIGH_TO_LOW_TRANSITION);
        }
        else
        {
            GPIO_selectInterruptEdge(GPIO_PORT_P2,GPIO_PIN2,GPIO_LOW_TO_HIGH_TRANSITION);
        }
    }

    if (GPIO_getInterruptStatus(GPIO_PORT_P2 , GPIO_PIN3))
    {
        GPIO_toggleOutputOnPin(GPIO_PORT_P1,GPIO_PIN0);
        if (GPIO_INPUT_PIN_HIGH == GPIO_getInputPinValue(GPIO_PORT_P2 , GPIO_PIN3))
        {
            GPIO_selectInterruptEdge(GPIO_PORT_P2,GPIO_PIN3,GPIO_HIGH_TO_LOW_TRANSITION);
        }
        else
        {
            GPIO_selectInterruptEdge(GPIO_PORT_P2,GPIO_PIN3,GPIO_LOW_TO_HIGH_TRANSITION);
        }
    }

    if (GPIO_getInterruptStatus(GPIO_PORT_P2 , GPIO_PIN7))
    {
        GPIO_toggleOutputOnPin(GPIO_PORT_P1,GPIO_PIN1);
        if (GPIO_INPUT_PIN_HIGH == GPIO_getInputPinValue(GPIO_PORT_P2 , GPIO_PIN7))
        {
            GPIO_selectInterruptEdge(GPIO_PORT_P2,GPIO_PIN7,GPIO_HIGH_TO_LOW_TRANSITION);
        }
        else
        {
            GPIO_selectInterruptEdge(GPIO_PORT_P2,GPIO_PIN7,GPIO_LOW_TO_HIGH_TRANSITION);
        }
    }

    GPIO_clearInterrupt(GPIO_PORT_P2,GPIO_PIN2 + GPIO_PIN3+GPIO_PIN7);
}

Some old problem occured. This situation happened last week, but it could not be repeated yesterday, so it was not sent out.

The code can be debugged step by step till:_ _bis_SR_register(LPM3_bits + GIE);

But when I press the button, it just happens.

It works right now.

Thanks for the explanation.