This thread has been locked.

If you have a related question, please click the "Ask a related question" button in the top right corner. The newly created question will be automatically linked to this question.

  • Resolved

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

Intellectual 260 points
Curtis Harrison

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

Intellectual 260 points
Curtis Harrison
  • Expert 3830 points
    Robert Wessels

    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);
    }
  }
}

  • Intellectual 260 points
    Curtis Harrison

    In reply to Robert Wessels:

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

    Thanks

    Curtis

  • Expert 3830 points
    Robert Wessels

    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

  • Intellectual 260 points
    Curtis Harrison

    In reply to Robert Wessels:

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

    Thanks

    Curtis

  • Expert 3830 points
    Robert Wessels

    In reply to Curtis Harrison:

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

    Robert

  • Intellectual 260 points
    Curtis Harrison

    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

This thread has been locked.

If you have a related question, please click the "Ask a related question" button in the top right corner. The newly created question will be automatically linked to this question.

All content and materials on this site are provided "as is". TI and its respective suppliers and providers of content make no representations about the suitability of these materials for any purpose and disclaim all warranties and conditions with regard to these materials, including but not limited to all implied warranties and conditions of merchantability, fitness for a particular purpose, title and non-infringement of any third party intellectual property right. TI and its respective suppliers and providers of content make no representations about the suitability of these materials for any purpose and disclaim all warranties and conditions with respect to these materials. No license, either express or implied, by estoppel or otherwise, is granted by TI. Use of the information on this site may require a license from a third party, or a license from TI.

Content on this site may contain or be subject to specific guidelines or limitations on use. All postings and use of the content on this site are subject to the Terms of Use of the site; third parties using this content agree to abide by any limitations or guidelines and to comply with the Terms of Use of this site. TI, its suppliers and providers of content reserve the right to make corrections, deletions, modifications, enhancements, improvements and other changes to the content and materials, its products, programs and services at any time or to move or discontinue any content, products, programs, or services without notice.

TI Worldwide | Contact Us | my.TI Login | Site Map | Corporate Citizenship | mobile m.ti.com (Mobile Version)

TI is a global semiconductor design and manufacturing company. Innovate with 100,000+ analog ICs and
embedded processors, along with software, tools and the industry’s largest sales/support staff.

© Copyright 1995-2018 Texas Instruments Incorporated. All rights reserved.
Trademarks | Privacy Policy | Terms of Use