[#4206] sport varialbes exported twice warning on trunk

Document created by Aaronwu Employee on Aug 29, 2013
Version 1Show Document
  • View in full screen mode

[#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

    Tags

 

File Name     File Type     File Size     Posted By

No Files Were Found

Attachments

    Outcomes