From: Greg Ungerer <gerg@snapgear.com>
To: "Uwe Kleine-König" <u.kleine-koenig@pengutronix.de>
Cc: Artem Bityutskiy <artem.bityutskiy@linux.intel.com>,
linux-mtd@lists.infradead.org, kernel@pengutronix.de,
Greg Ungerer <gerg@uclinux.org>
Subject: Re: [PATCH] [RFC] mtd/uclinux: support ROM and allow passing the base address
Date: Tue, 9 Oct 2012 20:37:09 +1000 [thread overview]
Message-ID: <5073FE55.2000402@snapgear.com> (raw)
In-Reply-To: <20121009084418.GH18531@pengutronix.de>
Hi Uwe,
On 10/09/2012 06:44 PM, Uwe Kleine-König wrote:
> On Tue, Oct 09, 2012 at 01:29:46PM +1000, Greg Ungerer wrote:
>> On 09/10/12 01:25, Uwe Kleine-K÷nig wrote:
>>> This allows to put the filesystem at a defined address in ROM allowing
>>> to save more precious RAM.
>>>
>>> I think it's save to default to ROM because the intention of using the
>>> uclinux map is to use a romfs and so mtd-ram doesn't give you anything
>>> that mtd-rom doesn't.
>>>
>>> Signed-off-by: Uwe Kleine-K├Ànig <u.kleine-koenig@pengutronix.de>
>>
>> Looks good to me, so:
>>
>> Acked-by: Greg Ungerer <gerg@uclinux.org>
>>
>> Couple of things to be aware of. Artem currently has a couple of
>> patches against this file (looks like at least 1 of them is currently
>> in linux-next). It will probably conflict with your changes, should
>> be easy enough to fix up.
> If you tell me a branch to test, I can do so.
Sure. All three patches I sent to the linux-mtd list are here:
The following changes since commit 979570e02981d4a8fc20b3cc8fd651856c98ee9d:
Linus Torvalds (1):
Linux 3.6-rc7
are available in the git repository at:
git://git.kernel.org/pub/scm/linux/kernel/git/gerg/m68knommu.git cfmmu
Greg Ungerer (3):
mtd: fix wrong usage of ioremap_nocache() in uclinux.c map driver
mtd: allow uclinux map driver to be used on any ColdFire CPU platform
mtd: clean up uclinux.c map driver
drivers/mtd/maps/Kconfig | 2 +-
drivers/mtd/maps/uclinux.c | 42
+++++++++++++++++++-----------------------
2 files changed, 20 insertions(+), 24 deletions(-)
As far as I can see Artem has pushed the first 2 into for linux-next,
but not the third. I am hoping he hops in here and lets us know what
his intentions are with the third...
Regards
Greg
------------------------------------------------------------------------
Greg Ungerer -- Principal Engineer EMAIL: gerg@snapgear.com
SnapGear Group, McAfee PHONE: +61 7 3435 2888
8 Gardner Close, FAX: +61 7 3891 3630
Milton, QLD, 4064, Australia WEB: http://www.SnapGear.com
next prev parent reply other threads:[~2012-10-09 10:47 UTC|newest]
Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-10-08 15:25 [PATCH] [RFC] mtd/uclinux: support ROM and allow passing the base address Uwe Kleine-König
2012-10-09 3:29 ` Greg Ungerer
2012-10-09 8:44 ` Uwe Kleine-König
2012-10-09 10:37 ` Greg Ungerer [this message]
2012-10-10 7:58 ` Uwe Kleine-König
2012-10-12 4:46 ` Mike Frysinger
2012-10-12 5:57 ` Greg Ungerer
2012-10-12 7:22 ` Uwe Kleine-König
2012-10-12 16:09 ` Mike Frysinger
2012-10-12 16:22 ` Mike Frysinger
2012-10-15 14:00 ` Artem Bityutskiy
2012-10-11 4:21 ` Mike Frysinger
2012-10-11 6:13 ` Uwe Kleine-König
2012-10-12 7:41 ` Uwe Kleine-König
2012-10-12 16:18 ` Mike Frysinger
2012-10-12 7:41 ` [PATCH 1/2 RFC v2] " Uwe Kleine-König
2012-10-12 7:41 ` [PATCH 2/2] mtd/uclinux: add a comment about why uclinux_ram_map must not be static Uwe Kleine-König
2012-10-12 16:23 ` Mike Frysinger
2012-10-15 4:15 ` Greg Ungerer
2012-10-15 4:14 ` [PATCH 1/2 RFC v2] mtd/uclinux: support ROM and allow passing the base address Greg Ungerer
2012-10-15 6:58 ` Uwe Kleine-König
2012-10-16 1:13 ` Greg Ungerer
2012-10-16 6:56 ` Uwe Kleine-König
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=5073FE55.2000402@snapgear.com \
--to=gerg@snapgear.com \
--cc=artem.bityutskiy@linux.intel.com \
--cc=gerg@uclinux.org \
--cc=kernel@pengutronix.de \
--cc=linux-mtd@lists.infradead.org \
--cc=u.kleine-koenig@pengutronix.de \
/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.