Re: [PATCH v2 1/3] phy: core: Move runtime PM reference counting to the parent device

2018-01-02 Thread Ulf Hansson
On 24 December 2017 at 13:00, Rafael J. Wysocki wrote: > On Saturday, December 23, 2017 4:09:33 PM CET Ulf Hansson wrote: >> [...] >> >> > >> > So IMO the changes you are proposing make sense regardless of the >> > genpd issue, because they generally simplify the phy code, but

Re: [PATCH v2 1/3] phy: core: Move runtime PM reference counting to the parent device

2018-01-02 Thread Ulf Hansson
On 24 December 2017 at 13:00, Rafael J. Wysocki wrote: > On Saturday, December 23, 2017 4:09:33 PM CET Ulf Hansson wrote: >> [...] >> >> > >> > So IMO the changes you are proposing make sense regardless of the >> > genpd issue, because they generally simplify the phy code, but the >> > additional

Re: [PATCH v2 1/3] phy: core: Move runtime PM reference counting to the parent device

2017-12-24 Thread Rafael J. Wysocki
On Saturday, December 23, 2017 4:09:33 PM CET Ulf Hansson wrote: > [...] > > > > > So IMO the changes you are proposing make sense regardless of the > > genpd issue, because they generally simplify the phy code, but the > > additional use_runtime_pm field in struct phy represents redundant > >

Re: [PATCH v2 1/3] phy: core: Move runtime PM reference counting to the parent device

2017-12-24 Thread Rafael J. Wysocki
On Saturday, December 23, 2017 4:09:33 PM CET Ulf Hansson wrote: > [...] > > > > > So IMO the changes you are proposing make sense regardless of the > > genpd issue, because they generally simplify the phy code, but the > > additional use_runtime_pm field in struct phy represents redundant > >

Re: [PATCH v2 1/3] phy: core: Move runtime PM reference counting to the parent device

2017-12-23 Thread Ulf Hansson
[...] > > So IMO the changes you are proposing make sense regardless of the > genpd issue, because they generally simplify the phy code, but the > additional use_runtime_pm field in struct phy represents redundant > information (manipulating reference counters shouldn't matter if > runtime PM is

Re: [PATCH v2 1/3] phy: core: Move runtime PM reference counting to the parent device

2017-12-23 Thread Ulf Hansson
[...] > > So IMO the changes you are proposing make sense regardless of the > genpd issue, because they generally simplify the phy code, but the > additional use_runtime_pm field in struct phy represents redundant > information (manipulating reference counters shouldn't matter if > runtime PM is

Re: [PATCH v2 1/3] phy: core: Move runtime PM reference counting to the parent device

2017-12-23 Thread Rafael J. Wysocki
On Sat, Dec 23, 2017 at 1:37 PM, Ulf Hansson wrote: > On 23 December 2017 at 02:35, Rafael J. Wysocki wrote: >> On Thu, Dec 21, 2017 at 11:50 AM, Ulf Hansson wrote: >>> On 21 December 2017 at 02:39, Rafael J. Wysocki

Re: [PATCH v2 1/3] phy: core: Move runtime PM reference counting to the parent device

2017-12-23 Thread Rafael J. Wysocki
On Sat, Dec 23, 2017 at 1:37 PM, Ulf Hansson wrote: > On 23 December 2017 at 02:35, Rafael J. Wysocki wrote: >> On Thu, Dec 21, 2017 at 11:50 AM, Ulf Hansson wrote: >>> On 21 December 2017 at 02:39, Rafael J. Wysocki wrote: On Wed, Dec 20, 2017 at 3:09 PM, Ulf Hansson wrote: >

Re: [PATCH v2 1/3] phy: core: Move runtime PM reference counting to the parent device

2017-12-23 Thread Rafael J. Wysocki
On Thu, Dec 21, 2017 at 2:39 AM, Rafael J. Wysocki wrote: > On Wed, Dec 20, 2017 at 3:09 PM, Ulf Hansson wrote: >> The runtime PM deployment in the phy core is deployed using the phy core >> device, which is created by the phy core and assigned as a

Re: [PATCH v2 1/3] phy: core: Move runtime PM reference counting to the parent device

2017-12-23 Thread Rafael J. Wysocki
On Thu, Dec 21, 2017 at 2:39 AM, Rafael J. Wysocki wrote: > On Wed, Dec 20, 2017 at 3:09 PM, Ulf Hansson wrote: >> The runtime PM deployment in the phy core is deployed using the phy core >> device, which is created by the phy core and assigned as a child device of >> the phy provider device. >>

Re: [PATCH v2 1/3] phy: core: Move runtime PM reference counting to the parent device

2017-12-23 Thread Ulf Hansson
On 23 December 2017 at 02:35, Rafael J. Wysocki wrote: > On Thu, Dec 21, 2017 at 11:50 AM, Ulf Hansson wrote: >> On 21 December 2017 at 02:39, Rafael J. Wysocki wrote: >>> On Wed, Dec 20, 2017 at 3:09 PM, Ulf Hansson

Re: [PATCH v2 1/3] phy: core: Move runtime PM reference counting to the parent device

2017-12-23 Thread Ulf Hansson
On 23 December 2017 at 02:35, Rafael J. Wysocki wrote: > On Thu, Dec 21, 2017 at 11:50 AM, Ulf Hansson wrote: >> On 21 December 2017 at 02:39, Rafael J. Wysocki wrote: >>> On Wed, Dec 20, 2017 at 3:09 PM, Ulf Hansson wrote: The runtime PM deployment in the phy core is deployed using the

Re: [PATCH v2 1/3] phy: core: Move runtime PM reference counting to the parent device

2017-12-22 Thread Rafael J. Wysocki
On Sat, Dec 23, 2017 at 2:35 AM, Rafael J. Wysocki wrote: > On Thu, Dec 21, 2017 at 11:50 AM, Ulf Hansson wrote: >> On 21 December 2017 at 02:39, Rafael J. Wysocki wrote: >>> On Wed, Dec 20, 2017 at 3:09 PM, Ulf Hansson

Re: [PATCH v2 1/3] phy: core: Move runtime PM reference counting to the parent device

2017-12-22 Thread Rafael J. Wysocki
On Sat, Dec 23, 2017 at 2:35 AM, Rafael J. Wysocki wrote: > On Thu, Dec 21, 2017 at 11:50 AM, Ulf Hansson wrote: >> On 21 December 2017 at 02:39, Rafael J. Wysocki wrote: >>> On Wed, Dec 20, 2017 at 3:09 PM, Ulf Hansson wrote: The runtime PM deployment in the phy core is deployed using

Re: [PATCH v2 1/3] phy: core: Move runtime PM reference counting to the parent device

2017-12-22 Thread Rafael J. Wysocki
On Thu, Dec 21, 2017 at 11:50 AM, Ulf Hansson wrote: > On 21 December 2017 at 02:39, Rafael J. Wysocki wrote: >> On Wed, Dec 20, 2017 at 3:09 PM, Ulf Hansson wrote: >>> The runtime PM deployment in the phy core is deployed using

Re: [PATCH v2 1/3] phy: core: Move runtime PM reference counting to the parent device

2017-12-22 Thread Rafael J. Wysocki
On Thu, Dec 21, 2017 at 11:50 AM, Ulf Hansson wrote: > On 21 December 2017 at 02:39, Rafael J. Wysocki wrote: >> On Wed, Dec 20, 2017 at 3:09 PM, Ulf Hansson wrote: >>> The runtime PM deployment in the phy core is deployed using the phy core >>> device, which is created by the phy core and

Re: [PATCH v2 1/3] phy: core: Move runtime PM reference counting to the parent device

2017-12-21 Thread Ulf Hansson
On 21 December 2017 at 02:39, Rafael J. Wysocki wrote: > On Wed, Dec 20, 2017 at 3:09 PM, Ulf Hansson wrote: >> The runtime PM deployment in the phy core is deployed using the phy core >> device, which is created by the phy core and assigned as a child

Re: [PATCH v2 1/3] phy: core: Move runtime PM reference counting to the parent device

2017-12-21 Thread Ulf Hansson
On 21 December 2017 at 02:39, Rafael J. Wysocki wrote: > On Wed, Dec 20, 2017 at 3:09 PM, Ulf Hansson wrote: >> The runtime PM deployment in the phy core is deployed using the phy core >> device, which is created by the phy core and assigned as a child device of >> the phy provider device. >> >>

Re: [PATCH v2 1/3] phy: core: Move runtime PM reference counting to the parent device

2017-12-20 Thread Rafael J. Wysocki
On Wed, Dec 20, 2017 at 3:09 PM, Ulf Hansson wrote: > The runtime PM deployment in the phy core is deployed using the phy core > device, which is created by the phy core and assigned as a child device of > the phy provider device. > > The behaviour around the runtime PM

Re: [PATCH v2 1/3] phy: core: Move runtime PM reference counting to the parent device

2017-12-20 Thread Rafael J. Wysocki
On Wed, Dec 20, 2017 at 3:09 PM, Ulf Hansson wrote: > The runtime PM deployment in the phy core is deployed using the phy core > device, which is created by the phy core and assigned as a child device of > the phy provider device. > > The behaviour around the runtime PM deployment cause some

[PATCH v2 1/3] phy: core: Move runtime PM reference counting to the parent device

2017-12-20 Thread Ulf Hansson
The runtime PM deployment in the phy core is deployed using the phy core device, which is created by the phy core and assigned as a child device of the phy provider device. The behaviour around the runtime PM deployment cause some issues during system suspend, in cases when the phy provider

[PATCH v2 1/3] phy: core: Move runtime PM reference counting to the parent device

2017-12-20 Thread Ulf Hansson
The runtime PM deployment in the phy core is deployed using the phy core device, which is created by the phy core and assigned as a child device of the phy provider device. The behaviour around the runtime PM deployment cause some issues during system suspend, in cases when the phy provider