From: Ctirad Fertr <c.fertr@gmail.com>
To: alsa-devel@alsa-project.org
Subject: Re: [PATCH] 1616(M) cardbus improvements
Date: Mon, 3 Dec 2007 00:33:25 +0100 [thread overview]
Message-ID: <200712030033.25351.c.fertr@gmail.com> (raw)
In-Reply-To: <s5hwss3iqqu.wl%tiwai@suse.de>
Dne Tuesday 27 of November 2007 17:43:05 jste napsal(a):
> The changes look almost good to me. But, if I understand correctly,
> the input source put isn't changed properly for EMU1616.
No, it's correct, because emu1010_input_dst[] structure is generic for
all EMU PCI cards.
> The patch below is the revised version with bunch of coding style
> fixes (at the next time, try scripts/checkpatch.pl in linux kernel
> tree before submission).
I will do that next time. Thank you.
> Let me know if it works for you. I'll apply to ALSA tree after your
> confirmation.
I works very well :)
Unfortunatelly, now I found that driver (or host card) reload causes kernel
ooops on 2.6.23 (2.6.22 is unaffected). This is NOT related to my patch. It
crashes even with 1.0.15 version. This is bad :( It crashes immeditely after
a firmware for the host card is loaded. Can someone confirm that?
> Oh, BTW, it'd be also appreciated if you can write an additional patch
> to add enum definitions for each emu_model type instead of digits.
> "if (emu_model == 3)" is no good style.
OK. I'll do it.
---
Ctirad
next prev parent reply other threads:[~2007-12-02 23:34 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-11-15 23:07 [PATCH] 1616(M) cardbus improvements Ctirad Fertr
2007-11-27 16:43 ` Takashi Iwai
2007-11-27 16:45 ` Takashi Iwai
2007-11-27 19:29 ` James Courtier-Dutton
2007-12-02 23:33 ` Ctirad Fertr [this message]
2007-12-03 9:29 ` Takashi Iwai
2007-12-03 23:43 ` Ctirad Fertr
2007-12-13 14:21 ` Takashi Iwai
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=200712030033.25351.c.fertr@gmail.com \
--to=c.fertr@gmail.com \
--cc=alsa-devel@alsa-project.org \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.