OpenAstroTracker-Firmware icon indicating copy to clipboard operation
OpenAstroTracker-Firmware copied to clipboard

Minor fixes for esp32 configurations

Open philharrisathome opened this issue 4 years ago • 3 comments

Minor fixes against the esp32 build:

  • Removed #define DISABLED & ENABLED in Constants.hpp due to clash with existing symbols in esp32 IDF. DISABLED & ENABLED don't appear anywhere else in the OAT codebase.
  • Removed unnecessary #include "Constants.hpp" in .\boards* . Constants is already included in the parent Configuration.hpp.
  • In Configuration.hpp default BOARD to new BOARD_UNKNOWN value and extend conditional tree at line 253. This is to prevent potential misconfigured hardware in case BOARD_TYPE is undefined: for an esp32 build, if BOARD_TYPE is undefined then Configuration.hpp would apply pin assignments for BOARD_AVR_MEGA2560. Mismatches were possible with other configurations also. With this change no pin assignments are made, however the configuration checks later in Configuration.hpp will provide a suitable message. @CTetford - you may want to review this.

philharrisathome avatar Jan 24 '21 14:01 philharrisathome

Mh ... could we sync the changes with hal ideas? Some of these changes already happened in the hal branch

andre-stefanov avatar Jan 24 '21 15:01 andre-stefanov

I can take a look at making sure these are also merged/addressed into hal branch. It would also be worth merging these into devel for now to fix the warnings on esp32 there. Has all new development switched to hal? I'm still working with the premise that devel feeds release, and that at some point we will switch to hal.

philharrisathome avatar Jan 24 '21 16:01 philharrisathome

Hi @andre-stefanov . Can we get this merged into develop? The build warnings and configuration conflicts fixed by this PR are still present.

philharrisathome avatar Feb 07 '21 13:02 philharrisathome