CCS C Software and Maintenance Offers
FAQFAQ   FAQForum Help   FAQOfficial CCS Support   SearchSearch  RegisterRegister 

ProfileProfile   Log in to check your private messagesLog in to check your private messages   Log inLog in 

CCS does not monitor this forum on a regular basis.

Please do not post bug reports on this forum. Send them to support@ccsinfo.com

Reading seconds value from ds1307 skipping with dspic30f4013
Goto page 1, 2  Next
 
Post new topic   Reply to topic    CCS Forum Index -> General CCS C Discussion
View previous topic :: View next topic  
Author Message
ahmed-agt



Joined: 24 Oct 2014
Posts: 9

View user's profile Send private message

Reading seconds value from ds1307 skipping with dspic30f4013
PostPosted: Mon Nov 05, 2018 8:22 am     Reply with quote

Code:
#include <testdspccs.h>

//----LCD Driver and Port 
#include <Flex_LCD420.C>
#define USE_RW_PIN 1
#define LCD_BL PIN_D1

//----------FAST = 100000 SDA = PIN_F2, SCL = PIN_F3, FORCE_HW
#define RTC_SDA  PIN_F2
#define RTC_SCL  PIN_F3
#use I2C(master, SDA = RTC_SDA, SCL = RTC_SCL, FAST = 100000)

//---- Analogue Control Interface
#define AMP_GAIN_SW PIN_A11
#define SENSOR_GND_SW PIN_D9

/*---------------------------------------*/
//---- Variable Names Declaration
char time[] = "TIME:  :  :     ";
char calendar[] = "DATE:  /  /20  ";

unsigned int16 int_count;

unsigned int8 seconds, second, second10, minute, minute10,
hour, hour10, date, date10, month, month10,
year, year10, day;


#INT_TIMER1
void Timer1_ISR(void)
{   
         //IFS0 = IFS0 & 0xFFF7;
         ++int_count;
         //if(int_count == 60) int_count = 0;
         
}



//----------------------------------
//
void ds1307_display(){
   
   second10 = (second & 0x70) >> 4;
   second = second & 0x0F;
   minute10 = (minute & 0x70) >> 4;
   minute = minute & 0x0F;
   hour10 = (hour & 0x30) >> 4;
   hour = hour & 0x0F;
   date10 = (date & 0x30) >> 4;
   date = date & 0x0F;
   month10 = (month & 0x10) >> 4;
   month = month & 0x0F;
   year10 = (year & 0xF0) >> 4;
   year = year & 0x0F;
   
   time[12] = second + 48;
   time[11] = second10 + 48;
   time[9] = minute + 48;
   time[8] = minute10 + 48;
   time[6] = hour + 48;
   time[5] = hour10 + 48;
   calendar[14] = year + 48;
   calendar[13] = year10 + 48;
   calendar[9] = month + 48;
   calendar[8] = month10 + 48;
   calendar[6] = date + 48;
   calendar[5] = date10 + 48;
   
   //lcd_gotoxy(1, 1); // Go to column 1 row 1
   printf(lcd_putc, time); // Display time
   //lcd_gotoxy(1, 2); // Go to column 1 row 2
   lcd_putc('\n');
   printf(lcd_putc, calendar); // Display calendar
}

void ds1307_write(unsigned int8 address, data_)
{
   i2c_start(); // Start I2C
   i2c_write(0xD0); // DS1307 address
   i2c_write(address); // Send register address
   i2c_write(data_); // Write data to the selected register
   i2c_stop(); // Stop I2C
}

void ds1307_read()
{
   i2c_start(); // Start I2C
   i2c_write(0xD0); // DS1307 address
   i2c_write(0x00); // Send register address
   
   i2c_start(); // Restart I2C
   i2c_write(0xD1); // Initialize data read
 
   second =i2c_read(1); // Read seconds from register 0  second
   minute =i2c_read(1); // Read minuts from register 1
   hour = i2c_read(1); // Read hour from register 2
   day = i2c_read(1); // Read day from register 3
   date = i2c_read(1); // Read date from register 4
   month = i2c_read(1); // Read month from register 5
   year = i2c_read(0); // Read year from register 6
   i2c_stop(); // Stop I2C
}





//-------------------------------

void main()
{
   int_count = restart_cause();
   
   set_tris_a(0x0000);
   set_tris_B(0X01FF);
   set_tris_c(0X00);
   set_tris_D(0X00);
   set_tris_F(0X00); 
   
   //----initialising LCD     
    lcd_init();   
    output_high(LCD_BL);
   
   
    lcd_putc('\f');
     lcd_putc("Loading......\n");
     printf(lcd_putc,"%u", int_count);
     delay_ms(5000);
   
    i2c_init(TRUE);
    ds1307_write(0x00, 0x03);
   
    /////-------------------------------
    lcd_putc('\f');
      lcd_putc("Loading......\n"); delay_ms(2000);
   int_count = 0;
   
   
   setup_timer1(TMR_INTERNAL|TMR_DIV_BY_256,65000);
   enable_interrupts(INTR_LEVEL1);
   set_timer1(0);
      enable_interrupts(INT_TIMER1);
       
      //enable_interrupts(INTR_GLOBAL);
   
   while(TRUE)
   {
      ds1307_read(); // Read data from DS1307 RTCC
      //int_count = get_timer1();
      lcd_putc('\f');
      lcd_putc("testing......\n");
      printf(lcd_putc,"%u\n", int_count);
     
     ds1307_display(); // Diaplay time and calendar
      delay_ms(500);
      output_toggle(AMP_GAIN_SW);
   }

}
Quote:
This attached code is use to interface dspic30f4013 with ds1307. Reading values from the rtc ds1307 was successful accept second value or reading from address 0x00 of the timer. kindly check the code if it is not having a bug. Using CCS 5.070


Last edited by ahmed-agt on Mon Nov 05, 2018 8:43 am; edited 1 time in total
temtronic



Joined: 01 Jul 2010
Posts: 9081
Location: Greensville,Ontario

View user's profile Send private message

PostPosted: Mon Nov 05, 2018 8:39 am     Reply with quote

yikes that's really complicated for just reading the RTC every second...
You can do it a whole lot easier... have the RTC generate an interrupt at a 1Hz rate. I've done that for years and none of my project 'skip seconds'.
Also by letting the RTC do it, you free up the timer1 for more important tasks.
As well the formatting to display the date/time is unneccasriy complicated. Use the CCS supplied DS1307 driver !

Jay
ahmed-agt



Joined: 24 Oct 2014
Posts: 9

View user's profile Send private message

Thanks for your respond.
PostPosted: Mon Nov 05, 2018 8:50 am     Reply with quote

Every things works fine timer1 interrupt, date and time. This code works on pic18f and pic16f. But on dspic30f4013, the second is reading zeros.
using mikroc, it work fine with same hardware.
ahmed-agt



Joined: 24 Oct 2014
Posts: 9

View user's profile Send private message

Re: Reading seconds value from ds1307 skipping with dspic30f
PostPosted: Mon Nov 05, 2018 9:08 am     Reply with quote

ahmed-agt wrote:
Code:
#include <testdspccs.h>

//----LCD Driver and Port 
#include <Flex_LCD420.C>
#define USE_RW_PIN 1
#define LCD_BL PIN_D1

//----------FAST = 100000 SDA = PIN_F2, SCL = PIN_F3, FORCE_HW
#define RTC_SDA  PIN_F2
#define RTC_SCL  PIN_F3
#use I2C(master, SDA = RTC_SDA, SCL = RTC_SCL, FAST = 100000)

//---- Analogue Control Interface
#define AMP_GAIN_SW PIN_A11
#define SENSOR_GND_SW PIN_D9

/*---------------------------------------*/
//---- Variable Names Declaration
char time[] = "TIME:  :  :     ";
char calendar[] = "DATE:  /  /20  ";

unsigned int16 int_count;

unsigned int8 seconds, second, second10, minute, minute10,
hour, hour10, date, date10, month, month10,
year, year10, day;


#INT_TIMER1
void Timer1_ISR(void)
{   
         //IFS0 = IFS0 & 0xFFF7;
         ++int_count;
         //if(int_count == 60) int_count = 0;
         
}



//----------------------------------
//
void ds1307_display(){
   
   second10 = (second & 0x70) >> 4;
   second = second & 0x0F;
   minute10 = (minute & 0x70) >> 4;
   minute = minute & 0x0F;
   hour10 = (hour & 0x30) >> 4;
   hour = hour & 0x0F;
   date10 = (date & 0x30) >> 4;
   date = date & 0x0F;
   month10 = (month & 0x10) >> 4;
   month = month & 0x0F;
   year10 = (year & 0xF0) >> 4;
   year = year & 0x0F;
   
   time[12] = second + 48;
   time[11] = second10 + 48;
   time[9] = minute + 48;
   time[8] = minute10 + 48;
   time[6] = hour + 48;
   time[5] = hour10 + 48;
   calendar[14] = year + 48;
   calendar[13] = year10 + 48;
   calendar[9] = month + 48;
   calendar[8] = month10 + 48;
   calendar[6] = date + 48;
   calendar[5] = date10 + 48;
   
   //lcd_gotoxy(1, 1); // Go to column 1 row 1
   printf(lcd_putc, time); // Display time
   //lcd_gotoxy(1, 2); // Go to column 1 row 2
   lcd_putc('\n');
   printf(lcd_putc, calendar); // Display calendar
}

void ds1307_write(unsigned int8 address, data_)
{
   i2c_start(); // Start I2C
   i2c_write(0xD0); // DS1307 address
   i2c_write(address); // Send register address
   i2c_write(data_); // Write data to the selected register
   i2c_stop(); // Stop I2C
}

void ds1307_read()
{
   i2c_start(); // Start I2C
   i2c_write(0xD0); // DS1307 address
   i2c_write(0x00); // Send register address
   
   i2c_start(); // Restart I2C
   i2c_write(0xD1); // Initialize data read
 
   second =i2c_read(1); // Read seconds from register 0  second
   minute =i2c_read(1); // Read minuts from register 1
   hour = i2c_read(1); // Read hour from register 2
   day = i2c_read(1); // Read day from register 3
   date = i2c_read(1); // Read date from register 4
   month = i2c_read(1); // Read month from register 5
   year = i2c_read(0); // Read year from register 6
   i2c_stop(); // Stop I2C
}





//-------------------------------

void main()
{
   int_count = restart_cause();
   
   set_tris_a(0x0000);
   set_tris_B(0X01FF);
   set_tris_c(0X00);
   set_tris_D(0X00);
   set_tris_F(0X00); 
   
   //----initialising LCD     
    lcd_init();   
    output_high(LCD_BL);
   
   
    lcd_putc('\f');
     lcd_putc("Loading......\n");
     printf(lcd_putc,"%u", int_count);
     delay_ms(5000);
   
    i2c_init(TRUE);
    ds1307_write(0x00, 0x03);
   
    /////-------------------------------
    lcd_putc('\f');
      lcd_putc("Loading......\n"); delay_ms(2000);
   int_count = 0;
   
   
   setup_timer1(TMR_INTERNAL|TMR_DIV_BY_256,65000);
   enable_interrupts(INTR_LEVEL1);
   set_timer1(0);
      enable_interrupts(INT_TIMER1);
       
      //enable_interrupts(INTR_GLOBAL);
   
   while(TRUE)
   {
      ds1307_read(); // Read data from DS1307 RTCC
      //int_count = get_timer1();
      lcd_putc('\f');
      lcd_putc("testing......\n");
      printf(lcd_putc,"%u\n", int_count);
     
     ds1307_display(); // Diaplay time and calendar
      delay_ms(500);
      output_toggle(AMP_GAIN_SW);
   }

}
Quote:
This attached code is use to interface dspic30f4013 with ds1307. Reading values from the rtc ds1307 was successful accept second value or reading from address 0x00 of the timer. kindly check the code if it is not having a bug. Using CCS 5.070


Please, can any one test this code for bug? Because, it works on other chip, and also on Mikroc. But on dspic30f4013 it return zeros for seconds while other values are ok.
temtronic



Joined: 01 Jul 2010
Posts: 9081
Location: Greensville,Ontario

View user's profile Send private message

PostPosted: Mon Nov 05, 2018 10:29 am     Reply with quote

Is Vbat of the DS1307 connected to a good coin cell battery ?
Ttelmah



Joined: 11 Mar 2010
Posts: 19195

View user's profile Send private message

PostPosted: Mon Nov 05, 2018 11:23 am     Reply with quote

You claim the interrupts are working, but they are not enabled in what you post. You have also left out the chip configuration, so we cannot tell if it is setup reasonably.
What clock are you running?.
How is this generated?.
What voltage are you running?.
What pullups on the I2C bus?.
ahmed-agt



Joined: 24 Oct 2014
Posts: 9

View user's profile Send private message

PostPosted: Mon Nov 05, 2018 12:02 pm     Reply with quote

Ttelmah wrote:
You claim the interrupts are working, but they are not enabled in what you post. You have also left out the chip configuration, so we cannot tell if it is setup reasonably.
What clock are you running?.
How is this generated?.
What voltage are you running?.
What pullups on the I2C bus?.


Note, the chip is dspic30f4013. to configure the timer1 in interrupt mode, T1IE enable it without the INTR_GLOBAL or the two together. i observed this recently when I am having Trap stack issue.
I reduce the code to only part giving me problem, however all are working accept that seconds issue.
The system running at 64Mhz using 8Mhz crystal with xt_PLL x8.

These is the configuration use from the header file--
Code:

#include <30F4013.h>
#DEVICE ADC=12
#device ICSP=1
#fuses XT_PLL8
#use delay(clock=64000000,crystal=8000000)
#FUSES MCLR
#FUSES NOWDT                    //No Watch Dog Timer
//#FUSES CKSFSM                   //Clock Switching is enabled, fail Safe clock monitor is enabled
#FUSES NOCKSFSM                 //Clock Switching is disabled, fail Safe clock monitor is disabled
#FUSES NOBROWNOUT
Ttelmah



Joined: 11 Mar 2010
Posts: 19195

View user's profile Send private message

PostPosted: Mon Nov 05, 2018 12:12 pm     Reply with quote

One thing to try. After every I2C_stop() in the code add delay_us(2).

I2C requires a minimum time between an I2C stop and a subsequent I2C start. Typically a couple of uSec. At normal processor speeds the time involved in leaving a routine and entering the next gives enough delay. But at 32MIPS, you probably do not have enough delay.

Update that. Make it delay_us(4). Just checked the datasheet, and the ds1307, requires a minimum of 4.7uSec, so 2uSec is not enough.
ahmed-agt



Joined: 24 Oct 2014
Posts: 9

View user's profile Send private message

PostPosted: Mon Nov 05, 2018 12:19 pm     Reply with quote

temtronic wrote:
Is Vbat of the DS1307 connected to a good coin cell battery ?

Button Battery good, minutes and hours are running fine and accurate.
ahmed-agt



Joined: 24 Oct 2014
Posts: 9

View user's profile Send private message

PostPosted: Mon Nov 05, 2018 12:23 pm     Reply with quote

Ttelmah wrote:
One thing to try. After every I2C_stop() in the code add delay_us(2).

I2C requires a minimum time between an I2C stop and a subsequent I2C start. Typically a couple of uSec. At normal processor speeds the time involved in leaving a routine and entering the next gives enough delay. But at 32MIPS, you probably do not have enough delay.

Update that. Make it delay_us(4). Just checked the datasheet, and the ds1307, requires a minimum of 4.7uSec, so 2uSec is not enough.


Thank you for these observation. i had done that before but no good outcome since ccs compiler do consider some of this timing issues. However I will retry YOUR suggestion.
Ttelmah



Joined: 11 Mar 2010
Posts: 19195

View user's profile Send private message

PostPosted: Mon Nov 05, 2018 11:47 pm     Reply with quote

One other thing, your I2C setup should have slow=100000, not fast=100000.
Using 'fast' tells the compiler that the device supports fast timings. The DS1307, doesn't.
ahmed-agt



Joined: 24 Oct 2014
Posts: 9

View user's profile Send private message

PostPosted: Tue Nov 06, 2018 2:14 am     Reply with quote

Ttelmah wrote:
One other thing, your I2C setup should have slow=100000, not fast=100000.
Using 'fast' tells the compiler that the device supports fast timings. The DS1307, doesn't.


Thank you for your efforts. The reply i receive from CCS support late yesterday confirm that it is a bug from the compiler version. And the message from CCS -"This issue has been corrected and the fix will be available in the next compiler release we do"
Two suggestions were given as way out, i will post it after testing!
Ttelmah



Joined: 11 Mar 2010
Posts: 19195

View user's profile Send private message

PostPosted: Tue Nov 06, 2018 2:32 am     Reply with quote

Simple answer then, switch to using software I2C. On your chip this will make no difference at all (software can give over a MHz on a chip at this speed), and will fix the issue.
temtronic



Joined: 01 Jul 2010
Posts: 9081
Location: Greensville,Ontario

View user's profile Send private message

PostPosted: Tue Nov 06, 2018 6:33 am     Reply with quote

Another possible solution is to just slow down the clock. Try 8MHz, see if it performs 100%, if so X2 to 16MHz, test,works? great, try 32Mhz....

Unless you're performing a LOT of math, you may not need 64MHz speed.
What's interesting is you said both 16 and 18 series PICs worked fine but...you never said at what speed.
Jay
Ttelmah



Joined: 11 Mar 2010
Posts: 19195

View user's profile Send private message

PostPosted: Tue Nov 06, 2018 8:46 am     Reply with quote

Well (of course), the fastest PIC18, is only 16MIPS. Given he is running at 32MIPS, there will be big differences....
Display posts from previous:   
Post new topic   Reply to topic    CCS Forum Index -> General CCS C Discussion All times are GMT - 6 Hours
Goto page 1, 2  Next
Page 1 of 2

 
Jump to:  
You cannot post new topics in this forum
You cannot reply to topics in this forum
You cannot edit your posts in this forum
You cannot delete your posts in this forum
You cannot vote in polls in this forum


Powered by phpBB © 2001, 2005 phpBB Group