[#4206] sport varialbes exported twice warning on trunk

Submitted By: Mingquan Pan

Open Date

2008-06-30 06:16:09 Close Date

2008-07-11 04:56:58

Priority:

Medium Assignee:

Cliff Cai

Status:

Closed Fixed In Release:

N/A

Found In Release:

N/A Release:

Category:

N/A Board:

N/A

Processor:

N/A Silicon Revision:

Is this bug repeatable?:

Yes Resolution:

Fixed

Uboot version or rev.:

Toolchain version or rev.:

08r1-12

App binary format:

N/A

Summary: sport varialbes exported twice warning on trunk

Details:

sport varialbes exported twice warning on trunk.

make[3]: Entering directory `/home/test/work/cruise/checkouts/uclinux-dist/linux-2.6.x'

CHK include/linux/version.h

CHK include/linux/utsrelease.h

CALL scripts/checksyscalls.sh

CHK include/linux/compile.h

AS usr/initramfs_data.o

LD usr/built-in.o

GEN .version

CHK include/linux/compile.h

UPD include/linux/compile.h

CC init/version.o

LD init/built-in.o

LD .tmp_vmlinux1

KSYM .tmp_kallsyms1.S

AS .tmp_kallsyms1.o

LD .tmp_vmlinux2

KSYM .tmp_kallsyms2.S

AS .tmp_kallsyms2.o

LD vmlinux

SYSMAP System.map

SYSMAP .tmp_System.map

MODPOST vmlinux

OBJCOPY arch/blackfin/boot/vmlinux.bin

Building modules, stage 2.

GZIP arch/blackfin/boot/vmlinux.gz

MODPOST 34 modules

WARNING: sound/soc/blackfin/snd-soc-bf5xx-i2s: 'sport_done' exported twice. Previous export was in sound/soc/blackfin/snd-soc-bf5xx-ac97.ko

WARNING: sound/soc/blackfin/snd-soc-bf5xx-i2s: 'sport_init' exported twice. Previous export was in sound/soc/blackfin/snd-soc-bf5xx-ac97.ko

WARNING: sound/soc/blackfin/snd-soc-bf5xx-i2s: 'sport_set_err_callback' exported twice. Previous export was in sound/soc/blackfin/snd-soc-bf5xx-ac97.ko

WARNING: sound/soc/blackfin/snd-soc-bf5xx-i2s: 'sport_set_tx_callback' exported twice. Previous export was in sound/soc/blackfin/snd-soc-bf5xx-ac97.ko

WARNING: sound/soc/blackfin/snd-soc-bf5xx-i2s: 'sport_set_rx_callback' exported twice. Previous export was in sound/soc/blackfin/snd-soc-bf5xx-ac97.ko

WARNING: sound/soc/blackfin/snd-soc-bf5xx-i2s: 'sport_curr_offset_tx' exported twice. Previous export was in sound/soc/blackfin/snd-soc-bf5xx-ac97.ko

WARNING: sound/soc/blackfin/snd-soc-bf5xx-i2s: 'sport_curr_offset_rx' exported twice. Previous export was in sound/soc/blackfin/snd-soc-bf5xx-ac97.ko

WARNING: sound/soc/blackfin/snd-soc-bf5xx-i2s: 'sport_config_tx_dma' exported twice. Previous export was in sound/soc/blackfin/snd-soc-bf5xx-ac97.ko

WARNING: sound/soc/blackfin/snd-soc-bf5xx-i2s: 'sport_config_rx_dma' exported twice. Previous export was in sound/soc/blackfin/snd-soc-bf5xx-ac97.ko

WARNING: sound/soc/blackfin/snd-soc-bf5xx-i2s: 'sport_tx_stop' exported twice. Previous export was in sound/soc/blackfin/snd-soc-bf5xx-ac97.ko

WARNING: sound/soc/blackfin/snd-soc-bf5xx-i2s: 'sport_tx_start' exported twice. Previous export was in sound/soc/blackfin/snd-soc-bf5xx-ac97.ko

WARNING: sound/soc/blackfin/snd-soc-bf5xx-i2s: 'sport_rx_stop' exported twice. Previous export was in sound/soc/blackfin/snd-soc-bf5xx-ac97.ko

WARNING: sound/soc/blackfin/snd-soc-bf5xx-i2s: 'sport_rx_start' exported twice. Previous export was in sound/soc/blackfin/snd-soc-bf5xx-ac97.ko

WARNING: sound/soc/blackfin/snd-soc-bf5xx-i2s: 'sport_config_tx' exported twice. Previous export was in sound/soc/blackfin/snd-soc-bf5xx-ac97.ko

WARNING: sound/soc/blackfin/snd-soc-bf5xx-i2s: 'sport_config_rx' exported twice. Previous export was in sound/soc/blackfin/snd-soc-bf5xx-ac97.ko

WARNING: sound/soc/blackfin/snd-soc-bf5xx-i2s: 'sport_set_multichannel' exported twice. Previous export was in sound/soc/blackfin/snd-soc-bf5xx-ac97.ko

WARNING: sound/soc/blackfin/snd-soc-bf5xx-i2s: 'bf5xx_frame_to_pcm' exported twice. Previous export was in sound/soc/blackfin/snd-soc-bf5xx-ac97.ko

WARNING: sound/soc/blackfin/snd-soc-bf5xx-i2s: 'bf5xx_pcm_to_frame' exported twice. Previous export was in sound/soc/blackfin/snd-soc-bf5xx-ac97.ko

WARNING: sound/soc/blackfin/snd-soc-bf5xx-i2s: 'sport_handle' exported twice. Previous export was in sound/soc/blackfin/snd-soc-bf5xx-ac97.ko

UIMAGE arch/blackfin/boot/vmImage

Image Name: Linux-2.6.22.19-ADI-2008R1-svn49

Created: Tue Jul 1 01:46:45 2008

Image Type: Blackfin Linux Kernel Image (gzip compressed)

Data Size: 3887297 Bytes = 3796.19 kB = 3.71 MB

Follow-ups

--- Cliff Cai 2008-07-01 04:08:07

Hi all,

We now use one bf5xx-pcm.c to serve both I2S and AC97.

Of course,AC97 and I2S have different frame struct to hold the audio data and

the different "copy" function to fill the DMA buffer,currently we use

the same frame struct name(audio_frame) and same "copy" function name

(bf5xx_pcm_frame)for the two bus,and they are defined/declared in the two bus's

header respectively.Then we can include the relevant header to bf5xx-pcm.c

depending on which bus we use.(CONFIG_XXX_I2S or CONFIG_XXX_AC97).

So,it's not allowed to build two drivers using different bus at the same

time.An alternative way to solve this problem is introducing two

bf5xx-xxxpcm.c,one for I2S the other for AC97.

what's your opinion?

Cliff

--- Mike Frysinger 2008-07-01 07:11:22

the sport code should be a standalone module, not linked directly into all other

modules ... that way you can load both ac97 and i2s at the same time

placing a restriction that you can use only i2s or ac97 at the same time is

wrong ... we have multiple sports and our framework needs to support arbitrary

sport connections with arbitrary codec locations

--- Cliff Cai 2008-07-01 22:02:43

yes,I plan to make SPORT as a standalone module,but it's really hard to support

both AC97 and I2S at the same time only using one DMA driver(bf5xx-pcm.c).I

don't know if other CUP like PXA can support both AC97 and I2S at the same

time,I can see in the source tree that it have both AC97 and I2S DAI drivers but

only have one DMA driver.

Cliff

--- Mike Frysinger 2008-07-01 22:19:52

i dont think supporting AC97 and I2S on the same SPORT is realistic. you'd be

running AC97 on one SPORT and I2S on a different one, or multiple AC97s, or ...

--- Cliff Cai 2008-07-09 05:15:11

fix it now.Making SPORT as a standalone module.

--- Mingquan Pan 2008-07-11 04:56:57

Yes,fixed on trunk.Close.

Files

Changes

Commits

Dependencies

Duplicates

Associations