Fix the cli error messages to make it clearer when you need to provide more CSVs

enjoy-digital / litescope

Small footprint and configurable embedded FPGA logic analyzer

Other

167 stars 40 forks source link

Fix the cli error messages to make it clearer when you need to provide more CSVs #31

Closed sthornington closed 3 years ago

sthornington commented 3 years ago

The SoC CSR CSV is in fact required, but was defaulting to csr.csv. This was made more confusing by the error message which was complaining about an absence of regs.

Also, fix the trigger value parsing for hex values.

sthornington commented 3 years ago

I will have to look into why the CI test failed later...

sthornington commented 3 years ago

I will have to look into why the CI test failed later...

I'm actually not even sure how my changes to the CLI and analyzer could have caused a build failure...

enjoy-digital commented 3 years ago

Thanks @sthornington, sorry for the delay. This is merged.