Re: [PATCH 2/2] HID: i2c-hid: support the regulator

2016-09-14 Thread Benjamin Tissoires
On Sep 14 2016 or thereabouts, Brian Norris wrote: > On Wed, Sep 14, 2016 at 03:55:05PM +0800, Brian Norris wrote: > > The default behavior of regulator_get() is to provide a dummy regulator > > if none is found. So the pointer is never NULL, and it won't break > > devices without a regulator. If y

Re: [PATCH 2/2] HID: i2c-hid: support the regulator

2016-09-14 Thread Brian Norris
On Wed, Sep 14, 2016 at 03:55:05PM +0800, Brian Norris wrote: > The default behavior of regulator_get() is to provide a dummy regulator > if none is found. So the pointer is never NULL, and it won't break > devices without a regulator. If you don't want a dummy regulator you > would use regulator_g

Re: [PATCH 2/2] HID: i2c-hid: support the regulator

2016-09-14 Thread Brian Norris
Hi Benjamin, On Wed, Sep 14, 2016 at 09:36:03AM +0200, Benjamin Tissoires wrote: > On Sep 05 2016 or thereabouts, Caesar Wang wrote: > > diff --git a/drivers/hid/i2c-hid/i2c-hid.c b/drivers/hid/i2c-hid/i2c-hid.c > > index b3ec4f2..07cc7aa 100644 > > --- a/drivers/hid/i2c-hid/i2c-hid.c > > +++ b/dr

Re: [PATCH 2/2] HID: i2c-hid: support the regulator

2016-09-14 Thread Benjamin Tissoires
On Sep 05 2016 or thereabouts, Caesar Wang wrote: > From: Brian Norris > > In order to allow supporting the HID based devices that need power on/off > the regulator. We try to get a power-supply property from the > device tree. > > Signed-off-by: Brian Norris > Signed-off-by: Caesar Wang > Cc: