definieren von Portpins mit #define

[Kenia (Gast)]

Veruch mich grad im Programmieren und will in C die Portpins als 
eingänge mit #define im AVR Studio definieren.

Dies kann doch auf diesem Weg umgestzt werden:

#define INPUT_A PINB0

[Peter II (Gast)]

Kenia schrieb:
> Dies kann doch auf diesem Weg umgestzt werden:
>
> #define INPUT_A PINB0

nö, weil du damit überhaupt kein code erzeugst.

[Karl H. (kbuchegg)]

@Kenia

Lies dir das mal durch

http://www.mikrocontroller.net/articles/C-Präprozessor

[Falk B. (falk)]

http://www.mikrocontroller.net/articles/Soft-PWM#Erster_Versuch

[Kenia (Gast)]

Es geht darum später in der while schleife mit diesen  neuen Ausdrücken 
anstatt PINA0 PINA1 PINA2.. zu arbeiten

#define LaserA_Input PINA0
#define Messergebnis PORTB0


Also
statt
...
while
{
if (PINA0 ==1)
PORTB0 ==1;
}


will ich schreiben

while
{
if (LaserA_Input== 1)
Messergebnis ==1;
}

[troll (Gast)]

Die Grundidee ist richtig, allerdings kann zu mindestens der GCC nicht 
direkt auf Pins zugreifen. Du musst also eher sowas machen:

#define PortTaster PINB
#define PinTaster PINB0 //oder wie das auch definiert sein mag, eben die Bitnummer

if(PortTaster&(1<<PinTaster))
{
tu_was_sinnvolles
}

Oder hier gucken: Beitrag "Re: "Syntaxbefehl" erstellen"

[troll (Gast)]

Kenia schrieb:
> if (PINA0 ==1)
> PORTB0 ==1;
Uhhh, Vorsicht!
Vergleiche: ==
Zuweisungen: =

(Der Compiler sollte aber ne Warnung ausspucken.)

[..... (Gast)]

Oh Gott... :P

[Fabian O. (xfr)]

Definiere Dir lieber Inline-Funktionen mit klaren Namen, das ist 
lesbarer und auf jeden Mikrocontroller portabel:

static inline bool laser_is_on(void) {
  return PINB & (1 << PINB0);
}

static inline void set_result_pin(void) {
  PORTA |= (1 << PA0);
}

static inline void clear_result_pin(void) {
  PORTA &= ~(1 << PA0);
}

[...]

if (laser_is_on()) {
  set_result_pin();
}

[Falk B. (falk)]

Man kann es auch mit einem einfachen #define machen, etwa so

#define LASER_ON (PIND & (1<<PD4))    // #defines immer GROSS
#define RELAY_ON (PORTD |= (1<<PD3));
#define RELAY_OFF (PORTD &= ~(1<<PD3));

if (LASER_ON) {
  RELAY_ON
} else {
  RELAY_OFF
}

Einfach lesbar und selbsterklärend. Und wenn sich mal die Pinzuordung 
ändert, muss man nur EINMAL die defines ändern, der Rest vom Quelltext 
bleibt wie er ist.

[Bastler (Gast)]

Ich kann Falk Brunner nur zustimmen!

Irgendwann steigt man auf einen anderen Controller um.
Dann kann man den Code problemlos übernehmen und muss nur ein paar 
#defines ändern.

Ich hatte es so gemacht, der Umzug einer GLCD Ansteuerung dauerte nur 2 
Stunden.