From patchwork Fri Oct 9 00:01:55 2015 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: William Breathitt Gray X-Patchwork-Id: 528023 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 3F6FC140D9B for ; Fri, 9 Oct 2015 11:02:09 +1100 (AEDT) Authentication-Results: ozlabs.org; dkim=fail reason="signature verification failed" (2048-bit key; unprotected) header.d=gmail.com header.i=@gmail.com header.b=g5MliAHE; dkim-atps=neutral Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756977AbbJIACI (ORCPT ); Thu, 8 Oct 2015 20:02:08 -0400 Received: from mail-yk0-f175.google.com ([209.85.160.175]:33501 "EHLO mail-yk0-f175.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756432AbbJIACG (ORCPT ); Thu, 8 Oct 2015 20:02:06 -0400 Received: by ykft14 with SMTP id t14so64663378ykf.0; Thu, 08 Oct 2015 17:02:04 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=date:from:to:cc:subject:message-id:mime-version:content-type :content-disposition:user-agent; bh=/sZnN665ge3HscCnO72356iyf2flysvMOvu0nr9hH7Y=; b=g5MliAHEvnQhnY8b5vN4n+l7hFVQPqMTiBljsNCXkuO01csGBTK/cOYbe4dB1I7EKK Ag1T6VHnABR0GyonFrEyJ4Jz8paGWgXd97nYqhjem9eaF4Z4BepwL2xLwX0DmEvH6M+j cS3AK7xTyRdIVdE6RBq85O/wZ2eNNNQ569k24/SLJlaRVDXWgmUn5ZZWk64u1X7wdt2M JiNbNy0NiXQTY2YBUZDx5NGcQ+x4K7fW1nug2vhVSEwHszXYAagNZjD4BpIGXAQFkoOl wE1zM8PfeZgUxtxW7yy6sr/e6Xi0l653fOtDePIaAwOXfaW2cmsrltyAPutbaxXlmQoX 3a/A== X-Received: by 10.129.35.6 with SMTP id j6mr7281551ywj.8.1444348924218; Thu, 08 Oct 2015 17:02:04 -0700 (PDT) Received: from sophia (71-47-58-73.res.bhn.net. [71.47.58.73]) by smtp.gmail.com with ESMTPSA id i23sm32307865ywe.38.2015.10.08.17.02.02 (version=TLSv1.2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Thu, 08 Oct 2015 17:02:03 -0700 (PDT) Date: Thu, 8 Oct 2015 20:01:55 -0400 From: William Breathitt Gray To: linus.walleij@linaro.org, gnurou@gmail.com Cc: linux-kernel@vger.kernel.org, linux-gpio@vger.kernel.org Subject: [PATCH v8] gpio: Add GPIO support for the ACCES 104-IDIO-16 Message-ID: <20151009000155.GA1977@sophia> MIME-Version: 1.0 Content-Disposition: inline User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-gpio-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-gpio@vger.kernel.org The ACCES 104-IDIO-16 family of PC/104 utility boards feature 16 optically isolated inputs and 16 optically isolated FET solid state outputs. This driver provides GPIO support for these 32 channels of digital I/O. Change-of-State detection interrupts are not supported. GPIO 0-15 correspond to digital outputs 0-15, while GPIO 16-31 correspond to digital inputs 0-15. The base port address for the device may be set via the a_104_idio_16_base module parameter. Signed-off-by: William Breathitt Gray --- Changes in v8: - Remove value inversion before output since gpio driver should leave active-low configuration to the user - Use platform driver name for setting platform device name since they should match - Move gpio chip data population to the more fitting probe function drivers/gpio/Kconfig | 10 ++ drivers/gpio/Makefile | 1 + drivers/gpio/gpio-104-idio-16.c | 219 ++++++++++++++++++++++++++++++++++++++++ 3 files changed, 230 insertions(+) create mode 100644 drivers/gpio/gpio-104-idio-16.c diff --git a/drivers/gpio/Kconfig b/drivers/gpio/Kconfig index 8949b3f..dc843e0 100644 --- a/drivers/gpio/Kconfig +++ b/drivers/gpio/Kconfig @@ -684,6 +684,16 @@ config GPIO_SX150X endmenu +menu "ISA GPIO drivers" + +config GPIO_104_IDIO_16 + tristate "ACCES 104-IDIO-16 GPIO support" + depends on X86 + help + Enables GPIO support for the ACCES 104-IDIO-16 family. + +endmenu + menu "MFD GPIO expanders" config GPIO_ADP5520 diff --git a/drivers/gpio/Makefile b/drivers/gpio/Makefile index f79a7c4..6f2fea5 100644 --- a/drivers/gpio/Makefile +++ b/drivers/gpio/Makefile @@ -12,6 +12,7 @@ obj-$(CONFIG_GPIO_ACPI) += gpiolib-acpi.o # Device drivers. Generally keep list sorted alphabetically obj-$(CONFIG_GPIO_GENERIC) += gpio-generic.o +obj-$(CONFIG_GPIO_104_IDIO_16) += gpio-104-idio-16.o obj-$(CONFIG_GPIO_74X164) += gpio-74x164.o obj-$(CONFIG_GPIO_74XX_MMIO) += gpio-74xx-mmio.o obj-$(CONFIG_GPIO_ADNP) += gpio-adnp.o diff --git a/drivers/gpio/gpio-104-idio-16.c b/drivers/gpio/gpio-104-idio-16.c new file mode 100644 index 0000000..f21432a --- /dev/null +++ b/drivers/gpio/gpio-104-idio-16.c @@ -0,0 +1,219 @@ +/* + * GPIO driver for the ACCES 104-IDIO-16 family + * Copyright (C) 2015 William Breathitt Gray + * + * This program is free software; you can redistribute it and/or modify + * it under the terms of the GNU General Public License, version 2, as + * published by the Free Software Foundation. + * + * This program is distributed in the hope that it will be useful, but + * WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU + * General Public License for more details. + */ +#include +#include +#include +#include +#include +#include +#include +#include +#include +#include +#include + +static unsigned a_104_idio_16_base; +module_param(a_104_idio_16_base, uint, 0); +MODULE_PARM_DESC(a_104_idio_16_base, "ACCES 104-IDIO-16 base address"); + +/** + * struct a_104_idio_16_gpio - GPIO device private data structure + * @chip: instance of the gpio_chip + * @lock: synchronization lock to prevent gpio_set race conditions + * @base: base port address of the GPIO device + * @extent: extent of port address region of the GPIO device + * @out_state: output bits state + */ +struct a_104_idio_16_gpio { + struct gpio_chip chip; + spinlock_t lock; + unsigned base; + unsigned extent; + unsigned out_state; +}; + +static int a_104_idio_16_gpio_get_direction(struct gpio_chip *chip, + unsigned offset) +{ + if (offset > 15) + return 1; + + return 0; +} + +static int a_104_idio_16_gpio_direction_input(struct gpio_chip *chip, + unsigned offset) +{ + return 0; +} + +static int a_104_idio_16_gpio_direction_output(struct gpio_chip *chip, + unsigned offset, int value) +{ + chip->set(chip, offset, value); + return 0; +} + +static struct a_104_idio_16_gpio *to_a104idio16gp(struct gpio_chip *gc) +{ + return container_of(gc, struct a_104_idio_16_gpio, chip); +} + +static int a_104_idio_16_gpio_get(struct gpio_chip *chip, unsigned offset) +{ + struct a_104_idio_16_gpio *const a104idio16gp = to_a104idio16gp(chip); + const unsigned BIT_MASK = 1U << (offset-16); + + if (offset < 16) + return -EINVAL; + + if (offset < 24) + return !!(inb(a104idio16gp->base + 1) & BIT_MASK); + + return !!(inb(a104idio16gp->base + 5) & (BIT_MASK>>8)); +} + +static void a_104_idio_16_gpio_set(struct gpio_chip *chip, unsigned offset, + int value) +{ + struct a_104_idio_16_gpio *const a104idio16gp = to_a104idio16gp(chip); + const unsigned BIT_MASK = 1U << offset; + unsigned long flags; + + if (offset > 15) + return; + + spin_lock_irqsave(&a104idio16gp->lock, flags); + + if (value) + a104idio16gp->out_state |= BIT_MASK; + else + a104idio16gp->out_state &= ~BIT_MASK; + + if (offset > 7) + outb(a104idio16gp->out_state >> 8, a104idio16gp->base + 4); + else + outb(a104idio16gp->out_state, a104idio16gp->base); + + spin_unlock_irqrestore(&a104idio16gp->lock, flags); +} + +static int __init a_104_idio_16_probe(struct platform_device *pdev) +{ + struct a_104_idio_16_gpio *const a104idio16gp = pdev->dev.platform_data; + struct device *dev = &pdev->dev; + int err; + + const unsigned BASE = a_104_idio_16_base; + const unsigned EXTENT = 8; + const char *const NAME = dev_name(dev); + + if (!request_region(BASE, EXTENT, NAME)) { + dev_err(dev, "Unable to lock %s port addresses (0x%X-0x%X)\n", + NAME, BASE, BASE + EXTENT); + err = -EBUSY; + goto out_lock_a_104_idio_16_port; + } + + a104idio16gp->chip.label = NAME; + a104idio16gp->chip.dev = dev; + a104idio16gp->chip.owner = THIS_MODULE; + a104idio16gp->chip.base = -1; + a104idio16gp->chip.ngpio = 32; + a104idio16gp->chip.get_direction = a_104_idio_16_gpio_get_direction; + a104idio16gp->chip.direction_input = a_104_idio_16_gpio_direction_input; + a104idio16gp->chip.direction_output = a_104_idio_16_gpio_direction_output; + a104idio16gp->chip.get = a_104_idio_16_gpio_get; + a104idio16gp->chip.set = a_104_idio_16_gpio_set; + a104idio16gp->base = BASE; + a104idio16gp->extent = EXTENT; + a104idio16gp->out_state = 0xFFFF; + + spin_lock_init(&a104idio16gp->lock); + + dev_info(dev, "GPIO detected\n"); + + err = gpiochip_add(&a104idio16gp->chip); + if (err) { + dev_err(dev, "GPIO registering failed (%d)\n", err); + goto out_gpio_register; + } + + return 0; + +out_gpio_register: + release_region(BASE, EXTENT); +out_lock_a_104_idio_16_port: + return err; +} + +static int a_104_idio_16_remove(struct platform_device *pdev) +{ + struct a_104_idio_16_gpio *const a104idio16gp = pdev->dev.platform_data; + + gpiochip_remove(&a104idio16gp->chip); + release_region(a104idio16gp->base, a104idio16gp->extent); + + return 0; +} + +static struct platform_device *a_104_idio_16_device; + +static struct platform_driver a_104_idio_16_driver = { + .driver = { + .name = "104-idio-16", + }, + .remove = a_104_idio_16_remove +}; + +static void __exit a_104_idio_16_exit(void) +{ + dev_info(&a_104_idio_16_device->dev, "Exiting module\n"); + + platform_device_unregister(a_104_idio_16_device); + platform_driver_unregister(&a_104_idio_16_driver); +} + +static int __init a_104_idio_16_init(void) +{ + int err; + struct a_104_idio_16_gpio gp; + + a_104_idio_16_device = platform_device_register_data(NULL, + a_104_idio_16_driver.driver.name, -1, &gp, sizeof(gp)); + if (IS_ERR(a_104_idio_16_device)) { + err = PTR_ERR(a_104_idio_16_device); + goto out_platform_device; + } + + dev_info(&a_104_idio_16_device->dev, "Initializing module\n"); + + err = platform_driver_probe(&a_104_idio_16_driver, a_104_idio_16_probe); + if (err) + goto out_platform_driver; + + return 0; + +out_platform_driver: + platform_device_unregister(a_104_idio_16_device); +out_platform_device: + return err; +} + +module_init(a_104_idio_16_init); +module_exit(a_104_idio_16_exit); + +MODULE_AUTHOR("William Breathitt Gray "); +MODULE_DESCRIPTION("ACCES 104-IDIO-16 GPIO driver"); +MODULE_LICENSE("GPL");