Opened 14 years ago

Last modified 19 months ago

#6798 new bug

Master Ticket: Updating audio drivers to co-exist with OpenSound — at Version 6

Reported by: mmadia Owned by: korli
Priority: normal Milestone: Unscheduled
Component: Drivers/Audio/OpenSound Version: R1/Development
Keywords: Cc:
Blocked By: Blocking:
Platform: All

Description (last modified by mmu_man)

This is a master ticket for tracking the updates of the various audio drivers, such that they can co-exist with OpenSound. (un)reserve_device needs to be utilized by Haiku's drivers, such as browser:haiku/trunk/src/add-ons/kernel/drivers/audio/hda/driver.cpp#L56

		if (info.class_base == PCI_multimedia
			&& info.class_sub == PCI_hd_audio) {
#ifdef __HAIKU__
			if ((*gPci->reserve_device)(info.bus, info.device, info.function, "hda",
				&gCards[gNumCards]) < B_OK) {
				dprintf("HDA: Failed to reserve PCI:%d:%d:%d\n",
					info.bus, info.device, info.function);
				continue;
			}
#endif

and browser:haiku/trunk/src/add-ons/kernel/drivers/audio/hda/driver.cpp#L95

#ifdef __HAIKU__
		(*gPci->unreserve_device)(gCards[i].pci_info.bus, gCards[i].pci_info.device,
			gCards[i].pci_info.function, "hda", &gCards[i]);
#endif

Appears to utilize (un)reserve_device

Does not appear to utilize (un)reserve_device

  • ac97/geode
  • ac97/ich
  • ac97/ichaudio
  • module_driver
  • null (don't do. doesn't use PCI)
  • sb16 (don't do. ISA)
  • sis7018 (don't do. see comments)
  • usb_audio (don't do. see comments)

note: listing the devices here for now. new tickets could be created as patches are submitted for each device

note: it appears some drivers' uninit_driver() do not call unreserve_device

Change History (6)

comment:1 by mmadia, 14 years ago

Description: modified (diff)

in reply to:  description comment:2 by siarzhuk, 14 years ago

Replying to mmadia:

Do not spend time patching following:

  • sis7018

This one is legacy R3 compatible driver. Upcoming version will use multiaudio anyway. As soon as I have intention to finish it in my deveveloper branch. ;-)

  • usb_audio

This one doesn't share any resources on PCI due it's nature. :-D And will be replaced too after Haiku USB bus managers become full isochronous transfer support.

comment:3 by mmadia, 14 years ago

Description: modified (diff)

comment:4 by mmadia, 14 years ago

Description: modified (diff)

in reply to:  description comment:5 by siarzhuk, 14 years ago

Replying to mmadia:

  • sis7018 (don't do. see comments)

Mentioned in my comment "branch version" of sis7018 driver updated in hrev39350.

comment:6 by mmu_man, 14 years ago

Description: modified (diff)
Note: See TracTickets for help on using tickets.