• Join
  • Sign In with my.TI Login
Texas Instruments
  • Products
  • Applications
  • Tools & Software
  • Support & Community
  • Sample & Buy
  • About TI
Sample & Purchase Cart Sample & Purchase Cart
  • Search
  • Advanced
TI E2E™ Community
  • Support Forums
  • Blogs
  • Groups
  • Videos
  • 简体中文
  • More ...
TI Home » TI E2E Community » Support Forums » Microcontrollers » MSP430™ Microcontrollers » MSP430 Ultra-Low Power 16-bit Microcontroller Forum » Error in watchdog timer reset function in build 1.10.00.18 of MSP430ware
Share
MSP430™ Microcontrollers
  • Forum
  • Announcements
  • E2E Wiki
Options
  • Subscribe via RSS
MSP430 Resources
  • MSP430 Product Folder
  • MSP-EXP430G2 - MSP430 LaunchPad Value Line Development kit
  • MSP430 Getting Started Guide
  • MSP430 Microcontroller Projects
  • More Resources >
  • Error in watchdog timer reset function in build 1.10.00.18 of MSP430ware

    Error in watchdog timer reset function in build 1.10.00.18 of MSP430ware

    This question is answered
    Chester Gillon
    Posted by Chester Gillon
    on Mar 24 2012 07:16 AM
    Mastermind6305 points

    The wdt.c has a WDT_resetTimer function:

    //*****************************************************************************
    //
    //! Clears the timer counter of the Watchdog Timer.
    //!
    //! \param baseAddress is the base address of the WDT module.
    //!
    //! This function clears the watchdog timer to 0x0000h.
    //!
    //! \return None
    //
    //*****************************************************************************
    void WDT_resetTimer (unsigned int baseAddress)


    However, there is no function prototype for WDT_resetTimer in wdt.h

    wdt.h contains the following prototype, for which there is no corresponding function in wdt.c

    extern void WDT_clearCounter (unsigned int baseAddress);

    Therefore, when using driverlib had to call WDT_resetTimer but that causes the compiler to generate a "function declared implicitly" warning.

    Request that the driverlib be changed to provide the correct function prototype to remove the warning.

    Report Abuse
    • Reply
    You have posted to a forum that requires a moderator to approve posts before they are publicly available.
    All Replies
    • Chester Gillon
      Posted by Chester Gillon
      on Mar 25 2012 06:33 AM
      Mastermind6305 points

      On further investigation found that the WDT_resetTimer function doesn't actually reset the watchdog timer counter. Instead, it writes zero to the WDTCTL_L register which has the effect of selecting a watchdog clock source of SMCLK with a watchdog timer interval of Watchdog clock source /2G

      WDT_resetTimer is:

      void WDT_resetTimer (unsigned int baseAddress)
      {
         //Set Counter Clear bit
         unsigned char newWDTStatus =
           ( HWREGB(baseAddress + OFS_WDTCTL_L) & WDTCNTCL );

         HWREG(baseAddress + OFS_WDTCTL) = WDTPW + newWDTStatus;
      }

      To actually reset the watchdog timer counter needs to be: 

      void WDT_resetTimer (unsigned int baseAddress)
      {
         //Set Counter Clear bit
         unsigned char newWDTStatus =
           ( HWREGB(baseAddress + OFS_WDTCTL_L) | WDTCNTCL );

         HWREG(baseAddress + OFS_WDTCTL) = WDTPW + newWDTStatus;
      }

      Report Abuse
      • Reply
      You have posted to a forum that requires a moderator to approve posts before they are publicly available.
    • Jens-Michael Gross
      Posted by Jens-Michael Gross
      on Mar 26 2012 18:58 PM
      Guru139900 points

      Maybe that's why the function prototype was missing in teh header file: the function is buggy, so it wasn't "published". :)

      _____________________________________
      Before posting bug reports or ask for help, do at least quick scan over this article. It applies to any kind of problem reporting. On any forum. And/or look here.
      If you cannot discuss your problem in the public, feel free to start a private conversation: click on my name and then 'start conversation'. But please do so only if you really cannot do it in a public thread, as I usually read all threads. And I prefer to answer where others can profit from it (or contribute to it) too.

      Report Abuse
      • Reply
      You have posted to a forum that requires a moderator to approve posts before they are publicly available.
    • Kasthuri Annamalai
      Posted by Kasthuri Annamalai
      on Apr 11 2012 11:12 AM
      Verified Answer
      Verified by Chester Gillon
      Intellectual2040 points

      Hi Chester,

      Thanks for pointing this out. This is indeed a bug. We will fix this in the next release. We will update E2E when this fix gets included in the next release.

      Thanks,
      Kasthuri

      ---------------------------------------------------------------------------------------------------------

      Please click the Verify Answer button on this post if it answers your question.
      ---------------------------------------------------------------------------------------------------------

      Report Abuse
      • Reply
      You have posted to a forum that requires a moderator to approve posts before they are publicly available.
    • Chester Gillon
      Posted by Chester Gillon
      on Jun 16 2012 09:50 AM
      Mastermind6305 points

      This been fixed in MSP430ware 1.20.00.16. 

      Report Abuse
      • Reply
      You have posted to a forum that requires a moderator to approve posts before they are publicly available.
    TI E2E™ Community
    • Support Forums
    • Blogs
    • Videos
    • Groups
    • Site Support & Feedback
    • Settings
    TI E2E™ Community Groups
    • TI University Program
    • Make the Switch
    • Microcontroller Projects
    • Motor Drive & Control
    Other Communities
    • Deyisupport
    • Designsomething.org
    • beagleboard.org
    • TI on Element 14
    • TI on TechXchangeSM
    Other Technical & Support Resources
    • WEBENCH® Design Center
    • Product Information Centers
    • Technical Documents
    • TI Design Network
    • TI Technical Articles
    • TI Training

    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.

    Follow Us Texas Instruments on Facebook Texas Instruments on Twitter Texas Instruments on LinkedIn Texas Instruments on Google+
    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-2013 Texas Instruments Incorporated. All rights reserved.
    Trademarks | Privacy Policy | Terms of Use