Hi Linus,<br><br><div class="gmail_quote">2010/4/1 Linus Walleij <span dir="ltr">&lt;<a href="mailto:linus.ml.walleij@gmail.com">linus.ml.walleij@gmail.com</a>&gt;</span><br><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
2010/3/31 Lemetayer Jean Marie &lt;<a href="mailto:jeanmarie.lemetayer@altran.com">jeanmarie.lemetayer@altran.com</a>&gt;:<br>
<div class="im"><br>
&gt; +#ifdef CONFIG_VENDOR_TOOL_COMPATIBILITY<br>
&gt; +static struct nand_ecclayout nomadik_ecc_layout = {<br>
&gt; +       .eccbytes = 3 * 4,<br>
&gt; +       .eccpos = {<br>
&gt; +               0x28, 0x29, 0x2A,<br>
&gt; +               0x2B, 0x2C, 0x2D,<br>
&gt; +               0x2E, 0x2F, 0x30,<br>
&gt; +               0x31, 0x32, 0x33},<br>
&gt; +       .oobfree = {{0x02, 0x26}, {0x34, 0x0C} },<br>
&gt; +};<br>
&gt; +#else<br>
&gt;  static struct nand_ecclayout nomadik_ecc_layout = {<br>
&gt;         .eccbytes = 3 * 4,<br>
&gt;         .eccpos = { /* each subpage has 16 bytes: pos 2,3,4 hosts ECC */<br>
&gt; @@ -54,6 +65,7 @@ static struct nand_ecclayout nomadik_ecc_layout = {<br>
&gt;         /* let&#39;s keep bytes 5,6,7 for us, just in case we change ECC algo */<br>
&gt;         .oobfree = { {0x08, 0x08}, {0x18, 0x08}, {0x28, 0x08}, {0x38, 0x08}<br>
&gt; },<br>
&gt;  };<br>
&gt; +#endif<br>
<br>
</div>Acked-by: Linus Walleij &lt;<a href="mailto:linus.walleij@stericsson.com">linus.walleij@stericsson.com</a>&gt;<br>
<br>
I don&#39;t think any of this layout belongs in the driver itself frankly speaking,<br>
I am facing the need to use this in U300 and we use yet *another*<br>
ECC layout with the FSMC, so we have to add yet another #ifdef.<br>
However I can fix that up with a patch on top of this one after it&#39;s<br>
queued.<br>
<br>
Yours,<br>
Linus Walleij<br></blockquote><div><br>What king of modification would you do ?<br>Do you think it shall be queued before your submission ?<br><br>Regards,<br>JM<br></div></div>