Dear Scott Wood, In message <4e5bbe52.7080...@freescale.com> you wrote: > > >> +#ifdef CONFIG_NAND_U_BOOT > >> + do_fixup_by_path_string(fdt, "nor_flash", "status", "disabled"); > >> + do_fixup_by_path_string(fdt, "nand_flash", "status", "okay"); > >> +#endif > > > > What does ""okay" mean? This is not documented anywhere. > > It is documented in ePAPR and IEEE 1275.
Ah, thanks for poointing out. > > Is this supposed to mean "enabled"? > > Yes, or more specifically, "The device is believed to be operational." > > It's generally equivalent to having no status property at all. > > > Then please write "enabled" > > Please don't redefine well-established standards. Agreed. Sorry, I was not aware of this definition in ePAPR. But I guess most lreaders will - like me - fail to recognize this relation, and the presented documentation is more confusing than helpful. If the recommendation is to omit the status entry in the device tree, we should omit it in the code above, too. Otherwise, we should set status to okay in both cases. Having one here and the other there is at best confusing. 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: w...@denx.de Remember that Beethoven wrote his first symphony in C ... _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
