diff options
author | Giovanni Di Sirio <gdisirio@gmail.com> | 2017-11-14 13:18:59 +0000 |
---|---|---|
committer | Giovanni Di Sirio <gdisirio@gmail.com> | 2017-11-14 13:18:59 +0000 |
commit | ed415bb76f7563346a3f9b0ba3ef45f9a6898afb (patch) | |
tree | 3444e6eb788fe8be4b98d22ea7cf96a1806d6a9a /os/hal/lib/complex/mfs | |
parent | 1935d28be9747aac18d9338fb1444514d257a51a (diff) | |
download | ChibiOS-ed415bb76f7563346a3f9b0ba3ef45f9a6898afb.tar.gz ChibiOS-ed415bb76f7563346a3f9b0ba3ef45f9a6898afb.tar.bz2 ChibiOS-ed415bb76f7563346a3f9b0ba3ef45f9a6898afb.zip |
git-svn-id: svn://svn.code.sf.net/p/chibios/svn/trunk@10998 35acf78f-673a-0410-8e92-d51de3d6d3f4
Diffstat (limited to 'os/hal/lib/complex/mfs')
-rw-r--r-- | os/hal/lib/complex/mfs/mfs.c | 145 | ||||
-rw-r--r-- | os/hal/lib/complex/mfs/mfs.h | 6 |
2 files changed, 51 insertions, 100 deletions
diff --git a/os/hal/lib/complex/mfs/mfs.c b/os/hal/lib/complex/mfs/mfs.c index a1decd519..71bc8ab76 100644 --- a/os/hal/lib/complex/mfs/mfs.c +++ b/os/hal/lib/complex/mfs/mfs.c @@ -154,7 +154,7 @@ static mfs_error_t mfs_flash_read(MFSDriver *mfsp, flash_offset_t offset, ferr = flashRead(mfsp->config->flashp, offset, n, rp);
if (ferr != FLASH_NO_ERROR) {
- mfsp->state = MFS_READY;
+ mfsp->state = MFS_ERROR;
return MFS_ERR_FLASH_FAILURE;
}
@@ -182,7 +182,7 @@ static mfs_error_t mfs_flash_write(MFSDriver *mfsp, ferr = flashProgram(mfsp->config->flashp, offset, n, wp);
if (ferr != FLASH_NO_ERROR) {
- mfsp->state = MFS_READY;
+ mfsp->state = MFS_ERROR;
return MFS_ERR_FLASH_FAILURE;
}
@@ -192,7 +192,7 @@ static mfs_error_t mfs_flash_write(MFSDriver *mfsp, size_t chunk = n <= MFS_CFG_BUFFER_SIZE ? n : MFS_CFG_BUFFER_SIZE;
RET_ON_ERROR(mfs_flash_read(mfsp, offset, chunk, mfsp->buffer.data8));
if (memcmp((void *)mfsp->buffer.data8, (void *)wp, chunk)) {
- mfsp->state = MFS_READY;
+ mfsp->state = MFS_ERROR;
return MFS_ERR_FLASH_FAILURE;
}
n -= chunk;
@@ -314,17 +314,17 @@ static mfs_error_t mfs_bank_erase(MFSDriver *mfsp, mfs_bank_t bank) { ferr = flashStartEraseSector(mfsp->config->flashp, sector);
if (ferr != FLASH_NO_ERROR) {
- mfsp->state = MFS_READY;
+ mfsp->state = MFS_ERROR;
return MFS_ERR_FLASH_FAILURE;
}
ferr = flashWaitErase(mfsp->config->flashp);
if (ferr != FLASH_NO_ERROR) {
- mfsp->state = MFS_READY;
+ mfsp->state = MFS_ERROR;
return MFS_ERR_FLASH_FAILURE;
}
ferr = flashVerifyErase(mfsp->config->flashp, sector);
if (ferr != FLASH_NO_ERROR) {
- mfsp->state = MFS_READY;
+ mfsp->state = MFS_ERROR;
return MFS_ERR_FLASH_FAILURE;
}
@@ -363,7 +363,7 @@ static mfs_error_t mfs_bank_verify_erase(MFSDriver *mfsp, mfs_bank_t bank) { return MFS_ERR_NOT_ERASED;
}
if (ferr != FLASH_NO_ERROR) {
- mfsp->state = MFS_READY;
+ mfsp->state = MFS_ERROR;
return MFS_ERR_FLASH_FAILURE;
}
@@ -795,64 +795,26 @@ void mfsObjectInit(MFSDriver *mfsp) { *
* @param[in] mfsp pointer to the @p MFSDriver object
* @param[in] config pointer to the configuration
- *
- * @api
- */
-void mfsStart(MFSDriver *mfsp, const MFSConfig *config) {
-
- osalDbgCheck((mfsp != NULL) && (config != NULL));
- osalDbgAssert((mfsp->state == MFS_STOP) || (mfsp->state == MFS_READY),
- "invalid state");
-
- mfsp->config = config;
- mfsp->state = MFS_READY;
-}
-
-/**
- * @brief Deactivates a MFS driver.
- *
- * @param[in] mfsp pointer to the @p MFSDriver object
- *
- * @api
- */
-void mfsStop(MFSDriver *mfsp) {
-
- osalDbgCheck(mfsp != NULL);
- osalDbgAssert((mfsp->state == MFS_STOP) || (mfsp->state == MFS_READY),
- "invalid state");
-
- mfsp->config = NULL;
- mfsp->state = MFS_STOP;
-}
-
-/**
- * @brief Mounts a managed flash storage.
- * @details This functions checks the storage internal state and eventually
- * performs the required initialization or repair operations.
- *
- * @param[in] mfsp pointer to the @p MFSDriver object
* @return The operation status.
* @retval MFS_NO_ERROR if the operation has been successfully completed.
- * @retval MFS_WARN_REPAIR if the operation has been completed but a
- * repair has been performed.
- * @retval MFS_ERR_INV_STATE if the volume is mounted.
+ * @retval MFS_WARN_GC if the operation triggered a garbage collection.
* @retval MFS_ERR_FLASH_FAILURE if the flash memory is unusable because HW
- * failures. Causes the volume to be unmounted.
+ * failures. Makes the driver enter the @p MFS_ERROR state.
+ * @retval MFS_ERR_INTERNAL if an internal logic failure is detected.
*
* @api
*/
-mfs_error_t mfsMount(MFSDriver *mfsp) {
+mfs_error_t mfsStart(MFSDriver *mfsp, const MFSConfig *config) {
unsigned i;
- osalDbgCheck(mfsp != NULL);
- osalDbgAssert((mfsp->state == MFS_READY) || (mfsp->state == MFS_MOUNTED),
- "invalid state");
+ osalDbgCheck((mfsp != NULL) && (config != NULL));
+ osalDbgAssert((mfsp->state == MFS_STOP) || (mfsp->state == MFS_READY) ||
+ (mfsp->state == MFS_ERROR), "invalid state");
- if (mfsp->state != MFS_READY) {
- return MFS_ERR_INV_STATE;
- }
+ /* Storing configuration.*/
+ mfsp->config = config;
- /* Clearing state.*/
+ /* Resetting previous state.*/
mfs_state_reset(mfsp);
/* Attempting to mount the managed partition.*/
@@ -860,38 +822,38 @@ mfs_error_t mfsMount(MFSDriver *mfsp) { mfs_error_t err;
err = mfs_try_mount(mfsp);
+ if (err == MFS_ERR_INTERNAL) {
+ /* Special case, do not retry on internal errors but report
+ immediately.*/
+ mfsp->state = MFS_ERROR;
+ return err;
+ }
if (!MFS_IS_ERROR(err)) {
- mfsp->state = MFS_MOUNTED;
+ mfsp->state = MFS_READY;
return err;
}
}
+ /* Driver start failed.*/
+ mfsp->state = MFS_ERROR;
return MFS_ERR_FLASH_FAILURE;
-}
+}
/**
- * @brief Unmounts a managed flash storage.
+ * @brief Deactivates a MFS driver.
*
* @param[in] mfsp pointer to the @p MFSDriver object
- * @return The operation status.
- * @retval MFS_NO_ERROR if the operation has been successfully completed.
- * @retval MFS_ERR_INV_STATE if the volume is not mounted.
*
* @api
*/
-mfs_error_t mfsUnmount(MFSDriver *mfsp) {
+void mfsStop(MFSDriver *mfsp) {
osalDbgCheck(mfsp != NULL);
- osalDbgAssert((mfsp->state == MFS_READY) || (mfsp->state == MFS_MOUNTED),
- "invalid state");
+ osalDbgAssert((mfsp->state == MFS_STOP) || (mfsp->state == MFS_READY) ||
+ (mfsp->state == MFS_ERROR), "invalid state");
- if (mfsp->state != MFS_MOUNTED) {
- return MFS_ERR_INV_STATE;
- }
-
- mfsp->state = MFS_READY;
-
- return MFS_NO_ERROR;
+ mfsp->config = NULL;
+ mfsp->state = MFS_STOP;
}
/**
@@ -899,18 +861,16 @@ mfs_error_t mfsUnmount(MFSDriver *mfsp) { *
* @param[in] mfsp pointer to the @p MFSDriver object
* @return The operation status.
- * @retval MFS_ERR_INV_STATE if the volume is mounted.
+ * @retval MFS_ERR_INV_STATE if the driver is in not in @p MSG_READY state.
* @retval MFS_NO_ERROR if the operation has been successfully completed.
* @retval MFS_ERR_FLASH_FAILURE if the flash memory is unusable because HW
- * failures. Causes the volume to be unmounted.
+ * failures. Makes the driver enter the @p MFS_ERROR state.
*
* @api
*/
mfs_error_t mfsErase(MFSDriver *mfsp) {
osalDbgCheck(mfsp != NULL);
- osalDbgAssert((mfsp->state == MFS_READY) || (mfsp->state == MFS_MOUNTED),
- "invalid state");
if (mfsp->state != MFS_READY) {
return MFS_ERR_INV_STATE;
@@ -918,6 +878,7 @@ mfs_error_t mfsErase(MFSDriver *mfsp) { RET_ON_ERROR(mfs_bank_erase(mfsp, MFS_BANK_0));
RET_ON_ERROR(mfs_bank_erase(mfsp, MFS_BANK_1));
+ /* TODO: Remount.*/
return MFS_NO_ERROR;
}
@@ -933,12 +894,12 @@ mfs_error_t mfsErase(MFSDriver *mfsp) { * @param[out] buffer pointer to a buffer for record data
* @return The operation status.
* @retval MFS_NO_ERROR if the operation has been successfully completed.
- * @retval MFS_ERR_INV_STATE if the volume is not mounted.
+ * @retval MFS_ERR_INV_STATE if the driver is in not in @p MSG_READY state.
* @retval MFS_ERR_INV_SIZE if the passed buffer is not large enough to
* contain the record data.
* @retval MFS_ERR_NOT_FOUND if the specified id does not exists.
* @retval MFS_ERR_FLASH_FAILURE if the flash memory is unusable because HW
- * failures. Causes the volume to be unmounted.
+ * failures. Makes the driver enter the @p MFS_ERROR state.
*
* @api
*/
@@ -948,10 +909,8 @@ mfs_error_t mfsReadRecord(MFSDriver *mfsp, uint32_t id, osalDbgCheck((mfsp != NULL) && (id >= 1) && (id <= MFS_CFG_MAX_RECORDS) &&
(np != NULL) && (buffer != NULL));
- osalDbgAssert((mfsp->state == MFS_READY) || (mfsp->state == MFS_MOUNTED),
- "invalid state");
- if (mfsp->state != MFS_MOUNTED) {
+ if (mfsp->state != MFS_READY) {
return MFS_ERR_INV_STATE;
}
@@ -975,7 +934,7 @@ mfs_error_t mfsReadRecord(MFSDriver *mfsp, uint32_t id, /* Checking CRC.*/
crc = crc16(0xFFFFU, buffer, *np);
if (crc != mfsp->buffer.dhdr.fields.crc) {
- mfsp->state = MFS_READY;
+ mfsp->state = MFS_ERROR;
return MFS_ERR_FLASH_FAILURE;
}
@@ -993,11 +952,11 @@ mfs_error_t mfsReadRecord(MFSDriver *mfsp, uint32_t id, * @return The operation status.
* @retval MFS_NO_ERROR if the operation has been successfully completed.
* @retval MFS_WARN_GC if the operation triggered a garbage collection.
- * @retval MFS_ERR_INV_STATE if the volume is not mounted.
+ * @retval MFS_ERR_INV_STATE if the driver is in not in @p MSG_READY state.
* @retval MFS_ERR_OUT_OF_MEM if there is not enough flash space for the
* operation.
* @retval MFS_ERR_FLASH_FAILURE if the flash memory is unusable because HW
- * failures. Causes the volume to be unmounted.
+ * failures. Makes the driver enter the @p MFS_ERROR state.
*
* @api
*/
@@ -1008,10 +967,8 @@ mfs_error_t mfsWriteRecord(MFSDriver *mfsp, uint32_t id, osalDbgCheck((mfsp != NULL) && (id >= 1) && (id <= MFS_CFG_MAX_RECORDS) &&
(n > 0U) && (buffer != NULL));
- osalDbgAssert((mfsp->state == MFS_READY) || (mfsp->state == MFS_MOUNTED),
- "invalid state");
- if (mfsp->state != MFS_MOUNTED) {
+ if (mfsp->state != MFS_READY) {
return MFS_ERR_INV_STATE;
}
@@ -1082,9 +1039,9 @@ mfs_error_t mfsWriteRecord(MFSDriver *mfsp, uint32_t id, * @return The operation status.
* @retval MFS_NO_ERROR if the operation has been successfully completed.
* @retval MFS_WARN_GC if the operation triggered a garbage collection.
- * @retval MFS_ERR_INV_STATE if the volume is not mounted.
+ * @retval MFS_ERR_INV_STATE if the driver is in not in @p MSG_READY state.
* @retval MFS_ERR_FLASH_FAILURE if the flash memory is unusable because HW
- * failures. Causes the volume to be unmounted.
+ * failures. Makes the driver enter the @p MFS_ERROR state.
* @retval MFS_ERR_INTERNAL if an internal logic failure is detected.
*
* @api
@@ -1094,10 +1051,8 @@ mfs_error_t mfsEraseRecord(MFSDriver *mfsp, uint32_t id) { bool warning = false;
osalDbgCheck((mfsp != NULL) && (id >= 1) && (id <= MFS_CFG_MAX_RECORDS));
- osalDbgAssert((mfsp->state == MFS_READY) || (mfsp->state == MFS_MOUNTED),
- "invalid state");
- if (mfsp->state != MFS_MOUNTED) {
+ if (mfsp->state != MFS_READY) {
return MFS_ERR_INV_STATE;
}
@@ -1151,19 +1106,17 @@ mfs_error_t mfsEraseRecord(MFSDriver *mfsp, uint32_t id) { * @param[in] mfsp pointer to the @p MFSDriver object
* @return The operation status.
* @retval MFS_NO_ERROR if the operation has been successfully completed.
- * @retval MFS_ERR_INV_STATE if the volume is not mounted.
+ * @retval MFS_ERR_INV_STATE if the driver is in not in @p MSG_READY state.
* @retval MFS_ERR_FLASH_FAILURE if the flash memory is unusable because HW
- * failures. Causes the volume to be unmounted.
+ * failures. Makes the driver enter the @p MFS_ERROR state.
*
* @api
*/
mfs_error_t mfsPerformGarbageCollection(MFSDriver *mfsp) {
osalDbgCheck(mfsp != NULL);
- osalDbgAssert((mfsp->state == MFS_READY) || (mfsp->state == MFS_MOUNTED),
- "invalid state");
- if (mfsp->state != MFS_MOUNTED) {
+ if (mfsp->state != MFS_READY) {
return MFS_ERR_INV_STATE;
}
diff --git a/os/hal/lib/complex/mfs/mfs.h b/os/hal/lib/complex/mfs/mfs.h index 49b1a342b..66c5f57d4 100644 --- a/os/hal/lib/complex/mfs/mfs.h +++ b/os/hal/lib/complex/mfs/mfs.h @@ -132,7 +132,7 @@ typedef enum { MFS_UNINIT = 0,
MFS_STOP = 1,
MFS_READY = 2,
- MFS_MOUNTED = 3
+ MFS_ERROR = 3
} mfs_state_t;
/**
@@ -349,10 +349,8 @@ typedef struct { extern "C" {
#endif
void mfsObjectInit(MFSDriver *devp);
- void mfsStart(MFSDriver *devp, const MFSConfig *config);
+ mfs_error_t mfsStart(MFSDriver *devp, const MFSConfig *config);
void mfsStop(MFSDriver *devp);
- mfs_error_t mfsMount(MFSDriver *devp);
- mfs_error_t mfsUnmount(MFSDriver *devp);
mfs_error_t mfsErase(MFSDriver *mfsp);
mfs_error_t mfsReadRecord(MFSDriver *devp, uint32_t id,
size_t *np, uint8_t *buffer);
|