Commit 55f89a8a authored Nov 30, 2015 by Mika Westerberg Committed by Rafael J. Wysocki Dec 07, 2015

driver core: Do not overwrite secondary fwnode with NULL if it is set



If multiple devices share single firmware node like it is case with MFD
devices, the same firmware node (ACPI) is assigned to all of them. The
function also modifies the shared firmware node in order to preserve
secondary firmware node of the device in question.

If the new device which is sharing the firmware node does not have
secondary node it will be NULL which will be assigned to the secondary node
of the shared firmware node losing all built-in properties.

Prevent this by setting the secondary firmware node only if the replacement
is non-NULL.

Print also warning if someone tries to overwrite secondary node that has
already been assigned.

Signed-off-by: Mika Westerberg <mika.westerberg@linux.intel.com>
Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>

parent 00bbc1d8

drivers/base/core.c

+4 −1

Original line number	Diff line number	Diff line
		@@ -2261,7 +2261,10 @@ void set_primary_fwnode(struct device dev, struct fwnode_handle fwnode)
		if (fwnode_is_primary(fn))
		fn = fn->secondary;

		if (fn) {
		WARN_ON(fwnode->secondary);
		fwnode->secondary = fn;
		}
		dev->fwnode = fwnode;
		} else {
		dev->fwnode = fwnode_is_primary(dev->fwnode) ?

Admin message