> @@ -138,8 +136,10 @@ int coreboot_table_init(struct device *dev, void __iomem > *ptr) > ptr_entry += entry.size; > } > > - if (ret) > + if (ret) { > + bus_unregister(&coreboot_bus_type); > iounmap(ptr); > + }
nit: maybe cleaner to just do if (ret) coreboot_table_exit(); here? You're essentially writing the same code again.