Skip to content

RDKBACCL-1518 : Phase-I Enabling Remote-debugger in BPI-R4 reference platform#412

Open
SsandhyaR wants to merge 1 commit intordkcentral:developfrom
SsandhyaR:Remote-debugger
Open

RDKBACCL-1518 : Phase-I Enabling Remote-debugger in BPI-R4 reference platform#412
SsandhyaR wants to merge 1 commit intordkcentral:developfrom
SsandhyaR:Remote-debugger

Conversation

@SsandhyaR
Copy link
Contributor

Reason for change : Adding the code changes for Remote-debugger to work with static and dynamic json
Risks: Low
Test Procedure: Bulid the image, and test with static and dynamic json files using Remote controllers

@SsandhyaR SsandhyaR marked this pull request as draft March 5, 2026 06:58
@rdkcmf-jenkins
Copy link
Contributor

build-bpi-r4-broadband Console Log uploaded to Gist URL: https://gist.github.com/rdkcmf-jenkins/a56ab31d0a9869cde6f12cb16b7afc87

@SsandhyaR SsandhyaR marked this pull request as ready for review March 5, 2026 12:45
@rdkcmf-jenkins
Copy link
Contributor

build-bpi-r4-broadband Console Log uploaded to Gist URL: https://gist.github.com/rdkcmf-jenkins/a7411e6e977df0148e611309fcd8aa5b

@@ -0,0 +1,6 @@
FILESEXTRAPATHS:prepend := "${THISDIR}/files:"
SRC_URI_append = " file://rdkmMgr_dwutils.patch"
FILES_${PN}_remove = "${libdir}/librdmopenssl.la"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this line can be last line.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

okay Mani
Addressed

touch ${_TOPDIR}/meta-filogic/recipes-wifi/hal/mt76_hal_change
fi

if [ ! -e ${_TOPDIR}/meta-rdk/recipes-common/rdm-agent/rdm_hal_change ]; then
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Instead, is this okay to upstream the changes?

Copy link
Contributor Author

@SsandhyaR SsandhyaR Mar 5, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah Mani, will do upstream
But rdm-agent latest source code is depends on updated with rdk-logger latest code
If we do upstream, build breaks for other source components since we need to update the rdk-logger srcrev too for rdm-agent

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we need to align this once we standardize meta-rdk

…platform

Reason for change : Adding the code changes for Remote-debugger to work with static and dynamic json
Risks: Low
Test Procedure: Bulid the image, and test with static and dynamic json files using Remote controllers

Signed-off-by: ssiras826 <[email protected]>
@rdkcmf-jenkins
Copy link
Contributor

build-bpi-r4-broadband Console Log uploaded to Gist URL: https://gist.github.com/rdkcmf-jenkins/831941c30f210e3798793ff4d21e9ea8

Copy link
Contributor

@cpokuru cpokuru left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

will take it further once gerrit changes merged

@@ -0,0 +1,43 @@
diff --git a/src/rdm_downloadutils.c b/src/rdm_downloadutils.c
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we need to upstream this once we align with github meta-rdk

touch ${_TOPDIR}/meta-filogic/recipes-wifi/hal/mt76_hal_change
fi

if [ ! -e ${_TOPDIR}/meta-rdk/recipes-common/rdm-agent/rdm_hal_change ]; then
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we need to align this once we standardize meta-rdk

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants