-
Notifications
You must be signed in to change notification settings - Fork 13.7k
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
Lidar Lite V3 newer than 2016 is not recognized by Pixhawk 2.1 #12278
Comments
@ar1040 , I will put some time into this issue in the next week or so and report back here as I make progress. |
@ar1040, my hardware is a V3HP and fortunately/unfortunately it works as expected with current PX4:master firmware flashed to a pixhawk 4 (fmu-v5). Where did you obtain your newer units and are they the V3 or V3HP? |
@mcsauder, my hardware is a V3. I bought these units from robotshop.com, but they are original Garmins. I can use the PWM protocol and it works, but it only reads 12 meters max, versus 17 meters using I2C so I would prefer to have the extra 5 meters of distance. I think it is also worth mentioning I am using a Pixhawk Cube and not the Pixhawk 4. I am also using the 1.8.2 f13bbac firmware version. I would like to swap to the 1.9.2 firmware but there are still issues with takeoff and attitude that conflicts with my software, but this is a different issue in itself. |
Hi @ar1040 . This might be very difficult to resolve if we cannot move you to the current release firmware; there are many bugfixes and code changes in the past year that might complicate resolving the issue or render it irrelevant. Could you raise a new issue to deal with the reason you cannot update to new firmware? |
@mcsauder I would like to use the newest firmware version due to the other fixes. I will open a new issue regarding the firmware issues. If the changes for the Lidar are pushed to the latest firmware version, it would still be very helpful, as I plan to switch our units to the latest firmware. |
@ar1040 , I've flashed v1.8.2 and and see an issue that might be the same as yours with my fmu-v5 hardware, the ll40ls i2c driver fails to initialize properly in the So, I think you have two options: 1) Upgrade to v1.9.2, or 2) backport the current driver I have backported to v1.8.2 by copying the current PX4:master branch You'll need to add these #defines to
and this variable in the class:
and copy/paste the current As this appears to not be an issue with v1.9.2 and newer code, we can probably close out this issue. Let us know your thoughts. |
@mcsauder , I updated another one of my Pixhawk Cubes to v1.9.2 but I still cannot get a distance sensor reading. I normally test by opening QGC and clicking the widgets tab on the top left, then analyze. I then type "distance" in the search bar of the analyze window. If the sensor is recognized, it gives me 8 (I think) data sets to view. I choose the distance_sensor_current data set and view it to make sure it is reading the distance correctly. However, even after I updated to v1.9.2 I could not get my Lidar Lite V3 to register. |
@ar1040 , can you test again with today's current PX4:master branch? |
@mcsauder , I pulled the latest master via QGC (dropdown and selected master). The firmware shows up as 1.9.0dev. I am not sure if I am pulling the correct firmware version. But I tested it anyway. Still, only my 2016 lidar works and none of the 2019s work with I2C (I tried 5 that I had on hand). If I am not testing the correct firmware please let me know. |
Hello, I am having the same issue (also with the master branch). One thing I am noticing is that if I boot using an older Lidar I see the data measurements, and if I leave the drone on and connect the newer lidar it works as well. However, when I do a boot using the newer lidar it does not work. |
@ar1040 ,
I've tried mine on the bench connected to I2C A and it starts correctly at boot with |
I am using the only I2C port available on the Pixhawk 2.1 Cube that I am aware of. |
This is actually common problem as I am also met with this. The follows is what Garmin replied in email: Is this solved yet? |
I'm refactoring the driver to try to solve this... Once I get a PR in, would you all help me test? |
No, this isn't solved yet. Please feel free to submit a PR or see also #12695 |
Hi @HenryHuYu and @ar1040 , would you give current master branch a run and see if this issue is solved? Thanks! |
I think this is fixed now.... or at least I hope it is. Please re-open if it is still an issue! |
@mcsauder, I apologize for the late response. I did pull the master and I am still having the same problem as I can not get data from the lidar unless I start it with the "ll40ls start" command in the mavlink console. |
This issue has been automatically marked as stale because it has not had recent activity. Thank you for your contributions. |
Looks like a potential fix has been merged. Closing, please re-open if the issue persists. |
it seems my lidarlite has some problem, even in arduino it only gets nack message, not the same issue |
The Lidar Lite V3 rangefinder is an optional rangefinder to be used with the Pixhawk 2.1. It can be enabled by the parameter EKF2_RNG_AID=1 and the Lidar Lite V3 can be initialized by the parameter SENS_EN_LL40LS. I use the I2C protocol.
I currently have 3-4 Lidar Lite V3's that are dated 2016 on the label. These Lidars work every time. The tests I have ran:
The 2016 Lidars will pass all of the tests described above. The 2018-2019 Lidars will only pass the arduino test.
I decided to go straight to the source by contacting Garmin. Garmin's response was, "The newer units use a newer hardware design that uses a faster FPGA. The intent was for it to function as a drop-in replacement for the original, but there are some differences. We have made updates to the https://github.com/garmin/LIDARLite_Arduino_Library that are intended to account for the differences."
Also I have attached what Garmin sent me about the change.
LLv3-I2C-Guidelines.pdf
Do you guys plan to support this new change? When could I expect the change to be implemented if so? Is there anything I can do in the meantime to fix this? Any and all help is appreciated.
The text was updated successfully, but these errors were encountered: