AVR Freaks

Any idea ? If I want to make macro for clear bit, from these define.

I have a very similar I/O question actually which is if i set:
DDRA = 0x15;
PORTA = 0x15;

so that the port is set to 00010101 (3 output pins hi asserted, 5 input ports lo) why is it that when i physically link say pin1 to pin2 (simple switch) to set pin2 high, and then try to read pin2 with:
either
if(PINA == 0x17) //00010111 binary
{
do something
}

or
if(PINA == 0b00010111) //0x17 hex
{
do something
}

i get something which compiles fine but never works when i use the switch, I even tried storing PINA into an integer and sending it to the LCD to see what was going on :] Any Ideas?

IF (PINA & 0b00000010)
{
}
i am afraid does not work and neither does the safer:
if((PORTA & 0b00000010)==1)
{
}
or

if((PINA & 0b00000010)==1)
{
}

or with && instead of &
i hope there is nothing wrong with my board :(

I take your point and it is a good one about not getting too distracted about what i am doing but i still fail to see what is wrong with code i am trying to make work like this for example:

DDRA = 0x00;
PORTA = 0x00;
bit_clear(DDRA,BIT(0)|BIT(1)|BIT(2)|BIT(3)|BIT(4)|BIT(5)|BIT(6)|BIT(7));
bit_clear(PORTA,BIT(0)|BIT(1)|BIT(2)|BIT(3)|BIT(4)|BIT(5)|BIT(6)|BIT(7));
bit_set(DDRA,BIT(0)|BIT(2)|BIT(4));
bit_set(PORTA,BIT(0)|BIT(2)|BIT(4));

bit_clear(PINA , BIT(1));
if(bit_get(PINA , BIT(1)) &= 0)
{
LCD4_Clear(); // Clear LCD Screen
LCD4_Home();
LCD4_PutS("switch on");

/* physical connect pins 0 and 1 */

delay_ms(1000); // delay loop
}

The code is a bit messy but it is for testing purposes of my switch between pin 1 and 0 only. Is the Imagecraft C (ICC) compiler i am using to blame or my novice skiils? If i just use:

if(bit_get(PINA , BIT(1)))
{
as before
}
it does not work either and i am at a loss as to why. More time on the simulator for me i think but I have found that the only reliable port commands so far are simple ones like DDRA = 0x03; and the like.

I managed to solve my problem not by programming, but instead by using a hardware solution - i.e. making my keypad switches switch between pins set to high and the ground pin. I would have prefered to have switches that switched between pins that were set to lo and pins set to high (i wanted to set the lo pins high with a connection to high or the high pins low with a connection to low) but for some reason that does not work on my board.

Thanks for the help anyways folks. :)

I'm not sure what you really want here. As such, the definitions in declare the ports /timers etc as variables, ie ; PORTB, so you access these like any other variable, so as to why you want to reassign these to another variable for the purpose of meaningful names I don't quite understand. Just doing a #define like shown at the beginning of this thread simply gives another name to , say , PORTB (as mentioned, the preprocessor does the text subsititution so the compiler itself sees the same thing).

As for renaming ports etc, that was discussed above.