>>>>> "Daniel" == Daniel Mack <zonque@xxxxxxxxx> writes: Hi, D> + Layouts for 1-bit ecc: stored at beginning of spare area as romcode: D> + D> + "hw-romcode" gpmc method & romcode layout D> + "bch4" 4-bit BCH ecc code D> + "bch8" 8-bit BCH ecc code D> + >> >> These are not 1-bit - Well, romcode might be depending on SoC. Looking at >> omap2.c it seems to be an alias for hw, so that isn't really helpful on >> E.G. am33xx where you should select bch8 to have something compatible >> with the romcode. Daniel> Not sure whether I follow you here. drivers/mtd/nand/omap2.c handles Daniel> cases for pdata->ecc_opt == OMAP_ECC_BCH8_CODE_HW, so the DT bindings Daniel> need to offer a way to set it. Daniel> Or are you purely referring to the comments only? Yes, but the document states (the first line I'm quotinge) 'Layouts for 1-bit ecc'. The other comment was about hw-romcode not being a very good name, as it apparently means the 1bit hamming code and ECC layout used on the older omap3, and not E.G. the bch8/elm layout used by E.G. am335x. D> + - ti,nand-ecc-use-elm: Property without value to specify that the D> + hardware error correction mode should be used. >> >> It's called the error location module. Like you pointed out yourself, >> this property isn't really nice. I haven't looked at the elm bindings in >> detail, but couldn't the nand driver automatically use the elm if the >> device tree provides a node for it? Perhaps the elm should be a subnode >> of the nand one? Daniel> That is to be discussed for the elm driver then. I'm not sure though Daniel> whether the elm should always be used when its DT node is present. Daniel> For now, I would leave it the way it is and have an optional Daniel> "ti,nand-ecc-use-elm". I just added some more lines of documenation to Daniel> describe the fact that the elm driver is needed in order to make it work. As discussed elsewhere, the elm also changes the ECC layout, so why not just have bchN (software bchN layout) / bchN-elm selections instead of the seperate ti,nand-ecc-use-elm? -- 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
