Thread: Interface DS1302(RTC) with 8051 to run time on lcd. Time run but got Problem,help pls

In DS1302InputByte and DS1302OutputByte functions, you set DS1302_CLK to 1, then the next command, you set it to 0. You need to put a delay in between those two commands. A delay loop like you used in your other code should work, just make sure you loop enough times to satisfy the timing requirement.

Also, as for your array confusion, you really need to find any class notes and textbooks you have for programming C, and read through the array sections carefully. Do all the examples in the textbook. Find some tutorials on line: we have one here, and Google will turn up dozens more. Write some simple programs using arrays, but not on your embedded device, on your PC where it's easy to test and debug.

Hi,
yeah,i did think about what you explain.
The hint you gave saying that it is not 7 or 9.
So i think it should be 8,am i right?
Well i have change it to 8 for the array but the problem still remain there.

even i dont put array there. i change the code to print out "Time:" on the top row then follow by that running time on the second row. But that 85 problem still exist too. I show the code that i change below. Pls,let me know what else i can do. I really want to solve this error. I trying hard now,pls guide me. Thx

Code:

void LCD_TimerDisp()
{
	unsigned char alphabet;
	
    for(alphabet=0; alphabet<5; alphabet++)
	{
		LCD_print(msg1[alphabet]);
	}

	do
	{
	DS1302_GetTime(&CurrentTime);
	TimeToStr(&CurrentTime);
	DateToStr(&CurrentTime);
	LCD_command(0xc0);
	for(alphabet=0; alphabet<8; alphabet++)
	{
		LCD_print(CurrentTime.TimeString[alphabet]);
	}
	LCD_command(0x06);
    for(alphabet=0; alphabet<5; alphabet++)
	{
		LCD_print(msg1[alphabet]);
	}
	delay(1000);
	}
	while(1);
}

For the msg1 ,i have define it as "time:"

Okay, so we've established you can reliably print text to the LCD. You have no problem printing a string like "time:". That means that all the code you wrote is fine. The problem is with the DS1302 code. Just because you found it on some website doesn't mean it's correct or you can't change it. The reason I brought up the timing thing is because if the guy who wrote it was working on a 8 MHz chip and you're working on a 40 MHz chip, your processor is 5x faster, meaning each pulse you send on DS1302_CLK is 1/5 the time. What was sufficient for the clock on his slow processor might not be on your faster processor. That's why I suggested the delay loop:

Code:

void DS1302InputByte(unsigned char d) 	//ʵʱʱÖÓдÈëÒ»×Ö½Ú(ÄÚ²¿º¯Êý) 
{  
    unsigned char i; 
    ACC = d; 
    for(i=8; i>0; i--) 
    { 
        DS1302_IO = ACC0;           	//Ï൱ÓÚ»ã±àÖÐµÄ RRC 
        DS1302_CLK = 1; 
        delay(10);  // delay 10 units
        DS1302_CLK = 0; 
        ACC = ACC >> 1;  
    }  
} 
 
unsigned char DS1302OutputByte(void) 	//ʵʱʱÖÓ¶ÁÈ¡Ò»×Ö½Ú(ÄÚ²¿º¯Êý) 
{  
    unsigned char i; 
    for(i=8; i>0; i--) 
    { 
        ACC = ACC >>1;         			//Ï൱ÓÚ»ã±àÖÐµÄ RRC  
        ACC7 = DS1302_IO; 
        DS1302_CLK = 1; 
        delay(10);  // delay 10 units
        DS1302_CLK = 0; 
    }  
    return(ACC);  
}

Note, I'm not sure this is the problem, but it can't hurt to try at this point. I also don't know what an acceptable delay value is. Maybe it's 10, maybe 1000. You'll have to play around with it, and it might not work at all. Other than that, I don't know what to tell you. Perhaps there's some wiring issue with your DS1302. It doesn't look like it, but the schematic in your other post is hard to read.

Well,thank for re-commanding me to add delay at the input and output areas.
I have tried that but the LCD print " 00:00:00" now. So i have remove the delay.

What can i do to solve the 85 problem?? I have try many ways but non of it works. The 85 still jump about when the timing is run.
What i know is the RTC chip should not have problem because the time can run.
Any suggestions? I will try and see what i can do about it.
Thanks.

I have tried both. Both of them works. So there is a problem with my DS1302_GetTime input code??
I don't understand what you mean by 50ms, i mean the delay.
So is it something wrong with my delay now? I have to add in delay or change the length of it??
Sorry,i don't really get what you mean.
can you tell me what should i try or do now??

Thanks.

Hello,
I have tried what i was told to do in post #18.
After i add delay in the DS1302InputByte and DS1302OutputByte,
the Lcd only displayed " 00:00:00 " & the time did not run at all.
I have also tried to play around by changing the length of the delay
but it wont make also different at all.

Well i not sure why that happen. Like what you have say, i think i understand what you mean.
Is there any other place which you think i can add in delay or any code to change??

Thanks