Apologies. I am just getting used to the git-send-email mailer for emailing threaded patch sets and made a few oversights. I will correct these, add Signed-off-by line and resubmit as thread. Olivia On Tue, Feb 02, 2021 at 12:41:04PM +0100, Takashi Iwai wrote: > On Tue, 02 Feb 2021 01:02:37 +0100, > Olivia Mackintosh wrote: > > > > Like the DJM-750, ensure that the format control message is passed to > > the device when opening a stream. It seems as though fmt->sync_ep is not > > always set when this function is called hence the passing of the value > > at the call site. If this can be fixed, fmt->sync_up should be used as > > the wvalue. > > Could you give your Signed-off-by line? > > > +static int pioneer_djm_set_format_quirk(struct snd_usb_substream *subs, > > + u16 windex) > > +{ > > + unsigned int cur_rate = subs->data_endpoint->cur_rate; > > + u8 sr[3]; > > + // Convert to little endian > > + sr[0] = cur_rate&0xff; > > + sr[1] = (cur_rate>>8)&0xff; > > + sr[2] = (cur_rate>>16)&0xff; > > Better to have some appropriate spaces around operators. > > Also, instead of open-code, you can use cpu_to_le32() and pass its > pointer, too. > > > thanks, > > Takashi