Hi Mark, On Wed, Aug 7, 2013 at 9:45 AM, Mark Brown <[email protected]> wrote: > On Fri, Aug 02, 2013 at 12:59:24AM -0700, Felipe F. Tonello wrote: > >> +int jack_ctl_integer_info(struct snd_kcontrol *kcontrol, >> + struct snd_ctl_elem_info *uinfo) >> +{ >> + uinfo->type = SNDRV_CTL_ELEM_TYPE_INTEGER; >> + uinfo->count = 0x10000U; >> + uinfo->value.integer.min = 0; >> + uinfo->value.integer.max = 0xffff; >> + return 0; >> +} > > I'd expected to see us creating multiple boolean controls here. I don't > have a great problem with doing things this way but it's surprising and > I worry about confusing existing userspace, Takashi?
Yes, it makes more sense. I got confused with another talk we had previously, that's why I end up doing as an int. I will wait for more comments about these patches and will submit a v3. Thanks, Felipe Tonello -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
