Eingang bzw. Ausgangsbit als Variable deklarieren

[Daniel P. (dpointeck)]

Hallo zusammen,

für mich ist die AVR programmierung Neuland und wollte von euch wissen 
ob es möglich ist für Ein/Ausgangsbits Variablen zu vergeben?

z.B.:

statt:

if (PIND & (1<<PIND2))
{
 PORTB = 0x01;
}

so:

if(Taster == 1)
{
 LED =1;
}

weil so wäre der Code verständlicher

[ThomasL (Gast)]

Hi

Ja das geht, falls ich dich richtig verstehe...

#define Taster (PIND & (1<<PIND2))

Für die LED geht das dann genauso.

[Daniel P. (dpointeck)]

Hallo und danke,

ja stimmt schon.
Nur hab ich hier das Problem das der Taster hier als high abgespeichert 
wird.
ich will eigentlich nir für das 3.Bit am PORTD eine Variable definieren.

[Johannes M. (johnny-m)]

Daniel P. wrote:
> if (PIND & (1<<PIND2))
> {
>  PORTB = 0x01;
> }
>
> so:
>
> if(Taster == 1)
> {
>  LED =1;
> }
Nein. Geht so nicht, weil (PIND & (1<<PIND2)) nie gleich 1 ist! Es ist 
allerhöchstens gleich "1 << PIND2", also "1 << 2" oder 4.

Wenn Du allerdings das "==1" weglässt, dann wird (PIND & (1<<PIND2)) als 
Wahrheitswert ausgewertet, und dann klappt es. Also einfach den 
Vergleich weglassen.

[Johannes M. (johnny-m)]

Daniel P. wrote:
> Nur hab ich hier das Problem das der Taster hier als high abgespeichert
> wird.
Vestehe nicht ganz, was Du damit sagen möchtest.

[Das Tier (Gast)]

Nun, Taster gedrückt Pin ist high.

[Daniel P. (dpointeck)]

Also das ist so gemeint das ich eigentlich nur immer den Zustand des 3. 
Bit am PORTD abfragen will das wäre eigentlich nur 1 bzw. 0.
Dies funktioniert nicht weil es in der Realität 0 bzw. 4 ist.

Also sieht das ganze momentan so aus:

#define Taster (PIND &(1<<PD2))

int main(void)
{

  DDRD = 0x00;
  PORTD = 0xff;


  DDRB = 0xff;
  PORTB = 0x00;

  while (1)
  {
    if (Taster == 4)
    {
      PORTB = 0x00;
    }
    else
    {
      PORTB = 0x01;
    }

  }

}

für mich sehe aber das ganze schöner aus wenn ich den Taster mit
if (Taster == 1) abfragen könnte.

[yalu (Gast)]

Es gibt verschiedene Möglichkeiten, den Code transparenter zu machen:

Die einfachste besteht darin, für den Port und das Portbit
aussagekräftigere Namen zu vergeben:

#define TASTERPORT PIND
#define TASTERBIT  PIND2

if (TASTERPORT & (1<<TASTERBIT))
// ...

Man kann auch Makros definieren, über die auf die Portbits wie auf
Variablen zugeriffen werden kann, dazu muss man aber etwas weiter
ausholen:

// Allgemeine Definitionen:

struct bits {
  uint8_t b0:1;
  uint8_t b1:1;
  uint8_t b2:1;
  uint8_t b3:1;
  uint8_t b4:1;
  uint8_t b5:1;
  uint8_t b6:1;
  uint8_t b7:1;
};

#define BIT(r,n) (((volatile struct bits *)&r)->b##n)


// Konkrete Definitionen der einzelnen Portbits:

#define TASTER BIT(PIND,  2)
#define LED    BIT(PORTD, 0)


// Verwendung der Makros:

// ...
  if(TASTER)  // oder auch: if(TASTER == 1)
    LED = 1;

Obwohl das etwas komplizierter aussieht, erzeugt auch hier der GCC
optimalen Code, nämliche exakt zwei Befehle (einen für die Abfrage und
einen für die Zuweisung).

Das Ganze wurde schon einmal hier behandelt:

  Beitrag "sbit macro für avr-gcc"

Was die Lesbarkeit des Codes betrifft: Geübte Mikrocontrollerprogrammie-
rer haben sich schon so sehr an die Notation mit den Bitoperatoren (&,
|, ~ und <<) gewöhnt, dass auch die erste Variante sofort verstanden
wird.

[Michael A. (micha54)]

Hallo,

Abfagen auf (Taster=1) erledigt man in C besser mit (Taster!=0)
In dieser Version ist es wurst, welches bit tatsächlich benutzt wird.

Gruss,
Michael

[Daniel P. (dpointeck)]

Danke für die prompten Antworten.
am einfachsten ist wohl die Variante if(Taster != 0) abzufragen.
werde aber die anderen auch noch testen.

also Danke nochmal an alle

[Johannes M. (johnny-m)]

Michael Appelt wrote:
> Abfagen auf (Taster=1) erledigt man in C besser mit (Taster!=0)

if(Taster != 0)

 macht exakt dasselbe wie

if(Taster)

Und das hatte ich oben schon mal angesprochen. Wozu also der zusätzliche 
Schreibaufwand?

[max (Gast)]

wenn du umbedingt ==1 willst dann mach:

#define Taster ((PIND &(1<<PD2)) >>PD2)

  ;)

[Johannes M. (johnny-m)]

max wrote:
> wenn du umbedingt ==1 willst dann mach:
>
> #define Taster ((PIND &(1<<PD2)) >>PD2)
>
>   ;)
Auaaaa... Komplizierter geht's fast net.

#define Taster ((PIND >> PD2) & 1)

Wenn Du natürlich gerne irgendwelche Sachen hin- und herschiebst, möchte 
ich Dich nicht daran hindern;-)

[Klaus (Gast)]

Ich rate dir, diese Idee nicht weiter zu verfolgen. Auch wenn es 
zunächst ein wenig umständlich aussieht, wie in C die Pins gesetzt 
werden, so geht es nunmal. Daran gewöhnt man sich und dann ist es auch 
gar kein Problem mehr.

Man kann jetzt krampfhaft versuchen einen anderen Weg zu finden, anstatt 
es einfach zu akzeptieren, aber das is verschwendete Zeit ;)

[yalu (Gast)]

@Johannes M.:
Auch wenn's kommisch klingt: Die doppelte Schieberei von Max wird vom
Compiler komplett eliminiert und durch einen einfachen SBIS-Befehl
ersetzt, während bei deiner Variante der Shift zur Laufzeit ausgeführt
wird, was sind je nach Bitnummer bis zu vier Befehle zusätzlich
bedeutet.

[Johannes M. (johnny-m)]

yalu wrote:
> @Johannes M.:
> Auch wenn's kommisch klingt: Die doppelte Schieberei von Max wird vom
> Compiler komplett eliminiert und durch einen einfachen SBIS-Befehl
> ersetzt, während bei deiner Variante der Shift zur Laufzeit ausgeführt
> wird, was sind je nach Bitnummer bis zu vier Befehle zusätzlich
> bedeutet.
Hmmm, klingt komisch, is aber so...

Hast natürlich Recht! Bei meiner Variante sind dem Optimizer die Hände 
gebunden. Aber abgesehen davon würde ich immer noch einfach den 
(überflüssigen) Vergleich weglassen. Dann kann man sich das 
zurückschieben schlicht schenken.

[Peter D. (peda)]

Klaus wrote:

> Man kann jetzt krampfhaft versuchen einen anderen Weg zu finden, anstatt
> es einfach zu akzeptieren, aber das is verschwendete Zeit ;)

Das ist doch kein Krampf, das ist astreines ANSI-C.

Ich finde auch, seitdem ich es verwende, hat sich die Lesbarkeit 
deutlich verbessert.
Und Schreibaufwand spart es obendrein.

Hier noch ein Beispiel mit Bitvariablen nach Yalu:

http://www.mikrocontroller.net/attachment/30300/lcd_drv.zip


Peter