Skip to content

Commit

Permalink
pcm: implement snd_pcm_hw_params_get_sync() and obsolete snd_pcm_info…
Browse files Browse the repository at this point in the history
…_get_sync()

Use the new clock source mechanism to get information about
similar PCM clock sources for PCM streams.

Link: https://lore.kernel.org/linux-sound/[email protected]/
Signed-off-by: Jaroslav Kysela <[email protected]>
  • Loading branch information
perexg committed Aug 20, 2024
1 parent 9b6dfb3 commit b154d91
Show file tree
Hide file tree
Showing 3 changed files with 32 additions and 5 deletions.
4 changes: 4 additions & 0 deletions include/pcm.h
Original file line number Diff line number Diff line change
Expand Up @@ -508,6 +508,9 @@ typedef union _snd_pcm_sync_id {
unsigned int id32[4];
} snd_pcm_sync_id_t;

/** synchronization ID size (see snd_pcm_hw_params_get_sync) */
#define SND_PCM_HW_PARAMS_SYNC_SIZE 16

/** Infinite wait for snd_pcm_wait() */
#define SND_PCM_WAIT_INFINITE (-1)
/** Wait for next i/o in snd_pcm_wait() */
Expand Down Expand Up @@ -747,6 +750,7 @@ int snd_pcm_hw_params_get_rate_numden(const snd_pcm_hw_params_t *params,
unsigned int *rate_den);
int snd_pcm_hw_params_get_sbits(const snd_pcm_hw_params_t *params);
int snd_pcm_hw_params_get_fifo_size(const snd_pcm_hw_params_t *params);
const unsigned char * snd_pcm_hw_params_get_sync(const snd_pcm_hw_params_t *params);

#if 0
typedef struct _snd_pcm_hw_strategy snd_pcm_hw_strategy_t;
Expand Down
3 changes: 2 additions & 1 deletion include/sound/uapi/asound.h
Original file line number Diff line number Diff line change
Expand Up @@ -426,7 +426,8 @@ struct snd_pcm_hw_params {
unsigned int rate_num; /* R: rate numerator */
unsigned int rate_den; /* R: rate denominator */
snd_pcm_uframes_t fifo_size; /* R: chip FIFO size in frames */
unsigned char reserved[64]; /* reserved for future */
unsigned char sync[16]; /* R: synchronization ID (perfect sync - one clock source) */
unsigned char reserved[48]; /* reserved for future */
};

enum {
Expand Down
30 changes: 26 additions & 4 deletions src/pcm/pcm.c
Original file line number Diff line number Diff line change
Expand Up @@ -494,8 +494,8 @@ software parameter.
There are two functions allowing link multiple streams together. In the
case, the linking means that all operations are synchronized. Because the
drivers cannot guarantee the synchronization (sample resolution) on hardware
lacking this feature, the #snd_pcm_info_get_sync() function
returns synchronization ID - #snd_pcm_sync_id_t, which is equal
lacking this feature, the #snd_pcm_hw_params_get_sync() function
returns 16-byte synchronization ID, which is equal
for hardware synchronized streams. When the #snd_pcm_link()
function is called, all operations managing the stream state for these two
streams are joined. The opposite function is #snd_pcm_unlink().
Expand Down Expand Up @@ -3948,6 +3948,25 @@ int snd_pcm_hw_params_get_fifo_size(const snd_pcm_hw_params_t *params)
return params->fifo_size;
}

/**
* \brief Get hardware synchronization ID from a PCM info container
* \param params Configuration space
* \return 16-byte synchronization ID (use #SND_PCM_HW_PARAMS_SYNC_SIZE)
*
* This synchronization ID determines the similar clocks for the
* PCM stream between multiple devices (including different cards).
* "All zeros" means "not set". The contents of the ID can be used
* only for a comparison with the contents of another ID returned
* from this function. Applications should not do a comparison with
* hard-coded values, because the implementation generating such
* synchronization IDs may be changed in future.
*/
const unsigned char *snd_pcm_hw_params_get_sync(const snd_pcm_hw_params_t *params)
{
assert(params);
return params->sync;
}

/**
* \brief Fill params with a full configuration space for a PCM
* \param pcm PCM handle
Expand Down Expand Up @@ -7332,17 +7351,20 @@ unsigned int snd_pcm_info_get_subdevices_avail(const snd_pcm_info_t *obj)
}

/**
* \brief Get hardware synchronization ID from a PCM info container
* \brief (DEPRECATED) Get hardware synchronization ID from a PCM info container
* \param obj PCM info container
* \return hardware synchronization ID
*/
snd_pcm_sync_id_t snd_pcm_info_get_sync(const snd_pcm_info_t *obj)
{
snd_pcm_sync_id_t res;
assert(obj);
memcpy(&res, &obj->sync, sizeof(res));
bzero(&res, sizeof(res));
return res;
}
#ifndef DOC_HIDDEN
link_warning(snd_pcm_info_get_sync, "Warning: snd_pcm_info_get_sync is deprecated, consider to use snd_pcm_hw_params_get_sync");
#endif

/**
* \brief Set wanted device inside a PCM info container (see #snd_ctl_pcm_info)
Expand Down

0 comments on commit b154d91

Please sign in to comment.