Re: [PATCH] ARM: tegra20: Store CPU "resettable" status in IRAM

2015-03-11 Thread Dmitry Osipenko
11.03.2015 13:29, Thierry Reding пишет: On Thu, Jan 15, 2015 at 01:58:57PM +0300, Dmitry Osipenko wrote: Commit 7232398abc6a ("ARM: tegra: Convert PMC to a driver") changed tegra_resume() location storing from late to early and, as a result, broke suspend on Tegra20. PMC scratch register 41 is

Re: [PATCH] ARM: tegra20: Store CPU "resettable" status in IRAM

2015-03-11 Thread Thierry Reding
On Thu, Jan 15, 2015 at 01:58:57PM +0300, Dmitry Osipenko wrote: > Commit 7232398abc6a ("ARM: tegra: Convert PMC to a driver") changed > tegra_resume() > location storing from late to early and, as a result, broke suspend on > Tegra20. > PMC scratch register 41 is used by tegra LP1 resume code

Re: [PATCH] ARM: tegra20: Store CPU resettable status in IRAM

2015-03-11 Thread Thierry Reding
On Thu, Jan 15, 2015 at 01:58:57PM +0300, Dmitry Osipenko wrote: Commit 7232398abc6a (ARM: tegra: Convert PMC to a driver) changed tegra_resume() location storing from late to early and, as a result, broke suspend on Tegra20. PMC scratch register 41 is used by tegra LP1 resume code for

Re: [PATCH] ARM: tegra20: Store CPU resettable status in IRAM

2015-03-11 Thread Dmitry Osipenko
11.03.2015 13:29, Thierry Reding пишет: On Thu, Jan 15, 2015 at 01:58:57PM +0300, Dmitry Osipenko wrote: Commit 7232398abc6a (ARM: tegra: Convert PMC to a driver) changed tegra_resume() location storing from late to early and, as a result, broke suspend on Tegra20. PMC scratch register 41 is

Re: [PATCH] ARM: tegra20: Store CPU "resettable" status in IRAM

2015-01-20 Thread Dmitry Osipenko
19.01.2015 21:26, Dmitry Osipenko пишет: 19.01.2015 21:00, Dmitry Osipenko пишет: 19.01.2015 20:45, Stephen Warren пишет: On 01/19/2015 10:41 AM, Dmitry Osipenko wrote: 19.01.2015 20:26, Stephen Warren пишет: Hopefully this works out. I suppose it's unlikely anyone will be running code on

Re: [PATCH] ARM: tegra20: Store CPU resettable status in IRAM

2015-01-20 Thread Dmitry Osipenko
19.01.2015 21:26, Dmitry Osipenko пишет: 19.01.2015 21:00, Dmitry Osipenko пишет: 19.01.2015 20:45, Stephen Warren пишет: On 01/19/2015 10:41 AM, Dmitry Osipenko wrote: 19.01.2015 20:26, Stephen Warren пишет: Hopefully this works out. I suppose it's unlikely anyone will be running code on

Re: [PATCH] ARM: tegra20: Store CPU "resettable" status in IRAM

2015-01-19 Thread Alexandre Courbot
On Mon, Jan 19, 2015 at 11:12 PM, Thierry Reding wrote: > On Thu, Jan 15, 2015 at 01:58:57PM +0300, Dmitry Osipenko wrote: >> Commit 7232398abc6a ("ARM: tegra: Convert PMC to a driver") changed >> tegra_resume() >> location storing from late to early and, as a result, broke suspend on >>

Re: [PATCH] ARM: tegra20: Store CPU "resettable" status in IRAM

2015-01-19 Thread Dmitry Osipenko
19.01.2015 21:00, Dmitry Osipenko пишет: 19.01.2015 20:45, Stephen Warren пишет: On 01/19/2015 10:41 AM, Dmitry Osipenko wrote: 19.01.2015 20:26, Stephen Warren пишет: Hopefully this works out. I suppose it's unlikely anyone will be running code on the AVP upstrem, so any potential conflict

Re: [PATCH] ARM: tegra20: Store CPU "resettable" status in IRAM

2015-01-19 Thread Dmitry Osipenko
19.01.2015 20:45, Stephen Warren пишет: On 01/19/2015 10:41 AM, Dmitry Osipenko wrote: 19.01.2015 20:26, Stephen Warren пишет: Hopefully this works out. I suppose it's unlikely anyone will be running code on the AVP upstrem, so any potential conflict with AVP's usage of IRAM isn't likely to

Re: [PATCH] ARM: tegra20: Store CPU "resettable" status in IRAM

2015-01-19 Thread Stephen Warren
On 01/19/2015 10:41 AM, Dmitry Osipenko wrote: 19.01.2015 20:26, Stephen Warren пишет: Hopefully this works out. I suppose it's unlikely anyone will be running code on the AVP upstrem, so any potential conflict with AVP's usage of IRAM isn't likely to occur. I don't see how it can conflict

Re: [PATCH] ARM: tegra20: Store CPU "resettable" status in IRAM

2015-01-19 Thread Dmitry Osipenko
19.01.2015 20:26, Stephen Warren пишет: Hopefully this works out. I suppose it's unlikely anyone will be running code on the AVP upstrem, so any potential conflict with AVP's usage of IRAM isn't likely to occur. I don't see how it can conflict with AVP code. First KB of IRAM is reserved for

Re: [PATCH] ARM: tegra20: Store CPU "resettable" status in IRAM

2015-01-19 Thread Stephen Warren
On 01/19/2015 07:12 AM, Thierry Reding wrote: On Thu, Jan 15, 2015 at 01:58:57PM +0300, Dmitry Osipenko wrote: Commit 7232398abc6a ("ARM: tegra: Convert PMC to a driver") changed tegra_resume() location storing from late to early and, as a result, broke suspend on Tegra20. PMC scratch register

Re: [PATCH] ARM: tegra20: Store CPU "resettable" status in IRAM

2015-01-19 Thread Dmitry Osipenko
19.01.2015 17:12, Thierry Reding пишет: On Thu, Jan 15, 2015 at 01:58:57PM +0300, Dmitry Osipenko wrote: Commit 7232398abc6a ("ARM: tegra: Convert PMC to a driver") changed tegra_resume() location storing from late to early and, as a result, broke suspend on Tegra20. PMC scratch register 41 is

Re: [PATCH] ARM: tegra20: Store CPU "resettable" status in IRAM

2015-01-19 Thread Thierry Reding
On Thu, Jan 15, 2015 at 01:58:57PM +0300, Dmitry Osipenko wrote: > Commit 7232398abc6a ("ARM: tegra: Convert PMC to a driver") changed > tegra_resume() > location storing from late to early and, as a result, broke suspend on > Tegra20. > PMC scratch register 41 is used by tegra LP1 resume code

Re: [PATCH] ARM: tegra20: Store CPU resettable status in IRAM

2015-01-19 Thread Alexandre Courbot
On Mon, Jan 19, 2015 at 11:12 PM, Thierry Reding thierry.red...@gmail.com wrote: On Thu, Jan 15, 2015 at 01:58:57PM +0300, Dmitry Osipenko wrote: Commit 7232398abc6a (ARM: tegra: Convert PMC to a driver) changed tegra_resume() location storing from late to early and, as a result, broke

Re: [PATCH] ARM: tegra20: Store CPU resettable status in IRAM

2015-01-19 Thread Thierry Reding
On Thu, Jan 15, 2015 at 01:58:57PM +0300, Dmitry Osipenko wrote: Commit 7232398abc6a (ARM: tegra: Convert PMC to a driver) changed tegra_resume() location storing from late to early and, as a result, broke suspend on Tegra20. PMC scratch register 41 is used by tegra LP1 resume code for

Re: [PATCH] ARM: tegra20: Store CPU resettable status in IRAM

2015-01-19 Thread Dmitry Osipenko
19.01.2015 17:12, Thierry Reding пишет: On Thu, Jan 15, 2015 at 01:58:57PM +0300, Dmitry Osipenko wrote: Commit 7232398abc6a (ARM: tegra: Convert PMC to a driver) changed tegra_resume() location storing from late to early and, as a result, broke suspend on Tegra20. PMC scratch register 41 is

Re: [PATCH] ARM: tegra20: Store CPU resettable status in IRAM

2015-01-19 Thread Dmitry Osipenko
19.01.2015 20:26, Stephen Warren пишет: Hopefully this works out. I suppose it's unlikely anyone will be running code on the AVP upstrem, so any potential conflict with AVP's usage of IRAM isn't likely to occur. I don't see how it can conflict with AVP code. First KB of IRAM is reserved for

Re: [PATCH] ARM: tegra20: Store CPU resettable status in IRAM

2015-01-19 Thread Stephen Warren
On 01/19/2015 07:12 AM, Thierry Reding wrote: On Thu, Jan 15, 2015 at 01:58:57PM +0300, Dmitry Osipenko wrote: Commit 7232398abc6a (ARM: tegra: Convert PMC to a driver) changed tegra_resume() location storing from late to early and, as a result, broke suspend on Tegra20. PMC scratch register

Re: [PATCH] ARM: tegra20: Store CPU resettable status in IRAM

2015-01-19 Thread Stephen Warren
On 01/19/2015 10:41 AM, Dmitry Osipenko wrote: 19.01.2015 20:26, Stephen Warren пишет: Hopefully this works out. I suppose it's unlikely anyone will be running code on the AVP upstrem, so any potential conflict with AVP's usage of IRAM isn't likely to occur. I don't see how it can conflict

Re: [PATCH] ARM: tegra20: Store CPU resettable status in IRAM

2015-01-19 Thread Dmitry Osipenko
19.01.2015 20:45, Stephen Warren пишет: On 01/19/2015 10:41 AM, Dmitry Osipenko wrote: 19.01.2015 20:26, Stephen Warren пишет: Hopefully this works out. I suppose it's unlikely anyone will be running code on the AVP upstrem, so any potential conflict with AVP's usage of IRAM isn't likely to

Re: [PATCH] ARM: tegra20: Store CPU resettable status in IRAM

2015-01-19 Thread Dmitry Osipenko
19.01.2015 21:00, Dmitry Osipenko пишет: 19.01.2015 20:45, Stephen Warren пишет: On 01/19/2015 10:41 AM, Dmitry Osipenko wrote: 19.01.2015 20:26, Stephen Warren пишет: Hopefully this works out. I suppose it's unlikely anyone will be running code on the AVP upstrem, so any potential conflict

[PATCH] ARM: tegra20: Store CPU "resettable" status in IRAM

2015-01-15 Thread Dmitry Osipenko
Commit 7232398abc6a ("ARM: tegra: Convert PMC to a driver") changed tegra_resume() location storing from late to early and, as a result, broke suspend on Tegra20. PMC scratch register 41 is used by tegra LP1 resume code for retrieving stored physical memory address of common resume function and

[PATCH] ARM: tegra20: Store CPU resettable status in IRAM

2015-01-15 Thread Dmitry Osipenko
Commit 7232398abc6a (ARM: tegra: Convert PMC to a driver) changed tegra_resume() location storing from late to early and, as a result, broke suspend on Tegra20. PMC scratch register 41 is used by tegra LP1 resume code for retrieving stored physical memory address of common resume function and in