[alsa-devel] Fwd: [PATCH 1/1] SPI : spi-pxa2xx : fix spi init of WM510205 codec via ACPI (resend)

[Christian Hartmann]

Hi,

@all

FYI: I can confirm that with the changed DSDT.dsl file and loading
the new custom DSDT table, I do not need this patch mentioned above.

So the patch is indeed only needed, as long as there is no
BIOS (DSDT change by manufacturer) update available.

In this case special case a custom DSDT table is loaded to
 fix the by ACPI wrongly set of SPIController to  1, which must be 0.

On the other hand I have now an tained kernel too :/ cause of the unset
"Select only drivers that don't need compile-time external firmware" and
 the ''Include Custom DSDT' options.


The dmesg message seems to be the same as with the patch applied but
with the current system DSDT table: here

[    0.000000] Linux version 4.1.0.60-rc0
(b23 at berkelium.fyedot.greenzone) (gcc version 4.8.3 20140911 (Red Hat
4.8.3-9) (GCC) ) #199 SMP Fri Jun 26
13:47:46 CEST 2015
...
[    0.000000] ACPI: Early table checksum verification disabled
...
[    5.916597] pxa2xx-spi 80860F0E:00: no DMA channels available, using PIO
[    5.916733] pxa2xx-spi 80860F0E:00: registered master spi32766 (dynamic)
[    5.926671] spi spi-WM510205:00: 8333333 Hz actual, PIO
[    5.926681] spi spi-WM510205:00: setup mode 0, 8 bits/w, 8000000 Hz max --> 0
[    5.926805] spi spi-WM510205:00: checking WM510205 with bmp180
[    5.926812] spi spi-WM510205:00: checking WM510205 with bmp181
[    5.926816] spi spi-WM510205:00: modalias WM510205 in id_table not
found, returns NULL
[    5.926843] arizona spi-WM510205:00: acpi_match_device() first,
than via spi_get_device_id().
[    5.926849] arizona spi-WM510205:00: matched ACPI ID and data
[    5.926853] arizona spi-WM510205:00: using 1 as type for arizona audio codec
[    5.926857] arizona spi-WM510205:00: regmap set to wm5102_spi
[    5.927665] arizona spi-WM510205:00: arizona_spi_probe done, call
and return of  arizona_dev_init
[    5.927759] spi-WM510205:00 supply AVDD not found, using dummy regulator
[    5.927782] spi-WM510205:00 supply DBVDD1 not found, using dummy regulator
[    5.927801] spi-WM510205:00 supply DCVDD not found, using dummy regulator
[    5.928958] arizona spi-WM510205:00: Unknown device ID: ffff
[    5.929098] pxa2xx-spi 80860F0E:00: registered child spi-WM510205:00

notice : some of these messages above are coming from local added dev_err();

cheers
chris


2015-06-26 13:58 GMT+02:00 Mark Brown <broonie at kernel.org>:
> On Fri, Jun 26, 2015 at 01:46:50PM +0200, Pierre-Louis Bossart wrote:
>> On 6/25/15 5:44 PM, Charles Keepax wrote:
>
>> >I think from the Linux side we can safely ignore those last two
>> >digits. I expect that all systems of the same make and model
>> >would report the same last two digits but that it might change
>> >between models if the Windows driver is expected to something
>> >differently for that model.
>
>> Here's the summary of what a valid _HID  should look like. It took me a
>> couple of days to gather the rules from different sources, I hope it helps
>> others figure things out.
>
> None of which really affact the matching that the kernel does since
> whatever form the string is generated in we need to do a full string
> match (nor do they correspond to what it seems the Wolfson drivers have
> invented but that's a separate issue).  Yay standards.