include: install compress_ops.h as public header#30
include: install compress_ops.h as public header#30mpratyus wants to merge 1 commit intoalsa-project:masterfrom
Conversation
Move tinycompress/compress_ops.h from noinst_HEADERS to nobase_include_HEADERS so it gets installed for users. Signed-off-by: Pratyush Meduri <mpratyus@qti.qualcomm.com>
|
thanks for the change. |
|
The motivation for this change is to make compress_ops.h available to external users and plugins that need to implement or register compress offload backends, rather than using tinycompress purely as a client library. |
Link: alsa-project#30 Fixes: 72dd4bb ("compress_hw_get_hpointer: use SNDRV_COMPRESS_AVAIL64") Signed-off-by: Jaroslav Kysela <perex@perex.cz>
We need more validation of passed ops structure to allow further API changes in future. This change looks for compress_plugin_mops symbol in the dynamic plugin library to make sure that the new magic and new get_tstamp64 members are handled correctly. Link: alsa-project#29 Link: alsa-project#30 Fixes: 0bd5530 ("compress_ops: add get_tstamp64") Signed-off-by: Jaroslav Kysela <perex@perex.cz>
|
I put some ideas to improve ops API to #31 before we make this header public. And yes, there is populate_compress_plugin_ops() function which tries to load and use external plugins on demand. |
3 participants
Move tinycompress/compress_ops.h from noinst_HEADERS to nobase_include_HEADERS so it gets installed for users.