Re: [PATCH v2 6/6] ARM: DaVinci: ASoC: Introduce platform driver model for dm644x, dm355

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

 



On Thu, Apr 16, 2009 at 06:45:33PM -0400, Naresh Medisetty wrote:

> +static struct platform_driver davinci_asoc_driver = {
> +	.probe		= davinci_i2s_probe,
> +	.remove		= davinci_i2s_remove,
> +	.driver		= {
> +		.name	= "davinci-asoc",
> +		.owner	= THIS_MODULE,
> +	},

I suspect that this isn't the best name for the driver - it should
probably be something like davinci-mcasp or possibly davinci-mcasp-audio
or similar.  This isn't the only possible audio device on the DaVinci
(there's an I2S device already) and more may be added in the future.

Other than that this looks OK.
_______________________________________________
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