From mboxrd@z Thu Jan 1 00:00:00 1970 From: Timur Tabi Subject: Re: [PATCH v8 02/11] ASoC: imx-pcm-fiq: Introduce pcm-fiq-params Date: Mon, 17 Jun 2013 06:39:55 -0500 Message-ID: <51BEF58B.8070405@tabi.org> References: <1371389117-19729-1-git-send-email-mpa@pengutronix.de> <1371389117-19729-3-git-send-email-mpa@pengutronix.de> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii"; Format="flowed" Content-Transfer-Encoding: 7bit Return-path: Received: from hrndva-omtalb.mail.rr.com (hrndva-omtalb.mail.rr.com [71.74.56.122]) by alsa0.perex.cz (Postfix) with ESMTP id 61B11261B1A for ; Mon, 17 Jun 2013 13:39:58 +0200 (CEST) In-Reply-To: <1371389117-19729-3-git-send-email-mpa@pengutronix.de> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: alsa-devel-bounces@alsa-project.org Sender: alsa-devel-bounces@alsa-project.org To: Markus Pargmann , linux-arm-kernel@lists.infradead.org Cc: Fabio Estevam , alsa-devel@alsa-project.org, Lars-Peter Clausen , devicetree-discuss@lists.ozlabs.org, Mark Brown , Liam Girdwood , Grant Likely , Sascha Hauer , Shawn Guo List-Id: alsa-devel@alsa-project.org Markus Pargmann wrote: > #ifdef CONFIG_SND_SOC_IMX_PCM_FIQ > -int imx_pcm_fiq_init(struct platform_device *pdev); > +int imx_pcm_fiq_init(struct platform_device *pdev, > + struct imx_pcm_fiq_params *params); > void imx_pcm_fiq_exit(struct platform_device *pdev); > #else > -static inline int imx_pcm_fiq_init(struct platform_device *pdev) > +static inline int imx_pcm_fiq_init(struct platform_device *pdev, > + struct imx_pcm_fiq_params *params); > { > return -ENODEV; > } This won't compile on PowerPC, because of the ; In file included from /home/b04825/git/linux.alsa2/sound/soc/fsl/fsl_ssi.c:53:0: /home/b04825/git/linux.alsa2/sound/soc/fsl/imx-pcm.h:65:1: error: expected identifier or '(' before '{' token /home/b04825/git/linux.alsa2/sound/soc/fsl/imx-pcm.h:63:19: warning: 'imx_pcm_fiq_init' used but never defined -- Timur Tabi From mboxrd@z Thu Jan 1 00:00:00 1970 From: timur@tabi.org (Timur Tabi) Date: Mon, 17 Jun 2013 06:39:55 -0500 Subject: [PATCH v8 02/11] ASoC: imx-pcm-fiq: Introduce pcm-fiq-params In-Reply-To: <1371389117-19729-3-git-send-email-mpa@pengutronix.de> References: <1371389117-19729-1-git-send-email-mpa@pengutronix.de> <1371389117-19729-3-git-send-email-mpa@pengutronix.de> Message-ID: <51BEF58B.8070405@tabi.org> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Markus Pargmann wrote: > #ifdef CONFIG_SND_SOC_IMX_PCM_FIQ > -int imx_pcm_fiq_init(struct platform_device *pdev); > +int imx_pcm_fiq_init(struct platform_device *pdev, > + struct imx_pcm_fiq_params *params); > void imx_pcm_fiq_exit(struct platform_device *pdev); > #else > -static inline int imx_pcm_fiq_init(struct platform_device *pdev) > +static inline int imx_pcm_fiq_init(struct platform_device *pdev, > + struct imx_pcm_fiq_params *params); > { > return -ENODEV; > } This won't compile on PowerPC, because of the ; In file included from /home/b04825/git/linux.alsa2/sound/soc/fsl/fsl_ssi.c:53:0: /home/b04825/git/linux.alsa2/sound/soc/fsl/imx-pcm.h:65:1: error: expected identifier or '(' before '{' token /home/b04825/git/linux.alsa2/sound/soc/fsl/imx-pcm.h:63:19: warning: 'imx_pcm_fiq_init' used but never defined -- Timur Tabi