Re: [PATCH v2 3/7] usb: phy: msm: Move regulator usage to managed resource allocation

2013-07-26 Thread Ivan T. Ivanov
On Fri, 2013-07-26 at 23:28 +0300, Felipe Balbi wrote:
> On Fri, Jul 26, 2013 at 03:31:34PM +0300, Ivan T. Ivanov wrote:
> > 
> > Hi Felipe, 
> > 
> > On Thu, 2013-07-25 at 16:43 +0300, Ivan T. Ivanov wrote:
> > > On Wed, 2013-07-24 at 15:39 +0300, Felipe Balbi wrote:
> > > > On Tue, Jul 09, 2013 at 06:47:09PM +0300, Ivan T. Ivanov wrote:
> > > > > From: "Ivan T. Ivanov" 
> > > > > 
> > > > > This patch move global regulators variables to driver state
> > > > > structire and move allocation of the regulators to be devm managed.
> > > > 
> > > > split into two patches please. One for moving the global regulators into
> > > > your structure and a separate patch to move to devm_*
> > > > 
> > 
> > Would you accept patch which convert all resources allocation to devm_
> > variants in single patch or you prefer separate patches for memory, 
> > clocks, regulators and irq?
> 
> devm_* all over the place should be fine ;-)
> 

Thanks, it is coming. 

Regards,
Ivan

--
To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH v2 3/7] usb: phy: msm: Move regulator usage to managed resource allocation

2013-07-26 Thread Felipe Balbi
On Fri, Jul 26, 2013 at 03:31:34PM +0300, Ivan T. Ivanov wrote:
> 
> Hi Felipe, 
> 
> On Thu, 2013-07-25 at 16:43 +0300, Ivan T. Ivanov wrote:
> > On Wed, 2013-07-24 at 15:39 +0300, Felipe Balbi wrote:
> > > On Tue, Jul 09, 2013 at 06:47:09PM +0300, Ivan T. Ivanov wrote:
> > > > From: "Ivan T. Ivanov" 
> > > > 
> > > > This patch move global regulators variables to driver state
> > > > structire and move allocation of the regulators to be devm managed.
> > > 
> > > split into two patches please. One for moving the global regulators into
> > > your structure and a separate patch to move to devm_*
> > > 
> 
> Would you accept patch which convert all resources allocation to devm_
> variants in single patch or you prefer separate patches for memory, 
> clocks, regulators and irq?

devm_* all over the place should be fine ;-)

-- 
balbi


signature.asc
Description: Digital signature


Re: [PATCH v2 3/7] usb: phy: msm: Move regulator usage to managed resource allocation

2013-07-26 Thread Ivan T. Ivanov

Hi Felipe, 

On Thu, 2013-07-25 at 16:43 +0300, Ivan T. Ivanov wrote:
> On Wed, 2013-07-24 at 15:39 +0300, Felipe Balbi wrote:
> > On Tue, Jul 09, 2013 at 06:47:09PM +0300, Ivan T. Ivanov wrote:
> > > From: "Ivan T. Ivanov" 
> > > 
> > > This patch move global regulators variables to driver state
> > > structire and move allocation of the regulators to be devm managed.
> > 
> > split into two patches please. One for moving the global regulators into
> > your structure and a separate patch to move to devm_*
> > 

Would you accept patch which convert all resources allocation to devm_
variants in single patch or you prefer separate patches for memory, 
clocks, regulators and irq?

Regards,
Ivan

> 
> Ok, will do and resend.
> 
> Ivan
> 
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majord...@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at  http://www.tux.org/lkml/


--
To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH v2 3/7] usb: phy: msm: Move regulator usage to managed resource allocation

2013-07-25 Thread Ivan T. Ivanov

On Wed, 2013-07-24 at 15:39 +0300, Felipe Balbi wrote:
> On Tue, Jul 09, 2013 at 06:47:09PM +0300, Ivan T. Ivanov wrote:
> > From: "Ivan T. Ivanov" 
> > 
> > This patch move global regulators variables to driver state
> > structire and move allocation of the regulators to be devm managed.
> 
> split into two patches please. One for moving the global regulators into
> your structure and a separate patch to move to devm_*
> 

Ok, will do and resend.

Ivan

--
To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH v2 3/7] usb: phy: msm: Move regulator usage to managed resource allocation

2013-07-24 Thread Felipe Balbi
On Tue, Jul 09, 2013 at 06:47:09PM +0300, Ivan T. Ivanov wrote:
> From: "Ivan T. Ivanov" 
> 
> This patch move global regulators variables to driver state
> structire and move allocation of the regulators to be devm managed.

split into two patches please. One for moving the global regulators into
your structure and a separate patch to move to devm_*

-- 
balbi


signature.asc
Description: Digital signature