[alsa-devel] [PATCH V2 1/2] ARM: pxa: spitz: register spitz-audio device
Arnd Bergmann
arnd at arndb.de
Wed Oct 22 10:46:06 CEST 2014
On Wednesday 22 October 2014 01:15:32 Dmitry Eremin-Solenikov wrote:
> Register spitz-audio device to be used by ASoC driver.
>
> [V2: Removed ifdefs around spitz_audio_init].
>
> Signed-off-by: Dmitry Eremin-Solenikov <dbaryshkov at gmail.com>
> ---
>
A more general note about your patch description above: when you
write about the changes between versions, put that below the ---
line under your Signed-off-by. Aside from that, your changelog
entry would actually benefit from more verbosity: explain what the
purpose of this is, not just what it does.
> /**************************************************************************
> ****>
> + * Audio devices
> +
> ***************************************************************************
> ***/ +static struct platform_device spitz_audio_device = {
> + .name = "spitz-audio",
> + .id = -1,
> +};
> +
> +static inline void spitz_audio_init(void)
> +{
> + platform_device_register(&spitz_audio_device);
> +}+
I realize that the file has more of these, but it would be better
for a number of reasons to use 'platform_device_register_simple'
instead of platform_device_register. statically defining platform
devices (or any other device) is not recommended.
Arnd
More information about the Alsa-devel
mailing list