diff mbox

[3/5] ASoC: omap-pcm: Include omap-pcm.h

Message ID 1403163631-26215-4-git-send-email-lars@metafoo.de (mailing list archive)
State Accepted
Commit 76a77f47123945f6e5a4dd167e54768012ef9669
Headers show

Commit Message

Lars-Peter Clausen June 19, 2014, 7:40 a.m. UTC
omap_pcm_platform_register() is declared in omap-pcm.h and defined in
omap-pcm.c. To make sure that the function signature matches for both omap-pcm.c
should include omap-pcm.h

Fixes the following warning from sparse:
	sound/soc/omap/omap-pcm.c:235:5: warning: symbol
	'omap_pcm_platform_register' was not declared. Should it be static?

Signed-off-by: Lars-Peter Clausen <lars@metafoo.de>
---
 sound/soc/omap/omap-pcm.c | 1 +
 1 file changed, 1 insertion(+)

Comments

Mark Brown June 21, 2014, 8 p.m. UTC | #1
On Thu, Jun 19, 2014 at 09:40:29AM +0200, Lars-Peter Clausen wrote:
> omap_pcm_platform_register() is declared in omap-pcm.h and defined in
> omap-pcm.c. To make sure that the function signature matches for both omap-pcm.c
> should include omap-pcm.h

Applied, thanks.
Lars-Peter Clausen June 30, 2014, 4:48 p.m. UTC | #2
On 06/21/2014 10:00 PM, Mark Brown wrote:
> On Thu, Jun 19, 2014 at 09:40:29AM +0200, Lars-Peter Clausen wrote:
>> omap_pcm_platform_register() is declared in omap-pcm.h and defined in
>> omap-pcm.c. To make sure that the function signature matches for both omap-pcm.c
>> should include omap-pcm.h
>
> Applied, thanks.

This patch somehow did not make it, the patch is not in asoc/next, nor is 
there a asoc/topic/omap or similar. Shall I resend?

Thanks,
- Lars
Mark Brown June 30, 2014, 9:54 p.m. UTC | #3
On Mon, Jun 30, 2014 at 06:48:33PM +0200, Lars-Peter Clausen wrote:
> On 06/21/2014 10:00 PM, Mark Brown wrote:

> >Applied, thanks.

> This patch somehow did not make it, the patch is not in asoc/next, nor is
> there a asoc/topic/omap or similar. Shall I resend?

It seems to be there?
Lars-Peter Clausen July 1, 2014, 5:57 a.m. UTC | #4
On 06/30/2014 11:54 PM, Mark Brown wrote:
> On Mon, Jun 30, 2014 at 06:48:33PM +0200, Lars-Peter Clausen wrote:
>> On 06/21/2014 10:00 PM, Mark Brown wrote:
>
>>> Applied, thanks.
>
>> This patch somehow did not make it, the patch is not in asoc/next, nor is
>> there a asoc/topic/omap or similar. Shall I resend?
>
> It seems to be there?
>

Now it is.

Thanks,
- Lars
diff mbox

Patch

diff --git a/sound/soc/omap/omap-pcm.c b/sound/soc/omap/omap-pcm.c
index 8d809f8..f4b05bc 100644
--- a/sound/soc/omap/omap-pcm.c
+++ b/sound/soc/omap/omap-pcm.c
@@ -31,6 +31,7 @@ 
 #include <sound/pcm_params.h>
 #include <sound/dmaengine_pcm.h>
 #include <sound/soc.h>
+#include <sound/omap-pcm.h>
 
 #ifdef CONFIG_ARCH_OMAP1
 #define pcm_omap1510()	cpu_is_omap1510()