Hello Mark,
I agree with your patch but the commit message is not completely accurate.
On 9/23/21 8:15 PM, Mark Brown wrote:
> In commit c46ed2281bbe ("tpm_tis_spi: add missing SPI device ID entries")
> we added SPI IDs for all the DT aliases to handle the fact that we always
> use SPI modaliases to load modules even when probed via DT however this
> added an alias for the non-idiomatic tpm_tis-spi compatible as tpm_tis_spi
The "tpm_tis_spi" entry was not introduced by the mentioned commit but it
has been in the driver since commit 0edbfea537d ("tpm/tpm_tis_spi: Add
support for spi phy").
What was missed in commit c46ed2281bbe was that the entries in the OF and
SPI tables didn't match and were different. So maybe the following instead:
"... however the mentioned commit missed that the SPI and OF device ID
entries did not match and were different and so DT nodes with compatible
"tcg,tpm_tis-spi" will not match".
> which doesn't actually match. Add an extra ID for tpm_tis-spi rather than
> just fix the existing one since what's currently there is going to be
> better for anyone actually using SPI IDs to instantiate.
>
> Fixes: c46ed2281bbe ("tpm_tis_spi: add missing SPI device ID entries")
> Fixes: 96c8395e2166 ("spi: Revert modalias changes")
Oh, I missed that the SPI core was modified to report OF modalias uevents
at some point. Yes, that won't work unless all the SPI drivers are fixed.
Anyway, for the change itself:
Reviewed-by: Javier Martinez Canillas <javierm@xxxxxxxxxx>
Best regards,
--
Javier Martinez Canillas
Linux Engineering
Red Hat
