All of lore.kernel.org
 help / color / mirror / Atom feed
From: David Brownell <david-b@pacbell.net>
To: Denis Turischev <denis@compulab.co.il>
Cc: Samuel Ortiz <sameo@linux.intel.com>,
	LKML <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH v3 3/3] gpio: add Intel SCH GPIO controller driver
Date: Sun, 21 Feb 2010 11:12:48 -0800	[thread overview]
Message-ID: <201002211112.48527.david-b@pacbell.net> (raw)
In-Reply-To: <4B812C21.8000602@compulab.co.il>

On Sunday 21 February 2010, Denis Turischev wrote:
> v2: there is no acpi_check_region, it will be implemented in mfd-core
> v3: patch refreshed against the latest Linus tree

Could such call really address the GPIO conflict issue I mentioned?

The AML bytecodes I looked at were writing directly to Southbridge
GPIO registers (or reading them), or relying on ACPI to mediate the
GPIO interrupts.  ISTR that button drivers, and code to switch into
or out of low power states, were good sources of such bad examples.

Calls like that should clearly be able to handle cases where ACPI
has a "Real" Driver (tm) ... e.g. for SMBus hardware.

I'm not sure what a good solution for this would be, short of just
not using ACPI ... which may not be practical, given the limited
degree of x86 board/system support for Linux.

I mention this mostly because when I looked at the issue in the
context of an ICHx GPIO driver, I didn't see a good solution to
the problem then ... and nothing seems to have changed meanwhile.

- Dave

  reply	other threads:[~2010-02-21 19:12 UTC|newest]

Thread overview: 73+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-02-11 10:24 [PATCH 0/3] Add Intel SCH GPIO driver Denis Turischev
2010-02-11 10:26 ` [PATCH 1/3] MFD: introduce lpc_sch for Intel SCH LPC bridge Denis Turischev
2010-02-11 10:26   ` Denis Turischev
2010-02-16 10:08   ` Samuel Ortiz
2010-02-16 10:08     ` Samuel Ortiz
2010-02-16 13:59     ` Pan, Jacob jun
2010-02-16 13:59       ` Pan, Jacob jun
2010-02-16 15:45       ` Jean Delvare
2010-02-16 17:19         ` Pan, Jacob jun
2010-02-16 17:19           ` Pan, Jacob jun
2010-02-16 17:29           ` Jean Delvare
2010-02-16 17:29             ` Jean Delvare
2010-02-16 19:57   ` David Brownell
2010-02-16 19:57     ` David Brownell
2010-02-16 21:49     ` Jean Delvare
2010-02-16 21:49       ` Jean Delvare
2010-02-17 10:03       ` Denis Turischev
2010-02-17 10:03         ` Denis Turischev
2010-02-17 10:44         ` Jean Delvare
2010-02-17 10:44           ` Jean Delvare
2010-02-17 12:35           ` Mike Rapoport
2010-02-17 12:35             ` Mike Rapoport
2010-02-17 14:37             ` Samuel Ortiz
2010-02-17 14:37               ` Samuel Ortiz
2010-02-18 17:42   ` [PATCH v2 " Denis Turischev
2010-02-18 17:42     ` Denis Turischev
2010-02-18 17:45     ` Randy Dunlap
2010-02-18 17:45       ` Randy Dunlap
2010-02-18 18:01       ` [PATCH v3 " Denis Turischev
2010-02-18 18:01         ` Denis Turischev
2010-02-18 18:08         ` Randy Dunlap
2010-02-18 18:08           ` Randy Dunlap
2010-02-19 10:30         ` Samuel Ortiz
2010-02-19 10:30           ` Samuel Ortiz
2010-02-23  8:26         ` Jean Delvare
2010-02-23  8:26           ` Jean Delvare
2010-02-23  9:25           ` Denis Turischev
2010-02-23  9:25             ` Denis Turischev
2010-02-23  9:41             ` Samuel Ortiz
2010-02-23  9:41               ` Samuel Ortiz
2010-02-11 10:28 ` [PATCH 2/3] i2c: convert i2c-isch to platform_device Denis Turischev
2010-02-11 10:28   ` Denis Turischev
2010-02-17 15:42   ` [PATCH v2 " Denis Turischev
2010-02-19 10:33     ` Samuel Ortiz
2010-02-19 10:33       ` Samuel Ortiz
2010-02-21 12:46     ` [PATCH v3 " Denis Turischev
2010-02-21 12:46       ` Denis Turischev
2010-02-23  7:00       ` Mike Rapoport
2010-02-23  7:00         ` Mike Rapoport
2010-02-23  8:12         ` Jean Delvare
2010-02-23  8:12           ` Jean Delvare
2010-02-23  8:20           ` Samuel Ortiz
2010-02-23  8:20             ` Samuel Ortiz
2010-02-23  8:24             ` Jean Delvare
2010-02-23  8:24               ` Jean Delvare
2010-02-28 19:00       ` Samuel Ortiz
2010-03-01 16:59         ` [PATCH v4 " Denis Turischev
2010-03-01 16:59           ` Denis Turischev
2010-03-02 10:06           ` Samuel Ortiz
2010-03-02 10:06             ` Samuel Ortiz
2010-02-11 10:28 ` [PATCH 3/3] gpio: add Intel SCH GPIO controller driver Denis Turischev
2010-02-11 10:28   ` Denis Turischev
2010-02-17 15:39   ` [PATCH v2 " Denis Turischev
2010-02-21 12:50     ` [PATCH v3 " Denis Turischev
2010-02-21 19:12       ` David Brownell [this message]
2010-02-24  6:49         ` Mike Rapoport
2010-03-01 18:09           ` Len Brown
2010-03-01 19:50             ` Alan Cox
2010-03-02  6:42             ` Mike Rapoport
2010-03-02  9:29               ` Denis Turischev
2010-02-28 19:01       ` Samuel Ortiz
2010-03-01 17:02         ` [PATCH v4 " Denis Turischev
2010-03-02 10:09           ` Samuel Ortiz

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=201002211112.48527.david-b@pacbell.net \
    --to=david-b@pacbell.net \
    --cc=denis@compulab.co.il \
    --cc=linux-kernel@vger.kernel.org \
    --cc=sameo@linux.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.