Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

AM32: G071 #249

Closed
stylesuxx opened this issue Aug 8, 2022 · 6 comments
Closed

AM32: G071 #249

stylesuxx opened this issue Aug 8, 2022 · 6 comments

Comments

@stylesuxx
Copy link
Owner

stylesuxx commented Aug 8, 2022

To make the g071 esc's compatible there are only two changes. The identifier : https://github.com/AlkaMotors/g071Bootloader/blob/f86fc79a05c7ddcc25d598f4b2c952204c98c674/Core/Src/main.c#L62

and eeprom location https://github.com/AlkaMotors/g071Bootloader/blob/f86fc79a05c7ddcc25d598f4b2c952204c98c674/Core/Src/main.c#L18

New MCU:

{
      "name": "G071",
      "signature": "0x3506",
      "page_size": 1024???,
      "flash_size": 65536???,
      "flash_offset": "0x08000000???",
      "firmware_start": "0x1000???"
    }
@stylesuxx stylesuxx changed the title AM32 G071 AM32: G071 Sep 18, 2022
@AlkaMotors
Copy link
Contributor

AlkaMotors commented Oct 15, 2022

@stylesuxx I have a couple mac users who would love to get this working. Those are the only changes that to make the g071 work. The eeprom layout is the same. Since there are now more and more arm mcu options, all esc's with 32 memory will report as the f051 mcu and 64 KB esc's will use the x3506 identifier. The page size can vary from 1kb to 2kb depeding on the mcu but the writing can be done in 1kb page size for all.

Maybe something like this?

{ "name": "ARM64K", "signature": "0x3506", "page_size": 1024, "flash_size": 65536, "flash_offset": "0x08000000", "firmware_start": "0x1000" }

As far as i can see the only line that would need to be changed depending on the identifier would be this ?
the offset for all 64k will be 0xF800.

const EEPROM_OFFSET = 0x7c00;

@stylesuxx
Copy link
Owner Author

Hey @AlkaMotors - that is perfect timing! I had to change eeprom offset for the BB51 too and was wondering if I should make the same changes to the AM32 config. I will do this either later today or tomorrow and you can give it a spin.

@stylesuxx
Copy link
Owner Author

@AlkaMotors I published the adaptations on the preview build, could you let me know if it works with the 64k? https://preview.esc-configurator.com/

@ljlzonda
Copy link

hi bro, just did some test with my G071 ESC setup, and it works fine for me!

@stylesuxx
Copy link
Owner Author

@ljlzonda awesome, thanks for the feedback!

@ljlzonda
Copy link

@ljlzonda awesome, thanks for the feedback!

settings all work out fine, but I found two missing settings comparing to standalone AM32 configurator , Sine Mode Power and Running Brake Level, also reported this issue with @AlkaMotors

stylesuxx added a commit that referenced this issue Oct 18, 2022
* Bluejay: BB51 adaptations  #265
* AM32: Add G071 support #249
* General: Make sure the correct slot is updated in the UI #269
@stylesuxx stylesuxx added this to the v0.27.0 milestone Oct 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants