Skip to content

RDKEMW-16814: reduce ble status prints and events #235

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
egalla204 wants to merge 7 commits into
base: develop
Choose a base branch
from
Open

RDKEMW-16814: reduce ble status prints and events #235

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
414b130
Release/1.1.12 (#200)
egalla204 Apr 9, 2026
02e0576
Merge branch 'release/1.1.12'
egalla204 Apr 9, 2026
6a7f4c3
RDKEMW-16814: reduce ble status prints and events
egalla204 May 14, 2026
57a79f5
apply some copilot review suggestions
egalla204 May 14, 2026
231d418
some more copilot comments
egalla204 May 15, 2026
22616f6
fix a comment
egalla204 May 18, 2026
05fb3fe
Merge branch 'develop' into RDKEMW-16814_reduce_ble_logging
egalla204 May 18, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
12 changes: 6 additions & 6 deletions src/attributes/ctrlm_attr_general.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -76,7 +76,7 @@ bool ctrlm_string_db_attr_t::read_db(ctrlm_db_ctx_t ctx) {
if(blob.read_db(ctx)) {
this->value = blob.to_string();
ret = true;
XLOGD_INFO("%s read from database: %s", this->get_name().c_str(), this->to_string().c_str());
XLOGD_DEBUG("%s read from database: %s", this->get_name().c_str(), this->to_string().c_str());
} else {
XLOGD_ERROR("failed to read from db <%s>", this->get_name().c_str());
}
Expand All @@ -90,7 +90,7 @@ bool ctrlm_string_db_attr_t::write_db(ctrlm_db_ctx_t ctx) {
if(blob.from_string(this->value)) {
if(blob.write_db(ctx)) {
ret = true;
XLOGD_INFO("%s written to database: %s", this->get_name().c_str(), this->to_string().c_str());
XLOGD_DEBUG("%s written to database: %s", this->get_name().c_str(), this->to_string().c_str());
} else {
XLOGD_ERROR("failed to write to db <%s>", this->get_name().c_str());
}
Expand Down Expand Up @@ -157,7 +157,7 @@ bool ctrlm_uint64_db_attr_t::read_db(ctrlm_db_ctx_t ctx) {
if(data.read_db(ctx)) {
this->set_value(data.get_uint64());
ret = true;
XLOGD_INFO("%s read from database: %s", this->get_name().c_str(), this->to_string().c_str());
XLOGD_DEBUG("%s read from database: %s", this->get_name().c_str(), this->to_string().c_str());
} else {
XLOGD_ERROR("failed to read from db <%s>", this->get_name().c_str());
}
Expand All @@ -169,7 +169,7 @@ bool ctrlm_uint64_db_attr_t::write_db(ctrlm_db_ctx_t ctx) {
ctrlm_db_uint64_t data(this->get_key(), this->get_table(), this->get_value());
if(data.write_db(ctx)) {
ret = true;
XLOGD_INFO("%s written to database: %s", this->get_name().c_str(), this->to_string().c_str());
XLOGD_DEBUG("%s written to database: %s", this->get_name().c_str(), this->to_string().c_str());
} else {
XLOGD_ERROR("failed to write to db <%s>", this->get_name().c_str());
}
Expand Down Expand Up @@ -237,7 +237,7 @@ bool ctrlm_ieee_db_addr_t::read_db(ctrlm_db_ctx_t ctx) {
if(data.read_db(ctx)) {
this->set_value(data.get_uint64());
ret = true;
XLOGD_INFO("%s read from database: %s", this->get_name().c_str(), this->to_string().c_str());
XLOGD_DEBUG("%s read from database: %s", this->get_name().c_str(), this->to_string().c_str());
} else {
XLOGD_ERROR("failed to read from db <%s>", this->get_name().c_str());
}
Expand All @@ -249,7 +249,7 @@ bool ctrlm_ieee_db_addr_t::write_db(ctrlm_db_ctx_t ctx) {
ctrlm_db_uint64_t data(this->get_key(), this->get_table(), this->get_value());
if(data.write_db(ctx)) {
ret = true;
XLOGD_INFO("%s written to database: %s", this->get_name().c_str(), this->to_string().c_str());
XLOGD_DEBUG("%s written to database: %s", this->get_name().c_str(), this->to_string().c_str());
} else {
XLOGD_ERROR("failed to write to db <%s>", this->get_name().c_str());
}
Expand Down
4 changes: 2 additions & 2 deletions src/attributes/ctrlm_attr_voice.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -256,7 +256,7 @@ bool ctrlm_voice_metrics_t::read_db(ctrlm_db_ctx_t ctx) {
this->utterances_exceeding_packet_loss_threshold_today = ((buf[39] << 24) | (buf[38] << 16) | (buf[37] << 8) | buf[36]);
this->utterances_exceeding_packet_loss_threshold_yesterday = ((buf[43] << 24) | (buf[42] << 16) | (buf[41] << 8) | buf[40]);
ret = true;
XLOGD_INFO("%s read from database: %s", this->get_name().c_str(), this->to_string().c_str());
XLOGD_DEBUG("%s read from database: %s", this->get_name().c_str(), this->to_string().c_str());
} else {
XLOGD_ERROR("data from db is too small <%s>", this->get_name().c_str());
}
Expand Down Expand Up @@ -322,7 +322,7 @@ bool ctrlm_voice_metrics_t::write_db(ctrlm_db_ctx_t ctx) {
if(blob.from_buffer(buf, sizeof(buf))) {
if(blob.write_db(ctx)) {
ret = true;
XLOGD_INFO("%s written to database: %s", this->get_name().c_str(), this->to_string().c_str());
XLOGD_DEBUG("%s written to database: %s", this->get_name().c_str(), this->to_string().c_str());
} else {
XLOGD_ERROR("failed to write to db <%s>", this->get_name().c_str());
}
Expand Down
4 changes: 2 additions & 2 deletions src/ble/ctrlm_ble_controller.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -58,15 +58,15 @@ ctrlm_obj_controller_ble_t::ctrlm_obj_controller_ble_t(ctrlm_controller_id_t con
wakeup_custom_list_(),
irdbs_supported_(0)
{
XLOGD_INFO("constructor - controller id <%u>", controller_id);
XLOGD_DEBUG("constructor - controller id <%u>", controller_id);

voice_metrics_->read_config();
ieee_address_->set_num_bytes(6);
last_key_time_->set_key("last_key_time"); //DB key needs to be updated for backwards compatibility
}

ctrlm_obj_controller_ble_t::ctrlm_obj_controller_ble_t() {
XLOGD_INFO("default constructor");
XLOGD_DEBUG("default constructor");
}

void ctrlm_obj_controller_ble_t::db_create() {
Expand Down
8 changes: 4 additions & 4 deletions src/ble/ctrlm_ble_controller_attr_version.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@ bool ctrlm_ble_sw_version_t::read_db(ctrlm_db_ctx_t ctx) {
if(blob.read_db(ctx)) {
if (this->from_string(blob.to_string())) {
ret = true;
XLOGD_INFO("%s read from database: %s", this->get_name().c_str(), this->to_string().c_str());
XLOGD_DEBUG("%s read from database: %s", this->get_name().c_str(), this->to_string().c_str());
} else {
XLOGD_ERROR("%s read from database failed to parse <%s> ", this->get_name().c_str(), blob.to_string().c_str());
}
Expand All @@ -58,7 +58,7 @@ bool ctrlm_ble_sw_version_t::write_db(ctrlm_db_ctx_t ctx) {
if(blob.from_string(ver_str)) {
if(blob.write_db(ctx)) {
ret = true;
XLOGD_INFO("%s written to database: %s", this->get_name().c_str(), this->to_string().c_str());
XLOGD_DEBUG("%s written to database: %s", this->get_name().c_str(), this->to_string().c_str());
} else {
XLOGD_ERROR("failed to write to db <%s>", this->get_name().c_str());
}
Expand Down Expand Up @@ -90,7 +90,7 @@ bool ctrlm_ble_hw_version_t::read_db(ctrlm_db_ctx_t ctx) {
if(blob.read_db(ctx)) {
if (this->from_string(blob.to_string())) {
ret = true;
XLOGD_INFO("%s read from database: %s", this->get_name().c_str(), this->to_string().c_str());
XLOGD_DEBUG("%s read from database: %s", this->get_name().c_str(), this->to_string().c_str());
} else {
XLOGD_ERROR("%s read from database failed to parse <%s> ", this->get_name().c_str(), blob.to_string().c_str());
}
Expand All @@ -108,7 +108,7 @@ bool ctrlm_ble_hw_version_t::write_db(ctrlm_db_ctx_t ctx) {
if(blob.from_string(ver_str)) {
if(blob.write_db(ctx)) {
ret = true;
XLOGD_INFO("%s written to database: %s", this->get_name().c_str(), this->to_string().c_str());
XLOGD_DEBUG("%s written to database: %s", this->get_name().c_str(), this->to_string().c_str());
} else {
XLOGD_ERROR("failed to write to db <%s>", this->get_name().c_str());
}
Expand Down
Loading
Loading