From: "Artem B. Bityutskiy" <dedekind@yandex.ru>
To: Josh Boyer <jwboyer@gmail.com>
Cc: linux-mtd@lists.infradead.org, haver@vnet.ibm.com,
"Jörn Engel" <joern@wohnheim.fh-wedel.de>,
"Marteo Tim" <tim.marteo@gmail.com>
Subject: Re: how to use jff2 on UBI layer?
Date: Mon, 31 Jul 2006 12:09:11 +0400 [thread overview]
Message-ID: <44CDBAA7.8020500@yandex.ru> (raw)
In-Reply-To: <625fc13d0607301228y2bfd2c62q462c508d57a32043@mail.gmail.com>
Josh Boyer wrote:
> It seems somewhat superfluous to add a generalized I/O layer to jffs2.
> It just doesn't sit right with me.
And after all, what do you mean by superfluous? One more function call?
NP - this is a technical question, I can easily turn that into a macro.
With my patch, JFFS2 I/O is clean and modularized. Examples:
You have (jffs2_io)->cm_needed field which tells you if this particular
I/O backend needs clean markers. E.g. - Dataflash and UBI does not need
them. Or do you prefer this setup_data_flash() stuff in wbuf.c???
Also, for eCos, you may put all the differences to io.c and don't spread
this over all sources.
Etc.
You just put all I/O-related stuff like this in one place. And depending
on the I/O backend you initialize it. And all this is in io.c file.
So, it only makes JFFS2 more understandable and cleaner. And it makes
sense even without UBI. Is this "superfluous" ? I think no.
--
Best Regards,
Artem B. Bityutskiy,
St.-Petersburg, Russia.
prev parent reply other threads:[~2006-07-31 8:09 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-07-10 2:50 how to use jff2 on UBI layer? Marteo Tim
2006-07-10 13:01 ` Frank Haverkamp
2006-07-12 2:21 ` Tim Marteo
2006-07-20 2:42 ` Kyungmin Park
2006-07-20 9:45 ` Jörn Engel
2006-07-21 6:42 ` Frank Haverkamp
2006-07-21 7:59 ` Jörn Engel
2006-07-24 10:46 ` Artem B. Bityutskiy
2006-07-24 11:40 ` Jörn Engel
2006-07-26 16:52 ` Artem B. Bityutskiy
2006-07-27 12:54 ` Artem B. Bityutskiy
2006-07-30 19:28 ` Josh Boyer
2006-07-31 7:21 ` Artem B. Bityutskiy
2006-07-31 7:43 ` Artem B. Bityutskiy
2006-07-31 9:29 ` Frank Haverkamp
2006-07-31 13:52 ` Josh Boyer
2006-07-31 8:09 ` Artem B. Bityutskiy [this message]
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=44CDBAA7.8020500@yandex.ru \
--to=dedekind@yandex.ru \
--cc=haver@vnet.ibm.com \
--cc=joern@wohnheim.fh-wedel.de \
--cc=jwboyer@gmail.com \
--cc=linux-mtd@lists.infradead.org \
--cc=tim.marteo@gmail.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.