[HDA] When controller has multiple codecs, make sure we try to use the one with audio features. This should improve laptop support. More to come ;)

git-svn-id: file:///srv/svn/repos/haiku/haiku/trunk@23295 a95241bf-73f2-0310-859d-f6bbb57e9c96

src/add-ons/kernel/drivers/audio/hda/driver.h

@@ -190,6 +190,7 @@ struct hda_controller_s {
 	rirb_t*		rirb;
 
 	hda_codec*		codecs[HDA_MAXCODECS];
+	hda_codec*		active_codec;
 	uint32			num_codecs;
 	
 	hda_stream*		streams[HDA_MAXSTREAMS];

src/add-ons/kernel/drivers/audio/hda/hda_controller.c

@@ -499,7 +499,14 @@ hda_hw_init(hda_controller* ctrlr)
 		if (ctrlr->codecsts & (1 << idx))
 			hda_codec_new(ctrlr, idx);
 
-	if (ctrlr->codecs[0] != NULL)
+	for (idx=0; idx < HDA_MAXCODECS; idx++) {
+		if (ctrlr->codecs[idx] && ctrlr->codecs[idx]->num_afgs) {
+			ctrlr->active_codec = ctrlr->codecs[idx];
+			break;
+		}
+	}
+
+	if (ctrlr->active_codec != NULL)
 		return B_OK;
 	else
 		rc = ENODEV;

src/add-ons/kernel/drivers/audio/hda/hda_multi_audio.c

@@ -272,7 +272,7 @@ multi_audio_control(void* cookie, uint32 op, void* arg, size_t len)
 	hda_codec* codec = (hda_codec*)cookie;
 	hda_afg* afg;
 
-	/* FIXME: Make sure we have a valid codec & afg... */
+	/* FIXME: We should simply pass the afg into here... */
 	if (!codec || codec->num_afgs == 0)
 		return ENODEV;
 

src/add-ons/kernel/drivers/audio/hda/hooks.c

@@ -47,8 +47,8 @@ static status_t
 hda_control (void* cookie, uint32 op, void* arg, size_t len)
 {
 	hda_controller* hc = (hda_controller*)cookie;
-	if (hc->codecs[0])
+	if (hc->active_codec)
-		return multi_audio_control(hc->codecs[0], op, arg, len);
+		return multi_audio_control(hc->active_codec, op, arg, len);
 
 	return B_BAD_VALUE;
 }