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

2010-12-05 Thread freddie_chopin
"David Brownell" 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 on. ? 4\/3!

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

2010-12-03 Thread David Brownell
--- On Fri, 12/3/10, Spencer Oliver wrote: > Date: Friday, December 3, 2010, 2:34 PM > On 03/12/2010 17:24, David Brownell > wrote: > > > > > > --- On Fri, 12/3/10, Spencer Oliver  > wrote: > Sorry i am slightly confused this patch as such has nothing to do with > the standard reset_config.

Re: [Openocd-development] [PATCH 1/2] add " cortex m3 reset config= = ..." 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 " cortex m3 reset config= = ..." to stm32.cfg

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

Re: [Openocd-development] [PATCH 1/2] add " cortex m3 reset config= = ..." 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 " cortex m3 reset config= = ..." 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 " cortex m3 reset config= = ..." to stm32.cfg

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

Re: [Openocd-development] [PATCH 1/2] add " cortex m3 reset config= = ..." 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 Oliver" 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 exam

Re: [Openocd-development] [PATCH 1/2] add " cortex m3 reset config = ..." 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. (The

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

2010-12-02 Thread Spencer Oliver
On 02/12/2010 12:42, freddie_cho...@op.pl wrote: "Spencer Oliver" 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

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

2010-12-02 Thread freddie_chopin
"Spencer Oliver" 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 patch would b

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

2010-12-02 Thread Spencer Oliver
On 02/12/2010 11:55, freddie_cho...@op.pl wrote: "Spencer Oliver" 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 use

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

2010-12-02 Thread freddie_chopin
"Spencer Oliver" 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 whom does that