Probleme mit GY-271 (HMC5883) Compass

[Bence (Gast)]

Ich bin Anfanger und habe Probleme mit gy-271 Modul(based on HMC5883 
chip). Der Wert der y-Achse ist immer -4096. Auserdem, trotz delay ist 
nur 100ms, ist der Program sehr langsam. Bitte hilf mir. Ich habe schon 
keine Idee mehr. Zum TWI benutzte ich Peter Fleurys I2C Master Library 
(http://homepage.hispeed.ch/peterfleury/doxygen/avr-gcc-libraries/group__pfleury__ic2master.html). 
Der Serialausgang sieht so aus:
...
X: 172
Y: -4096
Z: 415
X: 172
Y: -4096
Z: 413
X: 170
Y: -4096
Z: 415
...
Hat jemanden eine Idee, wie könnte ich ermitteln, ob der Chip 
kaputgegangen ist oder noch funktioniert. Das Datasheet befindet man 
hier: https://www.seeedstudio.com/wiki/images/4/42/HMC5883.pdf
Meine Code:

/*
 * twi.c
 *
 * Created: 2015.07.17. 15:43:34
 *  Author: Bence
 */ 

#define F_CPU 16000000
#define BAUD 9600
#define MYUBRR F_CPU/BAUD/16-1
#define rx_buffer_size 32
#define tx_buffer_size 32
#define compass_write 0x3C
#define compass_read 0x3D

#include <avr/io.h>
#include "i2cmaster.h"
#include <avr/interrupt.h>
#include <stdio.h>
#include <util/delay.h>


unsigned char rx_buffer[rx_buffer_size];
unsigned char tx_buffer[tx_buffer_size];
volatile unsigned char rx_read_pos=0;
volatile unsigned char tx_write_pos=0;
volatile unsigned char tx_load_pos=0;
volatile unsigned char available=0;

void initUSART();
char serialRead();
int serialWrite(char c, FILE *stream);
void delay(unsigned int ms);
void compass_init();

FILE uart_io= FDEV_SETUP_STREAM(serialWrite, serialRead, _FDEV_SETUP_RW);

int main(void)
{
  sei();
  initUSART();
  i2c_init();
  compass_init();
  int x=0;
  int y=0;
  int z=0;
    while(1)
    {
    i2c_start_wait(compass_write);
    i2c_write(0x03);
    i2c_rep_start(compass_read);
    
    x=(i2c_readAck()<<8);
    x|=i2c_readAck();
    
    y=((char)i2c_readAck()<<8);
    y|=(char)i2c_readAck();
    
    z=(i2c_readAck()<<8);
    z|=i2c_readNak();
    
        i2c_stop();
    
    printf("X: %i\n",x);
    printf("Y: %i\n",y);
    printf("Z: %i\n",z);
    delay(100);
    }
}

void initUSART()
{
  UBRR0H=((MYUBRR)>>8);
  UBRR0L=MYUBRR;
  
  UCSR0B=(1<<RXEN0)|(1<<RXCIE0)|(1<<TXEN0)|(1<<UDRIE0);
  UCSR0C=(1<<UCSZ00)|(1<<UCSZ01);
  stdin  = stdout = &uart_io;
}

void delay(unsigned int ms)
{
  for (unsigned int i=0;i<ms;i++)
  {
    _delay_ms(1);
  }
}

char serialRead()
{
  char c=0;
  while(available<=0);
  if(rx_read_pos>=available)
  {
    c=rx_buffer[rx_read_pos-available];
  }
  else
  {
    c=rx_buffer[rx_buffer_size+rx_read_pos-available];
  }
  available--;
  return c;
}

int serialWrite(char c, FILE *stream)
{
  if (c == '\n')
  {
    serialWrite('\r', 0);
  }
  tx_buffer[tx_load_pos]=c;
  tx_load_pos++;
  
  if (tx_load_pos>=tx_buffer_size)
  {
    tx_load_pos=0;
  }
  return 0;
}


void compass_init()
{
  i2c_start(compass_write);
  i2c_write(0x00);
  i2c_write(0b0001000);
  i2c_stop();

  i2c_start(compass_write);
  i2c_write(0x01); 
  i2c_write(0b00100000);
  i2c_stop();

  i2c_start(compass_write);
  i2c_write(0x02);
  i2c_write(0x00);
  i2c_stop();
}

ISR(USART_RX_vect)
{
  rx_buffer[rx_read_pos]=UDR0;
  rx_read_pos++;
  
  if(rx_read_pos>=rx_buffer_size)
  {
    rx_read_pos=0;
  }
  
  available++;
}

ISR(USART_UDRE_vect)
{
  if(tx_write_pos!=tx_load_pos)
  {
    UDR0=tx_buffer[tx_write_pos];
    tx_write_pos++;
    
    if (tx_write_pos>=tx_buffer_size)
    {
      tx_write_pos=0;
    }
  }
}

Danke für die Hilfe im voraus.

[Bence (Gast)]

Ich benutze atmega328p.

[Alex R. (itaxel)]

Bence schrieb:

y=((char)i2c_readAck()<<8);
y|=(char)i2c_readAck();

Hallo Bence,

entferne mal das "(char)" und schaue was dann rauskommt.

gr. Alex

[Bence (Gast)]

Ich hatte es nachträglich ausprobiert, aber es funktioniert so auch 
nicht, und es bleibte in meiner Code.

[Karl H. (kbuchegg)]

Bence schrieb:
> Ich hatte es nachträglich ausprobiert, aber es funktioniert so auch
> nicht, und es bleibte in meiner Code.

Er ist trotzdem keine gute Idee.
Ein unnötiger Cast sollte nie im Code sein. Im besten Fall bringt er 
nichts, im schlimmsten Fall kann er sich zu einer bösen Falle ausarten.

Unglücklicherweise löst das dein Problem mit den immer gleichen Werten 
nicht.
Das ist sowieso eigenartig, denn laut Datenblatt kann es ein Ergebnis 
von -4096 überhaupt nicht geben. Der Wertebereich läuft von -2048 bis 
+2047. -4096 ist da ausserhalb.

> Auserdem, trotz delay ist nur 100ms, ist der Program sehr langsam.

Das ist logisch.
Dein µC arbeitet sich an der Bearbeitung des UART Data Register Empty 
Interrupts zu Tode. Das ist ein bischen zu naiv angesetzt.
Beheb das erst mal, denn solange du den µC mit diesen Interrupts 
zumüllst, hast du nirgends ein vernünftiges Timing.
Wenn du schon beim Fleury warst, der hat auch schöne UART Funktionen.

[Bence T. (bence)]

Danke ich werde Fleurys UART library ausprobieren.
Ich habe im Datenblatt habe ich etwas über das Wert -4096 gefunden,aber 
ich weiß es nicht, wie ich damit diese Probleme lösen könnte.

"In the event the ADC reading overflows or underflows for the given 
channel,or if there is a math overflow during the bias measurement, this 
data register  will contain the value -4096 in 2’s complement form. This 
register value will clear when after the next valid measurement is 
made."

Ich verstehe nicht,wie das ADC Lesen überlaufen oder unterlaufen kann, 
und warum es keine gültige Messung machen kann.