Speaker class is missing here. Will it be added?

In the examples folder, there's an example for the MPU6886 imu. Unfortunately, it does not compile, because 'class M5Tough' has no member named 'IMU'. Looking at the code, that is correct, there's no reference whatsoever to an IMU in the declaration of the M5 class. According to the documentation, I'm not even sure the M5Though does have an IMU/MPU.

Can you clearify whether that example is supposed to work? Is there an IMU in the M5Though, and if so, what type is it?

I'm currently building a project on the Tough. I'm unsure which library to use though to draw on the LCD and access GPIOs.

Is the M5Tough the legacy project which supports more right now but will be deprecated eventually?

Can I run M5Tough without Arduino using just esp-idf?

M5Unified.h: No such file or directory

I encountered various errors/problems when trying to build/flash the ..\examples\core2_for_aws\FactoryTest\FactoryTest.ino, on a desktop PC with MS Windows 11 Pro and running Arduino IDE 1.8.19. A part of the errors have the same origin as discussed in issue #3: the lack of an IMU in the M5Stack TOUGH.

Starting with the following warning:

                 from C:\Users\pauls2\Documenten\Arduino\2022-03-03_M5Stack_TOUGH_FactoryTest\2022-03-03_M5Stack_TOUGH_FactoryTest.ino:21:
C:\Users\<Username>\Documents\Arduino\libraries\FastLED\src/fastspi.h:145:23: note: #pragma message: No hardware SPI pins defined.  All SPI access will default to bitbanged output
 #      pragma message "No hardware SPI pins defined.  All SPI access will default to bitbanged output"

I did not do anything to solve this warning.

Then Errors related to a non existant IMU:

FactoryTest:448:12: error: 'class M5Tough' has no member named 'IMU'
    if (M5.IMU.Init() == 0) {

FactoryTest:612:8: error: 'class M5Tough' has no member named 'IMU'
     M5.IMU.getAccelData(&accX, &accY, &accZ);
        ^~~
C:\Users\<Username>\Documents\Arduino\libraries\M5Tough-master\examples\core2_for_aws\FactoryTest\FactoryTest.ino: In function 'void MPU6886Page()':
FactoryTest:1755:12: error: 'class M5Tough' has no member named 'IMU'
         M5.IMU.getAccelData(&accX, &accY, &accZ);

For this fact I removed: a) the checkIMUInit() function; b) the checkIMUInit() function; c) the MPU6886Page() function.

Then:

FactoryTest:473:31: error: call of overloaded 'begin(int, int, int)' is ambiguous
     Wire1.begin(21, 22, 100000);
                               ^
In file included from C:\Users\<Username>\Documenten\Arduino\libraries\ArduinoECCX08\src/ECCX08.h:24,
                 from C:\Users\<Username>\Documenten\Arduino\libraries\ArduinoECCX08\src/ArduinoECCX08.h:23,
                 from C:\Users\<Username>\Documenten\Arduino\libraries\M5Tough-master\examples\core2_for_aws\FactoryTest\FactoryTest.ino:13:
C:\Users\<Username>\AppData\Local\Arduino15\packages\esp32\hardware\esp32\2.0.2\libraries\Wire\src/Wire.h:79:10: note: candidate: 'bool TwoWire::begin(int, int, uint32_t)'
     bool begin(int sda=-1, int scl=-1, uint32_t frequency=0); // returns true, if successful init of i2c bus
          ^~~~~
C:\Users\<Username>\AppData\Local\Arduino15\packages\esp32\hardware\esp32\2.0.2\libraries\Wire\src/Wire.h:80:10: note: candidate: 'bool TwoWire::begin(uint8_t, int, int, uint32_t)'
     bool begin(uint8_t slaveAddr, int sda=-1, int scl=-1, uint32_t frequency=0);

To solve this ambiguity error:
In line 478 I changed: Wire1.begin(21, 22, 10000); into: Wire1.begin((uint8_t)21, 22, 10000); . The same in line 1947.

Then:
../FactoryTest.ino:466:49: warning: control reaches end of non-void function [-Wreturn-type]
int checkAETCC608AInit() causes a warning because the function does not return anything.
I changed it into: void checkAETCC608AInit().

After all these corrections the sketch built and flashed to my M5Stack TOUGH, but the unit switched off immediately after only showing the text "motor test" at the bottom of the display and sounding a dualtone bleep.

A next build failed with the error that the dynaming memory was full (102%).
The sketch uses 1344717 bytes of the program storage. The maximum is 1310720 bytes.
In an attempt to solve this I changed, where possible, Serial.print() commands with static texts into Serial.print(F("...")); however there were not that many of these commands. More in number were combined Serial.printf() commands.

I didn't find yet a solution to the memory full error.

While using the soft Buttons on M5Tough in a more dynamic way (not just as global objects, but rather as local variables in functions with a loop), the following issues were seen:

1. When a Button object is deleted, pointers to it may remain in two locations (M5Buttons::event.button and M5Buttons::_finger[i].button) and may lead to overwriting free memory (or the stack) via M5.update().

A possible fix may be to add the following to the end of the method M5Buttons::delHandlers (which is called by the Button destructor with nullptr for fn):

  if (button && !fn) { // when deleting a button
   if (event.button == button) {
     event.button = nullptr;
   }
   for(int fIdx = 0; fIdx < 2; ++fIdx) {
     if (_finger[fIdx].button == button) {
       _finger[fIdx].button = nullptr;
     }
   }
 }

 
There is a similar problem with Gestures, but this was not used by us and the following fix (also for M5Buttons::delHandlers) was not tested:

  if (gesture && !fn) { // when deleting a gesture
   if (event.gesture == gesture) {
     event.gesture = nullptr;
   }
 }

2. Some members of Button are not explicitly initialized. This can obviously be a problem if the object is not global/static. An error was observed in a program with several buttons on the stack: on the first M5.update(), a fake event with partially random data was reported to one handler function.
   The following statements may be added to the beginning of Button::init() to make the initialization at least equivalent to global/static default initialization:

  _changed = false;
 _longPressing = _cancelled = _manuallyRead = false;
 _setState = 0;
 _lastRepeat = _lastLongPress = 0;
 _finger = 0;
 _hidden = false;
 _name[sizeof(_name) - 1] = 0; // for safety if input was too long

Furthermore, the struct Finger should have a constructor:

in M5Button.h, add constructor declaration in “struct Finger {..”:

struct Finger {
  Finger();

and in M5Button.cpp, add:

#ifdef _M5TOUCH_H_
Finger::Finger() {
  startTime = 0;
  tapTime = 0;
  button = nullptr;
}
#endif