CapnBry / CRServoF

CRSF to PWM Servo converter for STM32F103
GNU General Public License v3.0
217 stars 55 forks source link

Target for CJMCU1038 - Purple Pill #12

Closed derFrickler closed 3 years ago

derFrickler commented 3 years ago

I have some of these Purple Pill boards lying around: https://stm32-base.org/boards/STM32F103C8T6-Purple-Pill.html My Idea was to use them for your project, as they are small and have a proto area which makes it easy to solder a 6x3 Pin Header for the Servos.

I tried to make a new Target for the board using the pins that are broken out on the board.

#elif defined(TARGET_CJMCU1038)
      #define DPIN_LED        PB_11
      #define LED_INVERTED    1
      #define APIN_VBAT       PB_0
      #define USART_INPUT     USART1  // UART1 RX=PA10 TX=PA9
      #define OUTPUT_PIN_MAP  PA_0, PA_1, PA_2, PA_3, PB_3, PB_2, PB_1, PA_4 
 #endif

I am able to compile and flash the code, it seems to connect to the receiver but i don't get the right servo outputs. I enabled some printlines in your code and the serial console says:

ADC=0 0V
ADC=0 0V
6666
�␌␔␀␀␀␀␀␄␀␀␀␀��␌␔␀␀␀␀␀␄␀␀␀␀ADC=0 0V
ADC=0 0V
ADC=0 0V
ADC=0 0V
tentative conn
30dBm

And then it seems stuck.

On pins PA_0, PA_1, PA_2, PA_3 i can see a 50Hz servo signal with 1700ms. The other pins show nothing. I guess its some problem due to the used timers for my pins that need to be changed. Maybe you can point me in the right direction.

Thanks a lot Johannes

CapnBry commented 3 years ago

The pins you use aren't just randomly selected. You have to match them to a hardware timer because they are not assignable.

derFrickler commented 3 years ago

Thanks for the fast reply! I just chose the pins that were broken out on the board where i needed them, did not care about the timers. Its my first STM32 project, i only did AVR and Arduino before where that has never been an issue for me. Can you give me a hint on where to find that info and how to change the timers?

Thanks a lot Johannes

CapnBry commented 3 years ago

It is in the platform's peripheralpins.cpp for that MCU, or the datasheet for the original reference. Or since it is the same chip you can just google bluepill pinout and look at the callouts there where it says PWMx/y where X is the timer and Y is the channel.

derFrickler commented 3 years ago

Thanks, Found the info and managed to get that done, had 2 pins in use that did not support hardware PWM.

Now i get the correct receiver values and have a PWM on all pins - but still only 1700ms independent of the receiver values. servoSetUs seems to get the wrong µs values, but the serial debugging in packetChannels show the right values . Must be somewhere around where it messes up. https://github.com/CapnBry/CRServoF/blob/caddba34d603e75e339e7f37d813cf1ccaafcb49/src/main.cpp#L74 I will check that out this evening.

CapnBry commented 3 years ago

Good catch! I apparently put this bug in when I added the inverted output a couple weeks ago. Tested the inverted case, did not test that I left an assignment off the regular case. I just pushed the change to fix it, thanks!

derFrickler commented 3 years ago

OK, that explains a lot, was thinking how that line of code was supposed to work....

derFrickler commented 3 years ago

So, my prototype is running. I will add the target for that and a wiring diagram if you want. IMG_20210923_171618 Will try to add an option to show the cell-voltage (divide by recognized cell number) instead of the full battery voltage in the telemetry as well.

CapnBry commented 3 years ago

Nice! Sure if you want to do a PR for the target and include a markdown table for the pinout like there is a in the readme for the "F103" target. I guess maybe call this target purplepill with a comment that it is CJMCU-1038?

As far as cell voltage for VBatt, sure do this using a constexpr bool report_cell_voltage and a #define VBAT_MAX_CELL_VOLTAGE 430 lower down for being able to adjust the cell calculation. Keep all the math in integer centivolts.

JupaCreations commented 3 years ago

derFrickler are the SLC and SDA pins available to connect for example the BMP280 or better baro I2C board? Would that be possible to have a diy 8 channel LOS receiver with voltage and height measurement?

I have a blue pill to try but will also order this purple pill... :-)

What about a piggyback pcb to solder pins and other parts on?

Schematics and how to program would be welcome.

Thanks, Paco

CapnBry commented 3 years ago

derFrickler are the SLC and SDA pins available to connect for example the BMP280 or better baro I2C board? Would that be possible to have a diy 8 channel LOS receiver with voltage and height measurement?

I would be against turning this into some sort of flight controller by adding extra sensors, so do not add this to the PR for adding this target.

Paco: The hardware is exactly the same between the bluepill and this. This only has the place to solder servo pinheader on (but is missing some of the pins the bluepill version uses).

JupaCreations commented 3 years ago

Hello Brian,

I am not requesting a FC replacement solution to DIY. If that is the case I can 100% follow you objection.

For simple electric LOS gliding, battery voltage and variometer telemetry are the max needed. I like DIY solutions to tinker and learn from this all, just like the whole ELRS project. If I need more like stabilisation or more telemetry data I go the FC F 411 or F405 route anyway as that is more convenient.

The purple pill has smaller dimensions. For some applications that would be sufficient to have a smaller base setup.

In the end I need a plain/simple small 2 channel receiver too for a car for racing, so smaller dimensions are key here. Telemetry and FC are not allowed in those race classes.

Flying and racing is my passion.

Paco

derFrickler commented 3 years ago

OK, i did a pull request for the Purplepill target. https://github.com/CapnBry/CRServoF/pull/13

I added a wiring pic to my repo aswell, if you want i can do a pull request for that aswell: https://github.com/derFrickler/CRServoF/blob/master/doc/Purplepill_CJMCU1038_wiring.jpg

Next thing i will do is the cellcount detection and an report_cell_voltage option. Will do that in a separate request later.

Baro and other sensors is definitely not on my list, especialy for a Vario its muchmore than just reading sensor values...

P.S. Seems that Matec is bringing Servo PWM boards soon, even with Batt, Current and GPS telemetry. http://www.mateksys.com/?portfolio=crsf-pwm

CapnBry commented 3 years ago

Added to master now, sorry about the delay as I kept putting it off due to ELRS business. Thanks @derFrickler!