Skip to content
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

Jump to bottom

Fw logs with travis #6666

Merged
merged 44 commits into from
Jun 23, 2020
Merged

Fw logs with travis #6666

merged 44 commits into from
Jun 23, 2020

Conversation

remibettan
Copy link
Contributor

@remibettan remibettan commented Jun 22, 2020 •
edited
Loading

This PR includes FW logs and flash logs support via libRS API, for all devices (D4XX, L5XX and SR3XX).
Triggered by Jira ticket: DSO-14959

remibettan added 30 commits June 18, 2020 12:24
fw_logger class started
e21bdcc
API and example added - FW logs received without any filter and witho…
783fd9a 
…ut xml translation
FW logs parsed, using new line as delimiter
39326f0
Design change - fw logs returned to user as structure, even without p…
367cb72 
…arser
FW logs parsing works
7ead7a1
FW logs corrected to work with C API - remaining work for the parsing…
a599428 
… to be more user-friendly
API change - Getting logs one by one - parser not operable
e3b6c97
get_severity added, firmware_log_message struct improved
f09c831
fw logs messages parsed one by one, API added for timestamp, line, se…
8e1e606 
…verity, message, file name and thread name
fw log message provided as input to get firmware logs
5c651e9
Flash logs work - work to do in fw logger device - see comment
cc747b2
fw logs and flash logs implemented
dd69928
Parser made member of fw_logger, need to solve case for parsing witho…
2586328 
…ut init
rs-fw-logger tool using new API
c2f47de
fw logs api functions moved to dedicated header files
91e53b5
parse_log api change so that boolean value would be returned
0a69827
correct flash log example - to be deleted ater on
c8bbb53
flash logs pulling coorected to be same as fw logs
8588721
fw logger extension added to the ds5 devices, opcode removed from ctor
a8b9b7f
xml content sent to api instead of xml path
0b14689
modify fw-logger tool so that it could pull also flash logs
0a03822
fw logs code corrections
6fc4ee7
pids map added to fw logger, sr300 missing
08be3e7
removing flash logs example as it is already in rs-fw-logger tool
8f037b1
adding fw logs support for sr300
45ef2ff
removing not needed examples
a8d11e8
removing not needed unit tests
939d364
api functions names got shorter - fw instead of firmware
cc9bfa7
fw-logger tool readme updated, fw_logger_device improved
8b858ba
new line added at end of files
7c835f7
@ev-mp ev-mp mentioned this pull request Jun 22, 2020
Closed
@ev-mp ev-mp requested a review from dorodnic June 23, 2020 07:13
ev-mp
ev-mp approved these changes Jun 23, 2020
View reviewed changes
Copy link
Collaborator

@ev-mp ev-mp left a comment

Choose a reason for hiding this comment

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

Approved. Please open task to propagate the APIs to wrappers

@ev-mp ev-mp merged commit b70d087 into IntelRealSense:development Jun 23, 2020
@ev-mp
Temporal work-around for HW Monitor injection for SR300
5c27565 
Change-Id: I427e31accda8f88bd17b0da5fbd130998dce8133
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ev-mp ev-mp ev-mp approved these changes

@dorodnic dorodnic Awaiting requested review from dorodnic

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@remibettan @ev-mp