aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAnton Babushkin <anton.babushkin@me.com>2014-01-19 22:56:57 +0100
committerAnton Babushkin <anton.babushkin@me.com>2014-01-19 22:56:57 +0100
commit47c226988ccf0e90bda9fe7c106bcdaf8b2e67fd (patch)
treee9a102f4de8a5ba39a64f5d1d4af772f49a30481
parent5e3c365cd4809def0c5b21136a1b5741a98ae35e (diff)
downloadpx4-firmware-47c226988ccf0e90bda9fe7c106bcdaf8b2e67fd.tar.gz
px4-firmware-47c226988ccf0e90bda9fe7c106bcdaf8b2e67fd.tar.bz2
px4-firmware-47c226988ccf0e90bda9fe7c106bcdaf8b2e67fd.zip
sdlog2: code style fixed
-rw-r--r--src/modules/sdlog2/sdlog2.c17
1 files changed, 15 insertions, 2 deletions
diff --git a/src/modules/sdlog2/sdlog2.c b/src/modules/sdlog2/sdlog2.c
index 656575573..2319dbb3b 100644
--- a/src/modules/sdlog2/sdlog2.c
+++ b/src/modules/sdlog2/sdlog2.c
@@ -301,12 +301,15 @@ int create_log_dir()
int n = snprintf(log_dir, sizeof(log_dir), "%s/", log_root);
strftime(log_dir + n, sizeof(log_dir) - n, "%Y-%m-%d", &t);
mkdir_ret = mkdir(log_dir, S_IRWXU | S_IRWXG | S_IRWXO);
+
if (mkdir_ret == OK) {
warnx("log dir created: %s", log_dir);
+
} else if (errno != EEXIST) {
warn("failed creating new dir: %s", log_dir);
return -1;
}
+
} else {
/* look for the next dir that does not exist */
while (dir_number <= MAX_NO_LOGFOLDER) {
@@ -317,10 +320,12 @@ int create_log_dir()
if (mkdir_ret == 0) {
warnx("log dir created: %s", log_dir);
break;
+
} else if (errno != EEXIST) {
warn("failed creating new dir: %s", log_dir);
return -1;
}
+
/* dir exists already */
dir_number++;
continue;
@@ -352,8 +357,10 @@ int open_log_file()
gmtime_r(&gps_time_sec, &t);
strftime(log_file_name, sizeof(log_file_name), "%H_%M_%S.bin", &t);
snprintf(log_file_path, sizeof(log_file_path), "%s/%s", log_dir, log_file_name);
+
} else {
uint16_t file_number = 1; // start with file log001
+
/* look for the next file that does not exist */
while (file_number <= MAX_NO_LOGFILE) {
/* format log file path: e.g. /fs/microsd/sess001/log001.bin */
@@ -363,6 +370,7 @@ int open_log_file()
if (!file_exist(log_file_path)) {
break;
}
+
file_number++;
}
@@ -378,6 +386,7 @@ int open_log_file()
if (fd < 0) {
warn("failed opening log: %s", log_file_name);
mavlink_log_info(mavlink_fd, "[sdlog2] failed opening log: %s", log_file_name);
+
} else {
warnx("log file: %s", log_file_name);
mavlink_log_info(mavlink_fd, "[sdlog2] log file: %s", log_file_name);
@@ -605,8 +614,8 @@ int write_parameters(int fd)
}
case PARAM_TYPE_FLOAT:
- param_get(param, &value);
- break;
+ param_get(param, &value);
+ break;
default:
break;
@@ -700,6 +709,7 @@ int sdlog2_thread_main(int argc, char *argv[])
/* create log root dir */
int mkdir_ret = mkdir(log_root, S_IRWXU | S_IRWXG | S_IRWXO);
+
if (mkdir_ret != 0 && errno != EEXIST) {
err("failed creating log root dir: %s", log_root);
}
@@ -708,9 +718,11 @@ int sdlog2_thread_main(int argc, char *argv[])
const char *converter_in = "/etc/logging/conv.zip";
char *converter_out = malloc(64);
snprintf(converter_out, 64, "%s/conv.zip", log_root);
+
if (file_copy(converter_in, converter_out) != OK) {
warn("unable to copy conversion scripts");
}
+
free(converter_out);
/* initialize log buffer with specified size */
@@ -969,6 +981,7 @@ int sdlog2_thread_main(int argc, char *argv[])
gps_time = buf.gps_pos.time_gps_usec;
}
}
+
sdlog2_start_log();
}