From patchwork Sun Mar 3 18:58:15 2013 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Jean Delvare X-Patchwork-Id: 224564 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by ozlabs.org (Postfix) with ESMTP id B88862C030B for ; Mon, 4 Mar 2013 05:58:37 +1100 (EST) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753557Ab3CCS6Y (ORCPT ); Sun, 3 Mar 2013 13:58:24 -0500 Received: from zoneX.GCU-Squad.org ([194.213.125.0]:28244 "EHLO services.gcu-squad.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753199Ab3CCS6Y (ORCPT ); Sun, 3 Mar 2013 13:58:24 -0500 Received: from jdelvare.pck.nerim.net ([62.212.121.182] helo=endymion.delvare) by services.gcu-squad.org (GCU Mailer Daemon) with esmtpsa id 1UCE73-00009f-0Z (TLSv1:AES128-SHA:128) (envelope-from ) ; Sun, 03 Mar 2013 19:58:21 +0100 Date: Sun, 3 Mar 2013 19:58:15 +0100 From: Jean Delvare To: Peter Korsgaard Cc: Linux I2C , Wolfram Sang Subject: [PATCH] i2c-mux-gpio: Check gpio_direction_output return value Message-ID: <20130303195815.10ba20b1@endymion.delvare> X-Mailer: Claws Mail 3.7.10 (GTK+ 2.24.7; x86_64-suse-linux-gnu) Mime-Version: 1.0 Sender: linux-i2c-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-i2c@vger.kernel.org gpio_direction_output() may fail, check for that and deal with it appropriately. Also log an error message if gpio_request() fails. Signed-off-by: Jean Delvare Cc: Peter Korsgaard Cc: Wolfram Sang Acked-by: Peter Korsgaard --- drivers/i2c/muxes/i2c-mux-gpio.c | 17 ++++++++++++++--- 1 file changed, 14 insertions(+), 3 deletions(-) --- linux-3.7.orig/drivers/i2c/muxes/i2c-mux-gpio.c 2012-12-15 14:52:47.368991730 +0100 +++ linux-3.7/drivers/i2c/muxes/i2c-mux-gpio.c 2013-03-03 14:27:21.003905620 +0100 @@ -123,10 +123,21 @@ static int __devinit i2c_mux_gpio_probe( for (i = 0; i < pdata->n_gpios; i++) { ret = gpio_request(gpio_base + pdata->gpios[i], "i2c-mux-gpio"); - if (ret) + if (ret) { + dev_err(&pdev->dev, "Failed to request GPIO %d\n", + pdata->gpios[i]); goto err_request_gpio; - gpio_direction_output(gpio_base + pdata->gpios[i], - initial_state & (1 << i)); + } + + ret = gpio_direction_output(gpio_base + pdata->gpios[i], + initial_state & (1 << i)); + if (ret) { + dev_err(&pdev->dev, + "Failed to set direction of GPIO %d to output\n", + pdata->gpios[i]); + i++; + goto err_request_gpio; + } } for (i = 0; i < pdata->n_values; i++) {