[PATCH 1/2] mtd: nand: renumber conflicting BBT flags
Brian Norris
computersforpeace at gmail.com
Sat Apr 2 04:04:05 EDT 2011
Hello,
On 3/31/2011 5:58 AM, Artem Bityutskiy wrote:
> Hmm, it seems that the issue is that flags which belong to the same
> "space" should be in a single file. AFAICS, we have 2 spaces:
>
> 1. Chip flags
> 2. BBT flags
>
> They are 2 different things. But some of the flags are shared. And this
> is quite subtle thing.
>
> What I think we should do instead is to avoid sharing the same symbolic
> constant between 2 different spaces. Is this possible?
I'm not quite sure. Many of the "shared" options go into the
nand_chip.options field only so that they can later be copied to a
nand_bbt_descr.options field. I think this is only out of convenience so
that we can detect chip-based BBT options like 'scan 2nd page' before we
have actually allocated and assigned our bbt descriptors. For these
flags, we can use a new field in nand_chip, like a
"nand_chip.bbm_options". Then, many shared flags would really become
"early BBT flags" that could safely be copied over without conflict.
Does this make any sense or are there holes in my idea here? I can try
an RFC patch soon if that would help.
Thanks,
Brian
More information about the linux-mtd
mailing list