These are chat archives for SmingHub/Sming

13th
Sep 2016
frankdownunder
@frankdownunder
Sep 13 2016 02:00
@hreintke: rboot_Generic.cpp:
commandOutput->printf("Divide test %d, %d ");
probably should be:
commandOutput->printf("Divide test %d, %d \r\n", a, b); or similar.
Also - do you agree with me wrt IRAM_ATTR being nee3ded in Timer.h?
frankdownunder
@frankdownunder
Sep 13 2016 08:09
@hreintke Also in rboot_Generic, we have
else if (numToken = 2)
which is probably meant to be
else if (numToken == 2)
slaff
@slaff
Sep 13 2016 14:10

Porting axtls patch to RTOS is somewhere on my todolist, if slaff is not interested in that.

@Informatic axTLS for RTOS should be relatively easy. I haven't checked the development of the RTOS SDK but if the snmp functions are still not there then they can be ported from LWIP.

Piotr Dobrowolski
@Informatic
Sep 13 2016 17:09
@slaff thanks for your work on initial patch <3
hreintke
@hreintke
Sep 13 2016 21:05
@frankdownunder :
To which function you added the IRAM_ATTR attribute. The static processing function already has it.
frankdownunder
@frankdownunder
Sep 13 2016 21:07
@frankdownunder : I added it everwhere in Timer.h that had it before - just to test if it worked.
hreintke
@hreintke
Sep 13 2016 21:16
@frankdownunder :
You are right about rboot_generic. Leftover from my earlier testing. Will update.
hreintke
@hreintke
Sep 13 2016 21:23
Don't yet understand why the other functions would need the IRAM ATTR.
Difficult to test as with me it works, at least for the basic_rboot.
frankdownunder
@frankdownunder
Sep 13 2016 21:23
What does IRAM_ATTR do? I couldnt find the define for it
hreintke
@hreintke
Sep 13 2016 21:28
Functions without it are in flash and run from there.
IRAM_ATTR forces the function always to be in ram.
Needed for real interrupt routines as within them you cannot call "flash functions"
frankdownunder
@frankdownunder
Sep 13 2016 22:10
@hreintke I will retest and see if removal of IRAM_ATTR makes the LoadStoreAlignmentCause exception 9 come back.