Skip to content

Commit

Permalink
ASoC: samsung: Fix build on multiplatform
Browse files Browse the repository at this point in the history
PCM and S/PDIF drivers referenced mach headers for a trivial
data structure. This caused build errors on multiplatform builds
as machine headers are not accessible from driver files. Move the data
structure definition to the driver header and remove the dependency.
While at it rename the structure to avoid multiple definition errors
as the same structure is also used by the platform code.

Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
Signed-off-by: Mark Brown <broonie@linaro.org>
  • Loading branch information
Sachin Kamat authored and Mark Brown committed Apr 8, 2014
1 parent c159a85 commit 71e5222
Show file tree
Hide file tree
Showing 7 changed files with 18 additions and 16 deletions.
6 changes: 3 additions & 3 deletions sound/soc/samsung/ac97.c
Original file line number Diff line number Diff line change
Expand Up @@ -39,15 +39,15 @@ struct s3c_ac97_info {
};
static struct s3c_ac97_info s3c_ac97;

static struct s3c2410_dma_client s3c_dma_client_out = {
static struct s3c_dma_client s3c_dma_client_out = {
.name = "AC97 PCMOut"
};

static struct s3c2410_dma_client s3c_dma_client_in = {
static struct s3c_dma_client s3c_dma_client_in = {
.name = "AC97 PCMIn"
};

static struct s3c2410_dma_client s3c_dma_client_micin = {
static struct s3c_dma_client s3c_dma_client_micin = {
.name = "AC97 MicIn"
};

Expand Down
6 changes: 5 additions & 1 deletion sound/soc/samsung/dma.h
Original file line number Diff line number Diff line change
Expand Up @@ -14,8 +14,12 @@

#include <sound/dmaengine_pcm.h>

struct s3c_dma_client {
char *name;
};

struct s3c_dma_params {
struct s3c2410_dma_client *client; /* stream identifier */
struct s3c_dma_client *client; /* stream identifier */
int channel; /* Channel ID */
dma_addr_t dma_addr;
int dma_size; /* Size of the DMA transfer */
Expand Down
6 changes: 3 additions & 3 deletions sound/soc/samsung/i2s.c
Original file line number Diff line number Diff line change
Expand Up @@ -1211,10 +1211,10 @@ static int samsung_i2s_probe(struct platform_device *pdev)
pri_dai->dma_playback.dma_addr = regs_base + I2STXD;
pri_dai->dma_capture.dma_addr = regs_base + I2SRXD;
pri_dai->dma_playback.client =
(struct s3c2410_dma_client *)&pri_dai->dma_playback;
(struct s3c_dma_client *)&pri_dai->dma_playback;
pri_dai->dma_playback.ch_name = "tx";
pri_dai->dma_capture.client =
(struct s3c2410_dma_client *)&pri_dai->dma_capture;
(struct s3c_dma_client *)&pri_dai->dma_capture;
pri_dai->dma_capture.ch_name = "rx";
pri_dai->dma_playback.dma_size = 4;
pri_dai->dma_capture.dma_size = 4;
Expand All @@ -1233,7 +1233,7 @@ static int samsung_i2s_probe(struct platform_device *pdev)
}
sec_dai->dma_playback.dma_addr = regs_base + I2STXDS;
sec_dai->dma_playback.client =
(struct s3c2410_dma_client *)&sec_dai->dma_playback;
(struct s3c_dma_client *)&sec_dai->dma_playback;
sec_dai->dma_playback.ch_name = "tx-sec";

if (!np) {
Expand Down
5 changes: 2 additions & 3 deletions sound/soc/samsung/pcm.c
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,6 @@
#include <sound/pcm_params.h>

#include <linux/platform_data/asoc-s3c.h>
#include <mach/dma.h>

#include "dma.h"
#include "pcm.h"
Expand Down Expand Up @@ -132,11 +131,11 @@ struct s3c_pcm_info {
struct s3c_dma_params *dma_capture;
};

static struct s3c2410_dma_client s3c_pcm_dma_client_out = {
static struct s3c_dma_client s3c_pcm_dma_client_out = {
.name = "PCM Stereo out"
};

static struct s3c2410_dma_client s3c_pcm_dma_client_in = {
static struct s3c_dma_client s3c_pcm_dma_client_in = {
.name = "PCM Stereo in"
};

Expand Down
4 changes: 2 additions & 2 deletions sound/soc/samsung/s3c2412-i2s.c
Original file line number Diff line number Diff line change
Expand Up @@ -33,11 +33,11 @@
#include "regs-i2s-v2.h"
#include "s3c2412-i2s.h"

static struct s3c2410_dma_client s3c2412_dma_client_out = {
static struct s3c_dma_client s3c2412_dma_client_out = {
.name = "I2S PCM Stereo out"
};

static struct s3c2410_dma_client s3c2412_dma_client_in = {
static struct s3c_dma_client s3c2412_dma_client_in = {
.name = "I2S PCM Stereo in"
};

Expand Down
4 changes: 2 additions & 2 deletions sound/soc/samsung/s3c24xx-i2s.c
Original file line number Diff line number Diff line change
Expand Up @@ -31,11 +31,11 @@
#include "dma.h"
#include "s3c24xx-i2s.h"

static struct s3c2410_dma_client s3c24xx_dma_client_out = {
static struct s3c_dma_client s3c24xx_dma_client_out = {
.name = "I2S PCM Stereo out"
};

static struct s3c2410_dma_client s3c24xx_dma_client_in = {
static struct s3c_dma_client s3c24xx_dma_client_in = {
.name = "I2S PCM Stereo in"
};

Expand Down
3 changes: 1 addition & 2 deletions sound/soc/samsung/spdif.c
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,6 @@
#include <sound/pcm_params.h>

#include <linux/platform_data/asoc-s3c.h>
#include <mach/dma.h>

#include "dma.h"
#include "spdif.h"
Expand Down Expand Up @@ -94,7 +93,7 @@ struct samsung_spdif_info {
struct s3c_dma_params *dma_playback;
};

static struct s3c2410_dma_client spdif_dma_client_out = {
static struct s3c_dma_client spdif_dma_client_out = {
.name = "S/PDIF Stereo out",
};

Expand Down

0 comments on commit 71e5222

Please sign in to comment.