Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755452Ab0GaHKI (ORCPT ); Sat, 31 Jul 2010 03:10:08 -0400 Received: from mail-fx0-f46.google.com ([209.85.161.46]:41603 "EHLO mail-fx0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751161Ab0GaHKE (ORCPT ); Sat, 31 Jul 2010 03:10:04 -0400 DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=from:to:cc:subject:date:message-id:x-mailer; b=oib+arg+IYMrFTztoZ4VjV8s8ymf226PZSxZgWdTgeFUprmHzI/5tzOtaIeuX+LpXO vqtgewt/pXxR23NS3R+xUge5pJzpDeBFCAn0D2rqx+orN17xq8h7aYPm2Ek5m8NbisIq TFPDdqaOelXCf7oX6neLeo0s+jfZlszqgwtw4= From: Marek Vasut To: linux-arm-kernel@lists.infradead.org Cc: dmitry.torokhov@gmail.com, vapier@gentoo.org, pavel@ucw.cz, akpm@linux-foundation.org, khilman@deeprootsystems.com, linux-input@vger.kernel.org, linux-kernel@vger.kernel.org, eric.y.miao@gmail.com, Marek Vasut Subject: [PATCH v2] Input: Make ADS7846 independent on regulator Date: Sat, 31 Jul 2010 09:09:42 +0200 Message-Id: <1280560182-7071-1-git-send-email-marek.vasut@gmail.com> X-Mailer: git-send-email 1.7.1 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2768 Lines: 96 In case regulator was not found, ADS7846 failed to probe. This fixes a problem on some Sharp Zaurus devices, where there is no regulator present and the touchscreen is used. Signed-off-by: Marek Vasut --- drivers/input/touchscreen/ads7846.c | 38 +++++++++++++++++++--------------- 1 files changed, 21 insertions(+), 17 deletions(-) diff --git a/drivers/input/touchscreen/ads7846.c b/drivers/input/touchscreen/ads7846.c index a9fdf55..53b318b 100644 --- a/drivers/input/touchscreen/ads7846.c +++ b/drivers/input/touchscreen/ads7846.c @@ -791,7 +791,8 @@ static void ads7846_disable(struct ads7846 *ts) } } - regulator_disable(ts->reg); + if (ts->reg) + regulator_disable(ts->reg); /* we know the chip's in lowpower mode since we always * leave it that way after every request @@ -804,7 +805,8 @@ static void ads7846_enable(struct ads7846 *ts) if (!ts->disabled) return; - regulator_enable(ts->reg); + if (ts->reg) + regulator_enable(ts->reg); ts->disabled = 0; ts->irq_disabled = 0; @@ -1162,16 +1164,15 @@ static int __devinit ads7846_probe(struct spi_device *spi) ts->last_msg = m; ts->reg = regulator_get(&spi->dev, "vcc"); - if (IS_ERR(ts->reg)) { - err = PTR_ERR(ts->reg); - dev_err(&spi->dev, "unable to get regulator: %d\n", err); - goto err_free_gpio; - } - - err = regulator_enable(ts->reg); - if (err) { - dev_err(&spi->dev, "unable to enable regulator: %d\n", err); - goto err_put_regulator; + if (IS_ERR(ts->reg)) + ts->reg = NULL; + else { + err = regulator_enable(ts->reg); + if (err) { + dev_err(&spi->dev, "unable to enable regulator: %d\n", + err); + goto err_put_regulator; + } } if (request_irq(spi->irq, ads7846_irq, IRQF_TRIGGER_FALLING, @@ -1218,10 +1219,11 @@ static int __devinit ads7846_probe(struct spi_device *spi) err_free_irq: free_irq(spi->irq, ts); err_disable_regulator: - regulator_disable(ts->reg); + if (ts->reg) + regulator_disable(ts->reg); err_put_regulator: - regulator_put(ts->reg); - err_free_gpio: + if (ts->reg) + regulator_put(ts->reg); if (ts->gpio_pendown != -1) gpio_free(ts->gpio_pendown); err_cleanup_filter: @@ -1251,8 +1253,10 @@ static int __devexit ads7846_remove(struct spi_device *spi) /* suspend left the IRQ disabled */ enable_irq(ts->spi->irq); - regulator_disable(ts->reg); - regulator_put(ts->reg); + if (ts->reg) { + regulator_disable(ts->reg); + regulator_put(ts->reg); + } if (ts->gpio_pendown != -1) gpio_free(ts->gpio_pendown); -- 1.7.1 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/