Re: [PATCH] ALSA: ASoc: Add regulator support to CS4270 codec driver

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Nov 30, 2009 at 10:56 AM, Daniel Mack <daniel@xxxxxxxx> wrote:
> Signed-off-by: Daniel Mack <daniel@xxxxxxxx>
> Cc: Mark Brown <broonie@xxxxxxxxxxxxxxxxxxxxxxxxxxx>
> Cc: Timur Tabi <timur@xxxxxxxxxxxxx>
> Cc: Liam Girdwood <lrg@xxxxxxxxxxxxxxx>
> ---

Ack

> +static const char *supply_names[] = {
> +       "va", "vd", "vlc"
> +};

Minor nit -- this array would take up less space if it were an array
of strings instead of an array of pointers to strings.

static const char supply_names[4][] = {
       "va", "vd", "vlc"
};

-- 
Timur Tabi
Linux kernel developer at Freescale
_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel


[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux