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

Atmega32 support(mighty core) #598

Closed
Master811129 opened this Issue Apr 16, 2018 · 6 comments

Comments

Projects
None yet
4 participants
@Master811129

Master811129 commented Apr 16, 2018

hi guy, i have an issue with this library. when i coding with this lib it doesn't compile.
error: multiple library were found.

and when i change the model to arduoino-NG (atmega 8) it works very well.
please help

@focalintent

This comment has been minimized.

Member

focalintent commented Apr 17, 2018

What are you building it with, what version of the library are you using, what does the entire build output look like?

error: multiple library were found makes it sounds like you have multiple versions of FastLED installed.

I'm not aware of anyone using the mighty core - so it's likely that the library is missing pin definitions for your board (see platforms/avr/fastpin_avr.h) - but your error message isn't anything that comes out of FastLED itself, so I think you have some other build problems going on as well.

@obdevel

This comment has been minimized.

obdevel commented Jul 1, 2018

I'd welcome support for MightyCore too. Currently it results in a compilation error:

In file included from /Users/xxx/Documents/Arduino/libraries/FastLED/FastLED.h:51:0,
                 from /Users/xxx/Documents/Arduino/xyz_test_v1/xyz_test_v1.ino:8:
/Users/xxx/Documents/Arduino/libraries/FastLED/fastpin.h: In instantiation of 'class FastPin<13u>':
/Users/xxx/Documents/Arduino/libraries/FastLED/platforms/avr/clockless_trinket.h:96:49:   required from 'class ClocklessController<13u, 4, 10, 6, (EOrder)66u, 0, false, 10>'
/Users/xxx/Documents/Arduino/libraries/FastLED/chipsets.h:382:7:   required from 'class WS2812Controller800Khz<13u, (EOrder)66u>'
/Users/xxx/Documents/Arduino/libraries/FastLED/FastLED.h:94:34:   required from 'class NEOPIXEL<13u>'
/Users/xxx/Documents/Arduino/libraries/FastLED/FastLED.h:310:28:   required from 'static CLEDController& CFastLED::addLeds(CRGB*, int, int) [with CHIPSET = NEOPIXEL; unsigned char DATA_PIN = 13u]'
/Users/xxx/Documents/Arduino/xyz_test_v1/xyz_test_v1.ino:90:53:   required from here
/Users/xxx/Documents/Arduino/libraries/FastLED/fastpin.h:207:2: error: static assertion failed: Invalid pin specified
  static_assert(validpin(), "Invalid pin specified");
...
...
exit status 1
Error compiling for board ATmega644.
/Users/xxx/Documents/Arduino/libraries/FastLED/FastLED.h:17:21: note: #pragma message: FastLED version 3.001.007

Arduino 1.8.5, macOS 10.13.5, 644P, MightyCore 2.0.0, standard pinout.
Attempting to use pin PD5, which maps to Arduino pin 13. This pin has no special function on this board, other than being PWM-capable.

Happy to help out with the port if there is documentation available.

Thanks

@focalintent

This comment has been minimized.

Member

focalintent commented Jul 5, 2018

Again, pin definitions need to be added in platforms/avr/fastpin_avr.h for this board.

@obdevel

This comment has been minimized.

obdevel commented Jul 9, 2018

Ok, the 644P seems similar enough to the 1284P which is already supported. From my limited testing, I can compile and successfully run trivial tests by adding the 644 to the 1284 line in platforms/avr/fastpin_avr.h, e.g.

#elif defined(__AVR_ATmega1284P__) || defined(__AVR_ATmega644P__)

Maybe this will provide support for all the MightyCore supported MCUs. @Master811129 - do you want to try this ?

Are there any acceptance tests that can be run ? I'm using an own-design board so I don't have access to every pin to test.

@tmbomber383

This comment has been minimized.

tmbomber383 commented Sep 8, 2018

I use 644P processors as well. I'm in work on Saturday and tried using this library for the first time and ran into this exact, same, issue...
and came up with the exact, same, solution.
In fastpin_avr.h I modified line 183 from:
#elif defined(AVR_ATmega1284P)
to:
#elif defined(AVR_ATmega1284P) || defined(AVR_ATmega644P)
and I'm up and running.
So, confirmation on this being a solution and a second request that it be folded in to the next release.

@focalintent

This comment has been minimized.

Member

focalintent commented Sep 8, 2018

Just pushed FastLED 3.2.1 which adds this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment