gnupic@linuxhacker.org

Scott Dattalo wrote:

>[snip]
>
>I made a mistake. I essentialy issued an -ep instead of an -ef.
>
>
>  
>

I read somewhere that support for erasing only program memory space was 
removed from picstart's firmware. Anyway, it seems to me that the way 
it's currently implemented on picp would'nt work correctly on most flash 
parts anyway. I may be blindly guessing here.


> [snip]
>
>You may wish to consider this:
>
>It appears that the configuration fuses are read from the hex file, but 
>only the first word is getting programmed. At least when I attempt to 
>program the CP bits there's no effect. I glanced briefly at the code and 
>it appears that DoWritePgm only checks the case when an address is equal 
>to the start of configuration memory and *not* when it is in the range of 
>configuration memory. 
>  
>

Yes, that's the original picp's behaviour. I wasn't aware of this, 
because I only played with some hex files generated from mplab, which 
always had the write done at the boundary-start addresses. Thanks for 
pointing that out! :-)

In fact, there's a demand for region checks in various parts of the code 
yet.

>However, the command line option for writing the configuration bits looks
>like it'll write more of the configuration bits. OTOH, it still will not
>write all of them. picdev.c says that the configuration space for the
>18f452 is only 7 bytes. That's not true. There are 34 configuration bits
>spanning 13 configuration addresses. Perhaps the '7' refers to the number
>of 16-bit words and not 8-bit bytes? In any case, a -wc 0x0 only will 
>clear the first 7 or so configuration addresses.
>
>Scott
>
>  
>

All -w options other than -wp are broken to some extent (-wc is broken 
hard if I remember well). I plan to correct that soon.

The configuration size is in words. I remember checking the size against 
the datasheet info, but I'll recheck on that later.

I also plan some changes on command-line handling, but I'll do some sort 
of poll here first to grab some thoughs :-)


Cheers,

Alexandre