Libargus - Multiple CSI cameras still show same module string

[eh-steve]

Just bumping this issue as it’s still a problem in JP5.1

Multiple CSI cameras show same module string

Hi, I’m using a Xavier NX production module with JetPack 4.6, with 3 IMX219 cameras connected via a leeptop A205 carrier board; with the device-tree updated to be correct for this carrier board. All three cameras work fine using “nvgstcapture” with “–sensor_id” set to 0, 1 or 2. I’m now developing C++ code to access the cameras with LibArgus. The following code is meant to simply enumerate the cameras, and show the module-string for each camera: #include <vector> #include <stdio.h> #include…

While someone is looking at this, the Argus.h types specify a MAX_UUID_NAME_SIZE of 32 bytes but the sensor placement strings are longer than this. Please could someone increase the limit or decrease the length of the names?

[ShaneCCC]

@eh-steve
J5.1.1 should have the fixed.

Thanks

[eh-steve]

The moduleString bug is indeed fixed in 5.1.1, but

const uint32_t MAX_UUID_NAME_SIZE = 32;

is still present in Argus/UUID.h, while the UUIDs in Argus/Types.h have names which are longer than 32 bytes.

[ShaneCCC]

Suppose yes.
Please follow the MAX_UUID_NAME_SIZE

[eh-steve]

Is this aimed at me? The libargus headers maintained by nvidia need to be updated?

[ShaneCCC]

I mean the size is enough. Why do you need longer strings?

[eh-steve]

These UUID definitions are in the file supplied by nvidia (in file /usr/src/jetson_multimedia_api/include/Argus/Types.h):

DEFINE_UUID(SensorPlacement, SENSOR_PLACEMENT_REAR_OR_BOTTOM_OR_BOTTOM_LEFT, 01dba8b0,1946,11eb,8b6f,08,00,20,0c,9a,66);
DEFINE_UUID(SensorPlacement, SENSOR_PLACEMENT_FRONT_OR_TOP_OR_CENTER_LEFT,   01dba8b1,1946,11eb,8b6f,08,00,20,0c,9a,66);
DEFINE_UUID(SensorPlacement, SENSOR_PLACEMENT_CENTER_OR_CENTER_RIGHT,        01dba8b2,1946,11eb,8b6f,08,00,20,0c,9a,66);
DEFINE_UUID(SensorPlacement, SENSOR_PLACEMENT_TOP_LEFT,                      01dba8b3,1946,11eb,8b6f,08,00,20,0c,9a,66);
DEFINE_UUID(SensorPlacement, SENSOR_PLACEMENT_BOTTOM_RIGHT,                  01dba8b4,1946,11eb,8b6f,08,00,20,0c,9a,66);
DEFINE_UUID(SensorPlacement, SENSOR_PLACEMENT_TOP_RIGHT,                     01dba8b5,1946,11eb,8b6f,08,00,20,0c,9a,66);
DEFINE_UUID(SensorPlacement, SENSOR_PLACEMENT_UNKNOWN,                       01dba8b6,1946,11eb,8b6f,08,00,20,0c,9a,66);

Clearly SENSOR_PLACEMENT_REAR_OR_BOTTOM_OR_BOTTOM_LEFT is 46 characters which is longer than 32. This means that when you print this enum using getName(), it will be truncated to 32 characters (SENSOR_PLACEMENT_REAR_OR_BOTTOM_) which I doubt is the desired behaviour?

[ShaneCCC]

The moduleString bug is fixed any reason to use getName()?