Messages in this thread | | | Date | Tue, 1 Dec 2015 20:52:17 +0100 | From | Uwe Kleine-König <> | Subject | Re: [PATCH v2 2/3] tpm_tis: Use devm_ioremap_resource |
| |
Hello Jason,
On Tue, Dec 01, 2015 at 12:44:19PM -0700, Jason Gunthorpe wrote: > On Tue, Dec 01, 2015 at 08:22:40PM +0100, Uwe Kleine-König wrote: > > > + if (resource_size(&tpm_info.res) == 0) > > > + return -ENODEV; > > > + > > > > Does this result in an error message from the upper layers? > > I think so, yes. The probe will fail which causes the driver core to > report a message. > > The scenario this triggers is if the acpi stuff doesn't have a mem > resource, which is a firmware bug, I think. It could get a dedicated > print if that is what you are thinking?
The issue I saw is: There are three(?) ways the tpm could be bound. If one of the succeeds, the other two are expected to fail. But in this case an error message, that the tpm failed to be bound is at least misleading.
Best regards Uwe
-- Pengutronix e.K. | Uwe Kleine-König | Industrial Linux Solutions | http://www.pengutronix.de/ |
| |