Hi Doug,

thanks for tackling this issue! I agree it should be fixed, just
wondered about one thing:

While flags are often modified while under the "fwnode_link_lock",
this is not universally true.

Is it a possibility to use the lock in all code paths instead?
Because...

 	struct list_head consumers;
-	u8 flags;
+	unsigned long flags;

... this change costs some memory on every system. Maybe it can be
avoided?

Happy hacking,

   Wolfram