On Mon, Jan 16, 2017 at 4:17 PM, Dan Carpenter <dan.carpenter@xxxxxxxxxx> wrote: > On Sat, Jan 14, 2017 at 11:17:07PM +0530, Vaibhav Agarwal wrote: >> @@ -656,13 +660,13 @@ static int gbaudio_tplg_create_enum_kctl(struct gbaudio_module_info *gb, >> gb_enum = &ctl->info.value.enumerated; >> >> /* since count=1, and reg is dummy */ >> - gbe->max = gb_enum->items; >> + gbe->max = le32_to_cpu(gb_enum->items); >> gbe->texts = gb_generate_enum_strings(gb, gb_enum); >> >> /* debug enum info */ >> dev_dbg(gb->dev, "Max:%d, name_length:%d\n", gb_enum->items, > > This is printing little endian max. Just use gbe->max here. Yes, this makes more sense. I should have noticed this earlier :( Surely, I'll include this one & other suggested changes in v2 series. Thanks Dan :) -- vaibhav _______________________________________________ greybus-dev mailing list greybus-dev@xxxxxxxxxxxxxxxx https://lists.linaro.org/mailman/listinfo/greybus-dev
|