[U-Boot] [PATCH] mtdparts: Call nand_init() during mtdparts_init().

Wolfgang Denk wd at denx.de
Sat Oct 16 00:48:11 CEST 2010


Dear Scott Wood,

In message <20101015173533.4e1d8bcb at udp111988uds.am.freescale.net> you wrote:
>
> > This looks like a broken design to me.
> 
> What would you rather see in its place?

Good question. I have to admit that I don't know all this code and
it's embroilments too well. There is this with MTD and that without,
there is this for JFFS2 and that without. I lost track in that maze
long ago.

Eventually we should have some mtd_init() call which does this, but
then, I'm not sure if there might not be a case ot mtdparts without
MTD.

> > Assume we add this call here; would it then not also be needed in the
> > 'static' version of mtdparts_init() in "common/cmd_jffs2.c" (whatever
> > 'static' is supposed to mean) ?
> 
> Yes, it seems so.  Is there a good reason why jffs2 has its own
> implementation of this stuff?

Too many people working on differnt parts of the code, without
looking over their respective rims ?  Sorry, I never understood.
I think Stefan spent already some time to clean up parts of the mess,
but this probably needs more effort.

> yaffs_StartUp is in a similar situation.

Oh dear. It doesn't come to an end.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
What is wanted is not the will to believe,  but the will to find out,
which is the exact opposite.
		        -- Bertrand Russell, "Skeptical Essays", 1928


More information about the U-Boot mailing list