The clock is indeed initialized in board_dm355.c. The problem is the clock name 
is not given to the clk_get function. I hard coded the clock name to asp1(since 
it is dm355) and found out that this helps register the i2s and aic33x. but 
when I ran aplay and arecord, the clock is not correct and moreover there are 
lot or underruns detected. BTW it is also detected in the DM6446 case.


-----Original Message-----
From: Troy Kisky [mailto:[email protected]] 
Sent: Friday, July 10, 2009 3:48 PM
To: Mani, Arun
Cc: Mark Brown; [email protected]; 
[email protected]; Medisetty, Naresh
Subject: Re: [PATCH 07/26] davinci: dm646x: Adds McASP clock

Mani, Arun wrote:
> Hi,
> I am trying the latest 2.6.31 rc2 branch. I am trying to make the Audio work.
>  I found that the I2S clock is set to NULL. Because of this I am getting a 
> NODEV error.
> If I commented the check, I was able to map the I2S to the AIC33X. but I am 
> not getting any sound.
You need a clock, so don't comment out the check, rather give it one. ie.

+static struct snd_platform_data snd_data = {
+       .clk_name       = "asp0",
+};
+
 static struct i2c_board_info __initdata i2c_info[] =  {
        {
                I2C_BOARD_INFO("ths8200", 0x21),
@@ -230,6 +234,7 @@ static __init void board_init(void)
        platform_add_devices(davinci_devices,
                             ARRAY_SIZE(davinci_devices));
        davinci_serial_init(&uart_config);
+       dm644x_init_asp(&snd_data);


> I tried the aplay and arecord and got the read/write errors.
> 
> Any thoughts?
> 
> Thanks,
> Arun.
> 


_______________________________________________
Davinci-linux-open-source mailing list
[email protected]
http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source

Reply via email to