Re: [Openocd-development] [PATCH 1/2] add quot; cortex m3 reset config= = = ...quot; to stm32.cfg

2010-12-05 Thread freddie_chopin
David Brownell davi...@pacbell.net napisał(a): I think Freddie's comment (that it does have something to do with it) made confusion, then. Which comment? That would be explained by my trusting Freddie's comment to be accurate, i while it was instead ncorrect at the levels I was commenting

Re: [Openocd-development] [PATCH 1/2] add quot; cortex m3 reset config= = ...quot; to stm32.cfg

2010-12-03 Thread Spencer Oliver
On 02/12/2010 13:15, Spencer Oliver wrote: On 02/12/2010 12:42, freddie_cho...@op.pl wrote: Spencer Olivers...@spen-soft.co.uk napisał(a): As we know the current behaviour of cortex_m3 reset_config is to override the std 'reset_config' setting - this has undesired effects for people who

Re: [Openocd-development] [PATCH 1/2] add quot; cortex m3 reset config= = ...quot; to stm32.cfg

2010-12-03 Thread David Brownell
--- On Fri, 12/3/10, Spencer Oliver s...@spen-soft.co.uk wrote: On 02/12/2010 12:42, freddie_cho...@op.pl wrote: Spencer Olivers...@spen-soft.co.uk napisał(a): As we know the current behaviour of cortex_m3 reset_config is to override the std 'reset_config' setting - I've lost

Re: [Openocd-development] [PATCH 1/2] add quot; cortex m3 reset config= = ...quot; to stm32.cfg

2010-12-03 Thread Freddie Chopin
On 2010-12-03 12:49, Spencer Oliver wrote: On 02/12/2010 13:15, Spencer Oliver wrote: How about something like this: http://repo.or.cz/w/openocd/ntfreak.git/shortlog/refs/heads/stm32reset No objections to this then i will commit later today. If I understand your path correctly - it removes

Re: [Openocd-development] [PATCH 1/2] add quot; cortex m3 reset config= = ...quot; to stm32.cfg

2010-12-03 Thread Spencer Oliver
On 03/12/2010 19:13, Freddie Chopin wrote: On 2010-12-03 12:49, Spencer Oliver wrote: On 02/12/2010 13:15, Spencer Oliver wrote: How about something like this: http://repo.or.cz/w/openocd/ntfreak.git/shortlog/refs/heads/stm32reset No objections to this then i will commit later today. If I

Re: [Openocd-development] [PATCH 1/2] add quot; cortex m3 reset config= = ...quot; to stm32.cfg

2010-12-03 Thread Spencer Oliver
On 03/12/2010 17:24, David Brownell wrote: --- On Fri, 12/3/10, Spencer Olivers...@spen-soft.co.uk wrote: On 02/12/2010 12:42, freddie_cho...@op.pl wrote: Spencer Olivers...@spen-soft.co.uk napisał(a): As we know the current behaviour of cortex_m3 reset_config is to override the std

Re: [Openocd-development] [PATCH 1/2] add quot; cortex m3 reset config= = ...quot; to stm32.cfg

2010-12-03 Thread Freddie Chopin
On 2010-12-03 23:29, Spencer Oliver wrote: On 03/12/2010 19:13, Freddie Chopin wrote: On 2010-12-03 12:49, Spencer Oliver wrote: On 02/12/2010 13:15, Spencer Oliver wrote: How about something like this: http://repo.or.cz/w/openocd/ntfreak.git/shortlog/refs/heads/stm32reset No objections to

Re: [Openocd-development] [PATCH 1/2] add quot; cortex m3 reset config= = ...quot; to stm32.cfg

2010-12-03 Thread David Brownell
--- On Fri, 12/3/10, Spencer Oliver s...@spen-soft.co.uk wrote: Date: Friday, December 3, 2010, 2:34 PM On 03/12/2010 17:24, David Brownell wrote: --- On Fri, 12/3/10, Spencer Olivers...@spen-soft.co.uk  wrote: Sorry i am slightly confused this patch as such has nothing to do with

Re: [Openocd-development] [PATCH 1/2] add quot; cortex m3 reset config = ...quot; to stm32.cfg

2010-12-02 Thread freddie_chopin
Spencer Oliver s...@spen-soft.co.uk napisał(a): I thought we has decided to work on the ability to make openocd a bit more reset aware for cortex-m3. This is just for now, as maybe a new bug-fix release will be made. This fixes the case for some users, but will break it for others. For

Re: [Openocd-development] [PATCH 1/2] add quot; cortex m3 reset config = ...quot; to stm32.cfg

2010-12-02 Thread Spencer Oliver
On 02/12/2010 11:55, freddie_cho...@op.pl wrote: Spencer Olivers...@spen-soft.co.uk napisał(a): I thought we has decided to work on the ability to make openocd a bit more reset aware for cortex-m3. This is just for now, as maybe a new bug-fix release will be made. This fixes the

Re: [Openocd-development] [PATCH 1/2] add quot; cortex m3 reset config= = ...quot; to stm32.cfg

2010-12-02 Thread freddie_chopin
Spencer Oliver s...@spen-soft.co.uk napisał(a): As we know the current behaviour of cortex_m3 reset_config is to override the std 'reset_config' setting - this has undesired effects for people who expect srst for example to work. We need to change that functionality then adding your

Re: [Openocd-development] [PATCH 1/2] add quot; cortex m3 reset config= = ...quot; to stm32.cfg

2010-12-02 Thread Spencer Oliver
On 02/12/2010 12:42, freddie_cho...@op.pl wrote: Spencer Olivers...@spen-soft.co.uk napisał(a): As we know the current behaviour of cortex_m3 reset_config is to override the std 'reset_config' setting - this has undesired effects for people who expect srst for example to work.

Re: [Openocd-development] [PATCH 1/2] add quot; cortex m3 reset config = ...quot; to stm32.cfg

2010-12-02 Thread Peter Stuge
Spencer Oliver wrote: This does create an issue for all the luminary scripts however. They would need the 'reset_config srst_only' removing and this would break all the older scripts in the wild. With a significant improvement to our data model I think it's OK to break existing scripts. (They