>>>>> "Daniel" == Daniel Mack <zonque@xxxxxxxxx> writes: Hi, >>>> In omap2 driver NAND_ECC_HW ecc mode supports 3 ecc layout >>>> OMAP_ECC_HAMMING_CODE_HW_ROMCODE >>>> OMAP_ECC_BCH4_CODE_HW >>>> OMAP_ECC_BCH8_CODE_HW >>>> >>>> So selection of ecc layout data should come from DT not ecc mode. >>> >>> Ok, I see. I would still like to set them by string rather than magic >>> numbers that map to enum entries. Valid values would be "none", "hw", >>> "hw-romcode", "bch4" and "bch8". Are you ok with that? >> >> Ok, that's nice. Better use ecc_opt instead of ecc_mode. Daniel> I did some more extensive tests that include reading the same Daniel> nand pages from both U-Boot and the kernel with BCH8 ECC, and Daniel> it turns out that -> is_elm_used needs to be set in the pdata Daniel> in order to make this work. So what you're saying is that the choice of ELM or not is not just an optimization, it really changes the ECC layout? Perhaps it should be treated as a seperate layout (E.G. bch8-elm) then? -- Bye, Peter Korsgaard -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
