#endif
#include "mflash.h"
-#include "time_support.h"
-#include "fileio.h"
-#include "log.h"
+#include <target/target.h>
+#include <helper/time_support.h>
+#include <helper/fileio.h>
+#include <helper/log.h>
-static int s3c2440_set_gpio_to_output (mflash_gpio_num_t gpio);
-static int s3c2440_set_gpio_output_val (mflash_gpio_num_t gpio, u8 val);
-static int pxa270_set_gpio_to_output (mflash_gpio_num_t gpio);
-static int pxa270_set_gpio_output_val (mflash_gpio_num_t gpio, u8 val);
+static int s3c2440_set_gpio_to_output (struct mflash_gpio_num gpio);
+static int s3c2440_set_gpio_output_val (struct mflash_gpio_num gpio, uint8_t val);
+static int pxa270_set_gpio_to_output (struct mflash_gpio_num gpio);
+static int pxa270_set_gpio_output_val (struct mflash_gpio_num gpio, uint8_t val);
-static command_t *mflash_cmd;
+static struct mflash_bank *mflash_bank;
-static mflash_bank_t *mflash_bank;
-
-static mflash_gpio_drv_t pxa270_gpio = {
+static struct mflash_gpio_drv pxa270_gpio = {
.name = "pxa270",
.set_gpio_to_output = pxa270_set_gpio_to_output,
.set_gpio_output_val = pxa270_set_gpio_output_val
};
-static mflash_gpio_drv_t s3c2440_gpio = {
+static struct mflash_gpio_drv s3c2440_gpio = {
.name = "s3c2440",
.set_gpio_to_output = s3c2440_set_gpio_to_output,
.set_gpio_output_val = s3c2440_set_gpio_output_val
};
-static mflash_gpio_drv_t *mflash_gpio[] =
+static struct mflash_gpio_drv *mflash_gpio[] =
{
&pxa270_gpio,
&s3c2440_gpio,
#define PXA270_GPSR0 0x40E00018
#define PXA270_GPCR0 0x40E00024
-static int pxa270_set_gpio_to_output (mflash_gpio_num_t gpio)
+static int pxa270_set_gpio_to_output (struct mflash_gpio_num gpio)
{
- u32 addr, value, mask;
- target_t *target = mflash_bank->target;
+ uint32_t addr, value, mask;
+ struct target *target = mflash_bank->target;
int ret;
/* remove alternate function. */
return ret;
}
-static int pxa270_set_gpio_output_val (mflash_gpio_num_t gpio, u8 val)
+static int pxa270_set_gpio_output_val (struct mflash_gpio_num gpio, uint8_t val)
{
- u32 addr, value, mask;
- target_t *target = mflash_bank->target;
+ uint32_t addr, value, mask;
+ struct target *target = mflash_bank->target;
int ret;
mask = 0x1u << (gpio.num & 0x1F);
#define S3C2440_GPJCON 0x560000d0
#define S3C2440_GPJDAT 0x560000d4
-static int s3c2440_set_gpio_to_output (mflash_gpio_num_t gpio)
+static int s3c2440_set_gpio_to_output (struct mflash_gpio_num gpio)
{
- u32 data, mask, gpio_con;
- target_t *target = mflash_bank->target;
+ uint32_t data, mask, gpio_con;
+ struct target *target = mflash_bank->target;
int ret;
if (gpio.port[0] >= 'a' && gpio.port[0] <= 'h') {
} else if (gpio.port[0] == 'j') {
gpio_con = S3C2440_GPJCON;
} else {
- LOG_ERROR("invalid port %d%s", gpio.num, gpio.port);
+ LOG_ERROR("mflash: invalid port %d%s", gpio.num, gpio.port);
return ERROR_INVALID_ARGUMENTS;
}
return ret;
}
-static int s3c2440_set_gpio_output_val (mflash_gpio_num_t gpio, u8 val)
+static int s3c2440_set_gpio_output_val (struct mflash_gpio_num gpio, uint8_t val)
{
- u32 data, mask, gpio_dat;
- target_t *target = mflash_bank->target;
+ uint32_t data, mask, gpio_dat;
+ struct target *target = mflash_bank->target;
int ret;
if (gpio.port[0] >= 'a' && gpio.port[0] <= 'h') {
} else if (gpio.port[0] == 'j') {
gpio_dat = S3C2440_GPJDAT;
} else {
- LOG_ERROR("invalid port %d%s", gpio.num, gpio.port);
+ LOG_ERROR("mflash: invalid port %d%s", gpio.num, gpio.port);
return ERROR_INVALID_ARGUMENTS;
}
return ret;
}
-static int mg_hdrst(u8 level)
+static int mg_hdrst(uint8_t level)
{
return mflash_bank->gpio_drv->set_gpio_output_val(mflash_bank->rst_pin, level);
}
static int mg_init_gpio (void)
{
- mflash_gpio_drv_t *gpio_drv = mflash_bank->gpio_drv;
+ int ret;
+ struct mflash_gpio_drv *gpio_drv = mflash_bank->gpio_drv;
+
+ ret = gpio_drv->set_gpio_to_output(mflash_bank->rst_pin);
+ if (ret != ERROR_OK)
+ return ret;
- gpio_drv->set_gpio_to_output(mflash_bank->rst_pin);
- gpio_drv->set_gpio_output_val(mflash_bank->rst_pin, 1);
+ ret = gpio_drv->set_gpio_output_val(mflash_bank->rst_pin, 1);
- return ERROR_OK;
+ return ret;
}
-static int mg_dsk_wait(mg_io_type_wait wait, u32 time)
+static int mg_dsk_wait(mg_io_type_wait wait, uint32_t time)
{
- u8 status, error;
- target_t *target = mflash_bank->target;
- u32 mg_task_reg = mflash_bank->base + MG_REG_OFFSET;
- duration_t duration;
- long long t=0;
+ uint8_t status, error;
+ struct target *target = mflash_bank->target;
+ uint32_t mg_task_reg = mflash_bank->base + MG_REG_OFFSET;
+ int ret;
+ long long t = 0;
- duration_start_measure(&duration);
+ struct duration bench;
+ duration_start(&bench);
while (time) {
- target_read_u8(target, mg_task_reg + MG_REG_STATUS, &status);
+ ret = target_read_u8(target, mg_task_reg + MG_REG_STATUS, &status);
+ if (ret != ERROR_OK)
+ return ret;
if (status & mg_io_rbit_status_busy)
{
if (wait == mg_io_wait_bsy)
return ERROR_OK;
} else {
- switch(wait)
+ switch (wait)
{
case mg_io_wait_not_bsy:
return ERROR_OK;
/* Now we check the error condition! */
if (status & mg_io_rbit_status_error)
{
- target_read_u8(target, mg_task_reg + MG_REG_ERROR, &error);
+ ret = target_read_u8(target, mg_task_reg + MG_REG_ERROR, &error);
+ if (ret != ERROR_OK)
+ return ret;
- if (error & mg_io_rbit_err_bad_sect_num) {
- LOG_ERROR("sector not found");
- return ERROR_FAIL;
- }
- else if (error & (mg_io_rbit_err_bad_block | mg_io_rbit_err_uncorrectable)) {
- LOG_ERROR("bad block");
- return ERROR_FAIL;
- } else {
- LOG_ERROR("disk operation fail");
- return ERROR_FAIL;
- }
+ LOG_ERROR("mflash: io error 0x%02x", error);
+
+ return ERROR_MG_IO;
}
switch (wait)
}
}
- duration_stop_measure(&duration, NULL);
-
- t=duration.duration.tv_usec/1000;
- t+=duration.duration.tv_sec*1000;
+ ret = duration_measure(&bench);
+ if (ERROR_OK == ret)
+ t = duration_elapsed(&bench) * 1000.0;
+ else
+ LOG_ERROR("mflash: duration measurement failed: %d", ret);
if (t > time)
break;
}
- LOG_ERROR("timeout occured");
- return ERROR_FAIL;
+ LOG_ERROR("mflash: timeout occured");
+ return ERROR_MG_TIMEOUT;
}
-static int mg_dsk_srst(u8 on)
+static int mg_dsk_srst(uint8_t on)
{
- target_t *target = mflash_bank->target;
- u32 mg_task_reg = mflash_bank->base + MG_REG_OFFSET;
- u8 value;
+ struct target *target = mflash_bank->target;
+ uint32_t mg_task_reg = mflash_bank->base + MG_REG_OFFSET;
+ uint8_t value;
int ret;
if ((ret = target_read_u8(target, mg_task_reg + MG_REG_DRV_CTRL, &value)) != ERROR_OK)
return ret;
- if(on) {
+ if (on) {
value |= (mg_io_rbit_devc_srst);
} else {
value &= ~mg_io_rbit_devc_srst;
return ret;
}
-static int mg_dsk_io_cmd(u32 sect_num, u32 cnt, u8 cmd)
+static int mg_dsk_io_cmd(uint32_t sect_num, uint32_t cnt, uint8_t cmd)
{
- target_t *target = mflash_bank->target;
- u32 mg_task_reg = mflash_bank->base + MG_REG_OFFSET;
- u8 value;
+ struct target *target = mflash_bank->target;
+ uint32_t mg_task_reg = mflash_bank->base + MG_REG_OFFSET;
+ uint8_t value;
+ int ret;
- if (mg_dsk_wait(mg_io_wait_rdy_noerr, MG_OEM_DISK_WAIT_TIME_NORMAL) != ERROR_OK)
- return ERROR_FAIL;
+ ret = mg_dsk_wait(mg_io_wait_rdy_noerr, MG_OEM_DISK_WAIT_TIME_NORMAL);
+ if (ret != ERROR_OK)
+ return ret;
value = mg_io_rval_dev_drv_master | mg_io_rval_dev_lba_mode |((sect_num >> 24) & 0xf);
- target_write_u8(target, mg_task_reg + MG_REG_DRV_HEAD, value);
- target_write_u8(target, mg_task_reg + MG_REG_SECT_CNT, (u8)cnt);
- target_write_u8(target, mg_task_reg + MG_REG_SECT_NUM, (u8)sect_num);
- target_write_u8(target, mg_task_reg + MG_REG_CYL_LOW, (u8)(sect_num >> 8));
- target_write_u8(target, mg_task_reg + MG_REG_CYL_HIGH, (u8)(sect_num >> 16));
+ ret = target_write_u8(target, mg_task_reg + MG_REG_DRV_HEAD, value);
+ ret |= target_write_u8(target, mg_task_reg + MG_REG_SECT_CNT, (uint8_t)cnt);
+ ret |= target_write_u8(target, mg_task_reg + MG_REG_SECT_NUM, (uint8_t)sect_num);
+ ret |= target_write_u8(target, mg_task_reg + MG_REG_CYL_LOW, (uint8_t)(sect_num >> 8));
+ ret |= target_write_u8(target, mg_task_reg + MG_REG_CYL_HIGH, (uint8_t)(sect_num >> 16));
- target_write_u8(target, mg_task_reg + MG_REG_COMMAND, cmd);
+ if (ret != ERROR_OK)
+ return ret;
- return ERROR_OK;
+ return target_write_u8(target, mg_task_reg + MG_REG_COMMAND, cmd);
}
static int mg_dsk_drv_info(void)
{
- target_t *target = mflash_bank->target;
- u32 mg_buff = mflash_bank->base + MG_BUFFER_OFFSET;
+ struct target *target = mflash_bank->target;
+ uint32_t mg_buff = mflash_bank->base + MG_BUFFER_OFFSET;
+ int ret;
- if ( mg_dsk_io_cmd(0, 1, mg_io_cmd_identify) != ERROR_OK)
- return ERROR_FAIL;
+ if ((ret = mg_dsk_io_cmd(0, 1, mg_io_cmd_identify)) != ERROR_OK)
+ return ret;
- if ( mg_dsk_wait(mg_io_wait_drq, MG_OEM_DISK_WAIT_TIME_NORMAL) != ERROR_OK)
- return ERROR_FAIL;
+ ret = mg_dsk_wait(mg_io_wait_drq, MG_OEM_DISK_WAIT_TIME_NORMAL);
+ if (ret != ERROR_OK)
+ return ret;
- LOG_INFO("read drive info.");
+ LOG_INFO("mflash: read drive info");
if (! mflash_bank->drv_info)
- mflash_bank->drv_info = malloc(sizeof(mg_drv_info_t));
+ mflash_bank->drv_info = malloc(sizeof(struct mg_drv_info));
- target->type->read_memory(target, mg_buff, 2, sizeof(mg_io_type_drv_info) >> 1,
- (u8 *)&mflash_bank->drv_info->drv_id);
+ ret = target_read_memory(target, mg_buff, 2,
+ sizeof(mg_io_type_drv_info) >> 1,
+ (uint8_t *)&mflash_bank->drv_info->drv_id);
+ if (ret != ERROR_OK)
+ return ret;
- mflash_bank->drv_info->tot_sects = (u32)(mflash_bank->drv_info->drv_id.total_user_addressable_sectors_hi << 16)
+ mflash_bank->drv_info->tot_sects = (uint32_t)(mflash_bank->drv_info->drv_id.total_user_addressable_sectors_hi << 16)
+ mflash_bank->drv_info->drv_id.total_user_addressable_sectors_lo;
- target_write_u8(target, mflash_bank->base + MG_REG_OFFSET + MG_REG_COMMAND, mg_io_cmd_confirm_read);
-
- return ERROR_OK;
+ return target_write_u8(target, mflash_bank->base + MG_REG_OFFSET + MG_REG_COMMAND, mg_io_cmd_confirm_read);
}
static int mg_mflash_rst(void)
{
- mg_init_gpio();
+ int ret;
- mg_hdrst(0);
+ if ((ret = mg_init_gpio()) != ERROR_OK)
+ return ret;
- if (mg_dsk_wait(mg_io_wait_bsy, MG_OEM_DISK_WAIT_TIME_LONG) != ERROR_OK)
- return ERROR_FAIL;
+ if ((ret = mg_hdrst(0)) != ERROR_OK)
+ return ret;
- mg_hdrst(1);
+ if ((ret = mg_dsk_wait(mg_io_wait_bsy, MG_OEM_DISK_WAIT_TIME_LONG)) != ERROR_OK)
+ return ret;
- if (mg_dsk_wait(mg_io_wait_not_bsy, MG_OEM_DISK_WAIT_TIME_LONG) != ERROR_OK)
- return ERROR_FAIL;
+ if ((ret = mg_hdrst(1)) != ERROR_OK)
+ return ret;
- mg_dsk_srst(1);
+ if ((ret = mg_dsk_wait(mg_io_wait_not_bsy, MG_OEM_DISK_WAIT_TIME_LONG)) != ERROR_OK)
+ return ret;
- if (mg_dsk_wait(mg_io_wait_bsy, MG_OEM_DISK_WAIT_TIME_LONG) != ERROR_OK)
- return ERROR_FAIL;
+ if ((ret = mg_dsk_srst(1)) != ERROR_OK)
+ return ret;
- mg_dsk_srst(0);
+ if ((ret = mg_dsk_wait(mg_io_wait_bsy, MG_OEM_DISK_WAIT_TIME_LONG)) != ERROR_OK)
+ return ret;
- if (mg_dsk_wait(mg_io_wait_not_bsy, MG_OEM_DISK_WAIT_TIME_LONG) != ERROR_OK)
- return ERROR_FAIL;
+ if ((ret = mg_dsk_srst(0)) != ERROR_OK)
+ return ret;
+
+ if ((ret = mg_dsk_wait(mg_io_wait_not_bsy, MG_OEM_DISK_WAIT_TIME_LONG)) != ERROR_OK)
+ return ret;
LOG_INFO("mflash: reset ok");
static int mg_mflash_probe(void)
{
- if (mg_mflash_rst() != ERROR_OK)
- return ERROR_FAIL;
+ int ret;
- if (mg_dsk_drv_info() != ERROR_OK)
- return ERROR_FAIL;
+ if ((ret = mg_mflash_rst()) != ERROR_OK)
+ return ret;
- return ERROR_OK;
+ return mg_dsk_drv_info();
}
-static int mg_probe_cmd(struct command_context_s *cmd_ctx, char *cmd, char **args, int argc)
+COMMAND_HANDLER(mg_probe_cmd)
{
int ret;
ret = mg_mflash_probe();
if (ret == ERROR_OK) {
- command_print(cmd_ctx, "mflash (total %u sectors) found at 0x%8.8x",
- mflash_bank->drv_info->tot_sects, mflash_bank->base );
+ command_print(CMD_CTX, "mflash (total %" PRIu32 " sectors) found at 0x%8.8" PRIx32 "",
+ mflash_bank->drv_info->tot_sects, mflash_bank->base);
}
return ret;
}
-static int mg_mflash_do_read_sects(void *buff, u32 sect_num, u32 sect_cnt)
+static int mg_mflash_do_read_sects(void *buff, uint32_t sect_num, uint32_t sect_cnt)
{
- u32 i, address;
+ uint32_t i, address;
int ret;
- target_t *target = mflash_bank->target;
- u8 *buff_ptr = buff;
- duration_t duration;
+ struct target *target = mflash_bank->target;
+ uint8_t *buff_ptr = buff;
- if ( mg_dsk_io_cmd(sect_num, sect_cnt, mg_io_cmd_read) != ERROR_OK )
- return ERROR_FAIL;
+ if ((ret = mg_dsk_io_cmd(sect_num, sect_cnt, mg_io_cmd_read)) != ERROR_OK)
+ return ret;
address = mflash_bank->base + MG_BUFFER_OFFSET;
- duration_start_measure(&duration);
+ struct duration bench;
+ duration_start(&bench);
for (i = 0; i < sect_cnt; i++) {
- mg_dsk_wait(mg_io_wait_drq, MG_OEM_DISK_WAIT_TIME_NORMAL);
+ ret = mg_dsk_wait(mg_io_wait_drq, MG_OEM_DISK_WAIT_TIME_NORMAL);
+ if (ret != ERROR_OK)
+ return ret;
- target->type->read_memory(target, address, 2, MG_MFLASH_SECTOR_SIZE / 2, buff_ptr);
- buff_ptr += MG_MFLASH_SECTOR_SIZE;
+ ret = target_read_memory(target, address, 2, MG_MFLASH_SECTOR_SIZE / 2, buff_ptr);
+ if (ret != ERROR_OK)
+ return ret;
- target_write_u8(target, mflash_bank->base + MG_REG_OFFSET + MG_REG_COMMAND, mg_io_cmd_confirm_read);
+ buff_ptr += MG_MFLASH_SECTOR_SIZE;
- LOG_DEBUG("%u (0x%8.8x) sector read", sect_num + i, (sect_num + i) * MG_MFLASH_SECTOR_SIZE);
+ ret = target_write_u8(target, mflash_bank->base + MG_REG_OFFSET + MG_REG_COMMAND, mg_io_cmd_confirm_read);
+ if (ret != ERROR_OK)
+ return ret;
- duration_stop_measure(&duration, NULL);
+ LOG_DEBUG("mflash: %" PRIu32 " (0x%8.8" PRIx32 ") sector read", sect_num + i, (sect_num + i) * MG_MFLASH_SECTOR_SIZE);
- if ((duration.duration.tv_sec * 1000 + duration.duration.tv_usec / 1000) > 3000) {
- LOG_INFO("read %u'th sectors", sect_num + i);
- duration_start_measure(&duration);
+ ret = duration_measure(&bench);
+ if ((ERROR_OK == ret) && (duration_elapsed(&bench) > 3)) {
+ LOG_INFO("mflash: read %" PRIu32 "'th sectors", sect_num + i);
+ duration_start(&bench);
}
}
- ret = mg_dsk_wait(mg_io_wait_rdy, MG_OEM_DISK_WAIT_TIME_NORMAL);
-
- return ret;
+ return mg_dsk_wait(mg_io_wait_rdy, MG_OEM_DISK_WAIT_TIME_NORMAL);
}
-static int mg_mflash_read_sects(void *buff, u32 sect_num, u32 sect_cnt)
+static int mg_mflash_read_sects(void *buff, uint32_t sect_num, uint32_t sect_cnt)
{
- u32 quotient, residue, i;
- u8 *buff_ptr = buff;
+ uint32_t quotient, residue, i;
+ uint8_t *buff_ptr = buff;
+ int ret = ERROR_OK;
quotient = sect_cnt >> 8;
residue = sect_cnt % 256;
for (i = 0; i < quotient; i++) {
- LOG_DEBUG("sect num : %u buff : 0x%0lx", sect_num,
- (unsigned long)buff_ptr);
- mg_mflash_do_read_sects(buff_ptr, sect_num, 256);
+ LOG_DEBUG("mflash: sect num : %" PRIu32 " buff : %p",
+ sect_num, buff_ptr);
+ ret = mg_mflash_do_read_sects(buff_ptr, sect_num, 256);
+ if (ret != ERROR_OK)
+ return ret;
+
sect_num += 256;
buff_ptr += 256 * MG_MFLASH_SECTOR_SIZE;
}
if (residue) {
- LOG_DEBUG("sect num : %u buff : %0lx", sect_num,
- (unsigned long)buff_ptr);
- mg_mflash_do_read_sects(buff_ptr, sect_num, residue);
+ LOG_DEBUG("mflash: sect num : %" PRIx32 " buff : %p",
+ sect_num, buff_ptr);
+ return mg_mflash_do_read_sects(buff_ptr, sect_num, residue);
}
- return ERROR_OK;
+ return ret;
}
-static int mg_mflash_do_write_sects(void *buff, u32 sect_num, u32 sect_cnt,
+static int mg_mflash_do_write_sects(void *buff, uint32_t sect_num, uint32_t sect_cnt,
mg_io_type_cmd cmd)
{
- u32 i, address;
+ uint32_t i, address;
int ret;
- target_t *target = mflash_bank->target;
- u8 *buff_ptr = buff;
- duration_t duration;
+ struct target *target = mflash_bank->target;
+ uint8_t *buff_ptr = buff;
- if ( mg_dsk_io_cmd(sect_num, sect_cnt, cmd) != ERROR_OK )
- return ERROR_FAIL;
+ if ((ret = mg_dsk_io_cmd(sect_num, sect_cnt, cmd)) != ERROR_OK)
+ return ret;
address = mflash_bank->base + MG_BUFFER_OFFSET;
- duration_start_measure(&duration);
+ struct duration bench;
+ duration_start(&bench);
for (i = 0; i < sect_cnt; i++) {
ret = mg_dsk_wait(mg_io_wait_drq, MG_OEM_DISK_WAIT_TIME_NORMAL);
if (ret != ERROR_OK)
- LOG_ERROR("mg_io_wait_drq time out");
+ return ret;
- ret = target->type->write_memory(target, address, 2, MG_MFLASH_SECTOR_SIZE / 2, buff_ptr);
+ ret = target_write_memory(target, address, 2, MG_MFLASH_SECTOR_SIZE / 2, buff_ptr);
if (ret != ERROR_OK)
- LOG_ERROR("mem write error");
+ return ret;
+
buff_ptr += MG_MFLASH_SECTOR_SIZE;
ret = target_write_u8(target, mflash_bank->base + MG_REG_OFFSET + MG_REG_COMMAND, mg_io_cmd_confirm_write);
if (ret != ERROR_OK)
- LOG_ERROR("mg_io_cmd_confirm_write error");
-
- LOG_DEBUG("%u (0x%8.8x) sector write", sect_num + i, (sect_num + i) * MG_MFLASH_SECTOR_SIZE);
+ return ret;
- duration_stop_measure(&duration, NULL);
+ LOG_DEBUG("mflash: %" PRIu32 " (0x%8.8" PRIx32 ") sector write", sect_num + i, (sect_num + i) * MG_MFLASH_SECTOR_SIZE);
- if ((duration.duration.tv_sec * 1000 + duration.duration.tv_usec / 1000) > 3000) {
- LOG_INFO("wrote %u'th sectors", sect_num + i);
- duration_start_measure(&duration);
+ ret = duration_measure(&bench);
+ if ((ERROR_OK == ret) && (duration_elapsed(&bench) > 3)) {
+ LOG_INFO("mflash: wrote %" PRIu32 "'th sectors", sect_num + i);
+ duration_start(&bench);
}
}
return ret;
}
-static int mg_mflash_write_sects(void *buff, u32 sect_num, u32 sect_cnt)
+static int mg_mflash_write_sects(void *buff, uint32_t sect_num, uint32_t sect_cnt)
{
- u32 quotient, residue, i;
- u8 *buff_ptr = buff;
+ uint32_t quotient, residue, i;
+ uint8_t *buff_ptr = buff;
+ int ret = ERROR_OK;
quotient = sect_cnt >> 8;
residue = sect_cnt % 256;
for (i = 0; i < quotient; i++) {
- LOG_DEBUG("sect num : %u buff : %0lx", sect_num,
- (unsigned long)buff_ptr);
- mg_mflash_do_write_sects(buff_ptr, sect_num, 256, mg_io_cmd_write);
+ LOG_DEBUG("mflash: sect num : %" PRIu32 "buff : %p", sect_num,
+ buff_ptr);
+ ret = mg_mflash_do_write_sects(buff_ptr, sect_num, 256, mg_io_cmd_write);
+ if (ret != ERROR_OK)
+ return ret;
+
sect_num += 256;
buff_ptr += 256 * MG_MFLASH_SECTOR_SIZE;
}
if (residue) {
- LOG_DEBUG("sect num : %u buff : %0lx", sect_num,
- (unsigned long)buff_ptr);
- mg_mflash_do_write_sects(buff_ptr, sect_num, residue, mg_io_cmd_write);
+ LOG_DEBUG("mflash: sect num : %" PRIu32 " buff : %p", sect_num,
+ buff_ptr);
+ return mg_mflash_do_write_sects(buff_ptr, sect_num, residue, mg_io_cmd_write);
}
- return ERROR_OK;
+ return ret;
}
-static int mg_mflash_read (u32 addr, u8 *buff, u32 len)
+static int mg_mflash_read (uint32_t addr, uint8_t *buff, uint32_t len)
{
- u8 *sect_buff, *buff_ptr = buff;
- u32 cur_addr, next_sec_addr, end_addr, cnt, sect_num;
+ uint8_t *buff_ptr = buff;
+ uint8_t sect_buff[MG_MFLASH_SECTOR_SIZE];
+ uint32_t cur_addr, next_sec_addr, end_addr, cnt, sect_num;
+ int ret = ERROR_OK;
cnt = 0;
cur_addr = addr;
end_addr = addr + len;
- sect_buff = malloc(MG_MFLASH_SECTOR_SIZE);
-
if (cur_addr & MG_MFLASH_SECTOR_SIZE_MASK) {
next_sec_addr = (cur_addr + MG_MFLASH_SECTOR_SIZE) & ~MG_MFLASH_SECTOR_SIZE_MASK;
sect_num = cur_addr >> MG_MFLASH_SECTOR_SIZE_SHIFT;
- mg_mflash_read_sects(sect_buff, sect_num, 1);
+ ret = mg_mflash_read_sects(sect_buff, sect_num, 1);
+ if (ret != ERROR_OK)
+ return ret;
if (end_addr < next_sec_addr) {
memcpy(buff_ptr, sect_buff + (cur_addr & MG_MFLASH_SECTOR_SIZE_MASK), end_addr - cur_addr);
- LOG_DEBUG("copies %u byte from sector offset 0x%8.8x", end_addr - cur_addr, cur_addr);
+ LOG_DEBUG("mflash: copies %" PRIu32 " byte from sector offset 0x%8.8" PRIx32 "", end_addr - cur_addr, cur_addr);
cur_addr = end_addr;
} else {
memcpy(buff_ptr, sect_buff + (cur_addr & MG_MFLASH_SECTOR_SIZE_MASK), next_sec_addr - cur_addr);
- LOG_DEBUG("copies %u byte from sector offset 0x%8.8x", next_sec_addr - cur_addr, cur_addr);
+ LOG_DEBUG("mflash: copies %" PRIu32 " byte from sector offset 0x%8.8" PRIx32 "", next_sec_addr - cur_addr, cur_addr);
buff_ptr += (next_sec_addr - cur_addr);
cur_addr = next_sec_addr;
}
}
if (cnt)
- mg_mflash_read_sects(buff_ptr, sect_num, cnt);
+ if ((ret = mg_mflash_read_sects(buff_ptr, sect_num, cnt)) != ERROR_OK)
+ return ret;
buff_ptr += cnt * MG_MFLASH_SECTOR_SIZE;
cur_addr += cnt * MG_MFLASH_SECTOR_SIZE;
if (cur_addr < end_addr) {
sect_num = cur_addr >> MG_MFLASH_SECTOR_SIZE_SHIFT;
- mg_mflash_read_sects(sect_buff, sect_num, 1);
+ ret = mg_mflash_read_sects(sect_buff, sect_num, 1);
+ if (ret != ERROR_OK)
+ return ret;
+
memcpy(buff_ptr, sect_buff, end_addr - cur_addr);
- LOG_DEBUG("copies %u byte", end_addr - cur_addr);
+ LOG_DEBUG("mflash: copies %u byte", (unsigned)(end_addr - cur_addr));
}
}
- free(sect_buff);
-
- return ERROR_OK;
+ return ret;
}
-static int mg_mflash_write(u32 addr, u8 *buff, u32 len)
+static int mg_mflash_write(uint32_t addr, uint8_t *buff, uint32_t len)
{
- u8 *sect_buff, *buff_ptr = buff;
- u32 cur_addr, next_sec_addr, end_addr, cnt, sect_num;
+ uint8_t *buff_ptr = buff;
+ uint8_t sect_buff[MG_MFLASH_SECTOR_SIZE];
+ uint32_t cur_addr, next_sec_addr, end_addr, cnt, sect_num;
+ int ret = ERROR_OK;
cnt = 0;
cur_addr = addr;
end_addr = addr + len;
- sect_buff = malloc(MG_MFLASH_SECTOR_SIZE);
-
if (cur_addr & MG_MFLASH_SECTOR_SIZE_MASK) {
next_sec_addr = (cur_addr + MG_MFLASH_SECTOR_SIZE) & ~MG_MFLASH_SECTOR_SIZE_MASK;
sect_num = cur_addr >> MG_MFLASH_SECTOR_SIZE_SHIFT;
- mg_mflash_read_sects(sect_buff, sect_num, 1);
+ ret = mg_mflash_read_sects(sect_buff, sect_num, 1);
+ if (ret != ERROR_OK)
+ return ret;
if (end_addr < next_sec_addr) {
memcpy(sect_buff + (cur_addr & MG_MFLASH_SECTOR_SIZE_MASK), buff_ptr, end_addr - cur_addr);
- LOG_DEBUG("copies %u byte to sector offset 0x%8.8x", end_addr - cur_addr, cur_addr);
+ LOG_DEBUG("mflash: copies %" PRIu32 " byte to sector offset 0x%8.8" PRIx32 "", end_addr - cur_addr, cur_addr);
cur_addr = end_addr;
} else {
memcpy(sect_buff + (cur_addr & MG_MFLASH_SECTOR_SIZE_MASK), buff_ptr, next_sec_addr - cur_addr);
- LOG_DEBUG("copies %u byte to sector offset 0x%8.8x", next_sec_addr - cur_addr, cur_addr);
+ LOG_DEBUG("mflash: copies %" PRIu32 " byte to sector offset 0x%8.8" PRIx32 "", next_sec_addr - cur_addr, cur_addr);
buff_ptr += (next_sec_addr - cur_addr);
cur_addr = next_sec_addr;
}
- mg_mflash_write_sects(sect_buff, sect_num, 1);
+ ret = mg_mflash_write_sects(sect_buff, sect_num, 1);
+ if (ret != ERROR_OK)
+ return ret;
}
if (cur_addr < end_addr) {
}
if (cnt)
- mg_mflash_write_sects(buff_ptr, sect_num, cnt);
+ if ((ret = mg_mflash_write_sects(buff_ptr, sect_num, cnt)) != ERROR_OK)
+ return ret;
buff_ptr += cnt * MG_MFLASH_SECTOR_SIZE;
cur_addr += cnt * MG_MFLASH_SECTOR_SIZE;
if (cur_addr < end_addr) {
sect_num = cur_addr >> MG_MFLASH_SECTOR_SIZE_SHIFT;
- mg_mflash_read_sects(sect_buff, sect_num, 1);
+ ret = mg_mflash_read_sects(sect_buff, sect_num, 1);
+ if (ret != ERROR_OK)
+ return ret;
+
memcpy(sect_buff, buff_ptr, end_addr - cur_addr);
- LOG_DEBUG("copies %u byte", end_addr - cur_addr);
- mg_mflash_write_sects(sect_buff, sect_num, 1);
+ LOG_DEBUG("mflash: copies %" PRIu32 " byte", end_addr - cur_addr);
+ ret = mg_mflash_write_sects(sect_buff, sect_num, 1);
}
}
- free(sect_buff);
-
- return ERROR_OK;
+ return ret;
}
-static int mg_write_cmd(struct command_context_s *cmd_ctx, char *cmd, char **args, int argc)
+COMMAND_HANDLER(mg_write_cmd)
{
- u32 address, buf_cnt, cnt, res, i;
- u8 *buffer;
- fileio_t fileio;
- duration_t duration;
- char *duration_text;
+ uint32_t address, cnt, res, i;
+ uint8_t *buffer;
+ struct fileio fileio;
+ int ret;
- if (argc != 3) {
+ if (CMD_ARGC != 3) {
return ERROR_COMMAND_SYNTAX_ERROR;
}
- address = strtoul(args[2], NULL, 0);
+ COMMAND_PARSE_NUMBER(u32, CMD_ARGV[2], address);
- if (fileio_open(&fileio, args[1], FILEIO_READ, FILEIO_BINARY) != ERROR_OK) {
- return ERROR_FAIL;
- }
+ ret = fileio_open(&fileio, CMD_ARGV[1], FILEIO_READ, FILEIO_BINARY);
+ if (ret != ERROR_OK)
+ return ret;
buffer = malloc(MG_FILEIO_CHUNK);
if (!buffer) {
cnt = fileio.size / MG_FILEIO_CHUNK;
res = fileio.size % MG_FILEIO_CHUNK;
- duration_start_measure(&duration);
+ struct duration bench;
+ duration_start(&bench);
+ size_t buf_cnt;
for (i = 0; i < cnt; i++) {
- if (fileio_read(&fileio, MG_FILEIO_CHUNK, buffer, &buf_cnt) !=
+ if ((ret = fileio_read(&fileio, MG_FILEIO_CHUNK, buffer, &buf_cnt)) !=
ERROR_OK)
goto mg_write_cmd_err;
- if (mg_mflash_write(address, buffer, MG_FILEIO_CHUNK) != ERROR_OK)
+ if ((ret = mg_mflash_write(address, buffer, MG_FILEIO_CHUNK)) != ERROR_OK)
goto mg_write_cmd_err;
address += MG_FILEIO_CHUNK;
}
-
+
if (res) {
- if (fileio_read(&fileio, res, buffer, &buf_cnt) != ERROR_OK)
- goto mg_write_cmd_err;
- if (mg_mflash_write(address, buffer, res) != ERROR_OK)
+ if ((ret = fileio_read(&fileio, res, buffer, &buf_cnt)) != ERROR_OK)
+ goto mg_write_cmd_err;
+ if ((ret = mg_mflash_write(address, buffer, res)) != ERROR_OK)
goto mg_write_cmd_err;
}
- duration_stop_measure(&duration, &duration_text);
-
- command_print(cmd_ctx, "wrote %lli byte from file %s in %s (%f kB/s)",
- fileio.size, args[1], duration_text,
- (float)fileio.size / 1024.0 / ((float)duration.duration.tv_sec + ((float)duration.duration.tv_usec / 1000000.0)));
+ if (duration_measure(&bench) == ERROR_OK)
+ {
+ command_print(CMD_CTX, "wrote %zu byte from file %s "
+ "in %fs (%0.3f kB/s)", fileio.size, CMD_ARGV[1],
+ duration_elapsed(&bench), duration_kbps(&bench, fileio.size));
+ }
- free(duration_text);
free(buffer);
fileio_close(&fileio);
return ERROR_OK;
mg_write_cmd_err:
- duration_stop_measure(&duration, &duration_text);
- free(duration_text);
free(buffer);
fileio_close(&fileio);
- return ERROR_FAIL;
+ return ret;
}
-static int mg_dump_cmd(struct command_context_s *cmd_ctx, char *cmd, char **args, int argc)
+COMMAND_HANDLER(mg_dump_cmd)
{
- u32 address, size_written, size, cnt, res, i;
- u8 *buffer;
- fileio_t fileio;
- duration_t duration;
- char *duration_text;
+ uint32_t address, size, cnt, res, i;
+ uint8_t *buffer;
+ struct fileio fileio;
+ int ret;
- if (argc != 4) {
+ if (CMD_ARGC != 4) {
return ERROR_COMMAND_SYNTAX_ERROR;
}
- address = strtoul(args[2], NULL, 0);
- size = strtoul(args[3], NULL, 0);
+ COMMAND_PARSE_NUMBER(u32, CMD_ARGV[2], address);
+ COMMAND_PARSE_NUMBER(u32, CMD_ARGV[3], size);
+
+ ret = fileio_open(&fileio, CMD_ARGV[1], FILEIO_WRITE, FILEIO_BINARY);
+ if (ret != ERROR_OK)
+ return ret;
- if (fileio_open(&fileio, args[1], FILEIO_WRITE, FILEIO_BINARY) != ERROR_OK) {
- return ERROR_FAIL;
- }
-
buffer = malloc(MG_FILEIO_CHUNK);
if (!buffer) {
fileio_close(&fileio);
cnt = size / MG_FILEIO_CHUNK;
res = size % MG_FILEIO_CHUNK;
-
- duration_start_measure(&duration);
+ struct duration bench;
+ duration_start(&bench);
+
+ size_t size_written;
for (i = 0; i < cnt; i++) {
- if (mg_mflash_read(address, buffer, MG_FILEIO_CHUNK) != ERROR_OK)
+ if ((ret = mg_mflash_read(address, buffer, MG_FILEIO_CHUNK)) != ERROR_OK)
goto mg_dump_cmd_err;
- if (fileio_write(&fileio, MG_FILEIO_CHUNK, buffer, &size_written)
+ if ((ret = fileio_write(&fileio, MG_FILEIO_CHUNK, buffer, &size_written))
!= ERROR_OK)
goto mg_dump_cmd_err;
address += MG_FILEIO_CHUNK;
}
-
+
if (res) {
- if (mg_mflash_read(address, buffer, res) != ERROR_OK)
+ if ((ret = mg_mflash_read(address, buffer, res)) != ERROR_OK)
goto mg_dump_cmd_err;
- if (fileio_write(&fileio, res, buffer, &size_written) != ERROR_OK)
+ if ((ret = fileio_write(&fileio, res, buffer, &size_written)) != ERROR_OK)
goto mg_dump_cmd_err;
}
- duration_stop_measure(&duration, &duration_text);
-
- command_print(cmd_ctx, "dump image (address 0x%8.8x size %u) to file %s in %s (%f kB/s)",
- address, size, args[1], duration_text,
- (float)size / 1024.0 / ((float)duration.duration.tv_sec + ((float)duration.duration.tv_usec / 1000000.0)));
+ if (duration_measure(&bench) == ERROR_OK)
+ {
+ command_print(CMD_CTX, "dump image (address 0x%8.8" PRIx32 " "
+ "size %" PRIu32 ") to file %s in %fs (%0.3f kB/s)",
+ address, size, CMD_ARGV[1],
+ duration_elapsed(&bench), duration_kbps(&bench, size));
+ }
- free(duration_text);
free(buffer);
fileio_close(&fileio);
return ERROR_OK;
mg_dump_cmd_err:
- duration_stop_measure(&duration, &duration_text);
- free(duration_text);
free(buffer);
fileio_close(&fileio);
-
- return ERROR_FAIL;
+
+ return ret;
}
static int mg_set_feature(mg_feature_id feature, mg_feature_val config)
{
- target_t *target = mflash_bank->target;
- u32 mg_task_reg = mflash_bank->base + MG_REG_OFFSET;
+ struct target *target = mflash_bank->target;
+ uint32_t mg_task_reg = mflash_bank->base + MG_REG_OFFSET;
+ int ret;
- if (mg_dsk_wait(mg_io_wait_rdy_noerr, MG_OEM_DISK_WAIT_TIME_NORMAL)
+ if ((ret = mg_dsk_wait(mg_io_wait_rdy_noerr, MG_OEM_DISK_WAIT_TIME_NORMAL))
!= ERROR_OK)
- return ERROR_FAIL;
+ return ret;
- target_write_u8(target, mg_task_reg + MG_REG_FEATURE, feature);
- target_write_u8(target, mg_task_reg + MG_REG_SECT_CNT, config);
- target_write_u8(target, mg_task_reg + MG_REG_COMMAND,
+ ret = target_write_u8(target, mg_task_reg + MG_REG_FEATURE, feature);
+ ret |= target_write_u8(target, mg_task_reg + MG_REG_SECT_CNT, config);
+ ret |= target_write_u8(target, mg_task_reg + MG_REG_COMMAND,
mg_io_cmd_set_feature);
- return ERROR_OK;
+ return ret;
}
static int mg_is_valid_pll(double XIN, int N, double CLK_OUT, int NO)
double v2 = CLK_OUT * NO;
if (v1 <1000000 || v1 > 15000000 || v2 < 100000000 || v2 > 500000000)
- return ERROR_FAIL;
+ return ERROR_MG_INVALID_PLL;
return ERROR_OK;
}
{
int i, M;
- for (i = 1, M=0; i < 512; i <<= 1, feedback_div >>= 1)
+ for (i = 1, M = 0; i < 512; i <<= 1, feedback_div >>= 1)
M += (feedback_div & 1) * i;
return M + 2;
int i, NO;
for (i = 0, NO = 1; i < 2; ++i, output_div >>= 1)
- if(output_div & 1)
+ if (output_div & 1)
NO = NO << 1;
return NO;
CLK_OUT = XIN * ((double)M / N) / NO;
- if ((int)((CLK_OUT+ROUND) / DIV)
+ if ((int)((CLK_OUT + ROUND) / DIV)
== (int)(MG_PLL_CLK_OUT / DIV)) {
if (mg_is_valid_pll(XIN, N, CLK_OUT, NO) == ERROR_OK)
{
static int mg_verify_interface(void)
{
- u16 buff[MG_MFLASH_SECTOR_SIZE >> 1];
- u16 i, j;
- u32 address = mflash_bank->base + MG_BUFFER_OFFSET;
- target_t *target = mflash_bank->target;
+ uint16_t buff[MG_MFLASH_SECTOR_SIZE >> 1];
+ uint16_t i, j;
+ uint32_t address = mflash_bank->base + MG_BUFFER_OFFSET;
+ struct target *target = mflash_bank->target;
+ int ret;
for (j = 0; j < 10; j++) {
for (i = 0; i < MG_MFLASH_SECTOR_SIZE >> 1; i++)
buff[i] = i;
- target->type->write_memory(target, address, 2,
- MG_MFLASH_SECTOR_SIZE / 2, (u8 *)buff);
+ ret = target_write_memory(target, address, 2,
+ MG_MFLASH_SECTOR_SIZE / 2, (uint8_t *)buff);
+ if (ret != ERROR_OK)
+ return ret;
memset(buff, 0xff, MG_MFLASH_SECTOR_SIZE);
- target->type->read_memory(target, address, 2,
- MG_MFLASH_SECTOR_SIZE / 2, (u8 *)buff);
+ ret = target_read_memory(target, address, 2,
+ MG_MFLASH_SECTOR_SIZE / 2, (uint8_t *)buff);
+ if (ret != ERROR_OK)
+ return ret;
for (i = 0; i < MG_MFLASH_SECTOR_SIZE >> 1; i++) {
if (buff[i] != i) {
LOG_ERROR("mflash: verify interface fail");
- return ERROR_FAIL;
+ return ERROR_MG_INTERFACE;
}
}
}
LOG_INFO("mflash: verify interface ok");
- return ERROR_OK;
+ return ret;
}
static const char g_strSEG_SerialNum[20] = {
static int mg_storage_config(void)
{
- u8 buff[512];
+ uint8_t buff[512];
+ int ret;
- if (mg_set_feature(mg_feature_id_transmode, mg_feature_val_trans_vcmd)
+ if ((ret = mg_set_feature(mg_feature_id_transmode, mg_feature_val_trans_vcmd))
!= ERROR_OK)
- return ERROR_FAIL;
+ return ret;
mg_gen_ataid((mg_io_type_drv_info *)buff);
- if (mg_mflash_do_write_sects(buff, 0, 1, mg_vcmd_update_stgdrvinfo)
+ if ((ret = mg_mflash_do_write_sects(buff, 0, 1, mg_vcmd_update_stgdrvinfo))
!= ERROR_OK)
- return ERROR_FAIL;
+ return ret;
- if (mg_set_feature(mg_feature_id_transmode, mg_feature_val_trans_default)
+ if ((ret = mg_set_feature(mg_feature_id_transmode, mg_feature_val_trans_default))
!= ERROR_OK)
- return ERROR_FAIL;
+ return ret;
LOG_INFO("mflash: storage config ok");
- return ERROR_OK;
+ return ret;
}
static int mg_boot_config(void)
{
- u8 buff[512];
+ uint8_t buff[512];
+ int ret;
- if (mg_set_feature(mg_feature_id_transmode, mg_feature_val_trans_vcmd)
+ if ((ret = mg_set_feature(mg_feature_id_transmode, mg_feature_val_trans_vcmd))
!= ERROR_OK)
- return ERROR_FAIL;
+ return ret;
memset(buff, 0xff, 512);
buff[0] = mg_op_mode_snd; /* operation mode */
buff[1] = MG_UNLOCK_OTP_AREA;
buff[2] = 4; /* boot size */
- *((u32 *)(buff + 4)) = 0; /* XIP size */
+ *((uint32_t *)(buff + 4)) = 0; /* XIP size */
- if (mg_mflash_do_write_sects(buff, 0, 1, mg_vcmd_update_xipinfo)
+ if ((ret = mg_mflash_do_write_sects(buff, 0, 1, mg_vcmd_update_xipinfo))
!= ERROR_OK)
- return ERROR_FAIL;
+ return ret;
- if (mg_set_feature(mg_feature_id_transmode, mg_feature_val_trans_default)
+ if ((ret = mg_set_feature(mg_feature_id_transmode, mg_feature_val_trans_default))
!= ERROR_OK)
- return ERROR_FAIL;
+ return ret;
LOG_INFO("mflash: boot config ok");
- return ERROR_OK;
+ return ret;
}
static int mg_set_pll(mg_pll_t *pll)
{
- u8 buff[512];
+ uint8_t buff[512];
+ int ret;
memset(buff, 0xff, 512);
- *((u32 *)&buff[0]) = pll->lock_cyc; /* PLL Lock cycle */
- *((u16 *)&buff[4]) = pll->feedback_div; /* PLL Feedback 9bit Divider */
+ /* PLL Lock cycle and Feedback 9bit Divider */
+ memcpy(buff, &pll->lock_cyc, sizeof(uint32_t));
+ memcpy(buff + 4, &pll->feedback_div, sizeof(uint16_t));
buff[6] = pll->input_div; /* PLL Input 5bit Divider */
buff[7] = pll->output_div; /* PLL Output Divider */
- if (mg_set_feature(mg_feature_id_transmode, mg_feature_val_trans_vcmd)
+ if ((ret = mg_set_feature(mg_feature_id_transmode, mg_feature_val_trans_vcmd))
!= ERROR_OK)
- return ERROR_FAIL;
+ return ret;
- if (mg_mflash_do_write_sects(buff, 0, 1, mg_vcmd_wr_pll)
+ if ((ret = mg_mflash_do_write_sects(buff, 0, 1, mg_vcmd_wr_pll))
!= ERROR_OK)
- return ERROR_FAIL;
+ return ret;
- if (mg_set_feature(mg_feature_id_transmode, mg_feature_val_trans_default)
+ if ((ret = mg_set_feature(mg_feature_id_transmode, mg_feature_val_trans_default))
!= ERROR_OK)
- return ERROR_FAIL;
+ return ret;
LOG_INFO("mflash: set pll ok");
- return ERROR_OK;
+ return ret;
}
static int mg_erase_nand(void)
{
- if (mg_set_feature(mg_feature_id_transmode, mg_feature_val_trans_vcmd)
+ int ret;
+
+ if ((ret = mg_set_feature(mg_feature_id_transmode, mg_feature_val_trans_vcmd))
!= ERROR_OK)
- return ERROR_FAIL;
+ return ret;
- if (mg_mflash_do_write_sects(NULL, 0, 0, mg_vcmd_purge_nand)
+ if ((ret = mg_mflash_do_write_sects(NULL, 0, 0, mg_vcmd_purge_nand))
!= ERROR_OK)
- return ERROR_FAIL;
+ return ret;
- if (mg_set_feature(mg_feature_id_transmode, mg_feature_val_trans_default)
+ if ((ret = mg_set_feature(mg_feature_id_transmode, mg_feature_val_trans_default))
!= ERROR_OK)
- return ERROR_FAIL;
+ return ret;
LOG_INFO("mflash: erase nand ok");
- return ERROR_OK;
+ return ret;
}
-int mg_config_cmd(struct command_context_s *cmd_ctx, char *cmd,
- char **args, int argc)
+COMMAND_HANDLER(mg_config_cmd)
{
double fin, fout;
mg_pll_t pll;
+ int ret;
- if (mg_verify_interface() != ERROR_OK)
- return ERROR_FAIL;
+ if ((ret = mg_verify_interface()) != ERROR_OK)
+ return ret;
- if (mg_mflash_rst() != ERROR_OK)
- return ERROR_FAIL;
+ if ((ret = mg_mflash_rst()) != ERROR_OK)
+ return ret;
- switch (argc) {
+ switch (CMD_ARGC) {
case 2:
- if (!strcmp(args[1], "boot")) {
- if (mg_boot_config() != ERROR_OK)
- return ERROR_FAIL;
-
- return ERROR_OK;
- } else if (!strcmp(args[1], "storage")) {
- if (mg_storage_config() != ERROR_OK)
- return ERROR_FAIL;
-
- return ERROR_OK;
- } else
+ if (!strcmp(CMD_ARGV[1], "boot"))
+ return mg_boot_config();
+ else if (!strcmp(CMD_ARGV[1], "storage"))
+ return mg_storage_config();
+ else
return ERROR_COMMAND_NOTFOUND;
break;
case 3:
- if (!strcmp(args[1], "pll")) {
- fin = strtoul(args[2], NULL, 0);
-
- if (fin > MG_PLL_CLK_OUT)
- return ERROR_FAIL;
+ if (!strcmp(CMD_ARGV[1], "pll")) {
+ unsigned long freq;
+ COMMAND_PARSE_NUMBER(ulong, CMD_ARGV[2], freq);
+ fin = freq;
+
+ if (fin > MG_PLL_CLK_OUT) {
+ LOG_ERROR("mflash: input freq. is too large");
+ return ERROR_MG_INVALID_OSC;
+ }
fout = mg_calc_pll(fin, &pll);
- if (!fout)
- return ERROR_FAIL;
+ if (!fout) {
+ LOG_ERROR("mflash: cannot generate valid pll");
+ return ERROR_MG_INVALID_PLL;
+ }
- LOG_INFO("mflash: Fout=%u Hz, feedback=%u,"
+ LOG_INFO("mflash: Fout=%" PRIu32 " Hz, feedback=%u,"
"indiv=%u, outdiv=%u, lock=%u",
- (u32)fout, pll.feedback_div,
+ (uint32_t)fout, pll.feedback_div,
pll.input_div, pll.output_div,
pll.lock_cyc);
- if (mg_erase_nand() != ERROR_OK)
- return ERROR_FAIL;
-
- if (mg_set_pll(&pll) != ERROR_OK)
- return ERROR_FAIL;
+ if ((ret = mg_erase_nand()) != ERROR_OK)
+ return ret;
- return ERROR_OK;
+ return mg_set_pll(&pll);
} else
return ERROR_COMMAND_NOTFOUND;
break;
default:
return ERROR_COMMAND_SYNTAX_ERROR;
}
+}
- return ERROR_OK;
+static const struct command_registration mflash_exec_command_handlers[] = {
+ {
+ .name = "probe",
+ .handler = &mg_probe_cmd,
+ .mode = COMMAND_EXEC,
+ .help = "Detect bank configuration information",
+ },
+ {
+ .name = "write",
+ .handler = &mg_write_cmd,
+ .mode = COMMAND_EXEC,
+ .usage = "<num> <file> <address>",
+ .help = "Write a file at the specified address",
+ },
+ {
+ .name = "dump",
+ .handler = &mg_dump_cmd,
+ .mode = COMMAND_EXEC,
+ .usage = "<num> <file> <address> <size>",
+ .help = "Dump to a file from the specified address",
+ },
+ {
+ .name = "config",
+ .handler = &mg_config_cmd,
+ .mode = COMMAND_EXEC,
+ .usage = "<num> <stage>",
+ .help = "Dump to a file from the specified address",
+ },
+ COMMAND_REGISTRATION_DONE
+};
+
+int mflash_init_drivers(struct command_context *cmd_ctx)
+{
+ if (!mflash_bank)
+ return ERROR_OK;
+ return register_commands(cmd_ctx, NULL, mflash_exec_command_handlers);
}
-int mflash_init_drivers(struct command_context_s *cmd_ctx)
+COMMAND_HANDLER(handle_mflash_init_command)
{
- if (mflash_bank) {
- register_command(cmd_ctx, mflash_cmd, "probe", mg_probe_cmd, COMMAND_EXEC, NULL);
- register_command(cmd_ctx, mflash_cmd, "write", mg_write_cmd, COMMAND_EXEC,
- "mflash write <num> <file> <address>");
- register_command(cmd_ctx, mflash_cmd, "dump", mg_dump_cmd, COMMAND_EXEC,
- "mflash dump <num> <file> <address> <size>");
- register_command(cmd_ctx, mflash_cmd, "config", mg_config_cmd,
- COMMAND_EXEC, "mflash config <num> <stage>");
+ if (CMD_ARGC != 0)
+ return ERROR_COMMAND_SYNTAX_ERROR;
+
+ static bool mflash_initialized = false;
+ if (mflash_initialized)
+ {
+ LOG_INFO("'mflash init' has already been called");
+ return ERROR_OK;
}
+ mflash_initialized = true;
- return ERROR_OK;
+ LOG_DEBUG("Initializing mflash devices...");
+ return mflash_init_drivers(CMD_CTX);
}
-static int mg_bank_cmd(struct command_context_s *cmd_ctx, char *cmd, char **args, int argc)
+COMMAND_HANDLER(mg_bank_cmd)
{
- target_t *target;
- char *str;
+ struct target *target;
int i;
- if (argc < 4)
+ if (CMD_ARGC < 4)
{
return ERROR_COMMAND_SYNTAX_ERROR;
}
- if ((target = get_target(args[3])) == NULL)
+ if ((target = get_target(CMD_ARGV[3])) == NULL)
{
- LOG_ERROR("target '%s' not defined", args[3]);
+ LOG_ERROR("target '%s' not defined", CMD_ARGV[3]);
return ERROR_FAIL;
}
- mflash_bank = calloc(sizeof(mflash_bank_t), 1);
- mflash_bank->base = strtoul(args[1], NULL, 0);
- mflash_bank->rst_pin.num = strtoul(args[2], &str, 0);
+ mflash_bank = calloc(sizeof(struct mflash_bank), 1);
+ COMMAND_PARSE_NUMBER(u32, CMD_ARGV[1], mflash_bank->base);
+ /// @todo Verify how this parsing should work, then document it.
+ char *str;
+ mflash_bank->rst_pin.num = strtoul(CMD_ARGV[2], &str, 0);
if (*str)
- mflash_bank->rst_pin.port[0] = (u16)tolower(str[0]);
+ mflash_bank->rst_pin.port[0] = (uint16_t)
+ tolower((unsigned)str[0]);
mflash_bank->target = target;
for (i = 0; mflash_gpio[i] ; i++) {
- if (! strcmp(mflash_gpio[i]->name, args[0])) {
+ if (! strcmp(mflash_gpio[i]->name, CMD_ARGV[0])) {
mflash_bank->gpio_drv = mflash_gpio[i];
}
}
if (! mflash_bank->gpio_drv) {
- LOG_ERROR("%s is unsupported soc", args[0]);
- return ERROR_INVALID_ARGUMENTS;
+ LOG_ERROR("%s is unsupported soc", CMD_ARGV[0]);
+ return ERROR_MG_UNSUPPORTED_SOC;
}
return ERROR_OK;
}
-int mflash_register_commands(struct command_context_s *cmd_ctx)
+static const struct command_registration mflash_config_command_handlers[] = {
+ {
+ .name = "bank",
+ .handler = &mg_bank_cmd,
+ .mode = COMMAND_CONFIG,
+ .help = "configure a mflash device bank",
+ .usage = "<soc> <base> <RST pin> <target #>",
+ },
+ {
+ .name = "init",
+ .mode = COMMAND_CONFIG,
+ .handler = &handle_mflash_init_command,
+ .help = "initialize mflash devices",
+ },
+ COMMAND_REGISTRATION_DONE
+};
+static const struct command_registration mflash_command_handler[] = {
+ {
+ .name = "mflash",
+ .mode = COMMAND_ANY,
+ .help = "mflash command group",
+ .chain = mflash_config_command_handlers,
+ },
+ COMMAND_REGISTRATION_DONE
+};
+int mflash_register_commands(struct command_context *cmd_ctx)
{
- mflash_cmd = register_command(cmd_ctx, NULL, "mflash", NULL, COMMAND_ANY, NULL);
- register_command(cmd_ctx, mflash_cmd, "bank", mg_bank_cmd, COMMAND_CONFIG,
- "mflash bank <soc> <base> <RST pin> <target #>");
- return ERROR_OK;
+ return register_commands(cmd_ctx, NULL, mflash_command_handler);
}