Max has submitted this change and it was merged. Change subject: lc15-mgr: update parameter read/write ......................................................................
lc15-mgr: update parameter read/write Better match upstream lc15 code: * fsync on parameter write * use permanent storage directly for parameters Change-Id: Iac150bf2ebffc5fa9544b0f3b19cd647996fc8b6 Related: SYS#3728 --- M src/osmo-bts-litecell15/misc/lc15bts_par.c M src/osmo-bts-litecell15/misc/lc15bts_par.h 2 files changed, 4 insertions(+), 1 deletion(-) Approvals: Harald Welte: Looks good to me, approved Jenkins Builder: Verified diff --git a/src/osmo-bts-litecell15/misc/lc15bts_par.c b/src/osmo-bts-litecell15/misc/lc15bts_par.c index ab3e96b..13b0080 100644 --- a/src/osmo-bts-litecell15/misc/lc15bts_par.c +++ b/src/osmo-bts-litecell15/misc/lc15bts_par.c @@ -130,6 +130,8 @@ fclose(fp); return -EIO; } + + fsync(fp); fclose(fp); return 0; } @@ -161,6 +163,7 @@ rc = fwrite(buf, 1, size, fp); + fsync(fp); fclose(fp); return rc; diff --git a/src/osmo-bts-litecell15/misc/lc15bts_par.h b/src/osmo-bts-litecell15/misc/lc15bts_par.h index b9fe740..dd869d3 100644 --- a/src/osmo-bts-litecell15/misc/lc15bts_par.h +++ b/src/osmo-bts-litecell15/misc/lc15bts_par.h @@ -4,7 +4,7 @@ #include <osmocom/core/utils.h> #define FACTORY_ROM_PATH "/mnt/rom/factory" -#define USER_ROM_PATH "/mnt/rom/user" +#define USER_ROM_PATH "/mnt/storage/var/run/lc15bts-mgr" enum lc15bts_par { LC15BTS_PAR_TEMP_SUPPLY_MAX, -- To view, visit https://gerrit.osmocom.org/3272 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iac150bf2ebffc5fa9544b0f3b19cd647996fc8b6 Gerrit-PatchSet: 3 Gerrit-Project: osmo-bts Gerrit-Branch: master Gerrit-Owner: Max <msur...@sysmocom.de> Gerrit-Reviewer: Harald Welte <lafo...@gnumonks.org> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: Max <msur...@sysmocom.de> Gerrit-Reviewer: Minh-Quang Nguyen <minh-quang.ngu...@nutaq.com>