x1000: Improve NAND driver API

- Proper error codes are now returned from all functions. These codes will
  be used by a host-side flash tool for error reporting.

- nand_erase_block() was replaced by nand_erase_bytes(). The caller can't
  know how big an eraseblock is with the current API, so next best thing
  is to verify the correct alignment inside the call and reject the erase
  if it isn't properly aligned.

- Fixed typo in nandcmd_block_erase() which would cause an SFC error to be
  interpreted as success. Yikes.

Change-Id: Id4ac9b44fa7fc2fcb81ff19ba730df78457c0383

1 files changed, 18 insertions, 0 deletions

diff --git a/firmware/target/mips/ingenic_x1000/nand-x1000-err.h b/firmware/target/mips/ingenic_x1000/nand-x1000-err.h
new file mode 100644
index 0000000000..252d591062
--- /dev/null
+++ b/firmware/target/mips/ingenic_x1000/nand-x1000-err.h
@@ -0,0 +1,18 @@
+#ifndef __NAND_X1000_ERR_H__
+#define __NAND_X1000_ERR_H__
+
+/* Error codes which can be returned by the nand-x1000 API. These codes are
+ * also used by host-side tools, so we define them here to avoid polluting
+ * the namespace with useless X1000 APIs. */
+#define NANDERR_CHIP_UNSUPPORTED  (-1)
+#define NANDERR_WRITE_PROTECTED   (-2)
+#define NANDERR_UNALIGNED_ADDRESS (-3)
+#define NANDERR_UNALIGNED_LENGTH  (-4)
+#define NANDERR_READ_FAILED       (-5)
+#define NANDERR_ECC_FAILED        (-6)
+#define NANDERR_ERASE_FAILED      (-7)
+#define NANDERR_PROGRAM_FAILED    (-8)
+#define NANDERR_COMMAND_FAILED    (-9)
+#define NANDERR_OTHER             (-99)
+
+#endif /* __NAND_X1000_ERR_H__ */