t vago's MPGuino workspace thread

[josemapiro]

The characters of this new version are better, so it is easier to understand, maybe moved was the nature of the instant, to just a trace of the high, without points the side, (I) for (l), I think it would be easier to identify, thus no resemblance to (T) or with (C).
Now only lacking in the graphics menu.

Thank you
José Rodrigues

[bobale]

Excuse my ignorance, but does code from this first page in the thread run on Arduino Uno? And what compiler do you use to compile and upload code to the uC?

[t vago]

Yep, the code on the first page can run on an Arduino Uno, as long as you make the pin connections according to the MPGuino Wiki. The Uno is actually one of my test platforms, along with the Arduino Mega and the TinkerKit LCD module. You can use the Arduino IDE to upload, or the AVR development suite.

[NotJimCarrey]

I'm trying to build this in Arduino IDE 1.0.5 but get these errors:

mpguino_tav.ino:1025:38: error: ‘PORTF0’ was not declared in this scope
mpguino_tav.ino:1026:34: error: ‘PORTF1’ was not declared in this scope
mpguino_tav.ino:1027:36: error: ‘PORTE6’ was not declared in this scope
mpguino_tav.ino:1031:34: error: ‘PORTF4’ was not declared in this scope
In file included from mpguino_tav.ino:352:0:
mpguino_tav.ino:2972:61: error: variable ‘buttonPressAdrList’ must be const in order to be put into read-only section by means of ‘__attribute__((progmem))’
mpguino_tav.ino: In function ‘void __vector_21()’:
mpguino_tav.ino:3853:32: error: ‘MUX4’ was not declared in this scope
mpguino_tav.ino: In function ‘void LCD::init()’:
mpguino_tav.ino:4374:2: error: ‘DDRE’ was not declared in this scope
mpguino_tav.ino:4375:2: error: ‘DDRF’ was not declared in this scope
mpguino_tav.ino:4376:2: error: ‘PORTF’ was not declared in this scope
mpguino_tav.ino: In function ‘uint8_t LCD::writeNybble(uint8_t, uint8_t)’:
mpguino_tav.ino:4497:3: error: ‘PORTF’ was not declared in this scope
mpguino_tav.ino:4508:3: error: ‘PORTE’ was not declared in this scope
In file included from mpguino_tav.ino:352:0:
mpguino_tav.ino: At global scope:
mpguino_tav.ino:4753:34: error: variable ‘S64programList’ must be const in order to be put into read-only section by means of ‘__attribute__((progmem))’
mpguino_tav.ino: In function ‘int main()’:
mpguino_tav.ino:8026:17: error: ‘ADC7D’ was not declared in this scope
mpguino_tav.ino:8026:32: error: ‘ADC6D’ was not declared in this scope
mpguino_tav.ino:8027:2: error: ‘DIDR2’ was not declared in this scope
mpguino_tav.ino:8027:16: error: ‘ADC10D’ was not declared in this scope

Any ideas what I'm doing wrong?

[t vago]

The code is saved with this option enabled:

// if the below "#define"s are commented out, code will compile for an AtMega328-series processor
//#define ArduinoMega2560 true
#define TinkerkitLCDmodule true

That's at line 225. If you're compiling to an Arduino Mega2560, you'll need to comment out the "#define TinkerKitLCDmodule". If you're compiling to a base MPGuino board (or an Arduino Uno, or anything else), you'll need to comment out both #defines. Also, ensure that the Arduino 1.0.5 environment is set to compile to your target device.

Also, scroll down from line 225, read the comments, and comment out (or uncomment) the #defines for the other desired options.

Let me know how it goes.

[NotJimCarrey]

Thanks t vago, can't believe I missed that (Awesome work by the way)
Got it down to just 2 build errors:

In file included from mpguino_tav.ino:352:0:
mpguino_tav.ino:2972:61: error: variable ‘buttonPressAdrList’ must be const in order to be put into read-only section by means of ‘__attribute__((progmem))’
mpguino_tav.ino:4753:34: error: variable ‘S64programList’ must be const in order to be put into read-only section by means of ‘__attribute__((progmem))’

[t vago]

That's very strange... While I am happy that you like my version of this code, I'd like it more if we can get this code to actually compile for you.

In both the last stable version that I just downloaded, and my latest edit, the lines read as follows:

const char * buttonPressAdrList[(unsigned int)(screenSize)] PROGMEM = {

const uint8_t * S64programList[] PROGMEM = {

Also, I do not have any problem with compiling - I am also using the Arduino 1.0.5 IDE.

Please go to lines 2792 and 4753, and check that these lines are as what is listed above.

Also, can you tell me what board you are compiling to?

[NotJimCarrey]

Both lines are identical for me.
I'm trying to program an ATMEGA328P-PU via breadboard using an Arduino Micro as an ISP

[t vago]

Originally Posted by NotJimCarrey

Both lines are identical for me.
I'm trying to program an ATMEGA328P-PU via breadboard using an Arduino Micro as an ISP

Okay. Tried verifying to an Arduino Micro, still could not duplicate error. A Google search of "error: variable must be const in order to be put into read-only section by means of ‘__attribute__((progmem))’" turned up some results, mostly dealing with downgrading the underlying AVR compiler. One result did sound promising, though.

Try changing the two lines as follows:

const char * const buttonPressAdrList[(unsigned int)(screenSize)] PROGMEM = {

const uint8_t * const S64programList[] PROGMEM = {

The modifications do not cause any errors when I compile it, so maybe it will work for you?

[NotJimCarrey]

Sweet, that fixed it. Thanks