linux-gpio.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
To: David Cohen <david.a.cohen@intel.com>
Cc: Alexandre Courbot <acourbot@nvidia.com>,
	linux-gpio@vger.kernel.org,
	Linus Walleij <linus.walleij@linaro.org>,
	Mika Westerberg <mika.westerberg@linux.intel.com>,
	Sathyanarayanan Kuppuswamy <sathyanarayanan.kuppuswamy@intel.com>,
	Len Brown <lenb@kernel.org>
Subject: Re: [PATCH v1 0/3]
Date: Mon, 09 Dec 2013 11:19:57 +0200	[thread overview]
Message-ID: <1386580797.1871.105.camel@smile> (raw)
In-Reply-To: <52A10233.8030102@intel.com>

On Thu, 2013-12-05 at 14:46 -0800, David Cohen wrote:
> Hi Andy,
> 
> On 12/05/2013 08:36 AM, Andy Shevchenko wrote:
> > Here is a third [1] version of the SFI GPIO helpers. This series contains SFI
> 
> Your patch set subject says it's the first version :)

Will fix in next iteration.

> 
> > GPIO helpers along with the update of GPIO library to lookup for SFI resources.
> >
> > It has been tested on Medfield device on top of v3.13-rc2 + Alexandre's GPIO
> > descriptor API update [2], Mika's ACPI gpio patches [3] and my GPIO library
> > clean up [4].
> 
> Good test case. I'm able to extend it to Merrifield.

Be informed, that no driver is modified by this patch series. To test
you have to update one or more drivers. I've tested against gpio_keys,
but I'm going to make that solution upstreamable (it requires to update
gpio_keys driver itself first).

> 
> Br, David
> 
> >
> > [1] https://lkml.org/lkml/2013/6/5/316
> > [2] http://www.spinics.net/lists/kernel/msg1645907.html
> > [3] http://www.spinics.net/lists/linux-acpi/msg47572.html
> > [4] https://www.mail-archive.com/linux-gpio@vger.kernel.org/msg01515.html
> >
> > Andy Shevchenko (3):
> >   SFI: fix compiler warnings
> >   SFI: store GPIO table and export lookup function
> >   gpiolib: append SFI helpers for GPIO API
> >
> >  drivers/gpio/Kconfig       |   4 ++
> >  drivers/gpio/Makefile      |   1 +
> >  drivers/gpio/gpiolib-sfi.c |  28 +++++++++++
> >  drivers/gpio/gpiolib.c     |   3 ++
> >  drivers/gpio/gpiolib.h     |  13 +++++
> >  drivers/sfi/Makefile       |   2 +-
> >  drivers/sfi/sfi_acpi.c     |   9 ++--
> >  drivers/sfi/sfi_core.c     |  10 +++-
> >  drivers/sfi/sfi_core.h     |   3 ++
> >  drivers/sfi/sfi_gpio.c     | 123 +++++++++++++++++++++++++++++++++++++++++++++
> >  include/linux/sfi.h        |   8 +++
> >  11 files changed, 197 insertions(+), 7 deletions(-)
> >  create mode 100644 drivers/gpio/gpiolib-sfi.c
> >  create mode 100644 drivers/sfi/sfi_gpio.c
> >
> 

-- 
Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Intel Finland Oy


      reply	other threads:[~2013-12-09  9:21 UTC|newest]

Thread overview: 23+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-12-05 16:36 [PATCH v1 0/3] Andy Shevchenko
2013-12-05 16:36 ` [PATCH v1 1/3] SFI: fix compiler warnings Andy Shevchenko
2013-12-05 22:48   ` David Cohen
2013-12-09  8:52     ` Andy Shevchenko
2013-12-05 16:36 ` [PATCH v1 2/3] SFI: store GPIO table and export lookup function Andy Shevchenko
2013-12-05 23:07   ` David Cohen
2013-12-09  9:59     ` Andy Shevchenko
2013-12-06  1:51   ` Alex Courbot
2013-12-09  9:23     ` Andy Shevchenko
2013-12-05 16:36 ` [PATCH v1 3/3] gpiolib: append SFI helpers for GPIO API Andy Shevchenko
2013-12-05 23:20   ` David Cohen
2013-12-09 10:13     ` Andy Shevchenko
2013-12-06  1:52   ` Alex Courbot
2013-12-09 10:11     ` Andy Shevchenko
2013-12-10  3:00       ` Alex Courbot
2013-12-10 13:15         ` Andy Shevchenko
2013-12-10 15:21           ` David Cohen
2013-12-10 15:31             ` Andy Shevchenko
2013-12-11  2:47           ` Alex Courbot
2013-12-12  0:45             ` David Cohen
2013-12-12  1:46               ` Alex Courbot
2013-12-05 22:46 ` [PATCH v1 0/3] David Cohen
2013-12-09  9:19   ` Andy Shevchenko [this message]

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=1386580797.1871.105.camel@smile \
    --to=andriy.shevchenko@linux.intel.com \
    --cc=acourbot@nvidia.com \
    --cc=david.a.cohen@intel.com \
    --cc=lenb@kernel.org \
    --cc=linus.walleij@linaro.org \
    --cc=linux-gpio@vger.kernel.org \
    --cc=mika.westerberg@linux.intel.com \
    --cc=sathyanarayanan.kuppuswamy@intel.com \
    /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;
as well as URLs for NNTP newsgroup(s).