Hi George,

On Fri, May 20, 2016 at 11:05 PM, George McCollister george.mccollister@gmail.com wrote:

I'm working with an Advantech som-6867 (Baytrail com express CPU module) and som-db5800 carrier board (with ALC892 HD audio codec) as well as a custom carrier board (with CS4207 HD audio codec).

After adding DT entries for the correct HDA pad configuration, I got the ALC892 working by hacking over ALC262 verb table in arch/x86/cpu/baytrail/fsp_configs.c and am now investigating a permanent solution.

I noticed in Bin's patchset titled "x86: fsp: Move platform-specific config to chipset directory" his comment "But with this series, we are now able to get Azalia verb table from device tree, just like what we did for overriding UPD data configuration. This can be done in future patch set."

I also however noticed Simon's conflicting comment in arch/x86/cpu/baytrail/fsp_configs.c "This needs to be in ROM since if we put it in CAR, FSP init loses it when it drops CAR."

Guess the comment is out of date.

Is it safe to move Azalia verb tables into DT for Baytrail FSP?

Yes it is safe to move verb tables to DT, but I have not looked into this. Patches are welcome.

Regards, Bin