| Message ID | 20210923181520.8680-1-broonie@kernel.org (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | tpm_tis_spi: Add missing SPI ID | expand |
On Thu, 2021-09-23 at 19:15 +0100, 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 > 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") > Signed-off-by: Mark Brown <broonie@kernel.org> > --- > drivers/char/tpm/tpm_tis_spi_main.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/char/tpm/tpm_tis_spi_main.c b/drivers/char/tpm/tpm_tis_spi_main.c > index 54584b4b00d1..aaa59a00eeae 100644 > --- a/drivers/char/tpm/tpm_tis_spi_main.c > +++ b/drivers/char/tpm/tpm_tis_spi_main.c > @@ -267,6 +267,7 @@ static const struct spi_device_id tpm_tis_spi_id[] = { > { "st33htpm-spi", (unsigned long)tpm_tis_spi_probe }, > { "slb9670", (unsigned long)tpm_tis_spi_probe }, > { "tpm_tis_spi", (unsigned long)tpm_tis_spi_probe }, > + { "tpm_tis-spi", (unsigned long)tpm_tis_spi_probe }, > { "cr50", (unsigned long)cr50_spi_probe }, > {} > }; Reviewed-by: Jarkko Sakkinen <jarkko@kernel.org> I'll apply this. /Jarkko
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@redhat.com> Best regards,
diff --git a/drivers/char/tpm/tpm_tis_spi_main.c b/drivers/char/tpm/tpm_tis_spi_main.c index 54584b4b00d1..aaa59a00eeae 100644 --- a/drivers/char/tpm/tpm_tis_spi_main.c +++ b/drivers/char/tpm/tpm_tis_spi_main.c @@ -267,6 +267,7 @@ static const struct spi_device_id tpm_tis_spi_id[] = { { "st33htpm-spi", (unsigned long)tpm_tis_spi_probe }, { "slb9670", (unsigned long)tpm_tis_spi_probe }, { "tpm_tis_spi", (unsigned long)tpm_tis_spi_probe }, + { "tpm_tis-spi", (unsigned long)tpm_tis_spi_probe }, { "cr50", (unsigned long)cr50_spi_probe }, {} };
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 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") Signed-off-by: Mark Brown <broonie@kernel.org> --- drivers/char/tpm/tpm_tis_spi_main.c | 1 + 1 file changed, 1 insertion(+)