Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1642622 > unrolled thread
| Started by | Tony Lindgren <tony@atomide.com> |
|---|---|
| First post | 2017-05-16 17:50 +0200 |
| Last post | 2017-05-18 17:00 +0200 |
| Articles | 4 — 2 participants |
Back to article view | Back to linux.kernel
This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by
below is the oldest one visible, not the original post.
Re: [PATCH 0/8] ARM: OMAP2+: AM33XX/AM43XX: Add suspend-resume support Tony Lindgren <tony@atomide.com> - 2017-05-16 17:50 +0200
Re: [PATCH 0/8] ARM: OMAP2+: AM33XX/AM43XX: Add suspend-resume support Dave Gerlach <d-gerlach@ti.com> - 2017-05-16 18:00 +0200
Re: [PATCH 0/8] ARM: OMAP2+: AM33XX/AM43XX: Add suspend-resume support Dave Gerlach <d-gerlach@ti.com> - 2017-05-18 16:30 +0200
Re: [PATCH 0/8] ARM: OMAP2+: AM33XX/AM43XX: Add suspend-resume support Tony Lindgren <tony@atomide.com> - 2017-05-18 17:00 +0200
| From | Tony Lindgren <tony@atomide.com> |
|---|---|
| Date | 2017-05-16 17:50 +0200 |
| Subject | Re: [PATCH 0/8] ARM: OMAP2+: AM33XX/AM43XX: Add suspend-resume support |
| Message-ID | <tHMTw-7r2-7@gated-at.bofh.it> |
* Dave Gerlach <d-gerlach@ti.com> [170329 11:52]: > Hi Tony, > On 03/29/2017 12:13 PM, Tony Lindgren wrote: > > * Dave Gerlach <d-gerlach@ti.com> [170328 19:00]: > > > Hi, > > > This series contains the remaining code to enable suspend to mem and standby on > > > am335x and am437x platforms. It depends on the ti-emif-sram series sent here [1]. > > > > Looks good to me in general. To remove the dependency, can some > > of this be merged separately without breaking things before [1]? > > Maybe by disabling some of the PM states until [1] is in place? > > Thanks for the feedback. Actually, patch 7 is the only patch with a > dependency to [1]. The rest of the patches just add platform PM support > needed for suspend but won't do much of anything until patch 7 actually > enables the suspend states, so there is no reason why patches 1-6 and patch > 8 can't be merged now. I'm applying patches 1-5 into omap-for-v4.13/soc. Patch 6 needs ack from Russell. Regards, Tony
[toc] | [next] | [standalone]
| From | Dave Gerlach <d-gerlach@ti.com> |
|---|---|
| Date | 2017-05-16 18:00 +0200 |
| Message-ID | <tHN3c-7uo-19@gated-at.bofh.it> |
| In reply to | #1642622 |
On 05/16/2017 10:47 AM, Tony Lindgren wrote: > * Dave Gerlach <d-gerlach@ti.com> [170329 11:52]: >> Hi Tony, >> On 03/29/2017 12:13 PM, Tony Lindgren wrote: >>> * Dave Gerlach <d-gerlach@ti.com> [170328 19:00]: >>>> Hi, >>>> This series contains the remaining code to enable suspend to mem and standby on >>>> am335x and am437x platforms. It depends on the ti-emif-sram series sent here [1]. >>> >>> Looks good to me in general. To remove the dependency, can some >>> of this be merged separately without breaking things before [1]? >>> Maybe by disabling some of the PM states until [1] is in place? >> >> Thanks for the feedback. Actually, patch 7 is the only patch with a >> dependency to [1]. The rest of the patches just add platform PM support >> needed for suspend but won't do much of anything until patch 7 actually >> enables the suspend states, so there is no reason why patches 1-6 and patch >> 8 can't be merged now. > > I'm applying patches 1-5 into omap-for-v4.13/soc. Patch 6 needs > ack from Russell. > Thanks. I will resend patches 6-8 as a separate series. Regards, Dave > Regards, > > Tony >
[toc] | [prev] | [next] | [standalone]
| From | Dave Gerlach <d-gerlach@ti.com> |
|---|---|
| Date | 2017-05-18 16:30 +0200 |
| Message-ID | <tIuBb-3UN-9@gated-at.bofh.it> |
| In reply to | #1642632 |
Tony, On 05/16/2017 10:49 AM, Dave Gerlach wrote: > On 05/16/2017 10:47 AM, Tony Lindgren wrote: >> * Dave Gerlach <d-gerlach@ti.com> [170329 11:52]: >>> Hi Tony, >>> On 03/29/2017 12:13 PM, Tony Lindgren wrote: >>>> * Dave Gerlach <d-gerlach@ti.com> [170328 19:00]: >>>>> Hi, >>>>> This series contains the remaining code to enable suspend to mem and >>>>> standby on >>>>> am335x and am437x platforms. It depends on the ti-emif-sram series sent >>>>> here [1]. >>>> >>>> Looks good to me in general. To remove the dependency, can some >>>> of this be merged separately without breaking things before [1]? >>>> Maybe by disabling some of the PM states until [1] is in place? >>> >>> Thanks for the feedback. Actually, patch 7 is the only patch with a >>> dependency to [1]. The rest of the patches just add platform PM support >>> needed for suspend but won't do much of anything until patch 7 actually >>> enables the suspend states, so there is no reason why patches 1-6 and patch >>> 8 can't be merged now. >> >> I'm applying patches 1-5 into omap-for-v4.13/soc. Patch 6 needs >> ack from Russell. >> > > Thanks. I will resend patches 6-8 as a separate series. On second thought, it's probably best to keep patches 4 and 5 as part of the new series as well for review purposes, as they are not added to the build until patch 6 but also make use of the asm-offsets macros introduced in patch 6. I will send them along with patch 6-8 resend, making the new series patches 4-8. Regards, Dave > > Regards, > Dave > >> Regards, >> >> Tony >> >
[toc] | [prev] | [next] | [standalone]
| From | Tony Lindgren <tony@atomide.com> |
|---|---|
| Date | 2017-05-18 17:00 +0200 |
| Message-ID | <tIv4e-47X-21@gated-at.bofh.it> |
| In reply to | #1644596 |
* Dave Gerlach <d-gerlach@ti.com> [170518 07:23]: > Tony, > On 05/16/2017 10:49 AM, Dave Gerlach wrote: > > On 05/16/2017 10:47 AM, Tony Lindgren wrote: > > > * Dave Gerlach <d-gerlach@ti.com> [170329 11:52]: > > > > Hi Tony, > > > > On 03/29/2017 12:13 PM, Tony Lindgren wrote: > > > > > * Dave Gerlach <d-gerlach@ti.com> [170328 19:00]: > > > > > > Hi, > > > > > > This series contains the remaining code to enable suspend to mem and > > > > > > standby on > > > > > > am335x and am437x platforms. It depends on the ti-emif-sram series sent > > > > > > here [1]. > > > > > > > > > > Looks good to me in general. To remove the dependency, can some > > > > > of this be merged separately without breaking things before [1]? > > > > > Maybe by disabling some of the PM states until [1] is in place? > > > > > > > > Thanks for the feedback. Actually, patch 7 is the only patch with a > > > > dependency to [1]. The rest of the patches just add platform PM support > > > > needed for suspend but won't do much of anything until patch 7 actually > > > > enables the suspend states, so there is no reason why patches 1-6 and patch > > > > 8 can't be merged now. > > > > > > I'm applying patches 1-5 into omap-for-v4.13/soc. Patch 6 needs > > > ack from Russell. > > > > > > > Thanks. I will resend patches 6-8 as a separate series. > > On second thought, it's probably best to keep patches 4 and 5 as part of the > new series as well for review purposes, as they are not added to the build > until patch 6 but also make use of the asm-offsets macros introduced in > patch 6. I will send them along with patch 6-8 resend, making the new series > patches 4-8. OK dropping 4 and 5 today. Regards, Tony
[toc] | [prev] | [standalone]
Back to top | Article view | linux.kernel
csiph-web