http://forums.leaflabs.com/topic.php?id=550 A place to share, learn, and grow... en-US Fri, 22 Jan 2016 00:22:20 +0000 http://bbpress.org/?v=1.0.2 q http://forums.leaflabs.com/search.php http://forums.leaflabs.com/topic.php?id=550#post-3094 Sun, 12 Dec 2010 02:10:26 +0000 mbolivar 3094@http://forums.leaflabs.com/ <p>structuresound -- thanks!</p> <p>will be having a look at this once 0.0.9 is out. </p> http://forums.leaflabs.com/topic.php?id=550#post-3050 Wed, 08 Dec 2010 02:17:40 +0000 structuresound 3050@http://forums.leaflabs.com/ <p>mbolvar - I don't know much about contributing to repositories like github yet, so I've put the flash.c and flash.h files for you here <a href="http://www.structuresound.com/maple/" rel="nofollow">http://www.structuresound.com/maple/</a></p> <p>I would like to learn how to work with that kind of environment because I'm interested in contributing to the codebase, I'm just a little timid about it because I'm still very "novice" at coding in general.</p> <p>These don't have function definitions or implementations for the read/write functions that we've been discussing. </p> http://forums.leaflabs.com/topic.php?id=550#post-3046 Tue, 07 Dec 2010 23:11:26 +0000 mbolivar 3046@http://forums.leaflabs.com/ <p>gbulmer - no, nothing stopping you, of course.</p> <p>structuresound - is this up anywhere? </p> http://forums.leaflabs.com/topic.php?id=550#post-2968 Sat, 04 Dec 2010 01:36:06 +0000 structuresound 2968@http://forums.leaflabs.com/ <p>Also, in order to get things ready I wrapped the relevant functions from bootloader hardware.c/h into libmaple flash.c/h which requires a few of the defines from hardware.h, changing to the libmaple typdefs (i.e. u32 to uint32), include libmaple.h, and include usb_hardware.h</p> <p>This is my first time "porting" something albeit from one version of the same library to another, so I don't totally know what I'm doing but it seems to work great "in practice".</p> <p>One weird thing is that I now get some compiler warnings for redefines between usb_hardware and rcc neither of which I modified and didn't repeat any of those defines in flash.h. They don't seem to halt the compile so I'm not sure what's up.</p> <p>Probably wouldn't take much more effort to wrap a couple of overloaded easy read/write functions make sure the pageclear is never less than 0x08005000 and call it a user eeprom library. </p> http://forums.leaflabs.com/topic.php?id=550#post-2967 Sat, 04 Dec 2010 01:21:44 +0000 structuresound 2967@http://forums.leaflabs.com/ <p>bless you gbulmer, the union works like a charm. Only problem was that flashWriteWord doesn't want a pointer (it converts the address to a pointer internally, not sure why it's written this way). Here's what I did, to write use</p> <p>assuming myArray is the global variable I'm interested in storing<br /> also I hardcoded 64 in there instead of sizeOf probably will return to "macro'd" now that everything is squared away.</p> <pre><code>void writeFlash(){ uint32 flashPtr = (uint32)0x08015000L; flashErasePage(flashPtr); union { uint32 ui; uint8 ub[4]; } u; int i; u.ui = 0; for (i=0; i&lt;64; i++) { u.ub[i%4] = myArray[i]; if (i%4 == 3) { // time to write it flashWriteWord(flashPtr, u.ui); flashPtr += 0x04; u.ui = 0; } } }</code></pre> <p>to read the data back out of memory</p> <pre><code>void readFlash(){ uint32 flashPtr = (uint32)0x08015000L; union { uint32 ui; uint8 ub[4]; } u; int i; u.ui = 0; for (i=0; i&lt;64; i++) { if (i%4 == 0) { u.ui = *((uint32*)flashPtr); flashPtr += 0x04; } myArray[i] = u.ub[i%4]; } }</code></pre> http://forums.leaflabs.com/topic.php?id=550#post-2964 Fri, 03 Dec 2010 21:15:30 +0000 gbulmer 2964@http://forums.leaflabs.com/ <p>Argh!£%^&amp;*</p> <p>The formatting on this forum drives me nuts!<br /> I can't get the spaces in front of <code>uint32 *flashPtr = (uint32 *)0x08010000L;</code> to stay!</p> <p>GHAAARG!-( </p> http://forums.leaflabs.com/topic.php?id=550#post-2963 Fri, 03 Dec 2010 21:02:43 +0000 gbulmer 2963@http://forums.leaflabs.com/ <p>Spoiler alert<br /> - This contains a suggestion for code<br /> - ignore if you are having fun doing it your way</p> <p>If you have a byte array, you can do a truly horrible tricks with a pointer, but I won't encourage you (hint, use a pointer to the enum :-)</p> <p>Copy hardware.c and using my enum example above, do something like:<br /> <pre><code>uint32 *flashPtr = (uint32 *)0x08010000L; union { uint32 ui; uint8 ub[4]; } u; uint8 buff[100]; int i; u.ui = 0; for (i=0; i&lt;sizeof(buff); i++) { u.ub[i%4] = buff[i]; if (i%4 == 3) { // time to write it flashWriteWord(flashPtr, u.ui); ++flashPtr; u.ui = 0; } } // now tidy up any remainder of 1, 2, or 3 bytes // using flashWriteWord(flashPtr, u.ui);</code></pre> <p>(I didn't want to take away all the fun, so I left tidying up TODO :-)</p> <p>It isn't as compact as using a pointer, but avoids any issues of reading from non-aligned uint32's (which AFAIK is fine on Cortex-M3, but isn't supported on all ARM architectures)</p> <p>WARNING - I have not tested this code. </p> http://forums.leaflabs.com/topic.php?id=550#post-2961 Fri, 03 Dec 2010 20:25:58 +0000 structuresound 2961@http://forums.leaflabs.com/ <p>okay, cool that seems easier, I was trying this method:<br /> <pre><code>long l = 0; l |= b[0] &amp; 0xFF; l &lt;&lt;= 8; l |= b[1] &amp; 0xFF; l &lt;&lt;= 8; l |= b[2] &amp; 0xFF; l &lt;&lt;= 8; l |= b[3] &amp; 0xFF;</code></pre> <p>now the byte arrays I want to copy are larger than 4 bytes, maybe I can use a 32 bit array pointer to step through 4 bytes at a time?</p> <p>I'll try a direct implementation of hardware.c and see what I can come up with. </p> http://forums.leaflabs.com/topic.php?id=550#post-2959 Fri, 03 Dec 2010 19:34:41 +0000 gbulmer 2959@http://forums.leaflabs.com/ <p>mbolivar - I assume there is nothing to stop us just using hardware.c directly. Correct?<br /> Maybe, at some point in the future, factor out from hardware.c the flash functions, and put into their own file so that the Maple library API can use them too?</p> <p>structuresound - I'd just use hardware.c directly.</p> <p>All you need to do is pack 4 bytes into an unsigned int, or uint32 to avoid possible confusion. </p> <p>Packing can be done at the bit level with shifts, and's and or's, or you could try it with a <code>union</code>. You likely know this, but for the benefit of readers who follow ...</p> <p>Whenever I use a <code>union</code>, I always do a quick check to make sure it is going to work the way I want, e.g.</p> <pre><code>void loop() { union { uint32 ui; uint8 ub[4]; } u; if ( sizeof(u) == 4 ) { // Yipee, game on SerialUSB.print(&quot;sizeof(u) is correctly reported as 4 bytes: &quot;); SerialUSB.println(sizeof(u)); } else { SerialUSB.print(&quot;Unions will not work, sizeof(u) is not 4 bytes, it is reported as: &quot;); SerialUSB.println(sizeof(u)); } }</code></pre> <p>I do the check because the C standard says that the behaviour of unions is implementation defined, so it can vary from compiler to compiler. But, IMHO, if it does work, byte packing using a union is easier to understand than bit packing.</p> <p>EDIT: WARNING - I have not tested this code. </p> http://forums.leaflabs.com/topic.php?id=550#post-2957 Fri, 03 Dec 2010 18:41:40 +0000 mbolivar 2957@http://forums.leaflabs.com/ <p>structuresound -- yup, there's (currently) no libmaple support for writing flash, and there's no time-line in place for adding that support. as you discovered, we do it in the bootloader only. it's not on the critical path right now, although it's certainly reasonable for flash write support to get added to (at least) the libmaple C api. </p> http://forums.leaflabs.com/topic.php?id=550#post-2954 Fri, 03 Dec 2010 17:32:53 +0000 structuresound 2954@http://forums.leaflabs.com/ <p>Seems like there are two prerequisites to read/write memory</p> <p>1) Set up RCC<br /> 2) Unlock Flash</p> <p>The methods for this are integrated into the bootloader hardware.h but not in the normal libmaple.</p> <p>Does it seem like a better solution to borrow these from the bootloader or to try and port STM's eeprom.h so that it fits in libmaple?</p> <p>If anybody else is successfully reading/writing words to upper flash around say 0x08010000 I'd love to hear about it, otherwise I'll keep plugging away and report back if I have any success. </p> http://forums.leaflabs.com/topic.php?id=550#post-2942 Thu, 02 Dec 2010 22:06:31 +0000 structuresound 2942@http://forums.leaflabs.com/ <p>Any further developments on flash write / read?</p> <p>Working with flashWriteWord, but seems a little strange, I'm trying to deal with u8[]'s but that function seems to want u32's that then get chopped to two u16's to be written sequentially. I think I can do some bitwise copying and end up okay but I was wondering if anyone else had seen a simpler implementation for copying straight to memory with an index of 1 byte. </p>