On Thursday 14 November 2013 23:34:19 Sebastian Reichel wrote: > On Thu, Nov 14, 2013 at 07:05:06PM +0100, Pali Rohár wrote: > > On Thursday 14 November 2013 13:38:53 Sebastian Reichel wrote: > > > diff --git a/drivers/power/isp1704_charger.c > > > b/drivers/power/isp1704_charger.c index fc04d19..db96778 > > > 100644 > > > --- a/drivers/power/isp1704_charger.c > > > +++ b/drivers/power/isp1704_charger.c > > > @@ -28,6 +28,8 @@ > > > > > > #include <linux/platform_device.h> > > > #include <linux/power_supply.h> > > > #include <linux/delay.h> > > > > > > +#include <linux/of.h> > > > +#include <linux/of_gpio.h> > > > > > > #include <linux/usb/otg.h> > > > #include <linux/usb/ulpi.h> > > > > > > @@ -89,8 +91,8 @@ static void > > > isp1704_charger_set_power(struct isp1704_charger *isp, > > > bool on) { > > > > > > struct isp1704_charger_data *board = > > > > > > isp->dev->platform_data; > > > > > > - if (board && board->set_power) > > > - board->set_power(on); > > > + if (board) > > > + gpio_set_value(board->enable_gpio, on); > > > > > > } > > > > You need to check if enable_gpio in board data was defined > > or not. > > The device is not successful probed without valid enable_gpio. > > -- Sebastian Then, OK. -- Pali Rohár pali.rohar@xxxxxxxxx
Attachment:
signature.asc
Description: This is a digitally signed message part.