LKML Archive on lore.kernel.org
help / color / mirror / Atom feed
From: David Woodhouse <dwmw2@infradead.org>
To: Russell King - ARM Linux <linux@arm.linux.org.uk>
Cc: Sam Ravnborg <sam@ravnborg.org>,
Marcin Juszkiewicz <openembedded@haerwu.biz>,
Christoph Hellwig <hch@infradead.org>, Greg KH <greg@kroah.com>,
linux-arm-kernel@lists.arm.linux.org.uk,
linux-kernel@vger.kernel.org
Subject: Re: [PATCH] arch/arm/Kconfig: Make UIO available on ARM architecture
Date: Fri, 08 Feb 2008 10:18:31 +0000 [thread overview]
Message-ID: <1202465911.5469.240.camel@pmac.infradead.org> (raw)
In-Reply-To: <20080208094538.GC21852@flint.arm.linux.org.uk>
On Fri, 2008-02-08 at 09:45 +0000, Russell King - ARM Linux wrote:
> If we are serious about allowing ARM to use drivers/Kconfig, then let's
> not get distracted by perfection - by trying to do too many changes in
> one go.
>
> If, today, we conditionalise MTD or IDE on a certain set of symbols,
> then those conditions should be preserved in the first step - it should
> be a 1:1 translation.
That makes some sense. Currently you have:
if ALIGNMENT_TRAP || !CPU_CP15_MMU
source "drivers/mtd/Kconfig"
endif
> Later, if there's a need to improve it (as you're suggesting) that should
> be a *separate* change.
We can certainly improve the behaviour later, it's true -- it's not the
end of the world if we continue to have the whole of the MTD layer
turned off on platforms with alignment problems for now.
But still, it's HAVE_UNALIGNED_ACCESS which we want to depend on, not a
newly-invented HAVE_MTD. And there are other places we really ought to
be depending on HAVE_UNALIGNED_ACCESS too.
--
dwmw2
next prev parent reply other threads:[~2008-02-08 10:18 UTC|newest]
Thread overview: 30+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-02-07 12:38 Hans-Jürgen Koch
2008-02-07 12:58 ` Christoph Hellwig
2008-02-07 13:01 ` Sam Ravnborg
2008-02-07 15:09 ` Marcin Juszkiewicz
2008-02-07 16:05 ` Russell King - ARM Linux
2008-02-07 20:23 ` Sam Ravnborg
2008-02-07 20:29 ` Russell King - ARM Linux
2008-02-07 21:05 ` Sam Ravnborg
2008-02-08 9:04 ` David Woodhouse
2008-02-08 9:45 ` Russell King - ARM Linux
2008-02-08 10:18 ` David Woodhouse [this message]
2008-02-08 10:23 ` Russell King - ARM Linux
2008-02-08 10:43 ` David Woodhouse
2008-02-08 10:45 ` Russell King - ARM Linux
2008-02-12 1:51 ` Greg Ungerer
2008-02-08 10:58 ` David Woodhouse
2008-02-08 18:46 ` Sam Ravnborg
2008-02-08 18:52 ` David Woodhouse
2008-02-09 11:39 ` Sam Ravnborg
2008-02-07 14:21 ` Russell King - ARM Linux
2008-02-07 13:02 ` Russell King - ARM Linux
2008-02-07 13:12 ` Hans-Jürgen Koch
2008-02-11 8:25 ` Hans-Jürgen Koch
2008-02-11 14:39 ` Sam Ravnborg
2008-04-10 12:21 ` Uwe Kleine-König
2008-04-10 12:27 ` Russell King - ARM Linux
2008-04-10 12:57 ` Uwe Kleine-König
2008-04-10 13:10 ` Russell King - ARM Linux
2008-04-10 13:50 ` Uwe Kleine-König
2008-04-10 18:58 ` Sam Ravnborg
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=1202465911.5469.240.camel@pmac.infradead.org \
--to=dwmw2@infradead.org \
--cc=greg@kroah.com \
--cc=hch@infradead.org \
--cc=linux-arm-kernel@lists.arm.linux.org.uk \
--cc=linux-kernel@vger.kernel.org \
--cc=linux@arm.linux.org.uk \
--cc=openembedded@haerwu.biz \
--cc=sam@ravnborg.org \
--subject='Re: [PATCH] arch/arm/Kconfig: Make UIO available on ARM architecture' \
/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
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).