From: Wang Xingchao <xingchao.wang@linux.intel.com>
To: tiwai@suse.de
Cc: mengdong.lin@intel.com, alsa-devel@alsa-project.org,
Wang Xingchao <xingchao.wang@linux.intel.com>,
xingchao.wang@intel.com
Subject: [PATCH 2/2] ALSA: hda - Add cache connection info
Date: Fri, 21 Jun 2013 12:41:48 +0800 [thread overview]
Message-ID: <1371789708-10524-2-git-send-email-xingchao.wang@linux.intel.com> (raw)
In-Reply-To: <1371789708-10524-1-git-send-email-xingchao.wang@linux.intel.com>
Pin's connection list may change dynamically with hot-plug event
on Intel Haswell chip. Users would see connections be "0" in codec#.
when play audio on this pin, software choose connections from cache.
So add Cache connection info to avoid confuse.
Signed-off-by: Wang Xingchao <xingchao.wang@linux.intel.com>
---
sound/pci/hda/hda_codec.c | 13 +++++++++++++
sound/pci/hda/hda_codec.h | 2 ++
sound/pci/hda/hda_proc.c | 13 ++++++++++++-
3 files changed, 27 insertions(+), 1 deletion(-)
diff --git a/sound/pci/hda/hda_codec.c b/sound/pci/hda/hda_codec.c
index 55108b5..7517fd6 100644
--- a/sound/pci/hda/hda_codec.c
+++ b/sound/pci/hda/hda_codec.c
@@ -354,6 +354,19 @@ lookup_conn_list(struct hda_codec *codec, hda_nid_t nid)
return NULL;
}
+int lookup_cache_conn(struct hda_codec *codec,
+ hda_nid_t pin_nid, hda_nid_t *conn_list)
+{
+ struct hda_conn_list *p;
+
+ p = lookup_conn_list(codec, pin_nid);
+ if (p) {
+ memcpy(conn_list, p->conns, p->len * sizeof(hda_nid_t));
+ return p->len;
+ }
+ return 0;
+}
+
static int add_conn_list(struct hda_codec *codec, hda_nid_t nid, int len,
const hda_nid_t *list)
{
diff --git a/sound/pci/hda/hda_codec.h b/sound/pci/hda/hda_codec.h
index c93f902..145d184 100644
--- a/sound/pci/hda/hda_codec.h
+++ b/sound/pci/hda/hda_codec.h
@@ -963,6 +963,8 @@ snd_hda_get_num_conns(struct hda_codec *codec, hda_nid_t nid)
int snd_hda_get_num_raw_conns(struct hda_codec *codec, hda_nid_t nid);
int snd_hda_get_raw_connections(struct hda_codec *codec, hda_nid_t nid,
hda_nid_t *conn_list, int max_conns);
+int lookup_cache_conn(struct hda_codec *codec,
+ hda_nid_t pin_nid, hda_nid_t *conn_list);
int snd_hda_get_conn_list(struct hda_codec *codec, hda_nid_t nid,
const hda_nid_t **listp);
int snd_hda_override_conn_list(struct hda_codec *codec, hda_nid_t nid, int nums,
diff --git a/sound/pci/hda/hda_proc.c b/sound/pci/hda/hda_proc.c
index 0fee8fa..e52ad3c 100644
--- a/sound/pci/hda/hda_proc.c
+++ b/sound/pci/hda/hda_proc.c
@@ -504,6 +504,7 @@ static void print_conn_list(struct snd_info_buffer *buffer,
int conn_len)
{
int c, curr = -1;
+ hda_nid_t cache_conn[4];
if (conn_len > 1 &&
wid_type != AC_WID_AUD_MIX &&
@@ -511,7 +512,7 @@ static void print_conn_list(struct snd_info_buffer *buffer,
wid_type != AC_WID_POWER)
curr = snd_hda_codec_read(codec, nid, 0,
AC_VERB_GET_CONNECT_SEL, 0);
- snd_iprintf(buffer, " Connection: %d\n", conn_len);
+ snd_iprintf(buffer, " Raw Connection: %d\n", conn_len);
if (conn_len > 0) {
snd_iprintf(buffer, " ");
for (c = 0; c < conn_len; c++) {
@@ -521,6 +522,16 @@ static void print_conn_list(struct snd_info_buffer *buffer,
}
snd_iprintf(buffer, "\n");
}
+
+ /* Get Cache connections info */
+ conn_len = lookup_cache_conn(codec, nid, cache_conn);
+ if (conn_len > 0) {
+ snd_iprintf(buffer, " Cache Connection: %d\n", conn_len);
+ snd_iprintf(buffer, " ");
+ for (c = 0; c < conn_len; c++)
+ snd_iprintf(buffer, " 0x%02x", cache_conn[c]);
+ snd_iprintf(buffer, "\n");
+ }
}
static void print_gpio(struct snd_info_buffer *buffer,
--
1.8.1.2
next prev parent reply other threads:[~2013-06-21 5:10 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-06-21 4:41 [PATCH 1/2] ALSA: hda - Remove unused variable Wang Xingchao
2013-06-21 4:41 ` Wang Xingchao [this message]
2013-06-21 6:26 ` [PATCH 2/2] ALSA: hda - Add cache connection info Wang, Xingchao
2013-06-21 9:32 ` Takashi Iwai
2013-06-24 12:01 ` Wang, Xingchao
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=1371789708-10524-2-git-send-email-xingchao.wang@linux.intel.com \
--to=xingchao.wang@linux.intel.com \
--cc=alsa-devel@alsa-project.org \
--cc=mengdong.lin@intel.com \
--cc=tiwai@suse.de \
--cc=xingchao.wang@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 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.