Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

snd_hda(4) pin fixes realtek ALC286 #149

Closed
wants to merge 51 commits into from

Conversation

kstaring
Copy link
Contributor

As with previous patches, earlier branches are also contained in this one. Also, pin patches were ported provided the Linux sources were clear-- entries that looked suspiciously like human error (e.g. double entries, unknown pci vendor ids) weren't ported.

Maybe. Let's see if this is enough to get output working.
- Integrated Realtek patches into new structure.
- Ported Linux ALC880 pin patches.

Missing: GPIO and function patches. Some entries on Linux did not have a valid
name in Linux' SND_PCI_QUIRK definition. Did not port NULL-name SND_PCI_QUIRKs.
Also, unidentified vendor ids and vendor generic patches were not ported yet.
Skipped some out-ifdef'd code.
  the ALC269 table, ignored lots of other non-ALC269 entries)

Ignored some entries such as unidentified PCI vendor ids and a lot of
entries for other ALC series in the alc269_fixit array. For some reason
a lot of ALC patches are all thrown into the alc269 definitions-- messy.
The LSB in the pin configuration is not honoured yet, that's why some
pin configs weren't converted.
Merge branch 'hda_pin_fixes_realtek_alc260_268' into hda_pin_fixes_realtek_alc269
@bsdimp
Copy link
Member

bsdimp commented Jun 2, 2021

I've taken all these pull requests, squashed them and forward ported them to -current. https://reviews.freebsd.org/D30619 is the review. As such, I'm going to close all these here.

@bsdimp bsdimp closed this Jun 2, 2021
freebsd-git pushed a commit that referenced this pull request Jul 3, 2021
A number of structural changes:
  - Use decimal nid numbers instead of hex
  - updated the branch to incoorporate the suggestions made in the
    ALC280 pull request github thread
  - Convert magic pin values into strings.
  - Also update hdaa_patches to use clearer enums..
  - made pin patch type enum clearer, add macro for 'string' type
    patches
  - Added pin_patch structures to separate data from logic.
  - Integrated Realtek patches into new structure.

These incorporate fixes for ALC255, ALC256, ALC260, ALC262, ALC268,
ALC269, ALC280, ALC282, ALC283, ALC286, ALC290, ALC293, ALC296, ALC2880

And have definitions for a number of Dell and HP laptops.

Much of this data has been mined fromt he tables in the Linux driver.

imp squashed these into one commit because the changes from the github
pull requests no longer cleanly apply individually and made light style
changes after feedback from jhb.

Pull Request:		#139
Pull Request:		#140
Pull Request:		#141
Pull Request:		#142
Pull Request:		#143
Pull Request:		#144
Pull Request:		#145
Pull Request:		#146
Pull Request:		#147
Pull Request:		#148
Pull Request:		#149
Pull Request:		#150
Differential Revision:	https://reviews.freebsd.org/D30619
bsdjhb pushed a commit to bsdjhb/cheribsd that referenced this pull request Dec 14, 2021
A number of structural changes:
  - Use decimal nid numbers instead of hex
  - updated the branch to incoorporate the suggestions made in the
    ALC280 pull request github thread
  - Convert magic pin values into strings.
  - Also update hdaa_patches to use clearer enums..
  - made pin patch type enum clearer, add macro for 'string' type
    patches
  - Added pin_patch structures to separate data from logic.
  - Integrated Realtek patches into new structure.

These incorporate fixes for ALC255, ALC256, ALC260, ALC262, ALC268,
ALC269, ALC280, ALC282, ALC283, ALC286, ALC290, ALC293, ALC296, ALC2880

And have definitions for a number of Dell and HP laptops.

Much of this data has been mined fromt he tables in the Linux driver.

imp squashed these into one commit because the changes from the github
pull requests no longer cleanly apply individually and made light style
changes after feedback from jhb.

Pull Request:		freebsd/freebsd-src#139
Pull Request:		freebsd/freebsd-src#140
Pull Request:		freebsd/freebsd-src#141
Pull Request:		freebsd/freebsd-src#142
Pull Request:		freebsd/freebsd-src#143
Pull Request:		freebsd/freebsd-src#144
Pull Request:		freebsd/freebsd-src#145
Pull Request:		freebsd/freebsd-src#146
Pull Request:		freebsd/freebsd-src#147
Pull Request:		freebsd/freebsd-src#148
Pull Request:		freebsd/freebsd-src#149
Pull Request:		freebsd/freebsd-src#150
Differential Revision:	https://reviews.freebsd.org/D30619
emaste pushed a commit to emaste/freebsd that referenced this pull request Mar 24, 2022
A number of structural changes:
  - Use decimal nid numbers instead of hex
  - updated the branch to incoorporate the suggestions made in the
    ALC280 pull request github thread
  - Convert magic pin values into strings.
  - Also update hdaa_patches to use clearer enums..
  - made pin patch type enum clearer, add macro for 'string' type
    patches
  - Added pin_patch structures to separate data from logic.
  - Integrated Realtek patches into new structure.

These incorporate fixes for ALC255, ALC256, ALC260, ALC262, ALC268,
ALC269, ALC280, ALC282, ALC283, ALC286, ALC290, ALC293, ALC296, ALC2880

And have definitions for a number of Dell and HP laptops.

Much of this data has been mined fromt he tables in the Linux driver.

imp squashed these into one commit because the changes from the github
pull requests no longer cleanly apply individually and made light style
changes after feedback from jhb.

Pull Request:		freebsd#139
Pull Request:		freebsd#140
Pull Request:		freebsd#141
Pull Request:		freebsd#142
Pull Request:		freebsd#143
Pull Request:		freebsd#144
Pull Request:		freebsd#145
Pull Request:		freebsd#146
Pull Request:		freebsd#147
Pull Request:		freebsd#148
Pull Request:		freebsd#149
Pull Request:		freebsd#150
Differential Revision:	https://reviews.freebsd.org/D30619

(cherry picked from commit ef790cc)
(cherry picked from commit 29b4fa7)
(cherry picked from commit a10c85e)
(cherry picked from commit 86d69de)
freebsd-git pushed a commit that referenced this pull request Mar 24, 2022
A number of structural changes:
  - Use decimal nid numbers instead of hex
  - updated the branch to incoorporate the suggestions made in the
    ALC280 pull request github thread
  - Convert magic pin values into strings.
  - Also update hdaa_patches to use clearer enums..
  - made pin patch type enum clearer, add macro for 'string' type
    patches
  - Added pin_patch structures to separate data from logic.
  - Integrated Realtek patches into new structure.

These incorporate fixes for ALC255, ALC256, ALC260, ALC262, ALC268,
ALC269, ALC280, ALC282, ALC283, ALC286, ALC290, ALC293, ALC296, ALC2880

And have definitions for a number of Dell and HP laptops.

Much of this data has been mined fromt he tables in the Linux driver.

imp squashed these into one commit because the changes from the github
pull requests no longer cleanly apply individually and made light style
changes after feedback from jhb.

Pull Request:		#139
Pull Request:		#140
Pull Request:		#141
Pull Request:		#142
Pull Request:		#143
Pull Request:		#144
Pull Request:		#145
Pull Request:		#146
Pull Request:		#147
Pull Request:		#148
Pull Request:		#149
Pull Request:		#150
Differential Revision:	https://reviews.freebsd.org/D30619

(cherry picked from commit ef790cc)
(cherry picked from commit 29b4fa7)
(cherry picked from commit a10c85e)
(cherry picked from commit 86d69de)
@emaste emaste added the merged label Jun 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
3 participants