Re: [PATCH] media: i2c/adp1653: devicetree support for adp1653
On Tue 2014-12-23 21:49:04, Pavel Machek wrote: > On Tue 2014-12-23 15:23:25, Mauro Carvalho Chehab wrote: > > Em Wed, 3 Dec 2014 22:46:41 +0100 > > Pavel Machek escreveu: > > > > > > > > We are moving to device tree support on OMAP3, but that currently > > > breaks ADP1653 driver. This adds device tree support, plus required > > > documentation. > > > > > > Signed-off-by: Pavel Machek > > > > Please be sure to check your patch with checkpatch. There are several > > issues on it: Ok, you should have fixed version in your inbox. Happy holidays! Pavel -- (english) http://www.livejournal.com/~pavelmachek (cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH] media: i2c/adp1653: devicetree support for adp1653
On Tue 2014-12-23 15:23:25, Mauro Carvalho Chehab wrote: > Em Wed, 3 Dec 2014 22:46:41 +0100 > Pavel Machek escreveu: > > > > > We are moving to device tree support on OMAP3, but that currently > > breaks ADP1653 driver. This adds device tree support, plus required > > documentation. > > > > Signed-off-by: Pavel Machek > > Please be sure to check your patch with checkpatch. There are several > issues on it: > WARNING: DT compatible string "adi,adp1653" appears un-documented -- check > ./Documentation/devicetree/bindings/ > #78: FILE: arch/arm/boot/dts/omap3-n900.dts:572: > + compatible = "adi,adp1653"; Hmm. Take a look at part quoted below. Someone needs to fix checkpatch. Ccing authors. > ERROR: trailing whitespace > WARNING: line over 80 characters Will fix. > ERROR: trailing statements should be on next line > #177: FILE: drivers/media/i2c/adp1653.c:454: > + if (!child) return -EINVAL; I actually did these on purporse... the function is trivial, and by keeping returns on one line it fits into screen. I see you want it fixed below, so I'll do that for next version. Pavel > > --- /dev/null > > +++ b/Documentation/devicetree/bindings/media/i2c/adp1653.txt > > @@ -0,0 +1,38 @@ > > +* Analog Devices ADP1653 flash LED driver > > + > > +Required Properties: > > + > > + - compatible: Must contain one of the following > > +- "adi,adp1653" > > + > > + - reg: I2C slave address > > + > > + - gpios: References to the GPIO that controls the power for the chip. > > + > > +There are two led outputs available - flash and indicator. One led is > > +represented by one child node, nodes need to be named "flash" and > > "indicator". > > + > > +Required properties of the LED child node: > > +- max-microamp : see Documentation/devicetree/bindings/leds/common.txt > > + > > +Required properties of the flash LED child node: > > + > > +- flash-max-microamp : see > > Documentation/devicetree/bindings/leds/common.txt > > +- flash-timeout-microsec : see > > Documentation/devicetree/bindings/leds/common.txt > > + > > +Example: > > + > > +adp1653: led-controller@30 { > > +compatible = "adi,adp1653"; > > + reg = <0x30>; > > +gpios = <&gpio3 24 GPIO_ACTIVE_HIGH>; /* 88 */ > > + > > + flash { > > +flash-timeout-microsec = <50>; > > +flash-max-microamp = <32>; > > +max-microamp = <5>; > > + }; > > +indicator { > > +max-microamp = <17500>; > > + }; > > +}; -- (english) http://www.livejournal.com/~pavelmachek (cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH] media: i2c/adp1653: devicetree support for adp1653
Em Wed, 3 Dec 2014 22:46:41 +0100 Pavel Machek escreveu: > > We are moving to device tree support on OMAP3, but that currently > breaks ADP1653 driver. This adds device tree support, plus required > documentation. > > Signed-off-by: Pavel Machek Please be sure to check your patch with checkpatch. There are several issues on it: WARNING: DT compatible string "adi,adp1653" appears un-documented -- check ./Documentation/devicetree/bindings/ #78: FILE: arch/arm/boot/dts/omap3-n900.dts:572: + compatible = "adi,adp1653"; ERROR: trailing whitespace #136: FILE: drivers/media/i2c/adp1653.c:332: +^I^I^I$ ERROR: trailing whitespace #159: FILE: drivers/media/i2c/adp1653.c:436: +static int adp1653_of_init(struct i2c_client *client, struct adp1653_flash *flash, $ WARNING: line over 80 characters #159: FILE: drivers/media/i2c/adp1653.c:436: +static int adp1653_of_init(struct i2c_client *client, struct adp1653_flash *flash, ERROR: trailing statements should be on next line #177: FILE: drivers/media/i2c/adp1653.c:454: + if (!child) return -EINVAL; WARNING: line over 80 characters #178: FILE: drivers/media/i2c/adp1653.c:455: + if (of_property_read_u32(child, "flash-timeout-microsec", &val)) return -EINVAL; ERROR: trailing statements should be on next line #178: FILE: drivers/media/i2c/adp1653.c:455: + if (of_property_read_u32(child, "flash-timeout-microsec", &val)) return -EINVAL; WARNING: line over 80 characters #180: FILE: drivers/media/i2c/adp1653.c:457: + if (of_property_read_u32(child, "flash-max-microamp", &val)) return -EINVAL; ERROR: trailing statements should be on next line #180: FILE: drivers/media/i2c/adp1653.c:457: + if (of_property_read_u32(child, "flash-max-microamp", &val)) return -EINVAL; ERROR: trailing statements should be on next line #182: FILE: drivers/media/i2c/adp1653.c:459: + if (of_property_read_u32(child, "max-microamp", &val)) return -EINVAL; ERROR: trailing statements should be on next line #186: FILE: drivers/media/i2c/adp1653.c:463: + if (!child) return -EINVAL; ERROR: trailing statements should be on next line #187: FILE: drivers/media/i2c/adp1653.c:464: + if (of_property_read_u32(child, "max-microamp", &val)) return -EINVAL; ERROR: trailing whitespace #197: FILE: drivers/media/i2c/adp1653.c:474: +^I^Idev_err(&client->dev, "Error getting GPIO\n"); $ total: 9 errors, 4 warnings, 199 lines checked NOTE: whitespace errors detected, you may wish to use scripts/cleanpatch or scripts/cleanfile > > --- /dev/null > +++ b/Documentation/devicetree/bindings/media/i2c/adp1653.txt > @@ -0,0 +1,38 @@ > +* Analog Devices ADP1653 flash LED driver > + > +Required Properties: > + > + - compatible: Must contain one of the following > +- "adi,adp1653" > + > + - reg: I2C slave address > + > + - gpios: References to the GPIO that controls the power for the chip. > + > +There are two led outputs available - flash and indicator. One led is > +represented by one child node, nodes need to be named "flash" and > "indicator". > + > +Required properties of the LED child node: > +- max-microamp : see Documentation/devicetree/bindings/leds/common.txt > + > +Required properties of the flash LED child node: > + > +- flash-max-microamp : see Documentation/devicetree/bindings/leds/common.txt > +- flash-timeout-microsec : see > Documentation/devicetree/bindings/leds/common.txt > + > +Example: > + > +adp1653: led-controller@30 { > +compatible = "adi,adp1653"; > + reg = <0x30>; > +gpios = <&gpio3 24 GPIO_ACTIVE_HIGH>; /* 88 */ > + > + flash { > +flash-timeout-microsec = <50>; > +flash-max-microamp = <32>; > +max-microamp = <5>; > + }; > +indicator { > +max-microamp = <17500>; > + }; > +}; > diff --git a/arch/arm/boot/dts/omap3-n900.dts > b/arch/arm/boot/dts/omap3-n900.dts > index cfddc3d..11d8afd 100644 > --- a/arch/arm/boot/dts/omap3-n900.dts > +++ b/arch/arm/boot/dts/omap3-n900.dts > @@ -560,6 +567,22 @@ > > ti,usb-charger-detection = <&isp1704>; > }; > + > + adp1653: led-controller@30 { > + compatible = "adi,adp1653"; > + reg = <0x30>; > + gpios = <&gpio3 24 GPIO_ACTIVE_HIGH>; /* 88 */ > + > + flash { > + flash-timeout-microsec = <50>; > + flash-max-microamp = <32>; > + max-microamp = <5>; > + }; > + > + indicator { > + max-microamp = <17500>; > + }; > + }; > }; > > &i2c3 { > diff --git a/drivers/media/i2c/adp1653.c b/drivers/media/i2c/adp1653.c > index 873fe19..62601b2 100644 > --- a/drivers/media/i2c/adp1653.c > +++ b/drivers/media/i2c/adp1653.c > @@ -8,6 +8,7 @@ > * Contributors: > * Sakar
[PATCH] media: i2c/adp1653: devicetree support for adp1653
We are moving to device tree support on OMAP3, but that currently breaks ADP1653 driver. This adds device tree support, plus required documentation. Signed-off-by: Pavel Machek --- /dev/null +++ b/Documentation/devicetree/bindings/media/i2c/adp1653.txt @@ -0,0 +1,38 @@ +* Analog Devices ADP1653 flash LED driver + +Required Properties: + + - compatible: Must contain one of the following +- "adi,adp1653" + + - reg: I2C slave address + + - gpios: References to the GPIO that controls the power for the chip. + +There are two led outputs available - flash and indicator. One led is +represented by one child node, nodes need to be named "flash" and "indicator". + +Required properties of the LED child node: +- max-microamp : see Documentation/devicetree/bindings/leds/common.txt + +Required properties of the flash LED child node: + +- flash-max-microamp : see Documentation/devicetree/bindings/leds/common.txt +- flash-timeout-microsec : see Documentation/devicetree/bindings/leds/common.txt + +Example: + +adp1653: led-controller@30 { +compatible = "adi,adp1653"; + reg = <0x30>; +gpios = <&gpio3 24 GPIO_ACTIVE_HIGH>; /* 88 */ + + flash { +flash-timeout-microsec = <50>; +flash-max-microamp = <32>; +max-microamp = <5>; + }; +indicator { +max-microamp = <17500>; + }; +}; diff --git a/arch/arm/boot/dts/omap3-n900.dts b/arch/arm/boot/dts/omap3-n900.dts index cfddc3d..11d8afd 100644 --- a/arch/arm/boot/dts/omap3-n900.dts +++ b/arch/arm/boot/dts/omap3-n900.dts @@ -560,6 +567,22 @@ ti,usb-charger-detection = <&isp1704>; }; + + adp1653: led-controller@30 { + compatible = "adi,adp1653"; + reg = <0x30>; + gpios = <&gpio3 24 GPIO_ACTIVE_HIGH>; /* 88 */ + + flash { + flash-timeout-microsec = <50>; + flash-max-microamp = <32>; + max-microamp = <5>; + }; + + indicator { + max-microamp = <17500>; + }; + }; }; &i2c3 { diff --git a/drivers/media/i2c/adp1653.c b/drivers/media/i2c/adp1653.c index 873fe19..62601b2 100644 --- a/drivers/media/i2c/adp1653.c +++ b/drivers/media/i2c/adp1653.c @@ -8,6 +8,7 @@ * Contributors: * Sakari Ailus * Tuukka Toivonen + * Pavel Machek * * This program is free software; you can redistribute it and/or * modify it under the terms of the GNU General Public License @@ -34,9 +35,11 @@ #include #include #include +#include +#include #include #include #define TIMEOUT_MAX82 #define TIMEOUT_STEP 54600 #define TIMEOUT_MIN(TIMEOUT_MAX - ADP1653_REG_CONFIG_TMR_SET_MAX \ @@ -306,9 +318,18 @@ adp1653_init_device(struct adp1653_flash *flash) static int __adp1653_set_power(struct adp1653_flash *flash, int on) { - int ret; + int ret = 0; - ret = flash->platform_data->power(&flash->subdev, on); + if (flash->platform_data->power) + ret = flash->platform_data->power(&flash->subdev, on); + else { + gpio_set_value(flash->platform_data->power_gpio, on); + if (on) { + /* Some delay is apparently required. */ + udelay(20); + } + } + if (ret < 0) return ret; @@ -316,8 +337,13 @@ __adp1653_set_power(struct adp1653_flash *flash, int on) return 0; ret = adp1653_init_device(flash); - if (ret < 0) + if (ret >= 0) + return ret; + + if (flash->platform_data->power) flash->platform_data->power(&flash->subdev, 0); + else + gpio_set_value(flash->platform_data->power_gpio, 0); return ret; } @@ -407,21 +433,69 @@ static int adp1653_resume(struct device *dev) #endif /* CONFIG_PM */ +static int adp1653_of_init(struct i2c_client *client, struct adp1653_flash *flash, + struct device_node *node) +{ + u32 val; + struct adp1653_platform_data *pd; + enum of_gpio_flags flags; + int gpio; + struct device_node *child; + + if (!node) + return -EINVAL; + + pd = devm_kzalloc(&client->dev, sizeof(*pd), GFP_KERNEL); + if (!pd) + return -ENOMEM; + flash->platform_data = pd; + + child = of_get_child_by_name(node, "flash"); + if (!child) return -EINVAL; + if (of_property_read_u32(child, "flash-timeout-microsec", &val)) return -EINVAL; + pd->max_flash_timeout = val; + if (of_property_read_u32(child, "flash-max-microamp", &val)) return -EINVAL; + pd->max_flash_intensity = val