Re: [PATCH] ARM: hw_breakpoint: Enable debug powerdown only if system supports 'has_ossr'

2013-03-28 Thread Dietmar Eggemann
On 19/03/13 10:28, Will Deacon wrote: On Tue, Mar 19, 2013 at 06:39:38AM +, Santosh Shilimkar wrote: On Monday 18 March 2013 10:36 PM, Will Deacon wrote: Any chance you could follow up with your firmware/hardware guys about this please? I'd really like to understand how we end up in this st

Re: [PATCH] ARM: hw_breakpoint: Enable debug powerdown only if system supports 'has_ossr'

2013-03-25 Thread Santosh Shilimkar
On Monday 25 March 2013 04:19 PM, Will Deacon wrote: > On Mon, Mar 25, 2013 at 09:11:00AM +, Santosh Shilimkar wrote: >> Will, > > Hi Santosh, > >> Are you going to send the patch for 3.9-rcx ? As I said before without the >> patch OMAP4 CPUILDE is unusable because of that debug noise and hen

Re: [PATCH] ARM: hw_breakpoint: Enable debug powerdown only if system supports 'has_ossr'

2013-03-25 Thread Will Deacon
On Mon, Mar 25, 2013 at 09:11:00AM +, Santosh Shilimkar wrote: > Will, Hi Santosh, > Are you going to send the patch for 3.9-rcx ? As I said before without the > patch OMAP4 CPUILDE is unusable because of that debug noise and hence it > will be good to get that patch in It's in Russell's tre

Re: [PATCH] ARM: hw_breakpoint: Enable debug powerdown only if system supports 'has_ossr'

2013-03-25 Thread Santosh Shilimkar
Will, On Tuesday 19 March 2013 03:58 PM, Will Deacon wrote: > On Tue, Mar 19, 2013 at 06:39:38AM +, Santosh Shilimkar wrote: >> On Monday 18 March 2013 10:36 PM, Will Deacon wrote: [..] >>> Well, we could just add the warn_once prints but that doesn't stop debug >>> from breaking after the f

Re: [PATCH] ARM: hw_breakpoint: Enable debug powerdown only if system supports 'has_ossr'

2013-03-19 Thread Will Deacon
On Tue, Mar 19, 2013 at 06:39:38AM +, Santosh Shilimkar wrote: > On Monday 18 March 2013 10:36 PM, Will Deacon wrote: > > Any chance you could follow up with your firmware/hardware guys about this > > please? I'd really like to understand how we end up in this state in case we > > can do someth

Re: [PATCH] ARM: hw_breakpoint: Enable debug powerdown only if system supports 'has_ossr'

2013-03-18 Thread Santosh Shilimkar
On Monday 18 March 2013 10:36 PM, Will Deacon wrote: > On Mon, Mar 18, 2013 at 03:46:28PM +, Santosh Shilimkar wrote: >> On Monday 18 March 2013 08:37 PM, Will Deacon wrote: >>> That really sucks :( Does this affect all OMAP-based boards? >>> >> All OMAP4 based boards.. > > Brilliant. Is there

Re: [PATCH] ARM: hw_breakpoint: Enable debug powerdown only if system supports 'has_ossr'

2013-03-18 Thread Will Deacon
On Mon, Mar 18, 2013 at 03:46:28PM +, Santosh Shilimkar wrote: > On Monday 18 March 2013 08:37 PM, Will Deacon wrote: > > That really sucks :( Does this affect all OMAP-based boards? > > > All OMAP4 based boards.. Brilliant. Is there any way that the secure code can be fixed in future product

Re: [PATCH] ARM: hw_breakpoint: Enable debug powerdown only if system supports 'has_ossr'

2013-03-18 Thread Santosh Shilimkar
On Monday 18 March 2013 08:37 PM, Will Deacon wrote: > Hi Santosh, > > On Mon, Mar 18, 2013 at 06:51:30AM +, Santosh Shilimkar wrote: >> On Friday 15 March 2013 10:30 AM, Will Deacon wrote: >>> Furthermore, I was under the impression that hw_breakpoint did actually >>> work on panda, which imp

Re: [PATCH] ARM: hw_breakpoint: Enable debug powerdown only if system supports 'has_ossr'

2013-03-18 Thread Will Deacon
Hi Santosh, On Mon, Mar 18, 2013 at 06:51:30AM +, Santosh Shilimkar wrote: > On Friday 15 March 2013 10:30 AM, Will Deacon wrote: > > Furthermore, I was under the impression that hw_breakpoint did actually > > work on panda, which implies that a cold boot *does* manage to reset the > > registe

Re: [PATCH] ARM: hw_breakpoint: Enable debug powerdown only if system supports 'has_ossr'

2013-03-17 Thread Santosh Shilimkar
On Friday 15 March 2013 10:30 AM, Will Deacon wrote: > On Thu, Mar 14, 2013 at 01:08:00PM +0530, Santosh Shilimkar wrote: >> Will, > > Hi guys, > > I'm out of the office at the moment and have really terrible connectivity, > so I can't do too much until next week. However, I don't think adding th

Re: [PATCH] ARM: hw_breakpoint: Enable debug powerdown only if system supports 'has_ossr'

2013-03-14 Thread Will Deacon
On Thu, Mar 14, 2013 at 01:08:00PM +0530, Santosh Shilimkar wrote: > Will, Hi guys, I'm out of the office at the moment and have really terrible connectivity, so I can't do too much until next week. However, I don't think adding the has_ossr check is the right fix for this problem. > On Wednesda

Re: [PATCH] ARM: hw_breakpoint: Enable debug powerdown only if system supports 'has_ossr'

2013-03-14 Thread Kevin Hilman
Lokesh Vutla writes: > Commit {9a6eb31 ARM: hw_breakpoint: Debug powerdown support for self-hosted > debug} introduces debug powerdown support for self-hosted debug. > While merging the patch 'has_ossr' check was removed which > was needed for hardwares which doesn't support self-hosted debug. >

Re: [PATCH] ARM: hw_breakpoint: Enable debug powerdown only if system supports 'has_ossr'

2013-03-14 Thread Santosh Shilimkar
Will, On Wednesday 13 March 2013 05:59 PM, Lokesh Vutla wrote: > Hi Dietmar, > On Wednesday 13 March 2013 05:35 PM, Dietmar Eggemann wrote: >> On 13/03/13 06:52, Lokesh Vutla wrote: >>> Commit {9a6eb31 ARM: hw_breakpoint: Debug powerdown support for >>> self-hosted >>> debug} introduces debug powe

Re: [PATCH] ARM: hw_breakpoint: Enable debug powerdown only if system supports 'has_ossr'

2013-03-13 Thread Lokesh Vutla
Hi Dietmar, On Wednesday 13 March 2013 05:35 PM, Dietmar Eggemann wrote: On 13/03/13 06:52, Lokesh Vutla wrote: Commit {9a6eb31 ARM: hw_breakpoint: Debug powerdown support for self-hosted debug} introduces debug powerdown support for self-hosted debug. While merging the patch 'has_ossr' check wa

Re: [PATCH] ARM: hw_breakpoint: Enable debug powerdown only if system supports 'has_ossr'

2013-03-13 Thread Dietmar Eggemann
On 13/03/13 06:52, Lokesh Vutla wrote: Commit {9a6eb31 ARM: hw_breakpoint: Debug powerdown support for self-hosted debug} introduces debug powerdown support for self-hosted debug. While merging the patch 'has_ossr' check was removed which was needed for hardwares which doesn't support self-hosted

[PATCH] ARM: hw_breakpoint: Enable debug powerdown only if system supports 'has_ossr'

2013-03-12 Thread Lokesh Vutla
Commit {9a6eb31 ARM: hw_breakpoint: Debug powerdown support for self-hosted debug} introduces debug powerdown support for self-hosted debug. While merging the patch 'has_ossr' check was removed which was needed for hardwares which doesn't support self-hosted debug. Pandaboard (A9) is one such hardw