-
Notifications
You must be signed in to change notification settings - Fork 4.8k
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.
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
Remove L500 #12002
Remove L500 #12002
Conversation
Matching Win-Installer PR? |
Nope. Build succeeded. |
src/fw-update/fw-update-factory.cpp
Outdated
@@ -17,6 +15,13 @@ | |||
|
|||
namespace librealsense | |||
{ | |||
// We keep the L500 recovery identification to not confuse it with D400 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I vote to remove this too.
If I recall, the problem was that L515 With oud DFU on USB2 reported D400 recovery.
This mechanism protect flashing the D400 flash to it.
This is a very rare case and on this release we will even say we do not support L515..
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
But there is a chance that someone will brick their L515 in recovery mode.
If you're sure...
@@ -124,7 +123,6 @@ namespace librealsense | |||
{ | |||
camera_mesh.push_back(load_model(uncompress_d415_obj)); | |||
camera_mesh.push_back(load_model(uncompress_d435_obj)); | |||
camera_mesh.push_back(load_model(uncompress_L515_obj)); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
On this file please also change this:
if (starts_with(dev_name, "Intel RealSense L5")) index = 2;
if (starts_with(dev_name, "Intel RealSense D45")) index = 3;
to this:
if (starts_with(dev_name, "Intel RealSense D45")) index = 2;
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Right, fixed.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great!
Tracked on [LRS-615]