Svenska ElektronikForumet

Hur vet man vilken tid som man behöver vänta innan man ska läsa av och varför fungerar inte FOR loopen?

  while(1){
    Ow_Reset(&PORTA,1);                    // RESET
    Ow_Write(&PORTA,1,0xCC);               // SKIP_ROM
    Ow_Write(&PORTA,1,0x44);               // CONVERT_T
    Delay_us(120);

    Ow_Reset(&PORTA,1);                    // RESET
    Ow_Write(&PORTA,1,0xCC);               // SKIP_ROM
    Ow_Write(&PORTA,1,0xBE);               // READ_SCRATCHPAD
    Delay_ms(400);

/*
    for (i=1;i>=0;i++) {
      ROM_data[i] = Ow_Read(&PORTA,1);     // Spara temperaturen
      Usart_Write(ROM_data[i]);
    }
*/

    j = Ow_Read(&PORTA,1);     // Get temperature LSB
    temp = Ow_Read(&PORTA,1);  // Get temperature MSB
    temp <<= 8; temp += j;     // Form the result
    Usart_Write(temp); // Format and display result on LCD

    Delay_ms(1000);

[/i]

Såg att det var for-loopen som var felskriven.

Vad är det för fel på denna loop då?

Kod: Markera allt

    for (i=1;i>=0;i--) {
      ROM_data[i] = Ow_Read(&PORTA,1);     // Spara temperaturen
    }

Vill fylla Arrayen på pos 0 och 1 men den fastnar där och loopar hela tiden.

Antagligen är i en unsigned variabel vilket gör att den räknar 0-- vilket ger 255 vilket är >=0 vilket alltså är sant.

Dessutom är det en del overhead med bara 2 platser, alltså är det snabbare och "billigare" (räknat i minne) att göra:
ROM_data[1] = Ow_Read(&PORTA,1);
ROM_data[0] = Ow_Read(&PORTA,1);

Ska jag göra i till en signed?

I detta fall så blev det till att göra direkt som du skrev men jag tänkte för andra loopar som jag vill få till som ska räkna längre baklänges.

Det kan vara en lösning att göra den till en signed men man kan även ha kriterian: i < 255 eller om du vill vara HELT säker: i < 128.

Fick bli så här i stället:

Kod: Markera allt

  for (i=8;i>0;i--)
    ROM_nummer[i-1] = Ow_Read(&PORTA,1);     // Spara ROM nummer

for(i=8;i>0;) ROM_nummer[--i] = Ow_Read(&PORTA,1); // Spara ROM nummer

Denna version gör exakt det samma fast lite optimerat.
'--i' gör att i dekrementeras FÖRST och resultatet används SEDAN, på detta sätt sparar du CPU-tid vid att spola 'i-1'.

Får bara Routine ow_get_id signature do not match routine call signature när jag anropar nedanstående rutin:

Kod: Markera allt

unsigned short * ow_get_id() {

  signed short i;
  unsigned short ROM_id[8];

  Ow_Reset(OW_PORT,OW_PIN);                      // RESET
  Ow_Write(OW_PORT,OW_PIN,0x33);                 // READ_ROM
  Delay_us(120);                           // pause
  for (i=0;i<=7;i++)
    ROM_id[i] = Ow_Read(OW_PORT,OW_PIN);     // Spara ROM nummer

  return ROM_id;
  
}

Vad kan vara fel?

Det var fel i Header filen..

Svenska ElektronikForumet

1-Wire med PIC

1-Wire med PIC