From: "Artem B. Bityutskiy" <dedekind@yandex.ru>
To: Josh Boyer <jwboyer@gmail.com>
Cc: tglx@linutronix.de, linux-mtd@lists.infradead.org
Subject: Re: [Yaffs] bit error rates --> a vendor speaks
Date: Mon, 27 Feb 2006 20:21:56 +0300 [thread overview]
Message-ID: <44033534.7040708@yandex.ru> (raw)
In-Reply-To: <625fc13d0602270815l5c334b06u1a41a1ed7b7dc4ad@mail.gmail.com>
Josh, I believe you're only *partially* right
Please, switch to "architecture mode", leave this "low-level programmer"
mode. Forget about few bytes of DRAM for now, please! :-)
The bellow are different points which should help me to change your mind.
1. The region is an attribute of *flash chip*. The region is *not* an
attribute of MTD device. MTD device may describe the whole flash chip,
it may describe a part of flash chip (partitions), and it may describe
many flash chips (concatenation).
So, I state: the notion of region has nothing to do with the notion of
MTD device.
If you have a specific application, which works with a *flash chip* not
an *MTD device*, please, introduce another software object, sort of
"struct flash_chip". Let your applications to work with these "struct
flash_chip" objects.
Putting references to regions to the MTD device structure (struct
mtd_info) is *bad*.
2. Well, you're in ecstasy of the current approach (kidding :-) ). Then
explain me, why if, let's say JFFS3, opens an MTD device, it has access
to some weird "regions"? What's on earth is going on? This is definitely
error-prone. This is messy. This is bad design. JFFS3 does not want to
see regions.
3. Regions... How do you look at them?
We could look at them like this:
a). They are different entities, used for completely different purposes,
a rare program needs to use all of them simultaneously, they may have
different eraseblock size and other properties, they are just different.
Yes, it happened to be that they are on one single chip, but they are
still different.
Or we could look at them narrower:
b). A region is a kind of "appendix" to the flash chip, where the
bootcode or whatever may be put. There is not more then one (or really
few) regions per flash chip. There is only one big and main "data"
region, and one or few small auxiliary regions.
I reckon that the reality is more like b). And in this respect I see
your point. Indeed, what for should we spend more DRAM to describe this
unworthy "appendix" (aka region) by a distinct software object? Let's
push this all together!
I can agree with you *partially*, providing that chips like a). are
*not* going to appear. Partially means that I only agree that there *may
be* no reason to create a distinct MTD device for this poor appendix.
So I state: Ok, in b)-like chips, do not create a distinct MTD device.
Let's invent something different. But this must not be in struct
mtd_info anyway.
Do you see my points?
--
Best Regards,
Artem B. Bityutskiy,
St.-Petersburg, Russia.
next prev parent reply other threads:[~2006-02-27 17:22 UTC|newest]
Thread overview: 34+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <43EB96DC.3030900@eptar.com>
[not found] ` <35fb2e590602100558s2d868fa3o1752fbf3217439e4@mail.gmail.com>
[not found] ` <d97046180602151153g23064424x9e1ddf83a1d7ae4f@mail.gmail.com>
2006-02-16 1:32 ` [Yaffs] bit error rates --> a vendor speaks Charles Manning
2006-02-18 9:10 ` Thomas Gleixner
2006-02-18 16:31 ` Vitaly Wool
2006-02-19 8:22 ` Thomas Gleixner
2006-02-20 20:42 ` Charles Manning
2006-02-20 21:37 ` Thomas Gleixner
2006-02-20 22:40 ` Charles Manning
2006-02-20 23:18 ` Thomas Gleixner
2006-02-21 0:29 ` Jon Masters
2006-02-21 8:26 ` Thomas Gleixner
2006-02-21 9:35 ` Jörn Engel
2006-02-21 1:08 ` [Yaffs] bit error rates --> YAFFS for devices with no OOB Charles Manning
2006-02-21 2:12 ` Jon Masters
2006-02-22 0:38 ` Jamie Lokier
2006-02-21 12:14 ` [Yaffs] bit error rates --> a vendor speaks Artem B. Bityutskiy
2006-02-21 13:50 ` Josh Boyer
2006-02-21 14:36 ` Artem B. Bityutskiy
2006-02-21 14:49 ` Artem B. Bityutskiy
2006-02-21 11:59 ` Artem B. Bityutskiy
2006-02-21 12:06 ` Thomas Gleixner
2006-02-25 11:58 ` Artem B. Bityutskiy
2006-02-27 13:27 ` Josh Boyer
2006-02-27 16:01 ` Artem B. Bityutskiy
2006-02-27 16:15 ` Josh Boyer
2006-02-27 17:21 ` Artem B. Bityutskiy [this message]
2006-02-27 17:40 ` Josh Boyer
2006-02-18 18:11 ` Russ Dill
2006-02-19 0:29 ` Charles Manning
2006-02-19 5:08 ` Jon Masters
2006-02-19 8:29 ` Thomas Gleixner
2006-02-23 0:46 ` Russ Dill
2006-02-23 7:36 ` Thomas Gleixner
2006-02-23 8:31 ` Vitaly Wool
2006-02-24 9:51 ` Artem B. Bityutskiy
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=44033534.7040708@yandex.ru \
--to=dedekind@yandex.ru \
--cc=jwboyer@gmail.com \
--cc=linux-mtd@lists.infradead.org \
--cc=tglx@linutronix.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox