#56 closed defect (fixed)
Driver load displays ALSA_VERSION as the Uniaud version
| Reported by: | Brendan Oakley | Owned by: | Yoda |
|---|---|---|---|
| Priority: | normal | Milestone: | |
| Component: | UNIAUD32 | Version: | |
| Severity: | normal | Keywords: | |
| Cc: |
Description
ALSA_VERSION, in include/version.h, should contain the version of ALSA being used for the build, but the driver displays this when it loads the driver, so it instead contains the version of Uniaud. The output needs to be changed to report PRODUCT_VERSION for the Uniaud version instead. Displaying of ALSA_VERSION is optional, but if it is to be displayed at all, the text needs to clearly indicate that it is the ALSA version, not the Uniaud version, and the Uniaud version is not to be omitted.
The place in the code for the ALSA version could instead be alsa-kernel/include/sound/version.h, because this is where ALSA keeps it, so a full update of the ALSA code would necessarily update it there.
Note:
See TracTickets
for help on using tickets.
I remembered wrong about alsa-kernel/include/sound/version.h. That is not where ALSA keeps it. Best to leave it set in include/version.h.