| View previous topic :: View next topic |
| Author |
Message |
lgeorge123
Joined: 05 Dec 2004
Posts: 31
|
| cascade three 74hc595 with pic16f877a |
 Posted: Wed Oct 03, 2018 8:00 pm |
 |
|
I have three 74hc595d cascade with pic16f877a. The circuit part with 74hc595 is test with Arduino and is ok. The code is as follows:
| Code: |
#include <16f877a.h>
#fuses HS,NOWDT,NOPROTECT,NOLVP,NODEBUG
#use delay(clock=16000000)
void shiftOut(unsigned char val)
{
char j;
for(j = 0x80; j > 0; j = j >> 1)
{
if(val & j)
output_high(pin_e0); // data pin
else
output_low(pin_e0);
}
output_high(pin_e2); //clock pin
output_low(pin_e2);
}
void main() {
int8 h;
int8 p;
int8 u;
int32 d=70000;
h=(char)((0xff0000 & d)>>16);
p=(char)((0x00ff00&d)>>8);
u=(char)(0x0000ff &d);
output_low(pin_e1); // latch pin
shiftout(h);
shiftout(p);
shiftout(u);
output_high(pin_e1);
while(1);
}
|
The problem is the result is not as expected. Can someone help me !! |
|
 |
Ttelmah
Joined: 11 Mar 2010
Posts: 19498
|
|
| Posted: Thu Oct 04, 2018 1:15 am |
|
|
Why not just use the supplied 74595 driver. Just define your pins, tell it you have three 595's (#define NUMBER_OF_74594 3), and the work is all done.
Your maths for the values to output seems 'odd'.
Why % 70000?.
FF0000 % 70000 will give 0xc9E0. Shifting this right 16, gives 0.
If you just want the bytes from the values, why not just use make8, or a union?. |
|
|
lgeorge123
Joined: 05 Dec 2004
Posts: 31
|
| cascade three 74hc595 with pic16f877a |
| Posted: Thu Oct 04, 2018 2:58 am |
|
|
I change the code as follows:
| Code: | #include <16f877a.h>
#fuses HS,NOWDT,NOPROTECT,NOLVP,NODEBUG
#use delay(clock=16000000)
#include <74595.c>
#define EXP_OUT_ENABLE PIN_E1
#define EXP_OUT_CLOCK PIN_E2
#define EXP_OUT_DO PIN_E0
#define NUMBER_OF_74595 3
void main() {
int8 h;
int8 p;
int8 u;
int32 d=70000;
set_tris_e(0x00);
h=make8(d,0);
p=make8(d,1);
u=make8(d,2);
write_expanded_outputs(&h);
write_expanded_outputs(&p);
write_expanded_outputs(&u);
while(1);
}
the result is still not as expected ......... |
|
|
|
lgeorge123
Joined: 05 Dec 2004
Posts: 31
|
| cascade three 74hc595 with pic16f877a |
| Posted: Thu Oct 04, 2018 3:12 am |
|
|
| What I want to do is show 70000 as binary digit in three 74hc595 IC |
|
|
Ttelmah
Joined: 11 Mar 2010
Posts: 19498
|
|
| Posted: Thu Oct 04, 2018 3:17 am |
|
|
He is not showing 70000, but I repeat my question of why he is using this value to get a remainder from the bytes he passes. It means he will get screwy values and I suspect this is the real reason the values aren't as expected....
For his first value it is almost always going to give 0, since only a number between 65536, and 70000 can actually give a bit in the 3rd byte, and this is only ever going to give a value of 0 or 1 to this chip.
Looks mathematically completely wrong.... |
|
|
lgeorge123
Joined: 05 Dec 2004
Posts: 31
|
| cascade three 74hc595 with pic16f877a |
| Posted: Thu Oct 04, 2018 3:34 am |
|
|
| Sorry , my d value in ccs c language is int32; |
|
|
temtronic
Joined: 01 Jul 2010
Posts: 9221
Location: Greensville,Ontario
|
|
| Posted: Thu Oct 04, 2018 5:33 am |
|
|
shouldn't the defines for the 74595 vars BEFORE the driver gets loaded ?
I'm thinking (with a 24hr headache) that the data is being sent to the pins the driver defaults to B0,B1,B2 not the E0,E1,E2 ?? |
|
|
Ttelmah
Joined: 11 Mar 2010
Posts: 19498
|
|
| Posted: Thu Oct 04, 2018 7:42 am |
|
|
You'd be expecting 0b00000001 on the top chip, 0b00010001 on the second, and 0b0111000 on the third.
And Temtronc is right you _must_ define the settings before you load the driver. |
|
|
Ttelmah
Joined: 11 Mar 2010
Posts: 19498
|
|
| Posted: Thu Oct 04, 2018 7:54 am |
|
|
How it needs to be done:
| Code: |
#include <16f877a.h>
#fuses HS,NOWDT,NOPROTECT,NOLVP,NODEBUG
#use delay(clock=16000000)
#define EXP_OUT_ENABLE PIN_E1
#define EXP_OUT_CLOCK PIN_E2
#define EXP_OUT_DO PIN_E0
#define NUMBER_OF_74595 3
#include <74595.c>
void main()
{
int32 d=70000;
set_tris_e(0x00);
write_expanded_outputs((int8 *)&d);
//This automatically sends three bytes from d.
//Since value is stored LSB first, this sends the low three bytes
while(1)
;
}
|
The single call updates all three chips. |
|
|
|
FAQ 
Search
Register
Profile 
Log in to check your private messages 
Log in
