On Thu, Mar 8, 2012 at 9:48 AM, Mark Brown <broonie at opensource.wolfsonmicro.com> wrote: > /* dapm control */ > - short reg; /* negative reg = no direct dapm */ > + unsigned int reg; /* negative reg = no direct dapm */ Maybe you should update the comment now that this is unsigned int.