Messages in this thread |  | | From | Robert Jarzmik <> | Subject | Re: [PATCH 1/4] ASoC: wm9713: add binding for WM9713 codec | Date | Sat, 20 Feb 2016 19:22:04 +0100 |
| |
Mark Brown <broonie@kernel.org> writes:
> On Sat, Feb 20, 2016 at 03:37:56PM +0100, Robert Jarzmik wrote: > >> +WM9713 audio CODEC >> + >> +This devices supports I2C. > > No, it clearly doesn't... Right, it supports AC97.
> The problem with doing this is that since AC'97 is an enumerable bus we really > shouldn't need to list AC'97 CODECs in the device tree. Ok, I understand that.
> Instead we should be probing at runtime (as the non-ASoC AC'97 code does) or > something similar. When you say "non-ASoC AC'97 code", which file are you referring to ? Is it sound/pci/ac97/ac97_codec.c ?
If so is there already a table of tuples (AC97_VENDOR_ID1, AC97_VENDOR_ID2) -> (platform device, platform device data) and a matching mechanism already available to the ASoC drivers ?
Cheers.
-- Robert
|  |