Skip to contentSkip to conversation

Dell Latitude D820 + D/Port

PATCH In review Cory T. Tusar 2 participants 4 messages Jan 30, 2007

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 Dell Latitude D820 + D/Port

The attached patch adds support for the port replicator headphone output on a Dell Latitude D820 + D/Port.

Please CC me on any replies, as I am not subscribed to the alsa-devel list.

Thanks.

-Cory

Signature · 12 lines
Cory T. Tusar
Embedded Systems Engineer
Videon Central, Inc.
2171 Sandy Drive
State College, PA 16801
(814) 235-1111 x316
(814) 235-1118 fax


"Debugging is twice as hard as writing the code in the first place.
  Therefore, if you write the code as cleverly as possible, you are,
    by definition, not smart enough to debug it."  --Brian W. Kernighan

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your opinions on IT & business topics through brief surveys - and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV

_______________________________________________
Alsa-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/alsa-devel

#1 lore raw

At Tue, 30 Jan 2007 10:56:14 -0500,

Quoted 3 lines · Cory T. Tusar

The attached patch adds support for the port replicator headphone output on a Dell Latitude D820 + D/Port.

I cannot read the patch. Could you attach or embed a patch text?

thanks,

Takashi

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your opinions on IT & business topics through brief surveys - and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV

#2 lore raw
Quoted 5 lines · Takashi Iwai

At Tue, 30 Jan 2007 10:56:14 -0500,

Quoted 1 line · Cory T. Tusar

The attached patch adds support for the port replicator headphone output on a Dell Latitude D820 + D/Port.

I cannot read the patch. Could you attach or embed a patch text?

Hrmm...I sent it as an hg bundle as per the dev wiki. I'm not familiar with hg though, so I very well may have borked it.

Plaintext attached.

-Cory

Signature · 12 lines
Cory T. Tusar
Embedded Systems Engineer
Videon Central, Inc.
2171 Sandy Drive
State College, PA 16801
(814) 235-1111 x316
(814) 235-1118 fax


"Debugging is twice as hard as writing the code in the first place.
  Therefore, if you write the code as cleverly as possible, you are,
    by definition, not smart enough to debug it."  --Brian W. Kernighan

# HG changeset patch
# User Cory T. Tusar <[email protected]>
# Date 1169945686 18000
# Node ID e34c84e64c65931a9d5e8ebd2344b5e0a0fa9d25
# Parent 5f099a0fe6f0aac9e9b551bc36b3fdea8ed131e9
Support port replicator headphone output on Dell Latitude D820 + D/Port.

pci/hda/patch_sigmatel.c +2 -0
diff -r 5f099a0fe6f0 -r e34c84e64c65 pci/hda/patch_sigmatel.c
@@ -455,6 +455,8 @@ static struct snd_pci_quirk stac9200_cfg
 		      "Dell Latitude D620", STAC_REF),
 	SND_PCI_QUIRK(PCI_VENDOR_ID_DELL, 0x01cb,
 		      "Dell Latitude 120L", STAC_REF),
+	SND_PCI_QUIRK(PCI_VENDOR_ID_DELL, 0x01cc,
+		      "Dell Latitude D820", STAC_REF),
 	{} /* terminator */
 };
 

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your opinions on IT & business topics through brief surveys - and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV

_______________________________________________
Alsa-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/alsa-devel

#3 lore raw

At Tue, 30 Jan 2007 11:25:02 -0500,

Quoted 5 lines · Cory T. Tusar

[1 <text/plain; ISO-8859-1 (7bit)>]

Quoted 2 lines · Takashi Iwai

At Tue, 30 Jan 2007 10:56:14 -0500,
Cory T. Tusar wrote:

Quoted 7 lines
Quoted 4 lines
Quoted 2 lines

The attached patch adds support for the port replicator headphone output on a Dell Latitude D820 + D/Port.

I cannot read the patch. Could you attach or embed a patch text?

Hrmm...I sent it as an hg bundle as per the dev wiki. I'm not familiar with hg though, so I very well may have borked it.

Hm, it might be my use error. A text patch is always better to submit, because it makes eaiser to review and show the correction in the reply.

Anyway, I applied it to HG tree now. Thanks.

Takashi

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your opinions on IT & business topics through brief surveys - and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV