On Wed, 01 Apr 2020 17:25:38 +0200, sylvain.bertrand@xxxxxxxxx wrote: > > Hi, > > Lol, I was working on it too. I got almost exactly the same code related to the > addition of a configuration variable: > - in 'struct snd_pcm_direct_open_conf' I used the type 'snd_pcm_tstamp_type_t' > instead of 'int' for the added tstamp_type field. > - idem for the 'struct snd_pcm_direct' tstamp_type field. Heh, my initial patch was also with that strong type, but I changed it because I wanted to allow the "system default" value, which isn't represented with that type. thanks, Takashi