MEN-Mikro-Elektronik / 13MD05-90

MDIS5 System Package for Linux (including drivers)
Other
4 stars 4 forks source link

mdis_drivers_bbis_chameleon_com bugfix #229

Closed dpfeuffer closed 2 years ago

dpfeuffer commented 2 years ago

Note: This issue belongs to the internal problem report MAIN_PR007117 "15EM10Axx QDEC Hänger"

The following pull request shall be merged and tested: https://github.com/MEN-Mikro-Elektronik/mdis_drivers_bbis_chameleon_com/pull/3

This bugfix was necessary because of a customer problem under VxWorks. The fix was tested under VxWorks only but should be also required/usable under Linux. It is sufficient to perform the usual compilation/functional tests, there is no need for a special test to be carried out.

VictorParro commented 2 years ago

Please Dieter, can you confirm if running a Test Setup 2 is enough to validate this modification

dpfeuffer commented 2 years ago

It is sufficient when you run all the standard functional tests during the normal verification procedure of the next 13MD05-90_02_05 release. Test setup 2 is not suitable to test the chameleon BBIS driver, because the driver is not used. The chameleon BBIS handles chameleon FPGAs (e.g. at F215/G215 boards) but not M-Module carriers. My suggestion is to make a basic compilation test now. And a basic functional test with any test setup that uses the chameleon BBIS (if you have already one running). However, before the 02_05 release, all tests must be performed.

VictorParro commented 2 years ago

Thanks Dieter. It wasn't clear to me, which setup the BBIS Cahmeleon driver is used. To validate this modification, I will follow your suggestion.

GonzaloMartinR commented 2 years ago

Compilation and functional tests successfully performed to confirm the functionality of the changes.