2009-12-24 08:03:20     bf5xx-ac97.c question

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

2009-12-24 08:03:20     bf5xx-ac97.c question

Michail Kurochkin (BELARUS)

Message: 83917   

 

Hello,

After investigation of AD1980 problems in our custom hardware , i came to sound/soc/blackfin/bf5xx-ac97.c

 

I have some question.

 

What is a rationale of using [cmd_count]  array instead of static int variable

 

static int *cmd_count;

nextwrite[cmd_count[nextfrag]].ac97_data = data;

++cmd_count[nextfrag];

 

instead of static int offset_in_frag = 0;

nextwrite[offset_in_frag].ac97_data = data;

offset_in_frag++;

QuoteReplyEditDelete

 

 

2009-12-24 22:29:13     Re: bf5xx-ac97.c question

Cliff Cai (CHINA)

Message: 83922   

 

The original idea is every fragment has it's own counter to show the place where the command should be inserted to.So,use a static variable for all the fragments is also OK,only one more thing needs to be considered is that this value should not exceed the length of a fragment(the number of samples).see the detailed code in 2009r1.

 

Cliff

QuoteReplyEditDelete

 

 

2009-12-28 03:16:28     Re: bf5xx-ac97.c question

Michail Kurochkin (BELARUS)

Message: 83981   

 

Thanks.

Attachments

    Outcomes