richonguzman / LoRa_APRS_iGate

LoRa APRS iGATE for ESP32 Based Board with Rx + Tx capabilities
MIT License
208 stars 65 forks source link

Support modules with XTAL #133

Closed petrkr closed 1 month ago

petrkr commented 2 months ago

I making custom board for tracker and later mainly for iGate. I am using now SX1268 radio module which does not use TCXO but XTAL. https://www.aliexpress.com/item/1005005715303224.html

This add build time flag to tell RadioLib to use XTAL instead of TCXO.

Also one commit add error code from begin, which was helpful to se error -707

richonguzman commented 2 months ago

Hola Petr

both PR on igate and tracker are to add your new board to use this firmwares right?

I read the code and was not board_pins info for it

is this an esp32? s3, s2 , c3, c6 , gps?

petrkr commented 2 months ago

This is just pre-requests for usage of those boards.. also another boards, which will have lora module with Xtal instead of tcxo

petrkr commented 2 months ago

If you are curious about that board, so on that on which I testing it right now is here, c3.

But I think to do more things, just this xtal/tcxo is first not-breaking enhancementx which I will use in future, so why not support it already in main, it will provide less headache with future rebases

https://github.com/petrkr/LoRa_APRS_iGate/tree/olabboard-c3

richonguzman commented 2 months ago

I get your idea, but could you share the hole idea to do the mods:

name for the board to add it to platformio pin definition for boards_pinout.h for now

richonguzman commented 2 months ago

this is to help us both...

so I add also the xtal info as well

petrkr commented 2 months ago

I get your idea, but could you share the hole idea to do the mods:

name for the board to add it to platformio pin definition for boards_pinout.h for now

Thisbis not about specific MCU board yet, this is about specific LoRa module /stamp/ which that MCU board can use.

So it will affect all boards, which will use LoRa module which uses XTAL instead TCXO.

In short, that xtal true will turn off setting of Bias voltage for TCXO on DIO3 pin.

If you will set voltage and have xtal, you will get init error -707.

So this is lora module depend, not mcu board, thus it can not have board or pins definition. Because it does not rely on actual board yet.

I now testing it on esp32-c3, because I had it near hand. Final project, which i wanna do, will be eighter on esp32 or esp32-s3, I'll must check if you using SDK 4.x or already 5.x. for esp32 based systems. For what I want for s3 I need 5.x already. But that is out of this MR

richonguzman commented 2 months ago

Hi @petrkr ,

I added "octopus lab-add" branch for you to test: Xtal battery editing boards_pinout a minor things that should work

please let me know how it goes for you and if more things are needed (if you want find me at telegram with my github user)

petrkr commented 1 month ago

Seems this was done without merging, but with hand commit.