All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jaroslav Kysela <perex@perex.cz>
To: ALSA development <alsa-devel@alsa-project.org>
Cc: Takashi Iwai <tiwai@suse.de>
Subject: Re: [PATCH] ALSA: hda - Add DeviceID for Haswell HDA
Date: Mon, 16 Jul 2012 10:53:15 +0200	[thread overview]
Message-ID: <5003D67B.8080002@perex.cz> (raw)
In-Reply-To: <s5h1ukchze6.wl%tiwai@suse.de>

Date 16.7.2012 10:40, Takashi Iwai wrote:
> At Fri, 13 Jul 2012 19:48:22 +0200,
> Jaroslav Kysela wrote:
>>
>>
>> 00:03.0 0403: 8086:0c0c (rev 01)
>> 	Subsystem: 8086:2010
>> --
>> 00:1b.0 0403: 8086:8c20 (rev 01)
>> 	Subsystem: 8086:7270
>>
>>
>> Perhaps, we may add also some code to penalize the HDMI only HDA bridges
>> and allocate these cards on indexes 1+ by default (implement
>> AZX_DCAPS_INDEX1) . Comments? I'll prepare a patch upon an agreement.
> 
> Having index=0 for analog would be nice in general, but I'm afraid
> that it isn't always safe in the case of HD-audio (depending on the
> implementation, though, of course): 
> 
> - BIOS may disable the analog PCI part, e.g. if a device has only
>   HDMI/DP, or vice versa.
> 
> - There are already devices showing the same problem, e.g. AMD
>   on-board with AMD D-GPU, and users may have already added index
>   option for such.
> 
> IMO, we should provide a mechanism to advertise the preferred
> configuration from the driver side without fiddling the index.
> For example, exposing an info bit that this is digital-I/O only card,
> and let alsa-lib choose another one as default (unless explicitly
> specified).

It's not a bad idea, but it would be good to have also "standard"
plughw:0 and hw:0 devices available in the system.

Do you think that the PCI ID / PCI subsystem ID check is not sufficient
to determine the hw type before the codec probe?

						Jaroslav

-- 
Jaroslav Kysela <perex@perex.cz>
Linux Kernel Sound Maintainer
ALSA Project; Red Hat, Inc.

  reply	other threads:[~2012-07-16  8:52 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-06-13  2:23 [PATCH] ALSA: hda - Add DeviceID for Haswell HDA Wang Xingchao
2012-06-13  2:23 ` [PATCH] ALSA: hda - add Haswell HDMI pci id Wang Xingchao
2012-06-13  6:46 ` [PATCH] ALSA: hda - Add DeviceID for Haswell HDA Takashi Iwai
2012-06-13  8:05   ` Wang Xingchao
2012-06-13  8:14     ` Takashi Iwai
2012-06-13  8:31       ` Wang Xingchao
2012-06-13  8:43         ` Takashi Iwai
2012-07-13 17:48   ` Jaroslav Kysela
2012-07-14  9:25     ` Wang Xingchao
2012-07-16  8:20     ` David Henningsson
2012-07-16  8:42       ` Takashi Iwai
2012-07-16  8:49         ` Jaroslav Kysela
2012-07-16  8:40     ` Takashi Iwai
2012-07-16  8:53       ` Jaroslav Kysela [this message]
2012-07-16  9:02         ` Takashi Iwai
2012-07-16 14:05     ` 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=5003D67B.8080002@perex.cz \
    --to=perex@perex.cz \
    --cc=alsa-devel@alsa-project.org \
    --cc=tiwai@suse.de \
    /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.