Opened 8 years ago

Last modified 4 years ago

#6798 new bug

Master Ticket: Updating audio drivers to co-exist with OpenSound

Reported by: mmadia Owned by: mmu_man
Priority: normal Milestone: Unscheduled
Component: Drivers/Audio/OpenSound Version: R1/Development
Keywords: Cc:
Blocked By: Blocking:
Has a Patch: no 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 (8)

comment:1 Changed 8 years ago by mmadia

Description: modified (diff)

comment:2 in reply to:  description Changed 8 years ago by siarzhuk

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 Changed 8 years ago by mmadia

Description: modified (diff)

comment:4 Changed 8 years ago by mmadia

Description: modified (diff)

comment:5 in reply to:  description Changed 8 years ago by siarzhuk

Replying to mmadia:

  • sis7018 (don't do. see comments)

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

comment:6 Changed 8 years ago by mmu_man

Description: modified (diff)

comment:7 Changed 7 years ago by diver

Component: Drivers/AudioDrivers/Audio/OpenSound
Owner: changed from korli to mmu_man

comment:8 Changed 4 years ago by luroh

Milestone: R1Unscheduled

Moving out of R1 milestone (Prop #26).

Note: See TracTickets for help on using tickets.