On Fri, Aug 18, 2017 at 02:10:30PM -0400, Alex Deucher wrote:
> +++ b/sound/soc/amd/Kconfig
> @@ -2,3 +2,10 @@ config SND_SOC_AMD_ACP
> tristate "AMD Audio Coprocessor support"
> help
> This option enables ACP DMA support on AMD platform.
> +config SND_SOC_AMD_CZ_RT5645_MACH
> + tristate "AMD CZ support for RT5645"
Missing blank line between the stanzas.
> + select SND_SOC_RT5645
> + select SND_SOC_AMD_ACP
> + depends on I2C_DESIGNWARE_PLATFORM
No system dependencies of any kind? Looking at this I'd expect at least
CONFIG_ACPI || COMPILE_TEST. It's also unclear to me how the DesignWare
device is going to be instantiated here or if that should be a direct
depenency at all here.
> + ret = snd_soc_register_card(card);
devm_snd_soc_register_card() and then you don't need the remove
function.
> +static const struct acpi_device_id cz_audio_acpi_match[] = {
> + { "AMDI1002", 0 },
> + {},
> +};
> +
Missing MODULE_DEVICE_TABLE().
Attachment:
signature.asc
Description: PGP signature
_______________________________________________ Alsa-devel mailing list Alsa-devel@xxxxxxxxxxxxxxxx http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
