Message ID | 1401794126-22244-1-git-send-email-laurent.pinchart@ideasonboard.com |
---|---|
State | Accepted |
Headers | show |
On Tue, Jun 3, 2014 at 1:15 PM, Laurent Pinchart <laurent.pinchart@ideasonboard.com> wrote: > The ID-based GPIO API pushes handling of GPIO polarity to drivers. > Simplify the driver by switching to the descriptor-based GPIO API. > > This also fixes a mismatch between the pca954x DT bindings that document > a "reset-gpios" property and the driver that requests a "reset-gpio" > property. > > Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> Reviewed-by: Linus Walleij <linus.walleij@linaro.org> Yours, Linus Walleij -- To unsubscribe from this list: send the line "unsubscribe linux-i2c" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Tue, Jun 03, 2014 at 01:15:26PM +0200, Laurent Pinchart wrote: > The ID-based GPIO API pushes handling of GPIO polarity to drivers. > Simplify the driver by switching to the descriptor-based GPIO API. > > This also fixes a mismatch between the pca954x DT bindings that document > a "reset-gpios" property and the driver that requests a "reset-gpio" > property. > > Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> Applied to for-next, thanks!
On 06/03/2014 08:15 PM, Laurent Pinchart wrote: > The ID-based GPIO API pushes handling of GPIO polarity to drivers. > Simplify the driver by switching to the descriptor-based GPIO API. > > This also fixes a mismatch between the pca954x DT bindings that document > a "reset-gpios" property and the driver that requests a "reset-gpio" > property. > > Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> Reviewed-by: Alexandre Courbot <acourbot@nvidia.com> ----------------------------------------------------------------------------------- This email message is for the sole use of the intended recipient(s) and may contain confidential information. Any unauthorized review, use, disclosure or distribution is prohibited. If you are not the intended recipient, please contact the sender by reply email and destroy all copies of the original message. ----------------------------------------------------------------------------------- -- To unsubscribe from this list: send the line "unsubscribe linux-i2c" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/drivers/i2c/muxes/i2c-mux-pca954x.c b/drivers/i2c/muxes/i2c-mux-pca954x.c index 550bd36..c2c443f 100644 --- a/drivers/i2c/muxes/i2c-mux-pca954x.c +++ b/drivers/i2c/muxes/i2c-mux-pca954x.c @@ -186,7 +186,7 @@ static int pca954x_probe(struct i2c_client *client, { struct i2c_adapter *adap = to_i2c_adapter(client->dev.parent); struct pca954x_platform_data *pdata = dev_get_platdata(&client->dev); - struct device_node *np = client->dev.of_node; + struct gpio_desc *gpio; int num, force, class; struct pca954x *data; int ret; @@ -200,21 +200,10 @@ static int pca954x_probe(struct i2c_client *client, i2c_set_clientdata(client, data); - if (IS_ENABLED(CONFIG_OF) && np) { - enum of_gpio_flags flags; - int gpio; - - /* Get the mux out of reset if a reset GPIO is specified. */ - gpio = of_get_named_gpio_flags(np, "reset-gpio", 0, &flags); - if (gpio_is_valid(gpio)) { - ret = devm_gpio_request_one(&client->dev, gpio, - flags & OF_GPIO_ACTIVE_LOW ? - GPIOF_OUT_INIT_HIGH : GPIOF_OUT_INIT_LOW, - "pca954x reset"); - if (ret < 0) - return ret; - } - } + /* Get the mux out of reset if a reset GPIO is specified. */ + gpio = devm_gpiod_get(&client->dev, "reset"); + if (!IS_ERR(gpio)) + gpiod_direction_output(gpio, 0); /* Write the mux register at addr to verify * that the mux is in fact present. This also
The ID-based GPIO API pushes handling of GPIO polarity to drivers. Simplify the driver by switching to the descriptor-based GPIO API. This also fixes a mismatch between the pca954x DT bindings that document a "reset-gpios" property and the driver that requests a "reset-gpio" property. Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> --- drivers/i2c/muxes/i2c-mux-pca954x.c | 21 +++++---------------- 1 file changed, 5 insertions(+), 16 deletions(-)