LED1 OK. aber LED2 blinkt nicht mit voller Helligkeit

Mach aus

> DDRB = (1 << PB0);

mal

> DDRB = (1 << PB0) | (1 << PB1);

Da hätte man aber auch selbst drauf kommen können.

Super, vielen Dank
Bin/war blind.

Klasse....

Divide et impera

Mache 2 Tasks daraus:

1

#define F_CPU 1.2e6

2

3

#include <util/delay.h>

4

#include "sbit.h"

5

6

#define LED0    PORT_B0

7

#define LED0_oe  DDR_B0

8

#define LED1    PORT_B1

9

#define LED1_oe  DDR_B1

10

11

void task_0( void )

12

{

13

  static uint16_t cnt_10ms;

14

  switch( ++cnt_10ms ){

15

    case 100:   LED0 = 0;

16

                break;

17

    case 300:   LED0 = 1;

18

                cnt_10ms = 0;

19

                break;

20

  }

21

}

22

23

void task_1( void )

24

{

25

  static uint16_t cnt_10ms;

26

  switch( ++cnt_10ms ){

27

    case 250:   LED1 = 0;

28

                break;

29

    case 350:   LED1 = 1;

30

                cnt_10ms = 0;

31

                break;

32

  }

33

}

34

35

int main()

36

{

37

  LED0_oe = 1;

38

  LED1_oe = 1;

39

  for(;;){

40

    _delay_ms( 10 );      // main timebase: 10ms

41

    task_0();

42

    task_1();

43

  }

44

}

Wow, dankeschön Peter und sogar die sbit.h mitgesandt.
Das ist Profi

Bin gespannt und berichte gleich mal...

Danke Hilde,
bin in C absolut Neu und gebe dir Recht
Allerdings finde ich Peters Code auch übersichtlich, wenn ich denn den 
Code in einigen Jahren evtl. mal verstehen werde.:)

Danke für deine Gedanken
lg

meckermann schrieb:
> Klammer-Ebenen zuviel
> wird dann ist es echt Chaos.

Wenn man statt edlin einen vernünftigen Editor verwendet, bleiben beide 
Varianten und nach 4711 Schachtelebenen übersichtlich.

SCNR

P.S.: Ich bevorzuge auch Peter Variante.

meckermann schrieb:
> Bleibe konsistent beim Setzen der Klammer-Ebenen.

Ich benutze den Klammerungsstil als Unterscheidungsmerkmal.
Nur eine Funktion hat die '{' in einer extra Zeile.
Dadurch kann ich leichter erkennen, wann eine neue Funktion beginnt und 
wann nur ein conditional Block darin.

Joey N. schrieb:
> wenn ich denn den
> Code in einigen Jahren evtl. mal verstehen werde.:)

Warum so pessimistisch.
Frag ruhig, wenn Du etwas nicht verstehst.
Ich habe ihn absichtlich nicht kommentiert.
Ich hab auch nicht versucht den Zeitablauf aus Deinem Code zu 
extrahieren.
Ich denke aber, daß das Prinzip klar wird, wie man mehrere Tasks 
gleichzeitig und unabhängig voneinander ausführen kann.

Hallo Peter, Entschuldigung, das war  keine Kritik an dich.
Ich wollte nur zum Ausdruck bringen, das ich Neu bin und gerne so 
programmieren könnte wie du.


Das war sowieso mein nächstes Ziel, wie ich die  LEDs unabhängig 
voneinander ansteuern kann.

Ich bedanke ich mich nochmals für den tollen Code und frohes Neues.


lg

HildeK schrieb:

> #define SBIT_(port,pin) ((*(volatile struct bits*)&port).b##pin)
> #define SBIT(x,y)       SBIT_(x,y)
> [/c]
> speziell mit den Ausdrücken:
>

1

>  __attribute__((__packed__))

2

>

der Zweck von 'packed' besteht darin, dass der COmpiler nicht auf blöde 
Ideen kommt, und zwischen die einzelnen Strukturmember in

1

struct bits {

2

    uint8_t b0:1, b1:1, b2:1, b3:1, b4:1, b5:1, b6:1, b7:1;

3

} __attribute__((__packed__));

noch Alignment Bytes hineinpackt, weil er dir schnellstmöglichen zugriff 
auf die Member ermöglichen will. An dieser Stelle wollen wir den 
Compiler dazu zwingen, dass er alle 8 Bit Member in genau 1 Byte packt.

>

1

> ((*(volatile struct bits*)&port).b##pin)

2

>

https://gcc.gnu.org/onlinedocs/cpp/Concatenation.html

## 'hängt' einfach nur 2 Tokens zusammen und generiert daraus einen 
neuen Token.
Steht 'pin' für die Zahlenkonstante 5, dann generiert 'b##pin' daraus 
'b5'.

Hilde hats auch echt drauf, danke