mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Mathias Nyman <mathias.nyman@linux.intel.com>
To: Linus <linus.walleij@linaro.org>
Cc: <grant.likely@secretlab.ca>, <linux-kernel@vger.kernel.org>,
	Mathias Nyman <mathias.nyman@linux.intel.com>
Subject: [PATCH v2 0/1] gpio driver for Intel Baytrail platforms
Date: Thu, 13 Jun 2013 18:06:22 +0300	[thread overview]
Message-ID: <1371135983-12592-1-git-send-email-mathias.nyman@linux.intel.com> (raw)

This is the second version of the Intel baytrail GPIO driver.

After looking at the pinctrl subsystem that Linus W. suggested I think 
pinctrl suits platforms that don't have firmware configuring the pins
before the operating system is started, or when pins need to be configured
on the fly.

I'd still keep this driver under GPIO. Adding it to pinctrl
feels like adding more complexity without any bigger use for the features.

We expect BIOS to set all pin configurations correctly. 
The comments about pin muxing capabilities are removed from the driver.
The same firmware is anyway listing gpio resources in ACPI tables, so pin 
configurations should be correct. (The previous indication in the driver
about the need to configure pins was mostly because we're working with early 
develpment stage firmwares which still have small hickups)  

Changes since v1:
- generic cleanups suggested by Andy S.
- removed text about pin muxing
- added missing 44:th pin to SUS controller
- added level triggering option
- prevent "forever loop" in case pin is stuck in "interrupt triggered" status,     

Mathias Nyman (1):
  gpio: add Intel BayTrail gpio driver

 drivers/gpio/Kconfig         |   12 +
 drivers/gpio/Makefile        |    1 +
 drivers/gpio/gpio-baytrail.c |  547 ++++++++++++++++++++++++++++++++++++++++++
 3 files changed, 560 insertions(+), 0 deletions(-)
 create mode 100644 drivers/gpio/gpio-baytrail.c

-- 
1.7.4.1


             reply	other threads:[~2013-06-13 15:02 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-06-13 15:06 Mathias Nyman [this message]
2013-06-13 15:06 ` [PATCH v2 1/1] gpio: add Intel BayTrail gpio driver Mathias Nyman
2013-06-13 15:45 ` [PATCH v2 0/1] gpio driver for Intel Baytrail platforms Linus Walleij
2013-06-14 12:14   ` Mathias Nyman
2013-06-15 20:47     ` Linus Walleij

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1371135983-12592-1-git-send-email-mathias.nyman@linux.intel.com \
    --to=mathias.nyman@linux.intel.com \
    --cc=grant.likely@secretlab.ca \
    --cc=linus.walleij@linaro.org \
    --cc=linux-kernel@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

Powered by JetHome