于 2013年05月03日 07:39, Brian Norris 写道:
only run the extended parameter page function after that point.
sorry, I prefer to run the extended parameter page function here just following the case "p-> ecc_bits != 0xff". IMHO, the two cases should be put together as the onfi spec tells, rather be split out far away. The extended parameter page function is just the case "p->ecc_bits == 0xff".

If we put the nand_flash_detect_ext_param_page() after the point we reassign the chip->cmdfunc,
It's a little discrete in logic which makes the code not compact enough.

btw: Could you check you email client? I failed several times for the HTML issue.

thanks
Huang Shijie


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Reply via email to