Show Posts
Pages: [1]
1  Using Arduino / Programming Questions / Re: pulseIn bug on: January 21, 2013, 06:43:22 pm
Right now I use Arduino 1.0.3

File /usr/share/arduino/hardware/arduino/cores/arduino/Arduino.h has the following definitions:
Code:
#define clockCyclesPerMicrosecond() ( F_CPU / 1000000L )
#define clockCyclesToMicroseconds(a) ( (a) / clockCyclesPerMicrosecond() )
#define microsecondsToClockCycles(a) ( (a) * clockCyclesPerMicrosecond() )

F_CPU = 16 000 000
-----> clockCyclesPerMicrosecond = 16
"width" value (in my tests) was not more 736 (~550-736) ---> about 7-13 centimeters
(I temporary changed return value in wiring_pulse.c to get width from there)

File /usr/share/arduino/hardware/arduino/cores/arduino/wiring_pulse.c
Code:
return clockCyclesToMicroseconds(width * 21 + 16);

and problem still exists. smiley-sad-blue
2  Using Arduino / Programming Questions / Re: pulseIn bug on: January 21, 2013, 11:47:26 am
Very interesting.
I made several tests with this easy code:
Code:
unsigned long duration;
int counter = 0;

unsigned long before = 0;
void setup()
{
  pinMode (5, INPUT);
  Serial.begin(9600);
}
void loop()
{
  duration = pulseIn(5, LOW, 10000000L); // yes 10.000.000
  {
    Serial.print("D: ");
    Serial.println (duration);
    Serial.print("M: ");
    Serial.println(millis()-before);
    before = millis();
  }
}
not working. I got:
Code:
core.a(wiring_pulse.c.o): In function `pulseIn':
/usr/share/arduino/hardware/arduino/cores/arduino/wiring_pulse.c:68: undefined reference to `__muluhisi3'
collect2: error: ld returned 1 exit status

File /usr/share/arduino/hardware/arduino/cores/arduino/wiring_pulse.c line 68 (original) :
Code:
return clockCyclesToMicroseconds(width * 21 + 16);

If I change it to:
Code:
return clockCyclesToMicroseconds(width * 2 + 16);
(I know that there no sense)   smiley-red
...there no any compilation errors.
As you can see I just changed "21" to "2"  smiley-eek
BTW, code with "3" :
Code:
return clockCyclesToMicroseconds(width * 3 + 16);
not working too.   smiley-eek

any ideas? smiley-confuse

PS: Maximum value for "width" i have <= 736.

PPS: Gentoo Linux 64bit
3  Using Arduino / Programming Questions / Re: pulseIn bug on: January 20, 2013, 02:59:20 pm
Which board did you have selected?
Arduino Uno
4  Using Arduino / Programming Questions / Re: pulseIn bug on: January 20, 2013, 02:38:27 pm
What time out do you use?

Read this thread -http://arduino.cc/forum/index.php/topic,74813.msg563694.html#msg563694 -
See my reply msg10

Thanks again.

I use standard examples Examples --> Sensors --> Ping from arduino package.
It looks like mistake in example
Code:
duration = pulseIn(pingPin, HIGH);
here no timeout

Tried your testcode from your thread (post #10), got just one (not two like before) error on line 68:
Code:
core.a(wiring_pulse.c.o): In function `pulseIn':
/usr/share/arduino-0022/hardware/arduino/cores/arduino/wiring_pulse.c:68: undefined reference to `__muluhisi3'
collect2: error: ld returned 1 exit status

Your fix is working only on for 45-th line.  smiley-razz
 smiley
5  Using Arduino / Programming Questions / Re: pulseIn bug on: January 20, 2013, 01:13:55 pm
sure

file /usr/share/arduino-0022/hardware/arduino/cores/arduino/wiring_pulse.c:
Code:
/*
  wiring_pulse.c - pulseIn() function
  Part of Arduino - http://www.arduino.cc/

  Copyright (c) 2005-2006 David A. Mellis

  This library is free software; you can redistribute it and/or
  modify it under the terms of the GNU Lesser General Public
  License as published by the Free Software Foundation; either
  version 2.1 of the License, or (at your option) any later version.

  This library is distributed in the hope that it will be useful,
  but WITHOUT ANY WARRANTY; without even the implied warranty of
  MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
  Lesser General Public License for more details.

  You should have received a copy of the GNU Lesser General
  Public License along with this library; if not, write to the
  Free Software Foundation, Inc., 59 Temple Place, Suite 330,
  Boston, MA  02111-1307  USA

  $Id: wiring.c 248 2007-02-03 15:36:30Z mellis $
*/

#include "wiring_private.h"
#include "pins_arduino.h"

/* Measures the length (in microseconds) of a pulse on the pin; state is HIGH
 * or LOW, the type of pulse to measure.  Works on pulses from 2-3 microseconds
 * to 3 minutes in length, but must be called at least a few dozen microseconds
 * before the start of the pulse. */
unsigned long pulseIn(uint8_t pin, uint8_t state, unsigned long timeout)
{
        // cache the port and bit of the pin in order to speed up the
        // pulse width measuring loop and achieve finer resolution.  calling
        // digitalRead() instead yields much coarser resolution.
        uint8_t bit = digitalPinToBitMask(pin);
        uint8_t port = digitalPinToPort(pin);
        uint8_t stateMask = (state ? bit : 0);
        unsigned long width = 0; // keep initialization out of time critical area

        // convert the timeout from microseconds to a number of times through
        // the initial loop; it takes 16 clock cycles per iteration.
        unsigned long numloops = 0;
        unsigned long maxloops = microsecondsToClockCycles(timeout) / 16;

        // wait for any previous pulse to end
        while ((*portInputRegister(port) & bit) == stateMask)
                if (numloops++ == maxloops)
                        return 0;

        // wait for the pulse to start
        while ((*portInputRegister(port) & bit) != stateMask)
                if (numloops++ == maxloops)
                        return 0;

        // wait for the pulse to stop
        while ((*portInputRegister(port) & bit) == stateMask) {
                if (numloops++ == maxloops)
                        return 0;
                width++;
        }

        // convert the reading to microseconds. The loop has been determined
        // to be 20 clock cycles long and have about 16 clocks between the edge
        // and the start of the loop. There will be some error introduced by
        // the interrupt handlers.
        return clockCyclesToMicroseconds(width * 21 + 16);
}



problems in lines:
Code:
unsigned long maxloops = microsecondsToClockCycles(timeout) / 16;
...
return clockCyclesToMicroseconds(width * 21 + 16);

it looks like there is something with definition of  clockCyclesToMicroseconds and microsecondsToClockCycles in
/usr/share/arduino-0022/hardware/arduino/cores/arduino/wiring.h file
Code:
#define clockCyclesToMicroseconds(a) ( ((a) * 1000L) / (F_CPU / 1000L) )
#define microsecondsToClockCycles(a) ( ((a) * (F_CPU / 1000L)) / 1000L )

thanks
6  Using Arduino / Programming Questions / pulseIn bug on: January 20, 2013, 11:32:39 am
OS Gentoo Linux
Arduino 0022 or 1.0.3 (any)
Standard example: Sensors --> Ping
Getting error during "Verify" process :

Code:
core.a(wiring_pulse.c.o): In function `pulseIn':
/usr/share/arduino-0022/hardware/arduino/cores/arduino/wiring_pulse.c:45: undefined reference to `__muluhisi3'
/usr/share/arduino-0022/hardware/arduino/cores/arduino/wiring_pulse.c:68: undefined reference to `__muluhisi3'
collect2: error: ld returned 1 exit status

Any ideas...
Pages: [1]