RDKEMW-14121 : Added Mediatek Platform support in Memcapture Code Base#13
RDKEMW-14121 : Added Mediatek Platform support in Memcapture Code Base#13nhanasi merged 5 commits intordkcentral:mainfrom
Conversation
There was a problem hiding this comment.
Pull request overview
Adds a new MEDIATEK platform option to MemCapture so the tool can be invoked/configured for MediaTek-based devices.
Changes:
- Extend CLI help text and argument parsing to accept
--platform MEDIATEK. - Add
Platform::MEDIATEKto the platform enum. - Add initial MEDIATEK handling in
MemoryMetric(memory bandwidth condition and fragmentation column count).
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
main.cpp |
Adds MEDIATEK to usage text and parses --platform MEDIATEK into Platform::MEDIATEK. |
Platform.h |
Extends the Platform enum with MEDIATEK. |
MemoryMetric.cpp |
Attempts to include MEDIATEK in some platform-specific metric logic (bandwidth/fragmentation). |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Reason for change: Added MEDIATEK Platform support Test Procedure: please refer the ticket comments Risks: Medium Signed-off-by: Thamim Razith Abbas Ali <tabbas651@cable.comcast.com>
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 4 out of 4 changed files in this pull request and generated 3 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
meditek platform and added cma regions map and added getmemorybandwidth should be false in meditek
nhanasi
left a comment
nhanasi
left a comment
There was a problem hiding this comment.
Please raise review from feature branch to make official release as this method lands the code in the main branch
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 4 out of 4 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Reason for change: Added MEDIATEK Platform support
Test Procedure: please refer the ticket comments
Risks: Medium