[amlogic][rawnand] Add a TODO: for tuning of the controller delay.
It is set to 200us now, but testing different values for delay,
a delay as low as 20us works. Don't want to change this right now
before testing various combinations of controller/NAND that we need
to support. Also, this should be a controller specific parameter,
instead of a hardcoded constant that applies to all controllers.
Change-Id: Ie20a68814a81c1be5f364aac4b384ee3e77fd81b
diff --git a/system/dev/rawnand/aml-rawnand/aml-rawnand.c b/system/dev/rawnand/aml-rawnand/aml-rawnand.c
index 9f211df..2832564 100644
--- a/system/dev/rawnand/aml-rawnand/aml-rawnand.c
+++ b/system/dev/rawnand/aml-rawnand/aml-rawnand.c
@@ -942,6 +942,14 @@
raw_nand->controller_params.rand_mode = aml_params.rand_mode;
raw_nand->controller_params.options = NAND_USE_BOUNCE_BUFFER;
raw_nand->controller_params.bch_mode = aml_params.bch_mode;
+ /*
+ * TODO: Controller Delay Tuning.
+ * Experimentally, a 20us controller delay works, but need to
+ * 1) Test this across all Amlogic controlers/NAND combinations.
+ * 2) Check with Amlogic if this should be controller specific, and
+ * if so, find out what the delays should be for the different controllers,
+ * and make this a controller-based parameter.
+ */
raw_nand->controller_delay = 200;
/*