Elegante Pinabstraktion

Du hast mich nicht richtig verstanden.

Ich möchte die pins nur "Maskieren"

nicht nur für
LCD_LED = 1
sondern auch für
if(LCD_LED == 1)
{


Gruß Franz

nichtmal

#define LCD_cmd  PORTA
#define LCD_nRST 0

LCD_data |= (1 << LCD_nRST);

funktioniert obwohl man es über die suche oft so findet.

invalid operands to binary & (have 'PORT_t' and 'int')

Kann mir niemand helfen?

Gruß Franz

Hier ein paar Möglichkeiten:

http://www.nongnu.org/avr-libc/user-manual/FAQ.html#faq_port_pass

Franz schrieb:
> ... hast mich nicht richtig verstanden.
im ersten Teil hattest Du nichts anderes gefragt

> nicht nur für
> LCD_LED = 1
> sondern auch für
> if(LCD_LED == 1)

Codevision kann das. Macht den Code leider nicht besonders portierbar 
aber schön übersichtlich.

Rudi

Franz schrieb:
> nichtmal
>
> #define LCD_cmd  PORTA
> #define LCD_nRST 0
>
> LCD_data |= (1 << LCD_nRST);
>
> funktioniert obwohl man es über die suche oft so findet.
>
> invalid operands to binary & (have 'PORT_t' and 'int')
>
> Kann mir niemand helfen?
>
> Gruß Franz

Das versteh ich nicht ganz, LCD_data ist nirgendwo definiert.

1

#define LCD_DATA PORTA

2

#define LCD_NRST PA0

3

4

LCD_DATA |= (1 << LCD_NRST);

sollte aber funktionieren.

Für GCC, Verwendung:

1

#define SOME_PORT BITWISE_CHAR(PORTD, 3)

2

3

/* ... */

4

SOME_PORT = 1;

5

if (SOME_PORT == 1)

6

  ;

Mini Float schrieb:
> Aber gut, wenn ihr nicht wollt, dass euer Code portabel ist, dann eben
> nicht.
Über 'Portabilität' kann man sich in diesem Fall nun freilich drüber 
streiten...

Mit noch mehr Präprozessor kann man es portabel eindampfen:

1

#ifndef IO_H

2

#define IO_H

3

4

#include <avr/io.h>

5

6

/* IO ports */

7

#define IO_STRAIGHT    0

8

#define IO_INVERT      1

9

10

#define BUZZER    B, B1, IO_STRAIGHT

11

#define RELAIS    B, B5, IO_STRAIGHT

12

13

#define __IO_SET(port, bit, invert, state)      \

14

  if ( (invert) ^ ((state) ? 1 : 0) ) {         \

15

    PORT##port |= _BV(PORT##bit);               \

16

  }                                             \

17

  else {                                        \

18

    PORT##port &= ~_BV(PORT##bit);              \

19

  }

20

21

#define __IO_GET(port, bit, invert)             \

22

  ( (PORT##port & _BV(PORT##bit)) ? (1 ^ (invert)) : (0 ^ (invert)) )

23

24

25

#define IO_SET(port, state)    __IO_SET(port, state)

26

#define IO_GET(port)           __IO_GET(port)

27

28

#endif

Verwendung:

1

IO_SET(BUZZER, 1);

2

IO_SET(RELAIS, 0);

Und bitte, keine weiteren Kommentare dazu :-}

A. K. schrieb:
> Inwiefern ist io.h eigentlich essentiell abhängig von GCC? Das
> Inline-Verbot ist für die Frage hier irrelevant und die structs müssten
> eigentlich auch ohne "packed" schon frei von Löchern sein. Der Rest ist
> normales C.
Naja, das ist eigentlich der einzige Knackpunkt.

> Dass Bitfields allgemein als schwach portabel gelten ist klar, aber ganz
> so schlimm ist es dann doch nicht, wenn man damit Portpins anspricht und
> damit die Architektur sowieso festnagelt.
Darauf zielte ich nämlich weiter oben schon ab. Einen Abstraktionslayer 
zu abstrahieren, ist ziemlich sinnfrei. Irgendwo muss man ja an die 
Hardware anbauen.

Reine Lehre wäre dann wohl nur die vollständige Kapselung in Methoden, 
sodass man schlimmstenfalls noch mit etwas Programmlogik nachhelfen 
kann, wenn die Zuordnung der Ports nicht mehr direkt gegeben ist.