Hardware ESC 8xSERVO CONTROL on PIC (Oshonsoft BASIC)

jjw said:

??
I ran the original program/Rjenkinsgb from May 2022 in simulator and 100000us real time takes about 15 seconds.

Click to expand...

jjw said:

#136 works in SIM.
I had the oscilloscope (in SIM) for two first servos and they are ok.
To make it faster you can shorten the
if index> 15 to for example if index > 10

Click to expand...

jjw said:

Probably it works, but so slowly, that you don't wait long enough.
Have you defined WaitMs 1?
My SIM runs the servo loop / 8 servos in about one second.
To speed up:
From Options menu:
- don't use Basic program tracking.
- don't use oscilloscope, use microcontroller view instead.

Click to expand...

sagor1 said:

The code:

Code:

'For(uint8_t i = 0; i < num_servos; i++){
Dim num_servos(8) As Word  '?

Is originally a loop. You have to define num_servos as a byte (as in beginning of code) and set the value to =8, as that is the number of servos you are working with. Also, you have to define "i" as a byte somewhere at the beginning of the code as well.
Then you would have code that look like:

Code:

For i = 0 to num_servos-1 'for 8 servos, 0 to 7
....
rest of code
...
next i

If you want to speed up the SIM, you can try to define:
Define SIMULATION_WAITMS_VALUE = 0

Click to expand...

sagor1 said:

This code:

Code:

'For(uint8_t i = 0; i < num_servos; i++){
Dim num_servos(8) As Byte

is wrong.
num_servos is a byte value, and must be declared beforehand and initialized to a value of 8. It is NOT an array, just a value that declares the number of servos that you are going to process.

When declaring variables, it is best to put them all in one spot at the beginning, so you can see what is declared and what values or types. ie:

Code:

Dim i As Byte
Dim servo_pos(8) As Word
Dim servocount As Byte
Dim num_servos as Byte

' Fill in values for fixed variables
num_servos = 8  ' Declare number of servos to process in this code
.
.
.
etc

Also, you declare "servo_pos", but the code uses "servopos" in the interrupt routine. Pick one or the other.

Then, in the MAIN code, you need another loop to initalize the servo position. Original code starts as:

Code:

'For(uint8_t i = 0; i < num_servos; i++){
Dim num_servos(8) As Byte

'servopos[i] = 1.5 * 8000;
servo_pos(i) = 1.5 * 8000
.
.
.
etc

You need to put in a similar loop, similar to:

Code:

For i = 0 To num_servos - 1
[QUOTE]
servo_pos(i) = 1.5 * 8000
[/QUOTE]
.
.
.
etc
...
next i

Click to expand...

sagor1 said:

Hint:
Think about what is wrong with this part of your code. Remember that an interrupt routine is going to write/set values for the servo positions. Your main loop is all wrong...

Code:

servopos(i) = 1.5 * 8000

TRISB = 0

LATB = 0

INTCON.PEIE = 1  'PEIE/GIEL 1 = Enables all unmasked peripheral interrupts
INTCON.GIE = 1  'GIE/GIEH1 = Enables all unmasked interrupts

WaitUs 1

main_loop:

For i = 0 To num_servos - 1
    servopos(i) = 1.5 * 8000
    TRISB = 0
    LATB = 0
    INTCON.PEIE = 1
    INTCON.GIE = 1
    WaitUs 1
Next i

Goto main_loop

Click to expand...

jjw said:

Deleted.

Click to expand...

sagor1 said:

From my very basic understandiing of C (C++), the "&&" is a logical "AND". "++" means to increment the number after use. Things like "||" is a logical "OR".
Not sure about +=, but here is a site that explains all the standard operators:

Arduino - Operators

Arduino - Operators - An operator is a symbol that tells the compiler to perform specific mathematical or logical functions. C language is rich in built-in operators and provides the following types of operators ?

www.tutorialspoint.com

Click to expand...