• Resolved

I2C communication does not appear to be working correctly with Energia and CCS. How do I fix it?

I've just done a fresh install of CCS 6.1.3.00034.

I've also just installed Energia 0101E0017.

I am attempting to use code from an Arduino project to detect I2C devices.  The code is quite simple:

void findAddress()
{
  for (int a = fromAddress; a < toAddress; a++)
  {
    Wire.beginTransmission(a);
    Wire.write(0);
    int endStatus = Wire.endTransmission();

    if (endStatus == 0)
    {
      Serial.print("Found device at: ");
      printHex(a, true);
    }
  }
}

In the Arduino environmnet, it would output the address of all I2C devices that were connected.  However, this basically says that every address is a device..

Anyway, I believe there is a bug fix somewhere with the Energia project.  It is quite frustrating that we have to go find and install code hacks in order to get Energia to run and play nicely with CCS...

Perhaps there could be some encouragement by TI or somebody with more clout than me to get the latest Energia release to work well with the MSP430G2553 Launchpad.

Please let me know if there is a known fix for this problem, as it is currently stalling my project and making me ask why CCS has Energia as a project type when Energia doesn't appear to work with TI MSP430G2553 Launchpad right out of the box...

Thanks, and sorry if my post has turned into a slight rant..  Just frustrated fighting the tools that are supposed to make my job as a developer easier... :-< 

Thanks

Curtis

  • Hi Curtis,

    Sorry to hear this is not working for you. The issue was fixed after the release of Energia 17. Energia 18 took a bit longer than expected but will be out shortly.

    With that said, it's easy to patch Energia 17 to make this work.

    1. Make sure that you are using pins 14 (SCL) and 15 (SDA). A change in the BoosterPack header standard was introduced a while back which I had to adopt in Energia. This meant that by default I2C would be on pins 7 and 8. The G2553 does not have a hardware I2C on those pins and thus we had to implement a software version. This software version is not performing as it should and I will be taking it out in the future to not further cause issues for Energia users with I2C not always working properly on those pins. To set the default I2C back to pins 14 and 15 you have to call Wire.setModule(0) in setup() before Wire.begin(). Please see the example Sketch below.
    2. I am sure you already did so but just a checkpoint, make sure that you have pull-ups on SCL/SDA.
    3. copy the attached files to you Energia installation under hardware/msp430/cores/msp430
    4. make sure you remove the jumper for LED2 (P1.6) next to S2.

    With that in place you should not be able to run the Sketch below and successfully scan the bus.

    Robert @ Energia

    #include <Wire.h>
    
    void setup()
    {
      Serial.begin(9600);
      Serial.println("Start!");
      
      Wire.setModule(0);
      Wire.begin();
    }
    
    void loop()
    {
      findAddress();
      delay(1000);
    }
    
    uint8_t fromAddress = 0;
    uint8_t toAddress = 127;
    
    void findAddress()
    {
      for (uint8_t a = fromAddress; a < toAddress; a++)
      {
        Wire.beginTransmission(a);
        int endStatus = Wire.endTransmission();
    
        if (endStatus == 0)
        {
          Serial.print("Found device at: 0x");
          Serial.println(a, HEX);
        }
      }
    }

  • In reply to Robert Wessels:

    I'm having trouble finding the 'attached' files... Where are they?

    Thanks

    Curtis

  • In reply to Curtis Harrison:

    I really thought that I had attached them but must have forgotten. Sorry about that.

    twi.c

    twi_sw.c

  • In reply to Robert Wessels:

    The stars have aligned !!!  It works like a charm now !!!  Life are GROOVY !!!  Thanks a bunch Robert !!

    Thanks

    Curtis

  • In reply to Curtis Harrison:

    Great to hear that you got it working! Thanks for reporting back!

    Robert

  • In reply to Robert Wessels:

    Well, we've changed chips over to the MSP430G2553IRHB32R.  It's got 32 pins and ses pins 21 and 22 for I2C_SCL and I2C_SDA respectively.  This is a QFN package.

    My problem is that I cannot get I2C to work with this chip.  I've tried all of the above, but am using version 18 of energia.  I've replaced the pins_energia.h with the following and still cannot seem to get any response from my I2C devices.  Do you have any suggestions or recommendations?

    /*
    ************************************************************************
    * pins_energia.h
    *
    * Energia core files for MSP430
    * Copyright (c) 2012 Robert Wessels. All right reserved.
    *
    * Contribution: Rei VILO
    *
    ***********************************************************************
    Derived from:
    pins_arduino.h - Pin definition functions for Arduino
    Part of Arduino - http://www.arduino.cc/
    Copyright (c) 2007 David A. Mellis
    This library is free software; you can redistribute it and/or
    modify it under the terms of the GNU Lesser General Public
    License as published by the Free Software Foundation; either
    version 2.1 of the License, or (at your option) any later version.
    This library is distributed in the hope that it will be useful,
    but WITHOUT ANY WARRANTY; without even the implied warranty of
    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
    Lesser General Public License for more details.
    You should have received a copy of the GNU Lesser General
    Public License along with this library; if not, write to the
    Free Software Foundation, Inc., 59 Temple Place, Suite 330,
    Boston, MA 02111-1307 USA
    */
    #define __MSP430_HAS_PORT3_R__
    //#define IOT_SENSOR_NODE
    #ifndef Pins_Energia_h
    #define Pins_Energia_h
    #ifndef BV
    #define BV(x) (1 << (x))
    #endif

    #if defined(__MSP430_HAS_USCI__)
    static const uint8_t SS = 16; /* P2.4 */
    static const uint8_t SCK = 5; /* P1.5 */
    static const uint8_t MOSI = 22; /* P1.7 */
    static const uint8_t MISO = 21; /* P1.6 */
    static const uint8_t TWISDA = 22; /* P1.6 */
    static const uint8_t TWISCL = 21; /* P1.7 */
    static const uint8_t TWISCL1 = 11; /* P2.1 SW I2C */
    static const uint8_t TWISDA1 = 12; /* P2.2 SW I2C */
    static const uint8_t TWISDA0 = 23; /* P1.7 */
    static const uint8_t TWISCL0 = 22; /* P1.6 */
    static const uint8_t DEBUG_UARTRXD = 1; /* Receive Data (RXD) at P1.1 */
    static const uint8_t DEBUG_UARTTXD = 2; /* Transmit Data (TXD) at P1.2 */
    #define TWISDA1_SET_MODE (INPUT)
    #define TWISCL1_SET_MODE (INPUT)
    #define TWISDA0_SET_MODE (PORT_SELECTION0 | PORT_SELECTION1 /* | INPUT_PULLUP*/) /* do not enable the pull ups for this device */
    #define TWISCL0_SET_MODE (PORT_SELECTION0 | PORT_SELECTION1 /* | INPUT_PULLUP*/)
    #define TWISDA_SET_MODE (PORT_SELECTION0 | PORT_SELECTION1 /* | INPUT_PULLUP*/) /* do not enable the pull ups for this device */
    #define TWISCL_SET_MODE (PORT_SELECTION0 | PORT_SELECTION1 /* | INPUT_PULLUP*/)
    #define DEBUG_UARTRXD_SET_MODE (PORT_SELECTION0 | PORT_SELECTION1 | INPUT)
    #define DEBUG_UARTTXD_SET_MODE (PORT_SELECTION0 | PORT_SELECTION1 | OUTPUT)
    #define SPISCK_SET_MODE (PORT_SELECTION0 | PORT_SELECTION1)
    #define SPIMOSI_SET_MODE (PORT_SELECTION0 | PORT_SELECTION1)
    #define SPIMISO_SET_MODE (PORT_SELECTION0 | PORT_SELECTION1)
    #endif

    #define DEBUG_UART_MODULE_OFFSET 0x0

    #if defined(__MSP430_HAS_USI__)
    static const uint8_t SS = 16; /* P2.4 */
    static const uint8_t SCK = 5; /* P1.5 */
    static const uint8_t MOSI = 22; /* P1.7 */
    static const uint8_t MISO = 21; /* P1.6 */
    static const uint8_t TWISDA = 22; /* P1.6 */
    static const uint8_t TWISCL = 21; /* P1.7 */
    static const uint8_t DEBUG_UARTRXD = 1; /* Receive Data (RXD) at P1.1 */
    static const uint8_t DEBUG_UARTTXD = 2; /* Transmit Data (TXD) at P1.2 */
    #define TWISDA_SET_MODE (PORT_SELECTION0 | INPUT_PULLUP)
    #define TWISCL_SET_MODE (PORT_SELECTION0 | INPUT_PULLUP)
    #define DEBUG_UARTRXD_SET_MODE (PORT_SELECTION0 | INPUT)
    #define DEBUG_UARTTXD_SET_MODE (PORT_SELECTION0 | OUTPUT)
    #endif

    #define DEBUG_UART_MODULE 0x0

    static const uint8_t A0 = 31;
    static const uint8_t A1 = 1;
    static const uint8_t A2 = 2;
    static const uint8_t A3 = 3;
    static const uint8_t A4 = 4;
    static const uint8_t A5 = 5;
    static const uint8_t A6 = 21;
    static const uint8_t A7 = 22;
    static const uint8_t A10 = 128 + 10; // special. This is the internal temp sensor

    // +-+-+-+-+-+-+-+-+-+
    // VCC 1| |20 GND
    // (A0) P1.0 2| |19 XIN
    // (A1) P1.1 3| |18 XOUT
    // (A2) P1.2 4| |17 TEST
    // (A3) P1.3 5| |16 RST#
    // (A4) P1.4 6| |15 P1.7 (A7) (SCL) (MISO) depends on chip
    // (A5) P1.5 7| |14 P1.6 (A6) (SDA) (MOSI)
    // P2.0 8| |13 P2.5
    // P2.1 9| |12 P2.4
    // P2.2 10| |11 P2.3
    // +-+-+-+-+-+-+-+-+-+
    //

    // Pin names based on the silkscreen
    //
    static const uint8_t P1_1 = 1;
    static const uint8_t P1_2 = 2;
    static const uint8_t P1_3 = 3;
    static const uint8_t P1_4 = 4;
    static const uint8_t P1_5 = 5;
    static const uint8_t P3_1 = 6;
    static const uint8_t P3_0 = 7;
    static const uint8_t P2_0 = 9;
    static const uint8_t P2_1 = 10;
    static const uint8_t P2_2 = 11;
    static const uint8_t P3_2 = 12;
    static const uint8_t P3_3 = 13;
    static const uint8_t P3_4 = 14;
    static const uint8_t P2_3 = 15;
    static const uint8_t P2_4 = 16;
    static const uint8_t P2_5 = 17;
    static const uint8_t P3_5 = 18;
    static const uint8_t P3_6 = 19;
    static const uint8_t P3_7 = 20;
    static const uint8_t P1_6 = 21;
    static const uint8_t P1_7 = 22;
    static const uint8_t P2_7 = 25;
    static const uint8_t P2_6 = 26;
    static const uint8_t P1_0 = 31;

    static const uint8_t RED_LED = 25;
    static const uint8_t GREEN_LED = 26;

    #define PIN_MOSI P1_7
    #define PIN_MISO P1_6
    #define PIN_CLK P1_5
    #define PIN_CE P3_5
    #define PIN_nCS P3_6
    #define PIN_IRQ P2_5

    #define PIN_SCL P1_6
    #define PIN_SDA P1_7

    #define BATTERY_ADC A0

    static const uint8_t TEMPSENSOR = 10; // depends on chip


    #ifdef ARDUINO_MAIN

    const uint16_t port_to_input[] = {
    NOT_A_PORT,
    (uint16_t) &P1IN,
    (uint16_t) &P2IN,
    #ifdef __MSP430_HAS_PORT3_R__
    (uint16_t) &P3IN,
    #endif
    };

    const uint16_t port_to_output[] = {
    NOT_A_PORT,
    (uint16_t) &P1OUT,
    (uint16_t) &P2OUT,
    #ifdef __MSP430_HAS_PORT3_R__
    (uint16_t) &P3OUT,
    #endif
    };

    const uint16_t port_to_dir[] = {
    NOT_A_PORT,
    (uint16_t) &P1DIR,
    (uint16_t) &P2DIR,
    #ifdef __MSP430_HAS_PORT3_R__
    (uint16_t) &P3DIR,
    #endif
    };

    const uint16_t port_to_ren[] = {
    NOT_A_PORT,
    (uint16_t) &P1REN,
    (uint16_t) &P2REN,
    #ifdef __MSP430_HAS_PORT3_R__
    (uint16_t) &P3REN,
    #endif
    };

    const uint16_t port_to_sel0[] = { /* put this PxSEL register under the group of PxSEL0 */
    NOT_A_PORT,
    (uint16_t) &P1SEL,
    (uint16_t) &P2SEL,
    #ifdef __MSP430_HAS_PORT3_R__
    (uint16_t) &P3SEL,
    #endif
    };

    const uint16_t port_to_sel2[] = {
    NOT_A_PORT,
    #ifdef P1SEL2_
    (uint16_t) &P1SEL2,
    #else
    NOT_A_PORT,
    #endif
    #ifdef P2SEL2_
    (uint16_t) &P2SEL2,
    #else
    NOT_A_PORT,
    #endif
    #ifdef P3SEL2_
    (uint16_t) &P3SEL2,
    #else
    NOT_A_PORT,
    #endif
    };


    /*
    * Defines for devices with 2x TA3 timers (e.g. MSP430g2553). On the 20pin devices, upto 3 analog outputs are available
    * T0A1, T1A1 and T1A2
    */
    const uint8_t digital_pin_to_timer[] = {
    NOT_ON_TIMER, /* dummy */
    NOT_ON_TIMER, /* 1 - VCC */
    NOT_ON_TIMER, /* 2 - P1.0 */
    T0A0, /* 3 - P1.1, note: A0 output cannot be used with analogWrite */
    T0A1, /* 4 - P1.2 */
    NOT_ON_TIMER, /* 5 - P1.3 */
    NOT_ON_TIMER, /* 6 - P1.4 note: special case. Leaving as no timer due to difficulty determining if available */
    T0A0, /* 7 - P1.5 note: A0 output cannot be used with analogWrite */
    #if defined(__MSP430_HAS_T1A3__)
    T1A0, /* 8 - P2.0 note: A0 output cannot be used with analogWrite */
    T1A1, /* 9 - P2.1 */
    T1A1, /* 10 - P2.2 */
    T1A0, /* 11 - P2.3 note: A0 output cannot be used with analogWrite */
    T1A2, /* 12 - P2.4 */
    T1A2, /* 13 - P2.5 */
    #else
    NOT_ON_TIMER, /* 8 - P2.0 */
    NOT_ON_TIMER, /* 9 - P2.1 */
    NOT_ON_TIMER, /* 10 - P2.3 */
    NOT_ON_TIMER, /* 11 - P2.4 */
    NOT_ON_TIMER, /* 12 - P2.5 */
    NOT_ON_TIMER, /* 13 - P2.6 */
    #endif
    T0A1, /* 14 - P1.6 */
    NOT_ON_TIMER, /* 15 - P1.7 */
    NOT_ON_TIMER, /* 16 - /RESET */
    NOT_ON_TIMER, /* 17 - TEST */
    NOT_ON_TIMER, /* 18 - XOUT - P2.7 */
    T0A1, /* 19 - XIN - P2.6: */
    NOT_ON_TIMER, /* 20 - GND */
    };

    const uint8_t digital_pin_to_port[] = {
    NOT_A_PIN, /* dummy */
    P1, /* 1 */
    P1, /* 2 */
    P1, /* 3 */
    P1, /* 4 */
    P1, /* 5 */
    P3, /* 6 */
    P3, /* 7 */
    NOT_A_PIN, /* 8 */
    P2, /* 9 */
    P2, /* 10 */
    P2, /* 11 */
    P3, /* 12 */
    P3, /* 13 */
    P3, /* 14 */
    P2, /* 15 */
    P2, /* 16 */
    P2, /* 17 */
    P3, /* 18 */
    P3, /* 19 */
    P3, /* 20 */
    P1, /* 21 */
    P1, /* 22 */
    NOT_A_PIN, /* 23 */
    NOT_A_PIN, /* 24 */
    P2, /* 25 */
    P2, /* 26 */
    NOT_A_PIN, /* 27 */
    NOT_A_PIN, /* 28 */
    NOT_A_PIN, /* 29 */
    NOT_A_PIN, /* 30 */
    P1, /* 31 */
    NOT_A_PIN, /* 32 */
    };

    const uint8_t digital_pin_to_bit_mask[] = {
    NOT_A_PIN, /* 0, pin count starts at 1 */
    BV(1), /* 1, P1.1 */
    BV(2), /* 2, port P1.2 */
    BV(3), /* 3, port P1.3 */
    BV(4), /* 4, port P1.4 */
    BV(5), /* 5, port P1.5 */
    BV(1), /* 6, port P3.1 */
    BV(0), /* 7, port P3.0 */
    NOT_A_PIN, /* 8, NC */
    BV(0), /* 9, port P2.0 */
    BV(1), /* 10, port P2.1 */
    BV(2), /* 11, port P2.2 */
    BV(2), /* 12, port P3.2 */
    BV(3), /* 13, port P3.3 */
    BV(4), /* 14, port P3.4 */
    BV(3), /* 15, port P2.3 */
    BV(4), /* 16, P2.4 */
    BV(5), /* 17, P2.5 */
    BV(5), /* 18, P3.5 */
    BV(6), /* 19, P3.6 */
    BV(7), /* 20, P3.7 */
    BV(6), /* 21, P1.6 */
    BV(7), /* 22, P1.7 */
    NOT_A_PIN, /* 23, RST */
    NOT_A_PIN, /* 24, TEST */
    BV(7), /* 25, P2.7 */
    BV(6), /* 26, P2.6 */
    NOT_A_PIN, /* 27, GND */
    NOT_A_PIN, /* 28, GND */
    NOT_A_PIN, /* 29, VCC */
    NOT_A_PIN, /* 30, VCC */
    BV(0), /* 31, P1.0 */
    NOT_A_PIN, /* 32, NC */
    };
    const uint32_t digital_pin_to_analog_in[] = {
    NOT_ON_ADC, /* 0, pin count starts at 1 */
    1, /* 1, A1 */
    2, /* 2, A2 */
    3, /* 3, A3 */
    4, /* 4, A4 */
    5, /* 5, A5 */
    NOT_ON_ADC, /* 6, port P3.1 */
    NOT_ON_ADC, /* 7, port P3.0 */
    NOT_ON_ADC, /* 8, NC */
    NOT_ON_ADC, /* 9, port P2.0 */
    NOT_ON_ADC, /* 10, port P2.1 */
    NOT_ON_ADC, /* 11, port P2.2 */
    NOT_ON_ADC, /* 12, port P3.2 */
    NOT_ON_ADC, /* 13, port P3.3 */
    NOT_ON_ADC, /* 14, port P3.4 */
    NOT_ON_ADC, /* 15, port P2.3 */
    NOT_ON_ADC, /* 16, P2.4 */
    NOT_ON_ADC, /* 17, P2.5 */
    NOT_ON_ADC, /* 18, P3.5 */
    NOT_ON_ADC, /* 19, P3.6 */
    NOT_ON_ADC, /* 20, P3.7 */
    6, /* 21, A6 */
    7, /* 22, A7 */
    NOT_ON_ADC, /* 23, RST */
    NOT_ON_ADC, /* 24, TEST */
    NOT_ON_ADC, /* 25, P2.7 */
    NOT_ON_ADC, /* 26, P2.6 */
    NOT_ON_ADC, /* 27, GND */
    NOT_ON_ADC, /* 28, GND */
    NOT_ON_ADC, /* 29, VCC */
    NOT_ON_ADC, /* 30, VCC */
    0, /* 31, A0 */
    NOT_ON_ADC /* 32, NC */
    };

    #endif
    #endif

    Thanks

    Curtis