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
Attachment:
signature.asc
Description: Digital signature