On Fri, 2017-02-17 at 09:43 -0700, Jason Gunthorpe wrote: > On Thu, Feb 16, 2017 at 10:36:40PM +0000, Winkler, Tomas wrote: > > This is a BIOS issue + kernel ACPI platform driver refusing to > > handle > > overlapping memory space. The issue here is that the ACPI > > platform > > driver is not created, however this should not prevent the > > tpm_crb > > driver from working as it registers directly the ACPI device. > > For now > > you can suppress the message by adding the device in > > forbidden_id_list > > in drivers/acpi/acpi_platform.c > > > > > > I'm already testing a fixed BIOS, not sure whether and when it > > will be > > available for your particular device. > > Do you think we should go ahead with the patch I sent? Or should we > tell users they need a BIOS update? > > Jason
I would rather review a patch with proper commit message than just pick from an email thread in all cases with a proper explanation what the patch does and why it is the right way to fix the issue. I got an email privately and identified it as the very same issue. It was in a NUC and the author said that he had updated the BIOS to the latest version. That supports the view of doing some kind of workaround to the driver, perhaps the way you did it. /Jarkko ------------------------------------------------------------------------------ Check out the vibrant tech community on one of the world's most engaging tech sites, SlashDot.org! http://sdm.link/slashdot _______________________________________________ tpmdd-devel mailing list tpmdd-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/tpmdd-devel
