search

vedderb / bldc

The VESC motor control firmware
2.08k stars 1.31k forks source link

fix ZV freq for auto detection #713

Closed hackeybabel closed 4 months ago

hackeybabel commented 4 months ago

lower the zv freq for flux linkage measurement as the hardwares with phase shunts and v0-v7 enabled will trigger watchdog reset as you cannot run more than 30khz with v0-v7 enabled.

TechAUmNu commented 4 months ago

Better to just make the temporary config turn off V0 & V7 sampling.

hackeybabel commented 4 months ago

Better to just make the temporary config turn off V0 & V7 sampling.

But if with dual motors 25khz works fine why not use it with single motor as well.

TechAUmNu commented 4 months ago

I did a lot of testing and found that 40khz worked better when detecting low indictance motors. The 25khz is just a compromise because of the limitations of dual hardware.

https://github.com/vedderb/bldc/pull/533

TechAUmNu commented 4 months ago

Also only the wizard will set this to 40khz you can run it at whatever frequency you want from the foc page buttons.

So by just making sure that the conflicting setting is disabled temporarily during the wizard there is no change to functionality. Does your hwconfig have v0&v7 on by default?

vedderb commented 4 months ago

Is this ok? https://github.com/vedderb/bldc/commit/adfe4a736ea060a9fb462f82a719cb714d94a063

TechAUmNu commented 4 months ago

That looks good