diff options
| author | Arthur Heymans <arthur@aheymans.xyz> | 2023-08-22 13:16:40 +0200 |
|---|---|---|
| committer | Felix Held <felix-coreboot@felixheld.de> | 2023-09-15 13:46:46 +0000 |
| commit | 04c49a5e7cd92feb761d0d861472f12ccf6fc23b (patch) | |
| tree | 8ed4bc8c1c703f5cdc6e9d74143a7cb46abe6e5e | |
| parent | 77d8e0bec69f67e2eea84aeeee64abe7b470ef71 (diff) | |
drivers/pc80/tpm: Silence warnings when CONFIG_TPM is unset
Add some noop device_operations for devices below chip drivers/pc80/tpm
in the case CONFIG_TPM is unset. This avoids the warning "... missing
read_resources" when probing all devices for resources.
Signed-off-by: Arthur Heymans <arthur@aheymans.xyz>
Change-Id: Ibc37642a8b5f37b95c648141e03025a04ffa36bc
Reviewed-on: https://review.coreboot.org/c/coreboot/+/77386
Reviewed-by: Eric Lai <eric_lai@quanta.corp-partner.google.com>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
| -rw-r--r-- | src/drivers/pc80/tpm/tis.c | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/src/drivers/pc80/tpm/tis.c b/src/drivers/pc80/tpm/tis.c index ed9c8d0e0b..1ae6f1dc92 100644 --- a/src/drivers/pc80/tpm/tis.c +++ b/src/drivers/pc80/tpm/tis.c @@ -832,6 +832,11 @@ static struct device_operations lpc_tpm_ops = { #endif }; +static struct device_operations noop_tpm_ops = { + .read_resources = noop_read_resources, + .set_resources = noop_set_resources, +}; + static struct pnp_info pnp_dev_info[] = { { .flags = PNP_IRQ0 } }; @@ -841,6 +846,8 @@ static void enable_dev(struct device *dev) if (CONFIG(TPM)) pnp_enable_devices(dev, &lpc_tpm_ops, ARRAY_SIZE(pnp_dev_info), pnp_dev_info); + else + pnp_enable_devices(dev, &noop_tpm_ops, ARRAY_SIZE(pnp_dev_info), pnp_dev_info); } struct chip_operations drivers_pc80_tpm_ops = { |