Re: [PATCH v2 0/3] input: rmi4: Regulator supply support

2016-05-16 Thread Andrew Duggan
On 05/13/2016 03:29 PM, Bjorn Andersson wrote: On Thu 12 May 17:52 PDT 2016, Andrew Duggan wrote: On 05/11/2016 08:05 PM, Bjorn Andersson wrote: On Wed 11 May 16:30 PDT 2016, Andrew Duggan wrote: Hi Bjorn, On 05/10/2016 08:49 AM, Bjorn Andersson wrote: [..] So either we duplicate the

Re: [PATCH v2 0/3] input: rmi4: Regulator supply support

2016-05-16 Thread Andrew Duggan
On 05/13/2016 03:29 PM, Bjorn Andersson wrote: On Thu 12 May 17:52 PDT 2016, Andrew Duggan wrote: On 05/11/2016 08:05 PM, Bjorn Andersson wrote: On Wed 11 May 16:30 PDT 2016, Andrew Duggan wrote: Hi Bjorn, On 05/10/2016 08:49 AM, Bjorn Andersson wrote: [..] So either we duplicate the

Re: [PATCH v2 0/3] input: rmi4: Regulator supply support

2016-05-13 Thread Bjorn Andersson
On Thu 12 May 17:52 PDT 2016, Andrew Duggan wrote: > On 05/11/2016 08:05 PM, Bjorn Andersson wrote: > >On Wed 11 May 16:30 PDT 2016, Andrew Duggan wrote: > > > >>Hi Bjorn, > >> > >>On 05/10/2016 08:49 AM, Bjorn Andersson wrote: > >[..] > >>>So either we duplicate the regulator support in spi/i2c

Re: [PATCH v2 0/3] input: rmi4: Regulator supply support

2016-05-13 Thread Bjorn Andersson
On Thu 12 May 17:52 PDT 2016, Andrew Duggan wrote: > On 05/11/2016 08:05 PM, Bjorn Andersson wrote: > >On Wed 11 May 16:30 PDT 2016, Andrew Duggan wrote: > > > >>Hi Bjorn, > >> > >>On 05/10/2016 08:49 AM, Bjorn Andersson wrote: > >[..] > >>>So either we duplicate the regulator support in spi/i2c

Re: [PATCH v2 0/3] input: rmi4: Regulator supply support

2016-05-12 Thread Andrew Duggan
On 05/11/2016 08:05 PM, Bjorn Andersson wrote: On Wed 11 May 16:30 PDT 2016, Andrew Duggan wrote: Hi Bjorn, On 05/10/2016 08:49 AM, Bjorn Andersson wrote: [..] So either we duplicate the regulator support in spi/i2c or we make them optional in the core driver. Sounds like you prefer the

Re: [PATCH v2 0/3] input: rmi4: Regulator supply support

2016-05-12 Thread Andrew Duggan
On 05/11/2016 08:05 PM, Bjorn Andersson wrote: On Wed 11 May 16:30 PDT 2016, Andrew Duggan wrote: Hi Bjorn, On 05/10/2016 08:49 AM, Bjorn Andersson wrote: [..] So either we duplicate the regulator support in spi/i2c or we make them optional in the core driver. Sounds like you prefer the

Re: [PATCH v2 0/3] input: rmi4: Regulator supply support

2016-05-11 Thread Bjorn Andersson
On Wed 11 May 16:30 PDT 2016, Andrew Duggan wrote: > Hi Bjorn, > > On 05/10/2016 08:49 AM, Bjorn Andersson wrote: [..] > >So either we duplicate the regulator support in spi/i2c or we make them > >optional in the core driver. Sounds like you prefer the prior, i.e. v1 > >of my patch. > > Yes,

Re: [PATCH v2 0/3] input: rmi4: Regulator supply support

2016-05-11 Thread Bjorn Andersson
On Wed 11 May 16:30 PDT 2016, Andrew Duggan wrote: > Hi Bjorn, > > On 05/10/2016 08:49 AM, Bjorn Andersson wrote: [..] > >So either we duplicate the regulator support in spi/i2c or we make them > >optional in the core driver. Sounds like you prefer the prior, i.e. v1 > >of my patch. > > Yes,

Re: [PATCH v2 0/3] input: rmi4: Regulator supply support

2016-05-11 Thread Andrew Duggan
Hi Bjorn, On 05/10/2016 08:49 AM, Bjorn Andersson wrote: On Mon 09 May 17:36 PDT 2016, Andrew Duggan wrote: Hi Bjorn, On 05/06/2016 09:40 PM, Bjorn Andersson wrote: The first version of the regulator support patch suffered from being implemented in the transport driver, as a work around for

Re: [PATCH v2 0/3] input: rmi4: Regulator supply support

2016-05-11 Thread Andrew Duggan
Hi Bjorn, On 05/10/2016 08:49 AM, Bjorn Andersson wrote: On Mon 09 May 17:36 PDT 2016, Andrew Duggan wrote: Hi Bjorn, On 05/06/2016 09:40 PM, Bjorn Andersson wrote: The first version of the regulator support patch suffered from being implemented in the transport driver, as a work around for

Re: [PATCH v2 0/3] input: rmi4: Regulator supply support

2016-05-10 Thread Bjorn Andersson
On Mon 09 May 17:36 PDT 2016, Andrew Duggan wrote: > Hi Bjorn, > > On 05/06/2016 09:40 PM, Bjorn Andersson wrote: > >The first version of the regulator support patch suffered from being > >implemented in the transport driver, as a work around for resource > >availability > >racing (EPROBE_DEFER

Re: [PATCH v2 0/3] input: rmi4: Regulator supply support

2016-05-10 Thread Bjorn Andersson
On Mon 09 May 17:36 PDT 2016, Andrew Duggan wrote: > Hi Bjorn, > > On 05/06/2016 09:40 PM, Bjorn Andersson wrote: > >The first version of the regulator support patch suffered from being > >implemented in the transport driver, as a work around for resource > >availability > >racing (EPROBE_DEFER

Re: [PATCH v2 0/3] input: rmi4: Regulator supply support

2016-05-09 Thread Andrew Duggan
Hi Bjorn, On 05/06/2016 09:40 PM, Bjorn Andersson wrote: The first version of the regulator support patch suffered from being implemented in the transport driver, as a work around for resource availability racing (EPROBE_DEFER of the core driver) with the interrupt handler. After reconsidering

Re: [PATCH v2 0/3] input: rmi4: Regulator supply support

2016-05-09 Thread Andrew Duggan
Hi Bjorn, On 05/06/2016 09:40 PM, Bjorn Andersson wrote: The first version of the regulator support patch suffered from being implemented in the transport driver, as a work around for resource availability racing (EPROBE_DEFER of the core driver) with the interrupt handler. After reconsidering

[PATCH v2 0/3] input: rmi4: Regulator supply support

2016-05-06 Thread Bjorn Andersson
The first version of the regulator support patch suffered from being implemented in the transport driver, as a work around for resource availability racing (EPROBE_DEFER of the core driver) with the interrupt handler. After reconsidering the solutions discussed following that I concluded that the

[PATCH v2 0/3] input: rmi4: Regulator supply support

2016-05-06 Thread Bjorn Andersson
The first version of the regulator support patch suffered from being implemented in the transport driver, as a work around for resource availability racing (EPROBE_DEFER of the core driver) with the interrupt handler. After reconsidering the solutions discussed following that I concluded that the