[alsa-devel] [PATCH] ALSA: firewire: Neaten cmp_error function

Clemens Ladisch clemens at ladisch.de
Thu Oct 20 08:36:55 CEST 2011


Joe Perches wrote:
> Use an actual struct va_format instead of casting
> an array of pointers.

> -	dev_err(&c->resources.unit->device, "%cPCR%u: %pV",
> -		'i', c->pcr_index, &(struct va_format){ fmt, &va });

There is no cast.  This compound literal _is_ an actual
struct va_format.

> Move a constant char argument to the format string.

Good catch, but this module is going to be extended to handle
oPCRs, too.


Regards,
Clemens


More information about the Alsa-devel mailing list