Hi
Op 03-12-2021 om 23:58 schreef Pierre-Louis Bossart:
I would guess that indeed a return 0; is missing, but maybe it's time to remove this PCI code completely. I can't think of any user of the PCI parts of this driver.
Andy, Hans, Mark, Takashi, what do you think?
The Edison platform and actually some more based on Intel Merrifield are still alive and on the (second hand) market. But yes, I would rather focus on making SOF working there, but via PCI bus (or with ACPI, ASL code for which one should actually write down, currently it's a device with PCI interface only).
That said, Pierre, have you been able to setup your Edison to see anything on I2S with SOF?
I checked with oscilloscope signal was present but did not connect any actual codec.
No, I haven't touched my Edison boards since the initial integration with you and Ferry in 2020. I see that the firmware was updated to 1.8 and the kernel is 5.10+ now, so that should be easier than last time. We don't really need any change for the driver probe, PCI is just fine, what's missing is an ACPI recipe to enable audio functionality over the SSP pins.