velodyne
velodyne copied to clipboard
make driver portable to big-endian machines
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?
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.
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.
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.