Feel free to submit patches for these!  :)

Priority on regression fixes, but any fixes that don't de-stabilize
are still good to go.

- Dave


On Wednesday 20 January 2010, Edgar Grimberg wrote:
> 
> * I see some error messages for STR710 config:
> Error: command 'str7x' is already registered in '<global>' context
> Error: command 'disable_jtag' is already registered in 'str7x' context

That seems like a new one.  It's "just" scarey/cosmetic, yes?
I'd call that worth fixing, regardless; most users have no good
way to know if a given message is safe to ignore.


> * For STR710, I see wrong protected information in flash info:
> 
> > flash protect 0 0 last off
> cleared protection for sectors 0 through 7 on flash bank 0
> > flash info 0
> #0 : str7x at 0x40000000, size 0x00040000, buswidth 0, chipwidth 0
>     # 0: 0x00000000 (0x2000 8kB) protected
>     # 1: 0x00002000 (0x2000 8kB) protected
>     # 2: 0x00004000 (0x2000 8kB) protected
>     # 3: 0x00006000 (0x2000 8kB) protected
>     # 4: 0x00008000 (0x8000 32kB) protected
>     # 5: 0x00010000 (0x10000 64kB) protected
>     # 6: 0x00020000 (0x10000 64kB) protected
>     # 7: 0x00030000 (0x10000 64kB) protected

Is that a regression or a longstanding bug?  Judging by the
history in git, I'd guess it's the latter.


> * Formatting for mdh is wrong. Look at 0s:
> > mdh 0x0 32
> 0x00000000: beef beef beef beef beef beef beef beef beef beef beef
> beef beef beef beef beef
> 0x00000020: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00

Again, I'm thinking this is a longstanding bug.

Trivially fixable:  "%4.4x" should make that take 4 chars too.

Those lines should only have half as many value, too...
_______________________________________________
Openocd-development mailing list
Openocd-development@lists.berlios.de
https://lists.berlios.de/mailman/listinfo/openocd-development

Reply via email to