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/ | -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/