Opened 17 years ago

Closed 4 years ago

#57 closed defect (too-old)

ALSA source tree includes driver modules which do not compile

Reported by: Brendan Oakley Owned by:
Priority: low Milestone: AlsaResync
Component: Building Version:
Severity: normal Keywords:
Cc:

Description

In the makefile.os2's of the drivers, isa, and pci directories under alsa-kernel, there are driver source files listed that have been commented out. Edits have been made at some point to those source files, so a port was at least begun. AFAICT there does not appear to be support for these in the other places necessary to make them work even if they were not commented out of the makefiles.

An attempt to compile pci/ice1712 after the merge to 0.9.0rc4 failed with errors.

The problems with this scenario are:

  1. Hardware that could be supported by Uniaud is not.
  1. Uniaud is now tailored to work only with a few driver files with significant modifications. Tailoring it instead to work with a greater number of ALSA drivers with fewer modifications would reduce the work required to add support for new hardware, and reduce the chances for bugs peculiar to each hardware configuration.
  1. The present AlsaResync is merging these source files without any testing, which leaves room for error and reduces or nullifies the benefit of the work.

My proposed action at this point is to test whether these files compile successfully before the resync, and whether that has been broken by any stage of the resync, fixing as needed. Even though this does not in itself enable support for the hardware, it preserves that potential, reduces merge error, and also provides more clues about what in Uniaud outside of alsa-kernel might need done to improve general compatibility with ALSA.

Change History (3)

comment:1 by Brendan Oakley, 17 years ago

Milestone: AlsaResync

comment:2 by martini, 4 years ago

Hi

This ticket is too old. Please try the latest version and if the problem persist, please reopen the ticket.

Regards.

comment:3 by martini, 4 years ago

Resolution: too-old
Status: newclosed
Note: See TracTickets for help on using tickets.