alsa-devel.alsa-project.org archive mirror
 help / color / mirror / Atom feed
From: Michele Curti <michele.curti@gmail.com>
To: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Cc: Luka Karinja <luka.karinja@gmail.com>,
	Vinod Koul <vinod.koul@intel.com>,
	ao2@ao2.it, alsa-devel@alsa-project.org,
	Liam Girdwood <liam.r.girdwood@linux.intel.com>,
	Takashi Iwai <tiwai@suse.de>,
	"Nikula, Jarkko" <jarkko.nikula@intel.com>,
	Bard Liao <bardliao@realtek.com>
Subject: Re: writing an alsa driver
Date: Fri, 4 Mar 2016 12:07:17 +0100	[thread overview]
Message-ID: <20160304110717.GA2791@acervm> (raw)
In-Reply-To: <56D86467.9030703@linux.intel.com>

On Thu, Mar 03, 2016 at 10:20:55AM -0600, Pierre-Louis Bossart wrote:
> On 3/3/16 5:27 AM, Vinod Koul wrote:
> >On Thu, Mar 03, 2016 at 12:11:10PM +0100, Michele Curti wrote:
> >>Hi Vinod,
> >>
> >>On Thu, Jun 25, 2015 at 08:54:07AM +0530, Vinod Koul wrote:
> >>>>[    2.700468] sst-mfld-platform sst-mfld-platform: Slot control: codec_in rx deinterleaver codec_in1_1 doesn't have DAPM widget!!!
> >>>this is unexpected, I will check on this
> >>>
> >>>>...
> >>>>[    7.970937] intel_sst_acpi 80860F28:00: Wait timed-out condition:0x0, msg_id:0x1 fw_state 0x3
> >>>>[    7.970964] intel_sst_acpi 80860F28:00: fw returned err -16
> >>>Yes this is a problem, which FW are you using
> >
> >Have you tried the recent kernels, Pierre did a bunch of update and got many
> >things working.
> >
> >As for timeout, that simply means we have send a message to DSP which failed
> >to get a response. Perhpas the message had some bad element of FW had a
> >bug/crashed...
> 
> The firmware timeout is due to a change in the BIOS for BYT-CR devices that
> impacts the IPC interrupt number, I have a set of patches that fix this on
> Asus T100TAF but I still can't get any sound out.
>

Do you mean the .acpi_ipc_irq_index from 5 to 0?
If so, I also carry that fix in my patches.

Thanks,
Michele

On Thu, Mar 03, 2016 at 12:11:10PM +0100, Michele Curti wrote:
> 2. platform driver (patch 0002 at [1])
> 
> Used the sst-mfld-platform one (under sound/soc/intel/atom/sst), changing
> the .acpi_ipc_irq_index from 5 to 0 because the DSP interrupt order in
> the ACPI DSDT table of this laptop is different from the "normal" (see
> line 3745 of dsdt.dsl at [2]).
> 
> $ cat /proc/interrupts | grep sst
>             CPU0       CPU1       CPU2       CPU3       
>    1:         24          0          0          0   IO-APIC   29-fasteoi   intel_sst_driver
> 
> ...
> 
> [1] https://github.com/michelecurti/x205ta/tree/master/sound-patches
> [2] https://github.com/michelecurti/x205ta/tree/master/acpi-tables
> 

  reply	other threads:[~2016-03-04 11:07 UTC|newest]

Thread overview: 50+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-05-21 14:52 writing an alsa driver Michele Curti
2015-05-23 16:32 ` Michele Curti
2015-05-27  6:45   ` Takashi Iwai
2015-05-27  8:54     ` Michele Curti
2015-05-27 18:41       ` Liam Girdwood
2015-05-27 20:13         ` Michele Curti
2015-05-28 11:31           ` Liam Girdwood
2015-05-28 12:53             ` Michele Curti
2015-05-28 13:21               ` Liam Girdwood
2015-05-28 19:17                 ` Michele Curti
2015-05-29  9:42                   ` Liam Girdwood
2015-05-29 13:17                     ` Michele Curti
2015-06-01 11:06                       ` Liam Girdwood
2015-06-04  7:09                         ` Marcus Bannerman
2015-06-04  9:04                           ` Michele Curti
2015-06-04  8:59                         ` Michele Curti
2015-06-04 10:33                           ` Liam Girdwood
2015-06-04 17:44                             ` Juergen Bausa
2015-06-04 19:39                         ` Michele Curti
2015-06-05  7:00                           ` Jarkko Nikula
2015-06-08  8:26                             ` Michele Curti
2015-06-08  8:49                         ` Michele Curti
2015-06-13 13:47                           ` Luka Karinja
2015-06-15 16:46                             ` Michele Curti
2015-06-15 17:04                               ` Liam Girdwood
2015-06-16 20:50                                 ` Michele Curti
2015-06-17  9:16                                   ` Liam Girdwood
2015-06-17 21:34                                     ` Michele Curti
2015-06-18 11:14                                       ` Vinod Koul
2015-06-18 18:47                                         ` Michele Curti
2015-06-19  7:54                                           ` Vinod Koul
2015-06-19  8:12                                             ` Michele Curti
2015-06-17  6:39                                 ` Jarkko Nikula
2015-06-23  9:09                                 ` Michele Curti
2015-06-23  9:51                                   ` Liam Girdwood
2015-06-24 10:21                                   ` Vinod Koul
2015-06-24 11:31                                     ` Antonio Ospite
2015-06-24 11:40                                       ` Liam Girdwood
2015-06-24 16:04                                         ` Vinod Koul
2015-06-24 18:21                                     ` Michele Curti
2015-06-25  3:24                                       ` Vinod Koul
2015-06-25  7:39                                         ` Michele Curti
2016-03-03 11:11                                         ` Michele Curti
2016-03-03 11:27                                           ` Vinod Koul
2016-03-03 13:08                                             ` Michele Curti
2016-03-03 16:20                                             ` Pierre-Louis Bossart
2016-03-04 11:07                                               ` Michele Curti [this message]
  -- strict thread matches above, loose matches on Subject: below --
2015-07-22 20:05 Marco Bodega
2015-07-24 20:54 ` Michele Curti
2015-08-10 14:12 Johnny

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20160304110717.GA2791@acervm \
    --to=michele.curti@gmail.com \
    --cc=alsa-devel@alsa-project.org \
    --cc=ao2@ao2.it \
    --cc=bardliao@realtek.com \
    --cc=jarkko.nikula@intel.com \
    --cc=liam.r.girdwood@linux.intel.com \
    --cc=luka.karinja@gmail.com \
    --cc=pierre-louis.bossart@linux.intel.com \
    --cc=tiwai@suse.de \
    --cc=vinod.koul@intel.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).