tuupola / esp_effects

Old school demo effects for ESP32
MIT No Attribution
44 stars 5 forks source link

Some extra steps to get it working on M5Stack #5

Closed GunterO closed 4 years ago

GunterO commented 4 years ago

Hi, thanks for your library, really nice work! Anyway. I'm testing with a M5Stack Basic (black), on Windows 10 with IDF v4.1, and I had to take some extra steps to get it working. I'm quite a newbie here, so sorry if I made mistakes or are missing the obvious.

On compiling ($ idf.py build), I got these errors: image

To make them go away, I had to go into $ idf.py menuconfig and set this: image image

And then the compiler gave no errors.

On uploading it to the device, all the colors were inverted. So I had to set: image

Which made it run perfectly.

Is this as expected?

tuupola commented 4 years ago

Yes, you are required to configure settings to match your dev board or device. There are ready made config files for different boards. I use sdkconfig.m5stack with M5Stacks. It should work with the black version too. Instructions how to use ready made config files are in the README:

$ git clone git@github.com:tuupola/esp_effects.git --recursive
$ cd esp_effects
$ cp sdkconfig.m5stack sdkconfig
$ make -j8 flash
GunterO commented 4 years ago

Hi, thanks for your quick response. Yes, this is the first step I did: $ cp sdkconfig.m5stack sdkconfig But the e.g. CONFIG_AXP192_GPIO0_LDOIO0_VOLTAGE_BIT74 setting, where axp192_config.h is referring to isn't in there.

tuupola commented 4 years ago

Ah then it is a bug. I think I have updated the AXP192 library but not the default config files yet. Will check.

tuupola commented 4 years ago

I updated the AXP192 module to have default config values even when it is not used. Just tested and myself and both of the following now compile.

$ cp sdkconfig.m5stack sdkconfig
$ make -j8
$ cp sdkconfig.m5stack sdkconfig
$ idf.py build
tuupola commented 4 years ago

@GunterO Did this fix the problem for you too?

GunterO commented 4 years ago

Yes, it did, thanks! I'll close the issue.

tuupola commented 4 years ago

Thanks for the heads up! I had not compiled it for M5Stack for a while so missed the problem myself.