Matching time conditions

[Melanie]

You've set TRIS for PORTD.7 for Output?

You've checked that anything else that share that pin is switched off (eg any Analogue or other devices).

You've connected your LED the correct way around (with a current limiting Resistor) and it works?

Personally I like to use HIGH and LOW... eg...

HIGH PortD.7

eg

If AlarmH1=AlarmHour then
HIGH PortD.7
else
LOW PortD.7
endif

[malc-c]

You've set TRIS for PORTD.7 for Output?

You've checked that anything else that share that pin is switched off (eg any Analogue or other devices).

You've connected your LED the correct way around (with a current limiting Resistor) and it works?

Personally I like to use HIGH and LOW... eg...

HIGH PortD.7

eg

If AlarmH1=AlarmHour then
HIGH PortD.7
else
LOW PortD.7
endif

Scrub my last post - I used your HIGH LOW example and it worked

Thank you.....

[malc-c]

LOL - the LED just went out as the clock turned to 22:00 hrs

The best bit is I've learned more about datasheets through this thread, and some of the pitfalls of my poor coding

Thank you for your patience Mel - I do appreciate it

[Byte_Butcher]

Scrub my last post - I used your HIGH LOW example and it worked

If HIGH / LOW works, but PORTD.7 = 1 DOESN'T work, that sounds like a sure sign that you don't have your pin correctly set as an output.

Using HIGH or LOW automatically sets the pin as an output first before it sets it high or low

Check your TRIS settings again. PORTD.7 = 1 SHOULD work if the port is set up right....


steve

[malc-c]

Thanks Steve... I had the TRISO for port D set so no Idea !

One last thing (for tonight)

I added a section for minutes too and could get the LED to come on at say 22:34 and go out at 22:35.

AlarmH1=(RTCHour>>4)
AlarmH1=(AlarmH1&$03)*10
AlarmH1=AlarmH1+(RTCHour&$0F)

AlarmM1=(RTCMin>>4)
AlarmM1=(AlarmM1&$03)*10
AlarmM1=AlarmM1+(RTCMin&$0F)
    
AlarmHour1 = 22
Alarmmin1 = 34
	
	LCDOut ":",#(RTCMin>>4)&$0F,#RTCMin&$0F,":"
	LCDOut #(RTCSec>>4)&$0F,#RTCSec&$0F," "
	LCDOut $FE, $D4,"value = ",  # alarmh1, #alarmhour1

If AlarmH1 = AlarmHour1 and AlarmM1 = alarmmin1 then
HIGH PortD.7
else
LOW PortD.7
endif

So I added a a few more variables to try and set a time range but whilst it compiles it no longer lights the LED.

AlarmH1=(RTCHour>>4)
AlarmH1=(AlarmH1&$03)*10
AlarmH1=AlarmH1+(RTCHour&$0F)

AlarmM1=(RTCMin>>4)
AlarmM1=(AlarmM1&$03)*10
AlarmM1=AlarmM1+(RTCMin&$0F)
    
AlarmHour1 = 22
Alarmmin1 = 53
alarmhour2 = 22
Alarmmin2 = 55
	
	LCDOut ":",#(RTCMin>>4)&$0F,#RTCMin&$0F,":"
	LCDOut #(RTCSec>>4)&$0F,#RTCSec&$0F," "
	LCDOut $FE, $D4,"value = ",  # alarmh1, #alarmhour1

If AlarmH1 >= AlarmHour1 and AlarmH1 <= alarmhour2 and AlarmM1=>alarmmin1 and alarmM1<= alarmmin2 then
HIGH PortD.7
else
LOW PortD.7
endif

Any ideas ?

[Darrel Taylor]

Hi Malcolm,

I'm wondering if you missed the link I showed in post 11.

I really think that's what you're looking for.
<br>

[malc-c]

Hi Darrel,

Must of overlooked that... I'll have a read at work in the morning, and the see if I can implement the suggestions... It's getting late, my two little gray cells have gone to sleep...

[Melanie]

Malcolm, you can't have '&$03' for MINUTES (the way you have for HOURS)... but can you figure why and what value it should be if not $03? Correct that small omission and your code will run (there's incentive for you).