Gameboy Camera READ Problem

[Nicole F. (Gast)]

Hallo,

Mit folgendem Code versuche ich ein Bild aus der GameBoy Camera zu 
holen, leider vergeblichst. Denn der READ Pin wird von der Kamera nich 
hochgezogen.

while(!(PIND&(1<<CAM_READ)))
  {
    
    CLK_LOW();
    camStepDelay();
    CLK_HIGH();
    camStepDelay();
  }

Vielleicht hat jemand eine Idee woran es liegen könnte.

#define CAM_START  2
#define CAM_SIN    3
#define CAM_LOAD  4
#define CAM_RESET  5
#define CAM_CLK    6
#define CAM_READ  7

#define START_HIGH()  PORTD|=(1<<CAM_START)
#define SIN_HIGH()    PORTD|=(1<<CAM_SIN)
#define LOAD_HIGH()    PORTD|=(1<<CAM_LOAD)
#define RESET_HIGH()  PORTD|=(1<<CAM_RESET)
#define CLK_HIGH()    PORTD|=(1<<CAM_CLK)
#define READ_HIGH()    PORTD|=(1<<CAM_READ)


#define START_LOW()    PORTD&=~(1<<CAM_START)
#define SIN_LOW()    PORTD&=~(1<<CAM_SIN)
#define LOAD_LOW()    PORTD&=~(1<<CAM_LOAD)
#define RESET_LOW()    PORTD&=~(1<<CAM_RESET)
#define CLK_LOW()    PORTD&=~(1<<CAM_CLK)
#define READ_LOW()    PORTD&=~(1<<CAM_READ)

void cam_reset(void)
{
  RESET_LOW();
  CLK_LOW();
  camStepDelay();
  CLK_HIGH();
  camStepDelay();
  CLK_LOW();
  RESET_HIGH();
  camStepDelay();
  RESET_LOW();

}

void cam_write(unsigned char reg, unsigned char data){
  char i;


  for(i=0;i<3;i++)
  {
    CLK_LOW();
    
    if((reg & 0x04 ) > 0)
    SIN_HIGH();
    else
    SIN_LOW();
    reg = (reg<<1);
    camStepDelay();
    CLK_HIGH();
    camStepDelay();
    
  }


  for(i=0;i<8;i++)
  {
    CLK_LOW();

    if((data & 0x04 ) > 0)
    SIN_HIGH();
    else
    SIN_LOW();
    data = (data<<1);

    
    camStepDelay();
    CLK_HIGH();
    camStepDelay();
  }
  LOAD_HIGH();
  camStepDelay();
  CLK_LOW();
  camStepDelay();
  LOAD_LOW();
  SIN_LOW();
  
}

void cam_getphoto(void)
{
  cam_reset();

  cam_write(0,0x80);
  cam_write(1,0x0E);
  cam_write(2,0x06);
  cam_write(3,0x00);
  cam_write(4,0x01);
  cam_write(5,0x00);
  cam_write(6,0x01);
  cam_write(7,0x07);

  cam_start();
  
  cam_readsend();
}

void cam_start(void)
{
  
  
  START_HIGH();
  camStepDelay();
  CLK_HIGH();
  camStepDelay();
  START_HIGH();
  CLK_LOW();
  
  
  
  while(!(PIND&(1<<CAM_READ)))
  {
    
    CLK_LOW();
    camStepDelay();
    CLK_HIGH();
    camStepDelay();
  }
  
}


void cam_readsend(void){
  uint8_t pixel,line,value;
  

  for(line=0;line<123;line++)
  {
    for(pixel=1;pixel<129;pixel++)
    {
      CLK_LOW();
      
      //value = ADC_Read(5);
      char Buffer[7];
      itoa(ADC_Read(0),Buffer,10);
      uart_puts(Buffer);
      uart_putc(';');
      
      camStepDelay();
      CLK_HIGH();
      camStepDelay();
      
      
    }
  }
  uart_putc('!');
}

int main(void)
{
  

  DDRD |= (1<<CAM_START);
  DDRD |= (1<<CAM_SIN);
  DDRD |= (1<<CAM_LOAD);
  DDRD |= (1<<CAM_RESET);
  DDRD |= (1<<CAM_CLK);
  DDRD &= ~(1<<CAM_READ);


  
  uart_init( UART_BAUD_SELECT(UART_BAUD_RATE,F_CPU) );
  ADC_Init();
  sei();
  
  //sendData(2);
  
  cam_getphoto();
  
  while(1)
  {
    //TODO:: Please write your application code
  }
}

[Magnus M. (magnetus)]

[OT]
Vorbildlich kommentierter Code...
[/OT]

[Peter II (Gast)]

Magnus M. schrieb:
> Vorbildlich kommentierter Code...

und ziemlich unvollständig. Viele Funktionen Fehler und auch die 
globalen Variablen.

[Nicole F. (Gast)]

Peter II schrieb:
> Magnus M. schrieb:
>> Vorbildlich kommentierter Code...
>
> und ziemlich unvollständig. Viele Funktionen Fehler und auch die
> globalen Variablen.

Was fehlt denn?

OK, das hier hätte noch reinmüssen

unsigned char camClockSpeed = 0x0A;



void camStepDelay(){
  unsigned char u=camClockSpeed;
  while(u--) {__asm__ __volatile__ ("nop");}
}

Der Mega32 wird mit 16MHz getaktet

[Peter II (Gast)]

Nicole F. schrieb:
> Was fehlt denn?

ADC_Read

[Nicole F. (Gast)]

Peter II schrieb:
> Nicole F. schrieb:
>> Was fehlt denn?
>
> ADC_Read

Da liegt aber nicht das Problem. Mein Problem ist schon weit vorher. Die 
Kamera signalisiert mir nicht, das ich was zum lesen habe.
Also ist das jetzt erstmal unwichtig.

[Magnus M. (magnetus)]

Nicole F. schrieb:
> void cam_reset(void)
> {
>   RESET_LOW();
>   CLK_LOW();
>   camStepDelay();
>   CLK_HIGH();
>   camStepDelay();
>   CLK_LOW();
>   RESET_HIGH();
>   camStepDelay();
>   RESET_LOW();
>
> }

Dir ist aber schon klar, dass der RESET low-active ist?

[Nicole F. (Gast)]

Magnus M. schrieb:
> Dir ist aber schon klar, dass der RESET low-active ist?

Stimmt, steht auf Seite 2 im Datenblatt. Das hatte ich ja total 
übersehen.
DANKE!

void cam_reset(void)
{
  RESET_LOW();
  CLK_LOW();
  camStepDelay();
  CLK_HIGH();
  camStepDelay();
  CLK_LOW();
  RESET_HIGH();
  

}

Jetzt komme ich auch aus der While Schleife raus.
Aber irgendwie habe ich nur Müll als Ergbenis.

@ Peter. Jetzt sind wir beim ADC_READ angelangt

uint16_t ADC_Read( uint8_t channel )
{
  
  ADMUX = (ADMUX & ~(0x1F)) | (channel & 0x1F);
  //ADMUX = channel;
  
  ADCSRA |= (1<<ADSC);            // eine Wandlung "single conversion"
  while (ADCSRA & (1<<ADSC) ) {   // auf Abschluss der Konvertierung warten
  };
  return ADCW;                    // ADC auslesen und zurückgeben
}

[Nicole F. (Gast)]

Ich glaube meine cam_write Funktion  ist fehlerhaft

[Magnus M. (magnetus)]

Nicole F. schrieb:
> @ Peter. Jetzt sind wir beim ADC_READ angelangt

Und wo ist ADC_Init()?

[Peter II (Gast)]

  for(i=0;i<8;i++)
  {
    CLK_LOW();

    if((data & 0x04 ) > 0)
    SIN_HIGH();
    else
    SIN_LOW();
    data = (data<<1);

    
    camStepDelay();
    CLK_HIGH();
    camStepDelay();
  }
[c]

ich habe keine Ahnung von der  Gameboy Camera, aber der code ist merkwürdig. Hier werden vermutlich 8 bit gesendet. Warum prüft man da auf

[c]
if((data & 0x04 ) > 0)

üblicherweise wird das erste oder letzte bit von data geprüft.

[Magnus M. (magnetus)]

Peter II schrieb:
> ich habe keine Ahnung von der  Gameboy Camera, aber der code ist
> merkwürdig. Hier werden vermutlich 8 bit gesendet. Warum prüft man da
> auf
>
> if((data & 0x04 ) > 0)
>
> üblicherweise wird das erste oder letzte bit von data geprüft.

Das wollte ich auch gerade bemängeln. Sie(?) hat da wohl beim 
Programmieren wohl ein Copy&Paste aus der Registerschreibschleife ein 
paar zeilen darüber gemacht und vergessen die 0x04 nach 0x80 zu ändern. 
Dies sollte Sie natürlich spätestens jetzt nachholen!

Noch ein Tipp:

if((reg & 0x04 ) > 0)

Schreib das doch einfach als

if(reg & 0x04 )

Selbiges trifft natürlich auf die (data & 0x80) zu.

[Nicole F. (Gast)]

Magnus M. schrieb:
> Das wollte ich auch gerade bemängeln. Sie(?) hat da wohl beim
> Programmieren wohl ein Copy&Paste aus der Registerschreibschleife ein
> paar zeilen darüber gemacht und vergessen die 0x04 nach 0x80 zu ändern.
> Dies sollte Sie natürlich spätestens jetzt nachholen!

Stimmt, das sollte aber 0x08 heißen, oder?

[Peter II (Gast)]

Nicole F. schrieb:
> Stimmt, das sollte aber 0x08 heißen, oder?

warum? Du willst doch das höchste bit abfragen oder nicht?

also (1<<7) = 128 = 0x80

[Magnus M. (magnetus)]

Noch wat:

void cam_start(void)
{
  START_HIGH();
  camStepDelay();
  CLK_HIGH();
  camStepDelay();
  START_HIGH();      <-- Das sollte wohl mal ein START_LOW() werden?
  CLK_LOW();

[Nicole F. (Gast)]

Peter II schrieb:
> Nicole F. schrieb:
>> Stimmt, das sollte aber 0x08 heißen, oder?
>
> warum? Du willst doch das höchste bit abfragen oder nicht?
>
> also (1<<7) = 128 = 0x80

:-) stimmt

[Nicole F. (Gast)]

Hier noch die adc_init

void ADC_Init(void)
{

  
  ADMUX = (1<<REFS0);
  //ADMUX = (1<<REFS1) | (1<<REFS0);
  
  ADCSRA = (1<<ADPS1) | (1<<ADPS2) | (1<<ADPS0);     
  ADCSRA |= (1<<ADEN);                  
  
  
  ADCSRA |= (1<<ADSC);                  
  while (ADCSRA & (1<<ADSC) ) {         
  }
  
  (void) ADCW;
}

Bie 16MHz CPU Takt musste ich den größten Teiler nehmen

[Magnus M. (magnetus)]

Hast Du meinen anderen Beitrag auch gelesen und berücksichtigt?

Magnus M. schrieb:
> Noch wat:
(...)

[Nicole F. (Gast)]

Magnus M. schrieb:
> Hast Du meinen anderen Beitrag auch gelesen und berücksichtigt?
>
> Magnus M. schrieb:
>> Noch wat:
> (...)

Ja, auch diesen Flüchtigkeitsfehler entfernt.

[Peter II (Gast)]

Nicole F. schrieb:
> Ja, auch diesen Flüchtigkeitsfehler entfernt.

wie kann man nur so viel code schreiben ohne in stückchenweise zu testen 
und in betrieb zu nehmen?
Ich schreibe doch auch keine Betriebssystem und wundere mich an ende 
warum es nicht bootet.

[Magnus M. (magnetus)]

Und wie steht es nach all Dem um die Funktion Deines Konstruktes?

[Nicole F. (Gast)]

Magnus M. schrieb:
> Und wie steht es nach all Dem um die Funktion Deines Konstruktes?


Noch gefällt es mir nicht. Ich halte die Werte für zu nah beieinander.

999;1001;1000;998;998;998;998;999;999;999;1001;998;999;999;999;1001;998;999;999;999;1001;1000;998;998;998;998;999;999;999;1001;998;998;999;999;999;1001;1000;998;998;998;998;998;999;999;999;1001;998;999;999;999;1001;1000;998;998;998;998;998;999;999;999;1001;998;999;999;999;1001;1001;998;998;998;998;998;999;999;999;1001;998;999;999;999;1001;1001;999;998;998;998;998;999;999;999;1001;998;999;999;999;999;1001;998;999;999;999;1001;999;999;999;1001;998;999;999;999;1001;998;999;999;999;1001;1001;998;998;998;998;998;999;999;1001;998;998;999;999;1001;998;999;999;999;1001;1001;999;998;998;998;998;999;999;999;1001;998;999;999;999;1001;1000;998;998;998;998;998;999;999;1001;998;999;999;999;1001;998;999;999;999;1001;1001;998;998;998;998;998;999;999;1001;998;999;999;999;1001;998;999;999;999;1001;1001;999;998;998;998;998;999;999;999;1001;998;999;999;999;1001;1001;1000;998;999;1001;1001;999;998;998;998;998;998;999;999;999;1001;1001;998;998;998;998;998;999;999;999;1001;998;999;999;999;1001;1001;999;998;998;998;!

[Peter II (Gast)]

Nicole F. schrieb:
> Noch gefällt es mir nicht. Ich halte die Werte für zu nah beieinander.

das ist einfach rauschen vom ADC.

Hast du mal ein Oszi an den Anschluss gehalten, ob sich dort überhaupt 
etwas tut?

Wenn nicht vorhanden, einfach mal eine Wechselspannung an den ADC 
anlegen, dann sollte das ja an den werten erkennbar sein.

[Nicole F. (Gast)]

Der ADC funktioniert. Die CAM gibt aber nichts raus.

[Magnus M. (magnetus)]

Nicole F. schrieb:
> Der ADC funktioniert. Die CAM gibt aber nichts raus.

  cam_write(1,0x0E);  <-- ergibt ein Gain von 45,5dB !

Woher hast du die Registerwerte?

Was gibt deine Kamera aus wenn du die Linse zuhältst?

[Nicole F. (Gast)]

Es ändert sich nichts. Egal ob ich die Linse zuhalte, oder es gegen 
grelles Licht halte.

[Magnus M. (magnetus)]

1. Woher hast du eigentlich den ursprünglichen Code...?

2. Warte noch auf eine Antwort auf

Magnus M. schrieb:
> Nicole F. schrieb:
>> Der ADC funktioniert. Die CAM gibt aber nichts raus.
>   cam_write(1,0x0E);  <-- ergibt ein Gain von 45,5dB !
>
> Woher hast du die Registerwerte?

[Nicole F. (Gast)]

Die Werte sind von hier
http://www.angelfire.com/de3/juliprograms/amr/gbcam.htm

Die erste Frage verstehe ich nicht ganz

[Magnus M. (magnetus)]

Nicole F. schrieb:
> Die erste Frage verstehe ich nicht ganz

Von wem hast du den Code, den du uns im ersten Posting serviert hast?

[Nicole F. (Gast)]

Magnus M. schrieb:
> Von wem hast du den Code, den du uns im ersten Posting serviert hast?

Wie von wem habe ich den Code?
Das habe ich geschrieben. Ich hatte lediglich meine Register schreib 
funktion mit dem von ulrich radig's grafikkarten projekt inspirieren 
lassen.
Nachdem meins nicht richtig funktionierte.
Beitrag "Getaktete Übertragung"

[Nicole F. (Gast)]

Alles gut. Nachdem ich die CLK Frequenz und die Register angepasst habe, 
klappt alles gut.

[Magnus M. (magnetus)]

Nicole F. schrieb:
> Wie von wem habe ich den Code?
> Das habe ich geschrieben.

Hmmm... Wenn ich mir den Code im folgenden Link ansehe, hat das Ganze 
einen leicht plagiatartigen Beigeschmack.

http://www.triplay.de/zeigeprojekt.php?id=17#ansteuerung

Nicole F. schrieb:
> Alles gut. Nachdem ich die CLK Frequenz und die Register angepasst
> habe, klappt alles gut.

Ende gut, alles gut.

Herzlichen Glühstrumpf!

;)