Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1664789 > unrolled thread
| Started by | Alexandre Belloni <alexandre.belloni@free-electrons.com> |
|---|---|
| First post | 2017-06-13 14:30 +0200 |
| Last post | 2017-06-13 14:40 +0200 |
| Articles | 2 — 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 v2] clockevents/tcb_clksrc: implement suspend/resume Alexandre Belloni <alexandre.belloni@free-electrons.com> - 2017-06-13 14:30 +0200
Re: [PATCH v2] clockevents/tcb_clksrc: implement suspend/resume Daniel Lezcano <daniel.lezcano@linaro.org> - 2017-06-13 14:40 +0200
| From | Alexandre Belloni <alexandre.belloni@free-electrons.com> |
|---|---|
| Date | 2017-06-13 14:30 +0200 |
| Subject | Re: [PATCH v2] clockevents/tcb_clksrc: implement suspend/resume |
| Message-ID | <tRT7j-7UN-9@gated-at.bofh.it> |
On 13/06/2017 at 14:21:10 +0200, Daniel Lezcano wrote: > On 12/05/2017 20:22, Alexandre Belloni wrote: > > On sama5d2, power to the core may be cut while entering suspend mode. It is > > necessary to save and restore the TCB registers. > > > > Signed-off-by: Alexandre Belloni <alexandre.belloni@free-electrons.com> > > The context will be saved/restored on all the platforms (other than > sama5d2) using this timer even if the clock is not powered down. Is that ok? > Yes, that is fine. There is not much we can do here until we can determine what is the target suspend state from the drivers. > Should it be tagged for stable@? > No because support for this suspend mode is not yet upstream. > Side note: the config option is in drivers/misc, any particular reason > to have it there and not in the clocksource's Kconfig? > Mostly historical, this is changed in the other series. -- Alexandre Belloni, Free Electrons Embedded Linux and Kernel engineering http://free-electrons.com
[toc] | [next] | [standalone]
| From | Daniel Lezcano <daniel.lezcano@linaro.org> |
|---|---|
| Date | 2017-06-13 14:40 +0200 |
| Message-ID | <tRTgZ-7YB-9@gated-at.bofh.it> |
| In reply to | #1664789 |
On 13/06/2017 14:28, Alexandre Belloni wrote: > On 13/06/2017 at 14:21:10 +0200, Daniel Lezcano wrote: >> On 12/05/2017 20:22, Alexandre Belloni wrote: >>> On sama5d2, power to the core may be cut while entering suspend mode. It is >>> necessary to save and restore the TCB registers. >>> >>> Signed-off-by: Alexandre Belloni <alexandre.belloni@free-electrons.com> >> >> The context will be saved/restored on all the platforms (other than >> sama5d2) using this timer even if the clock is not powered down. Is that ok? >> > > Yes, that is fine. There is not much we can do here until we can > determine what is the target suspend state from the drivers. > >> Should it be tagged for stable@? >> > > No because support for this suspend mode is not yet upstream. > >> Side note: the config option is in drivers/misc, any particular reason >> to have it there and not in the clocksource's Kconfig? >> > > Mostly historical, this is changed in the other series. Ok, thanks. I've applied this patch for 4.13. -- Daniel -- <http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs Follow Linaro: <http://www.facebook.com/pages/Linaro> Facebook | <http://twitter.com/#!/linaroorg> Twitter | <http://www.linaro.org/linaro-blog/> Blog
[toc] | [prev] | [standalone]
Back to top | Article view | linux.kernel
csiph-web