Re: [PATCH V5] PCI: rcar: Use runtime PM to control controller clock

2018-05-21 Thread Marek Vasut
On 04/19/2018 12:00 PM, Geert Uytterhoeven wrote: > Hi Marek, > > On Tue, Apr 10, 2018 at 6:17 PM, Marek Vasut wrote: >> On 04/10/2018 05:28 PM, Geert Uytterhoeven wrote: >> The pairing looks as follows: >> >> .- rcar_pcie_parse_request_of_pci_ranges() >> |

Re: [PATCH V5] PCI: rcar: Use runtime PM to control controller clock

2018-05-21 Thread Marek Vasut
On 05/21/2018 03:03 PM, Lorenzo Pieralisi wrote: > On Mon, May 21, 2018 at 01:08:36PM +0200, Marek Vasut wrote: >> On 05/14/2018 05:49 PM, Lorenzo Pieralisi wrote: >>> On Mon, May 14, 2018 at 05:32:04PM +0200, Marek Vasut wrote: On 05/01/2018 12:55 PM, Lorenzo Pieralisi wrote: > On Fri,

Re: [PATCH V5] PCI: rcar: Use runtime PM to control controller clock

2018-05-21 Thread Lorenzo Pieralisi
On Mon, May 21, 2018 at 01:08:36PM +0200, Marek Vasut wrote: > On 05/14/2018 05:49 PM, Lorenzo Pieralisi wrote: > > On Mon, May 14, 2018 at 05:32:04PM +0200, Marek Vasut wrote: > >> On 05/01/2018 12:55 PM, Lorenzo Pieralisi wrote: > >>> On Fri, Apr 13, 2018 at 02:48:19PM +0200, Simon Horman wrote:

Re: [PATCH V5] PCI: rcar: Use runtime PM to control controller clock

2018-05-21 Thread Marek Vasut
On 05/14/2018 05:49 PM, Lorenzo Pieralisi wrote: > On Mon, May 14, 2018 at 05:32:04PM +0200, Marek Vasut wrote: >> On 05/01/2018 12:55 PM, Lorenzo Pieralisi wrote: >>> On Fri, Apr 13, 2018 at 02:48:19PM +0200, Simon Horman wrote: On Tue, Apr 10, 2018 at 06:17:04PM +0200, Marek Vasut wrote:

Re: [PATCH V5] PCI: rcar: Use runtime PM to control controller clock

2018-05-14 Thread Lorenzo Pieralisi
On Mon, May 14, 2018 at 05:32:04PM +0200, Marek Vasut wrote: > On 05/01/2018 12:55 PM, Lorenzo Pieralisi wrote: > > On Fri, Apr 13, 2018 at 02:48:19PM +0200, Simon Horman wrote: > >> On Tue, Apr 10, 2018 at 06:17:04PM +0200, Marek Vasut wrote: > >>> On 04/10/2018 05:28 PM, Geert Uytterhoeven

Re: [PATCH V5] PCI: rcar: Use runtime PM to control controller clock

2018-05-14 Thread Marek Vasut
On 05/01/2018 12:55 PM, Lorenzo Pieralisi wrote: > On Fri, Apr 13, 2018 at 02:48:19PM +0200, Simon Horman wrote: >> On Tue, Apr 10, 2018 at 06:17:04PM +0200, Marek Vasut wrote: >>> On 04/10/2018 05:28 PM, Geert Uytterhoeven wrote: >> >> ... >> >> rcar_pcie_get_resources() is called while the

Re: [PATCH V5] PCI: rcar: Use runtime PM to control controller clock

2018-05-01 Thread Lorenzo Pieralisi
On Fri, Apr 13, 2018 at 02:48:19PM +0200, Simon Horman wrote: > On Tue, Apr 10, 2018 at 06:17:04PM +0200, Marek Vasut wrote: > > On 04/10/2018 05:28 PM, Geert Uytterhoeven wrote: > > ... > > > >>> rcar_pcie_get_resources() is called while the device is > > >>> runtime-enabled/resumed, > > >>>

Re: [PATCH V5] PCI: rcar: Use runtime PM to control controller clock

2018-04-19 Thread Geert Uytterhoeven
Hi Marek, On Tue, Apr 10, 2018 at 6:17 PM, Marek Vasut wrote: > On 04/10/2018 05:28 PM, Geert Uytterhoeven wrote: > The pairing looks as follows: > > .- rcar_pcie_parse_request_of_pci_ranges() > | (pm_runtime_enable is here) > | .-

Re: [PATCH V5] PCI: rcar: Use runtime PM to control controller clock

2018-04-13 Thread Lorenzo Pieralisi
On Fri, Apr 13, 2018 at 02:48:19PM +0200, Simon Horman wrote: > On Tue, Apr 10, 2018 at 06:17:04PM +0200, Marek Vasut wrote: > > On 04/10/2018 05:28 PM, Geert Uytterhoeven wrote: > > ... > > > >>> rcar_pcie_get_resources() is called while the device is > > >>> runtime-enabled/resumed, > > >>>

Re: [PATCH V5] PCI: rcar: Use runtime PM to control controller clock

2018-04-13 Thread Simon Horman
On Tue, Apr 10, 2018 at 06:17:04PM +0200, Marek Vasut wrote: > On 04/10/2018 05:28 PM, Geert Uytterhoeven wrote: ... > >>> rcar_pcie_get_resources() is called while the device is > >>> runtime-enabled/resumed, > >>> pci_free_resource_list() is called while the device is runtime-disabled. > >

Re: [PATCH V5] PCI: rcar: Use runtime PM to control controller clock

2018-04-10 Thread Marek Vasut
On 04/10/2018 05:28 PM, Geert Uytterhoeven wrote: > Hi Marek, Hi, [...] The pairing looks as follows: .- rcar_pcie_parse_request_of_pci_ranges() | (pm_runtime_enable is here) | .- pm_runtime_get_sync() | | .- rcar_pcie_get_resources() >>> >>>

Re: [PATCH V5] PCI: rcar: Use runtime PM to control controller clock

2018-04-10 Thread Geert Uytterhoeven
Hi Marek, On Tue, Apr 10, 2018 at 5:25 PM, Marek Vasut wrote: > On 04/10/2018 04:42 PM, Geert Uytterhoeven wrote: >> On Tue, Apr 10, 2018 at 4:31 PM, Marek Vasut wrote: >>> On 04/09/2018 02:26 PM, Simon Horman wrote: On Mon, Apr 09, 2018 at

Re: [PATCH V5] PCI: rcar: Use runtime PM to control controller clock

2018-04-10 Thread Marek Vasut
On 04/10/2018 04:42 PM, Geert Uytterhoeven wrote: > Hi Marek, > > On Tue, Apr 10, 2018 at 4:31 PM, Marek Vasut wrote: >> On 04/09/2018 02:26 PM, Simon Horman wrote: >>> On Mon, Apr 09, 2018 at 01:47:38PM +0200, Geert Uytterhoeven wrote: On Mon, Apr 9, 2018 at 1:41 PM,

Re: [PATCH V5] PCI: rcar: Use runtime PM to control controller clock

2018-04-10 Thread Geert Uytterhoeven
Hi Marek, On Tue, Apr 10, 2018 at 4:31 PM, Marek Vasut wrote: > On 04/09/2018 02:26 PM, Simon Horman wrote: >> On Mon, Apr 09, 2018 at 01:47:38PM +0200, Geert Uytterhoeven wrote: >>> On Mon, Apr 9, 2018 at 1:41 PM, Simon Horman wrote: On Mon, Apr

Re: [PATCH V5] PCI: rcar: Use runtime PM to control controller clock

2018-04-10 Thread Marek Vasut
On 04/09/2018 02:26 PM, Simon Horman wrote: > On Mon, Apr 09, 2018 at 01:47:38PM +0200, Geert Uytterhoeven wrote: >> Hi Simon, Marek, >> >> On Mon, Apr 9, 2018 at 1:41 PM, Simon Horman wrote: >>> On Mon, Apr 09, 2018 at 10:20:05AM +0200, Marek Vasut wrote: On 04/09/2018

Re: [PATCH V5] PCI: rcar: Use runtime PM to control controller clock

2018-04-09 Thread Simon Horman
On Mon, Apr 09, 2018 at 01:47:38PM +0200, Geert Uytterhoeven wrote: > Hi Simon, Marek, > > On Mon, Apr 9, 2018 at 1:41 PM, Simon Horman wrote: > > On Mon, Apr 09, 2018 at 10:20:05AM +0200, Marek Vasut wrote: > >> On 04/09/2018 10:07 AM, Geert Uytterhoeven wrote: > >> > On

Re: [PATCH V5] PCI: rcar: Use runtime PM to control controller clock

2018-04-09 Thread Geert Uytterhoeven
Hi Simon, Marek, On Mon, Apr 9, 2018 at 1:41 PM, Simon Horman wrote: > On Mon, Apr 09, 2018 at 10:20:05AM +0200, Marek Vasut wrote: >> On 04/09/2018 10:07 AM, Geert Uytterhoeven wrote: >> > On Sun, Apr 8, 2018 at 3:09 PM, Marek Vasut wrote: >> >> From:

Re: [PATCH V5] PCI: rcar: Use runtime PM to control controller clock

2018-04-09 Thread Simon Horman
On Mon, Apr 09, 2018 at 10:20:05AM +0200, Marek Vasut wrote: > On 04/09/2018 10:07 AM, Geert Uytterhoeven wrote: > > Hi Marek, > > > > On Sun, Apr 8, 2018 at 3:09 PM, Marek Vasut wrote: > >> From: Dien Pham > >> > >> The controller clock can

Re: [PATCH V5] PCI: rcar: Use runtime PM to control controller clock

2018-04-09 Thread Marek Vasut
On 04/09/2018 10:07 AM, Geert Uytterhoeven wrote: > Hi Marek, > > On Sun, Apr 8, 2018 at 3:09 PM, Marek Vasut wrote: >> From: Dien Pham >> >> The controller clock can be switched off during suspend/resume, >> let runtime PM take care of that.

Re: [PATCH V5] PCI: rcar: Use runtime PM to control controller clock

2018-04-09 Thread Geert Uytterhoeven
Hi Marek, On Sun, Apr 8, 2018 at 3:09 PM, Marek Vasut wrote: > From: Dien Pham > > The controller clock can be switched off during suspend/resume, > let runtime PM take care of that. > > Signed-off-by: Dien Pham