sda = (d >> 7); für GCC

[Gast (Gast)]

Hallo,

ich versuche grade einen Beispielcode aus einem Datenblatt für meinen 
Mega8  umzuschreiben.
wie kann ich folgenden Codeschnipsel in eine GCC verständliche Form 
bringen?

sda = (d >> 7);

mfg und danke

Gast

[Benedikt K. (benedikt)]

Dies sollte beim gcc so funktionieren.

[Gast (Gast)]

Hallo,

ich schreib mal noch ein bischen mehr von meinem Code.

#define sda PC4
#define scl PC5


void i2c_write(unsigned char d)
{
  unsigned char i;

  PORTC &= ~(1<<PC5);
  for ( i=1; i<=8; i++)
    {
      sda = (d >> 7);
      PORTC |= (1<<scl);
      d = d << 1;
      PORTC &= ~(1<<scl);
    }
  PORTC |= (1<<sda);
  PORTC &= ~(1<<scl);
  PORTC |= (1<<scl);
}

als Fehelermeldung bekomme ich "error: invalid lvalue in assignment" für 
die Zeile mit "sda = (d >> 7);"
Alles andere scheint zumindest beim Comilieren Fehlerfrei zu sein.
Der Beispielcode aus dem Datenblatt ist Folgender:

void I2C_write(uchar d)
{
uchar i;
scl = 0;
for (i = 1; i <= 8; i++)
{
sda = (d >> 7);
scl = 1;
d = d << 1; /* increase scl high time */
scl = 0;
}
sda = 1; /* Release the sda line */
scl = 0;
scl = 1;
if(sda) printf("Ack bit missing %02X
",(unsigned int)d);
scl = 0;
}

[Thomas (Gast)]

sda ist keine Variable. Du kannst somit keinen Wert zuweisen.

[Peter D. (peda)]

Beim AVR mußt Du für Eingänge und Ausgänge verschiedene Register nehmen 
(PINx, PORTx).
Open Drain Ausgänge realisiert man mit dem DDRx-Register

Um Bitvariablen zu definieren, muß man unter GCC Bitfelder benutzen, 
dann gehts ganz einfach wie unter Keil C51:

#include <avr\io.h>
#include <inttypes.h>
#include <stdio.h>


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;
} __attribute__((__packed__));


#define SBIT(port,pin) ((*(volatile struct bits*)&port).b##pin)


#define scl     SBIT(PORTC, 5)
#define sda     SBIT(DDRC, 4)
#define sdain   SBIT(PINC, 4)
#define sdalevel SBIT(PORTC, 4)

#define uchar unsigned char


void I2C_write(uchar d)
{
uchar i;
sdalevel = 0;
scl = 0;
for (i = 1; i <= 8; i++)
{
sda = !(d >> 7);
scl = 1;
d = d << 1; /* increase scl high time */
scl = 0;
}
sda = 0; /* Release the sda line */
scl = 0;
scl = 1;
if(sdain) printf("Ack bit missing %02X",(unsigned int)d);
scl = 0;
}

Peter


P.S.:

d = d << 1; /* increase scl high time */

Dürfte beim AVR nicht reichen.
Außerdem verschiebt der AVR-GCC solche Intruktionen, da sie seiner 
Meinung nach keinen Einfluß auf die Ausführung haben.
IO-Zugriffe stellen leider keine Barrieren für Reordering dar.

[Volker (Gast)]

Peter hats zwar schon vorweggenommen, aber schau auch mal hier:

Beitrag "sbit macro für avr-gcc"


Gruß Volker