velodyne icon indicating copy to clipboard operation
velodyne copied to clipboard

make driver portable to big-endian machines

Open jack-oquin opened this issue 9 years ago • 2 comments

There are several places, mostly in unpacking raw data, where little-endian data access is assumed.

Eliminating those dependencies does not look difficult, but I only plan to do that work if someone with a big-endian platform actually requests it. Otherwise, it would not get adequate testing, so why bother?

jack-oquin avatar Nov 01 '15 19:11 jack-oquin

We plan to test a VLP16 under an ARM platform (Debian armhf). I think most ARM platforms are already little endian nowadays, so the extra work should not be justified. In any case, what are the chances of using a Velodyne sensor in a big endian system? How many people use a MIPS?

BTW: we could provide feedback on armhf if that is helpful.

jlblancoc avatar Nov 03 '15 21:11 jlblancoc

Feedback on ARM would be great. Please check whether it is actually little-endian, apparently that is implementation-dependent.

It should be relatively simple to make it portable to big-endian, as long as we have a reasonable test platform for it.

jack-oquin avatar Nov 03 '15 23:11 jack-oquin

Seeing as several places I have worked have used this driver without issue on ARMv8 machines, I don't think this requires any changes. Closing.

JWhitleyWork avatar Dec 04 '22 23:12 JWhitleyWork