From: Andrea Galbusera <andrea.galbusera@teamware•it>
To: avolkov@varma-el•com, dwmw2@infradead•org, dedekind@infradead•org
Cc: linux-mtd@lists•infradead.org, linuxppc-embedded@ozlabs•org
Subject: Re: Re: JFFS2 on Lite5200
Date: Tue, 24 Oct 2006 12:15:05 +0200 [thread overview]
Message-ID: <1161684905.4556.56.camel@localhost.localdomain> (raw)
Hi all!
First let me thank you all for your quick and kind help... I was out for
a few days so I take the time to reply only now.
On Tue, 2006-10-17 at 17:20 +0400, Andrey Volkov wrote:
> OMG, again same question,
Sorry Andrey for my too superficial search in the ml archive... Anyway
it seems this issue is somehow still open. ;-)
At moment I'm going with Andrey's patch (it works!) but I'd like to
understand better and maybe to help solving the problem. I see that
patching include/asm-ppc/io.h may not be the right way...
I'm pretty new to kernel hacking/developing so I got confused by your
discussion about the right place to patch things.
So a few questions arise. Do we need to change the physmap driver? Is
this somehow dependent on the target platform. If so can we put some
changes for Lite5200 only?
On Tue, 2006-10-17 at 17:54 +0400, Andrey Volkov wrote:
> There are two modes of operation for JFFS2 scan. Either it can use the
> MTD device's read() function to read from the flash into memory and
> scan
> from there, or -- IF the flash can be accessed as if it were normal
> memory -- it can use the point() method and skip the read() call by
> using a pointer directly into the flash.
>
> In your case, the flash cannot be accessed directly as if it were
> normal
> memory. So your map driver probably shouldn't be allowing point() to
> succeed.
>
> Having said that, I'm not entirely sure we have the code set up so
> that
> 'simple' map drivers _can_ prevent point(). We may need to fix that.
Looking at David's explanation about mapping driver... What does exacly
mean preventing point()/unpoint() to succeed? As a simple MTD _user_
(never digged inside the code) I don't see any hook to these functions
in drivers/mtd/maps/physmap.c so no easy way to set them to NULL.
--
Andrea Galbusera <andrea.galbusera@teamware•it>
reply other threads:[~2006-10-24 10:14 UTC|newest]
Thread overview: [no followups] expand[flat|nested] mbox.gz Atom feed
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=1161684905.4556.56.camel@localhost.localdomain \
--to=andrea.galbusera@teamware$(echo .)it \
--cc=avolkov@varma-el$(echo .)com \
--cc=dedekind@infradead$(echo .)org \
--cc=dwmw2@infradead$(echo .)org \
--cc=linux-mtd@lists$(echo .)infradead.org \
--cc=linuxppc-embedded@ozlabs$(echo .)org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
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