Issues trying to read descriptor with more than one configuration

[carpenterme]

Hi, this is my first GitHub issue post, so bear with me if this hard to read. My coworker and I are developing a AVB Manager GUI in C# visual studio using the open source library. However, we have noticed an issue reading in configurations from an entity using avdecccmdline. We believe that the READ_DESCRIPTOR command sent to the Entity is only reading the first configuration. This is a problem as we have multiple configurations on our entities. I've included some WireShark captures of a successful read_descriptor command with the configuration set to 0, and also have included a unsuccessful read_descriptor command capture when the current configuration was set to 1. Also, some screencaps in the case you don't have WireShark.

The problem in this one is that the configuration in the bad read should be 1 as that is the current configuration, but it is 0 in this case. This goes to show, once again, that the program is only picking up the first configuration. This is presenting some major delay in the development of our project,

Any insight or help is appreciated.

WireSharkCaptures.zip

[andrew-elder]

There could well be issues with either the command line application or the library itself when reading entities with multiple configurations. At the time of development there were no multi-configuration entities to test against. The intent was to design the library to support multiple configurations, but with nothing to test against, the end result is not guaranteed. I assume you have performed command line option to select which configuration is the current one?

[carpenterme]

I've tried using the command line option to change the configuration, but when I have tried changing the configuration it says it is an invalid new setting.

I have been successful changing the configurations using Riedel's AVB Manager though. That's what I've been using to change the configurations of the entities.

[andrew-elder]

You can see here https://github.com/audioscience/avdecc-lib/blob/staging/controller/lib/src/entity_descriptor_imp.cpp#L141 that the set configuration command is not implemented. I'd be happy to accept a patch that implements it.