Variable Register

Gast #500180
Lesenswert?
• ▲
▼
Hallo,

mein eigentliches Problem ist folgendes:

/*********************************************************************** 
*********************
                                  Definitionen:
************************************************************************ 
*********************/
#ifndef F_CPU
#define F_CPU 4000000
#endif


typedef unsigned char byte;

typedef struct
              {
        unsigned char rs  :1;    //PB0
        unsigned char rw  :1;    //PB1
        unsigned char en  :1;    //PB2
        unsigned char data:4;    //PB3-PB6
        }bit;


typedef union
              {
        byte  ports;
        bit   pin;
        }portb;

portb port;


/*********************************************************************** 
*********************
                                  Bibliotheken:
************************************************************************ 
*********************/
#include <avr/interrupt.h>
#include <stdlib.h>
#include <avr/io.h>
#include <util/delay.h>

/*********************************************************************** 
*********************
                                  Main-Funktion
************************************************************************ 
*********************/
int main (void)
{
unsigned int i;
DDRB=0xFF;
PORTB=port.ports;
while(1)
{
port.pin.rw=   1;
port.pin.rs=   1;
port.pin.en=   1;
port.pin.data=15;
PORTB=port.ports;
for(i=0;i<=100;i++)
{
_delay_ms(50);
}
port.pin.rs=  0;
port.pin.en=  0;
port.pin.rw=  0;
port.pin.data=0;
PORTB=port.ports;
for(i=0;i<=100;i++)
{
_delay_ms(50);
}
}
return 0;
}

Das Programm funktioniert nur dann, wenn ich jeweils nach dem ein/aus 
schalten der Bitfelder, die Zeile "PORTB=port.ports;" einfüge.
Ich wollte eigentlich diese Zeile global definieren, so dass ich sie 
nicht jeweils nach dem Schalten der Bitfelder ergänzen muss.

Hat jemand eine Idee?
Danke
#500181
Lesenswert?
• ▲
▼
Rufus t. Firefly wrote:
> Das wäre nur mit viel Aufwand und einem Linkerskript möglich, lässt sich
> aber mit einem #define viel einfacher hinpfuschen:
>
>    #define MeineVariable PORTB
>
> Allerdings ist von beiden Vorgehensweisen DEUTLICH abzuraten. Warum soll
> man im Sourcecode Portzugriffe verschleiern? Was spricht dagegen, PORTB
> auch so zu nennen?

Na ja, so dumm finde ich das gar nicht.

#define LCD_PORT PORTA
#define LCD_DDR  DDRA

int main()
{
  LCD_DDR = 0xFF;
  LCD_PORT = 0xFF;
}

hat ja doch einen gewissen dokumentarischen Zweck. Von der
Einfachheit einer möglichen Umstellung mal abgesehen.

Und manchmal, aber nur manchmal, kommt man dann tatsächlich in
die Verlegenheit eine Funktion schreiben zu müssen, der man
einen beliebigen Port übergibt und die auf diesem übergebenen
Port arbeitet. Dann übergibt man einen Zeiger.
Steht aber alles in der Doku zur avr-lib

RTFM
#500189
Lesenswert?
• ▲
▼
Ja.
Schreib dir eine Funktion dafür
1
void SetAll( uint8_t rw, uint8_t rs, uint8_t en, uint8_t data )
2
{
3
  port.pin.rs   =  rs;
4
  port.pin.en   =  en;
5
  port.pin.rw   =  rw;
6
  port.pin.data = data;
7

8
  PORTB = port.ports;
9
}
10

11
int main (void)
12
{
13
  unsigned int i;
14

15
  DDRB=0xFF;
16

17
  while(1)
18
  {
19
    SetAll( 1, 1, 1, 15 );
20
    for(i=0;i<=100;i++)
21
    {
22
      _delay_ms(50);
23
    }
24

25
    SetAll( 0, 0, 0, 0 );
26
    for(i=0;i<=100;i++)
27
    {
28
      _delay_ms(50);
29
    }
30
  }
31
  return 0;
32
}

Sieht doch (in main()) gleich besser aus.
#500194
Lesenswert?
• ▲
▼
Da du allerdings sowieso den kompletten Port neu
setzt, kannst du dir das ganze Bitfeld-Gerödel auch
sparen und ganz einfach machen:
1
void SetAll( uint8_t rw, uint8_t rs, uint8_t en, uint8_t data )
2
{
3
  PORTB |=  ( rs << PB) | ( rw << PB1 ) | ( en << PB2 ) | ( data << PB3 );
4
}

Bitfelder werden IMHO kräftig überschätzt, während Funktionen
kräftig unterschätzt werden.

Antwort schreiben

Hinweis: der ursprüngliche Beitrag ist mehr als 6 Monate alt. Bitte hier nur auf die ursprüngliche Frage antworten, für neue Fragen einen neuen Beitrag erstellen.

Formatierung
  • [c]C-Code[/c]
  • [code]Code in anderen Sprachen, ASCII-Zeichnungen[/code]
  • [math]Formel in LaTeX-Syntax[/math]
  • [[Titel]] – Link zu einem Artikel
  • Verweis auf einen anderen Beitrag: Rechtsklick auf dessen Beitrags-ID (#8078628), „Adresse kopieren“, in den Text einfügen

Mehr zur Formatierung

Längeren Sourcecode als Anhang, nicht in den Text. JPG nur für Fotos und Scans; Zeichnungen und Screenshots als PNG oder GIF. Siehe Bildformate.

Schon registriert? Anmelden

Wird nicht angezeigt. Wir schicken dir einen Link; sobald du damit bestätigst, erscheint dein Beitrag.

Mit dem Abschicken legst du einen kostenlosen Account an und erkennst die Nutzungsbedingungen und die Datenschutzerklärung an.