Re: [PATCH v8 1/5] PM / Runtime: Add getter for querying the IRQ safe option

2014-11-05 Thread Laurent Pinchart
Hi Russell, On Monday 03 November 2014 17:04:08 Russell King - ARM Linux wrote: > On Mon, Nov 03, 2014 at 09:57:28PM +0530, Vinod Koul wrote: > > On Sat, Nov 01, 2014 at 02:29:42AM +0200, Laurent Pinchart wrote: > >> Many other drivers suffer from the same problem. While I won't reject > >> your

Re: [PATCH v8 1/5] PM / Runtime: Add getter for querying the IRQ safe option

2014-11-05 Thread Vinod Koul
On Mon, Nov 03, 2014 at 06:59:37PM +0200, Laurent Pinchart wrote: > > > Many other drivers suffer from the same problem. While I won't reject your > > > proposed fix, I would prefer a more generic approach. > > > > > > One option that has been discussed previously was to use a work queue to > > >

Re: [PATCH v8 1/5] PM / Runtime: Add getter for querying the IRQ safe option

2014-11-05 Thread Vinod Koul
On Mon, Nov 03, 2014 at 05:04:08PM +, Russell King - ARM Linux wrote: > On Mon, Nov 03, 2014 at 09:57:28PM +0530, Vinod Koul wrote: > > On Sat, Nov 01, 2014 at 02:29:42AM +0200, Laurent Pinchart wrote: > > > Many other drivers suffer from the same problem. While I won't reject > > > your > >

Re: [PATCH v8 1/5] PM / Runtime: Add getter for querying the IRQ safe option

2014-11-05 Thread Vinod Koul
On Mon, Nov 03, 2014 at 05:04:08PM +, Russell King - ARM Linux wrote: On Mon, Nov 03, 2014 at 09:57:28PM +0530, Vinod Koul wrote: On Sat, Nov 01, 2014 at 02:29:42AM +0200, Laurent Pinchart wrote: Many other drivers suffer from the same problem. While I won't reject your proposed

Re: [PATCH v8 1/5] PM / Runtime: Add getter for querying the IRQ safe option

2014-11-05 Thread Vinod Koul
On Mon, Nov 03, 2014 at 06:59:37PM +0200, Laurent Pinchart wrote: Many other drivers suffer from the same problem. While I won't reject your proposed fix, I would prefer a more generic approach. One option that has been discussed previously was to use a work queue to delay starting

Re: [PATCH v8 1/5] PM / Runtime: Add getter for querying the IRQ safe option

2014-11-05 Thread Laurent Pinchart
Hi Russell, On Monday 03 November 2014 17:04:08 Russell King - ARM Linux wrote: On Mon, Nov 03, 2014 at 09:57:28PM +0530, Vinod Koul wrote: On Sat, Nov 01, 2014 at 02:29:42AM +0200, Laurent Pinchart wrote: Many other drivers suffer from the same problem. While I won't reject your proposed

Re: [PATCH v8 1/5] PM / Runtime: Add getter for querying the IRQ safe option

2014-11-03 Thread Russell King - ARM Linux
On Mon, Nov 03, 2014 at 09:57:28PM +0530, Vinod Koul wrote: > On Sat, Nov 01, 2014 at 02:29:42AM +0200, Laurent Pinchart wrote: > > Many other drivers suffer from the same problem. While I won't reject your > > proposed fix, I would prefer a more generic approach. > > > > One option that has

Re: [PATCH v8 1/5] PM / Runtime: Add getter for querying the IRQ safe option

2014-11-03 Thread Laurent Pinchart
Hi Vinod, On Monday 03 November 2014 21:57:28 Vinod Koul wrote: > On Sat, Nov 01, 2014 at 02:29:42AM +0200, Laurent Pinchart wrote: > > On Friday 31 October 2014 15:40:16 Krzysztof Kozlowski wrote: > >> On pią, 2014-10-31 at 15:22 +0100, Pavel Machek wrote: > >>> On Fri 2014-10-31 10:14:55,

Re: [PATCH v8 1/5] PM / Runtime: Add getter for querying the IRQ safe option

2014-11-03 Thread Vinod Koul
On Sat, Nov 01, 2014 at 02:29:42AM +0200, Laurent Pinchart wrote: > Hi Krzysztof, > > On Friday 31 October 2014 15:40:16 Krzysztof Kozlowski wrote: > > On pią, 2014-10-31 at 15:22 +0100, Pavel Machek wrote: > > > On Fri 2014-10-31 10:14:55, Krzysztof Kozlowski wrote: > > >> On pon, 2014-10-20 at

Re: [PATCH v8 1/5] PM / Runtime: Add getter for querying the IRQ safe option

2014-11-03 Thread Krzysztof Kozlowski
On sob, 2014-11-01 at 02:29 +0200, Laurent Pinchart wrote: > Hi Krzysztof, > > On Friday 31 October 2014 15:40:16 Krzysztof Kozlowski wrote: > > On pią, 2014-10-31 at 15:22 +0100, Pavel Machek wrote: > > > On Fri 2014-10-31 10:14:55, Krzysztof Kozlowski wrote: > > >> On pon, 2014-10-20 at 11:04

Re: [PATCH v8 1/5] PM / Runtime: Add getter for querying the IRQ safe option

2014-11-03 Thread Krzysztof Kozlowski
On sob, 2014-11-01 at 01:42 +0100, Rafael J. Wysocki wrote: > On Friday, October 31, 2014 11:04:52 PM Russell King - ARM Linux wrote: > > On Sat, Nov 01, 2014 at 12:11:05AM +0100, Rafael J. Wysocki wrote: > > > [CC list trimmed + added Kevin Hilman] > > > > > > On Monday, October 20, 2014

Re: [PATCH v8 1/5] PM / Runtime: Add getter for querying the IRQ safe option

2014-11-03 Thread Krzysztof Kozlowski
On sob, 2014-11-01 at 01:42 +0100, Rafael J. Wysocki wrote: On Friday, October 31, 2014 11:04:52 PM Russell King - ARM Linux wrote: On Sat, Nov 01, 2014 at 12:11:05AM +0100, Rafael J. Wysocki wrote: [CC list trimmed + added Kevin Hilman] On Monday, October 20, 2014 11:04:44 AM

Re: [PATCH v8 1/5] PM / Runtime: Add getter for querying the IRQ safe option

2014-11-03 Thread Krzysztof Kozlowski
On sob, 2014-11-01 at 02:29 +0200, Laurent Pinchart wrote: Hi Krzysztof, On Friday 31 October 2014 15:40:16 Krzysztof Kozlowski wrote: On pią, 2014-10-31 at 15:22 +0100, Pavel Machek wrote: On Fri 2014-10-31 10:14:55, Krzysztof Kozlowski wrote: On pon, 2014-10-20 at 11:04 +0200,

Re: [PATCH v8 1/5] PM / Runtime: Add getter for querying the IRQ safe option

2014-11-03 Thread Vinod Koul
On Sat, Nov 01, 2014 at 02:29:42AM +0200, Laurent Pinchart wrote: Hi Krzysztof, On Friday 31 October 2014 15:40:16 Krzysztof Kozlowski wrote: On pią, 2014-10-31 at 15:22 +0100, Pavel Machek wrote: On Fri 2014-10-31 10:14:55, Krzysztof Kozlowski wrote: On pon, 2014-10-20 at 11:04 +0200,

Re: [PATCH v8 1/5] PM / Runtime: Add getter for querying the IRQ safe option

2014-11-03 Thread Laurent Pinchart
Hi Vinod, On Monday 03 November 2014 21:57:28 Vinod Koul wrote: On Sat, Nov 01, 2014 at 02:29:42AM +0200, Laurent Pinchart wrote: On Friday 31 October 2014 15:40:16 Krzysztof Kozlowski wrote: On pią, 2014-10-31 at 15:22 +0100, Pavel Machek wrote: On Fri 2014-10-31 10:14:55, Krzysztof

Re: [PATCH v8 1/5] PM / Runtime: Add getter for querying the IRQ safe option

2014-11-03 Thread Russell King - ARM Linux
On Mon, Nov 03, 2014 at 09:57:28PM +0530, Vinod Koul wrote: On Sat, Nov 01, 2014 at 02:29:42AM +0200, Laurent Pinchart wrote: Many other drivers suffer from the same problem. While I won't reject your proposed fix, I would prefer a more generic approach. One option that has been

Re: [PATCH v8 1/5] PM / Runtime: Add getter for querying the IRQ safe option

2014-10-31 Thread Laurent Pinchart
Hi Krzysztof, On Friday 31 October 2014 15:40:16 Krzysztof Kozlowski wrote: > On pią, 2014-10-31 at 15:22 +0100, Pavel Machek wrote: > > On Fri 2014-10-31 10:14:55, Krzysztof Kozlowski wrote: > >> On pon, 2014-10-20 at 11:04 +0200, Krzysztof Kozlowski wrote: > >>> Add a simple getter

Re: [PATCH v8 1/5] PM / Runtime: Add getter for querying the IRQ safe option

2014-10-31 Thread Rafael J. Wysocki
On Friday, October 31, 2014 11:04:52 PM Russell King - ARM Linux wrote: > On Sat, Nov 01, 2014 at 12:11:05AM +0100, Rafael J. Wysocki wrote: > > [CC list trimmed + added Kevin Hilman] > > > > On Monday, October 20, 2014 11:04:44 AM Krzysztof Kozlowski wrote: > > > Add a simple getter

Re: [PATCH v8 1/5] PM / Runtime: Add getter for querying the IRQ safe option

2014-10-31 Thread Russell King - ARM Linux
On Sat, Nov 01, 2014 at 12:11:05AM +0100, Rafael J. Wysocki wrote: > [CC list trimmed + added Kevin Hilman] > > On Monday, October 20, 2014 11:04:44 AM Krzysztof Kozlowski wrote: > > Add a simple getter pm_runtime_is_irq_safe() for querying whether runtime > > PM IRQ safe was set or not. > > > >

Re: [PATCH v8 1/5] PM / Runtime: Add getter for querying the IRQ safe option

2014-10-31 Thread Rafael J. Wysocki
[CC list trimmed + added Kevin Hilman] On Monday, October 20, 2014 11:04:44 AM Krzysztof Kozlowski wrote: > Add a simple getter pm_runtime_is_irq_safe() for querying whether runtime > PM IRQ safe was set or not. > > Various bus drivers implementing runtime PM may use choose to suspend >

Re: [PATCH v8 1/5] PM / Runtime: Add getter for querying the IRQ safe option

2014-10-31 Thread Krzysztof Kozlowski
On pią, 2014-10-31 at 15:22 +0100, Pavel Machek wrote: > On Fri 2014-10-31 10:14:55, Krzysztof Kozlowski wrote: > > On pon, 2014-10-20 at 11:04 +0200, Krzysztof Kozlowski wrote: > > > Add a simple getter pm_runtime_is_irq_safe() for querying whether runtime > > > PM IRQ safe was set or not. > > >

Re: [PATCH v8 1/5] PM / Runtime: Add getter for querying the IRQ safe option

2014-10-31 Thread Pavel Machek
On Fri 2014-10-31 10:14:55, Krzysztof Kozlowski wrote: > On pon, 2014-10-20 at 11:04 +0200, Krzysztof Kozlowski wrote: > > Add a simple getter pm_runtime_is_irq_safe() for querying whether runtime > > PM IRQ safe was set or not. > > > > Various bus drivers implementing runtime PM may use choose

Re: [PATCH v8 1/5] PM / Runtime: Add getter for querying the IRQ safe option

2014-10-31 Thread Ulf Hansson
On 31 October 2014 10:33, Russell King - ARM Linux wrote: > On Fri, Oct 31, 2014 at 10:29:36AM +0100, Ulf Hansson wrote: >> On 31 October 2014 10:14, Krzysztof Kozlowski >> wrote: >> > On pon, 2014-10-20 at 11:04 +0200, Krzysztof Kozlowski wrote: >> >> Add a simple getter

Re: [PATCH v8 1/5] PM / Runtime: Add getter for querying the IRQ safe option

2014-10-31 Thread Russell King - ARM Linux
On Fri, Oct 31, 2014 at 10:29:36AM +0100, Ulf Hansson wrote: > On 31 October 2014 10:14, Krzysztof Kozlowski wrote: > > On pon, 2014-10-20 at 11:04 +0200, Krzysztof Kozlowski wrote: > >> Add a simple getter pm_runtime_is_irq_safe() for querying whether runtime > >> PM IRQ safe was set or not. >

Re: [PATCH v8 1/5] PM / Runtime: Add getter for querying the IRQ safe option

2014-10-31 Thread Krzysztof Kozlowski
On pią, 2014-10-31 at 10:29 +0100, Ulf Hansson wrote: > On 31 October 2014 10:14, Krzysztof Kozlowski wrote: > > On pon, 2014-10-20 at 11:04 +0200, Krzysztof Kozlowski wrote: > >> Add a simple getter pm_runtime_is_irq_safe() for querying whether runtime > >> PM IRQ safe was set or not. > >> > >>

Re: [PATCH v8 1/5] PM / Runtime: Add getter for querying the IRQ safe option

2014-10-31 Thread Ulf Hansson
On 31 October 2014 10:14, Krzysztof Kozlowski wrote: > On pon, 2014-10-20 at 11:04 +0200, Krzysztof Kozlowski wrote: >> Add a simple getter pm_runtime_is_irq_safe() for querying whether runtime >> PM IRQ safe was set or not. >> >> Various bus drivers implementing runtime PM may use choose to

Re: [PATCH v8 1/5] PM / Runtime: Add getter for querying the IRQ safe option

2014-10-31 Thread Krzysztof Kozlowski
On pon, 2014-10-20 at 11:04 +0200, Krzysztof Kozlowski wrote: > Add a simple getter pm_runtime_is_irq_safe() for querying whether runtime > PM IRQ safe was set or not. > > Various bus drivers implementing runtime PM may use choose to suspend > differently based on IRQ safeness status of child

Re: [PATCH v8 1/5] PM / Runtime: Add getter for querying the IRQ safe option

2014-10-31 Thread Krzysztof Kozlowski
On pon, 2014-10-20 at 11:04 +0200, Krzysztof Kozlowski wrote: Add a simple getter pm_runtime_is_irq_safe() for querying whether runtime PM IRQ safe was set or not. Various bus drivers implementing runtime PM may use choose to suspend differently based on IRQ safeness status of child driver

Re: [PATCH v8 1/5] PM / Runtime: Add getter for querying the IRQ safe option

2014-10-31 Thread Ulf Hansson
On 31 October 2014 10:14, Krzysztof Kozlowski k.kozlow...@samsung.com wrote: On pon, 2014-10-20 at 11:04 +0200, Krzysztof Kozlowski wrote: Add a simple getter pm_runtime_is_irq_safe() for querying whether runtime PM IRQ safe was set or not. Various bus drivers implementing runtime PM may use

Re: [PATCH v8 1/5] PM / Runtime: Add getter for querying the IRQ safe option

2014-10-31 Thread Krzysztof Kozlowski
On pią, 2014-10-31 at 10:29 +0100, Ulf Hansson wrote: On 31 October 2014 10:14, Krzysztof Kozlowski k.kozlow...@samsung.com wrote: On pon, 2014-10-20 at 11:04 +0200, Krzysztof Kozlowski wrote: Add a simple getter pm_runtime_is_irq_safe() for querying whether runtime PM IRQ safe was set or

Re: [PATCH v8 1/5] PM / Runtime: Add getter for querying the IRQ safe option

2014-10-31 Thread Russell King - ARM Linux
On Fri, Oct 31, 2014 at 10:29:36AM +0100, Ulf Hansson wrote: On 31 October 2014 10:14, Krzysztof Kozlowski k.kozlow...@samsung.com wrote: On pon, 2014-10-20 at 11:04 +0200, Krzysztof Kozlowski wrote: Add a simple getter pm_runtime_is_irq_safe() for querying whether runtime PM IRQ safe was

Re: [PATCH v8 1/5] PM / Runtime: Add getter for querying the IRQ safe option

2014-10-31 Thread Ulf Hansson
On 31 October 2014 10:33, Russell King - ARM Linux li...@arm.linux.org.uk wrote: On Fri, Oct 31, 2014 at 10:29:36AM +0100, Ulf Hansson wrote: On 31 October 2014 10:14, Krzysztof Kozlowski k.kozlow...@samsung.com wrote: On pon, 2014-10-20 at 11:04 +0200, Krzysztof Kozlowski wrote: Add a

Re: [PATCH v8 1/5] PM / Runtime: Add getter for querying the IRQ safe option

2014-10-31 Thread Pavel Machek
On Fri 2014-10-31 10:14:55, Krzysztof Kozlowski wrote: On pon, 2014-10-20 at 11:04 +0200, Krzysztof Kozlowski wrote: Add a simple getter pm_runtime_is_irq_safe() for querying whether runtime PM IRQ safe was set or not. Various bus drivers implementing runtime PM may use choose to suspend

Re: [PATCH v8 1/5] PM / Runtime: Add getter for querying the IRQ safe option

2014-10-31 Thread Krzysztof Kozlowski
On pią, 2014-10-31 at 15:22 +0100, Pavel Machek wrote: On Fri 2014-10-31 10:14:55, Krzysztof Kozlowski wrote: On pon, 2014-10-20 at 11:04 +0200, Krzysztof Kozlowski wrote: Add a simple getter pm_runtime_is_irq_safe() for querying whether runtime PM IRQ safe was set or not. Various

Re: [PATCH v8 1/5] PM / Runtime: Add getter for querying the IRQ safe option

2014-10-31 Thread Rafael J. Wysocki
[CC list trimmed + added Kevin Hilman] On Monday, October 20, 2014 11:04:44 AM Krzysztof Kozlowski wrote: Add a simple getter pm_runtime_is_irq_safe() for querying whether runtime PM IRQ safe was set or not. Various bus drivers implementing runtime PM may use choose to suspend differently

Re: [PATCH v8 1/5] PM / Runtime: Add getter for querying the IRQ safe option

2014-10-31 Thread Russell King - ARM Linux
On Sat, Nov 01, 2014 at 12:11:05AM +0100, Rafael J. Wysocki wrote: [CC list trimmed + added Kevin Hilman] On Monday, October 20, 2014 11:04:44 AM Krzysztof Kozlowski wrote: Add a simple getter pm_runtime_is_irq_safe() for querying whether runtime PM IRQ safe was set or not. Various

Re: [PATCH v8 1/5] PM / Runtime: Add getter for querying the IRQ safe option

2014-10-31 Thread Rafael J. Wysocki
On Friday, October 31, 2014 11:04:52 PM Russell King - ARM Linux wrote: On Sat, Nov 01, 2014 at 12:11:05AM +0100, Rafael J. Wysocki wrote: [CC list trimmed + added Kevin Hilman] On Monday, October 20, 2014 11:04:44 AM Krzysztof Kozlowski wrote: Add a simple getter

Re: [PATCH v8 1/5] PM / Runtime: Add getter for querying the IRQ safe option

2014-10-31 Thread Laurent Pinchart
Hi Krzysztof, On Friday 31 October 2014 15:40:16 Krzysztof Kozlowski wrote: On pią, 2014-10-31 at 15:22 +0100, Pavel Machek wrote: On Fri 2014-10-31 10:14:55, Krzysztof Kozlowski wrote: On pon, 2014-10-20 at 11:04 +0200, Krzysztof Kozlowski wrote: Add a simple getter

[PATCH v8 1/5] PM / Runtime: Add getter for querying the IRQ safe option

2014-10-20 Thread Krzysztof Kozlowski
Add a simple getter pm_runtime_is_irq_safe() for querying whether runtime PM IRQ safe was set or not. Various bus drivers implementing runtime PM may use choose to suspend differently based on IRQ safeness status of child driver (e.g. do not unprepare the clock if IRQ safe is not set).

[PATCH v8 1/5] PM / Runtime: Add getter for querying the IRQ safe option

2014-10-20 Thread Krzysztof Kozlowski
Add a simple getter pm_runtime_is_irq_safe() for querying whether runtime PM IRQ safe was set or not. Various bus drivers implementing runtime PM may use choose to suspend differently based on IRQ safeness status of child driver (e.g. do not unprepare the clock if IRQ safe is not set).