On Tue, Jan 17, 2017 at 11:22 PM, Mark Greer <mgreer@xxxxxxxxxxxxxxx> wrote: > On Tue, Jan 17, 2017 at 08:19:30PM +0530, Vaibhav Agarwal wrote: >> From: Vaibhav Agarwal <vaibhav.agarwal@xxxxxxxxxx> >> >> Proper byte order was completely disregarded for multi byte data shared >> between AP and module (and APB1). Fix this. >> >> Signed-off-by: Vaibhav Agarwal <vaibhav.agarwal@xxxxxxxxxx> >> --- > > Hi Vaibhav. > > I think you got them all except for this one in > audio_topology.c:gbcodec_mixer_dapm_ctl_put(): > >>>> max = info->value.integer.max; <<< > mask = (1 << fls(max)) - 1; > val = ucontrol->value.integer.value[0] & mask; > connect = !!val; Oh Ok. I'll include this change in v3. -- vaibhav > > Mark > -- _______________________________________________ greybus-dev mailing list greybus-dev@xxxxxxxxxxxxxxxx https://lists.linaro.org/mailman/listinfo/greybus-dev