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

Bitte melde dich an, um einen Beitrag zu schreiben.

oder

Mit Google-Account einloggen

Die Registrierung ist kostenlos und dauert nur eine Minute.

Jetzt registrieren