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 CCS Technical Support

Problem with passing array as an argument to function

 
Post new topic   Reply to topic    CCS Forum Index -> General CCS C Discussion
View previous topic :: View next topic  
Author Message
monsun



Joined: 17 Jan 2012
Posts: 17

View user's profile Send private message

Problem with passing array as an argument to function
PostPosted: Tue Jan 17, 2012 6:45 am     Reply with quote

Hello everyone
I'm trying to pass an array as an argument to function using pointer,
and after all day fight I can't make it working.

My pic is PIC18F6720

I have global variable
Code:

int1 key_array[5];

which values I would like to change using function
key_change(key_array).
Code:

void key_change(int1 *key[])
{
 int8 i;
   for(i=0;i<5;i++) *(key+i)=0;
   if(!input(PIN_G0)) {*key=1;}
   if(!input(PIN_G1)) {*(key+1)=1;}
   if(!input(PIN_G2)) {*(key+2)=1;}
   if(input(PIN_G3)) {*(key+3)=1;}
   if(input(PIN_G4)) {*(key+4)=1;}
}

The program compiles without any errors but it doesn't work like it should.
On my lcd display I see only the first value changing.
I tried the same function using global variable and it works
perfectly but I prefer to do this universal so that why I'm trying with pointers.

any ideas?
Ttelmah



Joined: 11 Mar 2010
Posts: 19520

View user's profile Send private message

PostPosted: Tue Jan 17, 2012 8:08 am     Reply with quote

Realistically I doubt if this can ever be made to work.
What you could do, is look at how 'pin numbers' are implemented. Here the number is the pointer to the memory address the pin is on, multiplied by 8, and then added to the bit number. The code divides the number by 8 before using it as a pointer.
Key to understand is that a 'pointer', is the address of a byte in memory. Given that an int1, is smaller than a byte, you basically can't have a 'pointer' to it directly....

Best Wishes
jeremiah



Joined: 20 Jul 2010
Posts: 1349

View user's profile Send private message

PostPosted: Tue Jan 17, 2012 9:17 am     Reply with quote

one option you might look at is bit fields in a struct. Depending on your compiler version, you could do something like:

Code:

//can use unsigned int8 instead if desired.
typedef union{
   unsigned int16 value;
   struct{
      unsigned int16 g0 : 1;
      unsigned int16 g1 : 1;
      unsigned int16 g2 : 1;
      unsigned int16 g3 : 1;
      unsigned int16 g4 : 1;
      unsigned int16    : 11;
   } bits;
} key_type;

static volatile key_type keys;  //will be initialized to 0

// the &key uses pass by reference to get same speed as a pointer
// but also avoids using pointer overhead in a lot of cases.  CCS does
// support this on some versions of its compilers.  You can use pointers
// if not though and just replace . operators with ->
void key_change(key_type &key);

void key_change(key_type &key){
   //I think it is ~...could be !...been a while
   key.bits.g0 = ~input(PIN_G0); 
   key.bits.g1 = ~input(PIN_G1);
   key.bits.g2 = ~input(PIN_G2);
   key.bits.g3 = ~input(PIN_G3);
   key.bits.g4 = ~input(PIN_G4);
}


Mind you this is off the top of my head without a compiler to test it, and it may not be the most efficient method. using the union gives the option of being able to use the values pin by pin or all at once as a single value. The order of the bits in the struct portion determines which order they are in the value portion. If I remember correctly, as done above, g0 will be bit0 of value, g1 will be bit1, etc.
Display posts from previous:   
Post new topic   Reply to topic    CCS Forum Index -> General CCS C Discussion All times are GMT - 6 Hours
Page 1 of 1

 
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