On 2018-04-30 13:55, Sebastian Reichel wrote:
Hi,
On Mon, Apr 30, 2018 at 09:57:42AM +0300, Peter Ujfalusi wrote:
[...] diff --git a/sound/soc/omap/sdma-pcm.h b/sound/soc/omap/sdma-pcm.h new file mode 100644 index 000000000000..ce13edfc52d8 --- /dev/null +++ b/sound/soc/omap/sdma-pcm.h @@ -0,0 +1,21 @@ +/* SPDX-License-Identifier: GPL-2.0 */ +/*
- Copyright (C) 2018 Texas Instruments Incorporated - http://www.ti.com
- Author: Peter Ujfalusi peter.ujfalusi@ti.com
- */
+#ifndef __SDMA_PCM_H__ +#define __SDMA_PCM_H__
+#if IS_ENABLED(CONFIG_SND_SDMA_SOC) +int sdma_pcm_platform_register(struct device *dev,
char *txdmachan, char *rxdmachan);+#else +static inline int sdma_pcm_platform_register(struct device *dev,
char *txdmachan, char *rxdmachan)+{
- return 0;
I would expect some error code instead?
Yeah, it could return -ENODEV. It is there so the McASP can be compiled for daVinci/am335x/am43xx where we do not have sDMA, only EDMA.
+} +#endif /* CONFIG_SND_SDMA_SOC */
+#endif /* __SDMA_PCM_H__ */
-- Sebastian
- Péter
Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki. Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki