From: Thomas Gleixner <tglx@linutronix.de>
To: Florian Fainelli <florian.fainelli@telecomint.eu>
Cc: Andres Salomon <dilinger@queued.net>, linux-kernel@vger.kernel.org
Subject: Re: [PATCH 1/4 resend] [x86] Add generic GPIO support to x86
Date: Thu, 25 Oct 2007 17:24:54 +0200 (CEST) [thread overview]
Message-ID: <alpine.LFD.0.9999.0710251720420.3186@localhost.localdomain> (raw)
In-Reply-To: <200710251430.03202.florian.fainelli@telecomint.eu>
On Thu, 25 Oct 2007, Florian Fainelli wrote:
> > It might be useful for everyone to check, whether the existing GPIO
> > functionality can be extended, reworked to match the needs of Geode as
> > well. Extending / reworking an existing interface is definitely
> > better than adding a new incompatible one.
>
> Of course. Just because Geode is the only x86 potential user of such an API
> does not mean it should impose its design. Remember that this patch is part
> of a serie to add support for the RDC R-321x SoC, which becomes another
> x86-family user.
>
> Looking at what Geode can do, and what RDC can do, I guess there are only few
> functions that are generic, all the drain, multiplexing, direction things are
> only available on the Geode board.
It does not matter whether Geode has 500 functions and RDC has 5. It
matters that we do _not_ want two different APIs which deal with the
same thing, where one can obviously be a subset of the other. So
either we can extend / rework the stuff that both parties are happy or
we are not going to merge either of them.
tglx
next prev parent reply other threads:[~2007-10-25 15:25 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-10-18 13:51 [PATCH 1/4 resend] [x86] Add generic GPIO support to x86 Florian Fainelli
2007-10-18 21:38 ` Andres Salomon
2007-10-19 12:01 ` Florian Fainelli
2007-10-19 21:32 ` Jordan Crouse
2007-10-21 16:06 ` Haavard Skinnemoen
2007-10-22 19:18 ` Andres Salomon
2007-10-22 19:59 ` Florian Fainelli
2007-10-25 8:18 ` Thomas Gleixner
2007-10-25 12:30 ` Florian Fainelli
2007-10-25 15:24 ` Thomas Gleixner [this message]
2008-02-13 22:02 ` Andrew Morton
2008-02-13 23:25 ` David Brownell
2008-02-14 1:55 ` David Brownell
2008-02-22 23:56 ` Anton Vorontsov
2008-02-23 0:51 ` David Brownell
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=alpine.LFD.0.9999.0710251720420.3186@localhost.localdomain \
--to=tglx@linutronix.de \
--cc=dilinger@queued.net \
--cc=florian.fainelli@telecomint.eu \
--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 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.