Hi Adam,

One minor note below if you end up re-rolling for other reasons, but
regardless of that:

Acked-by: Jeremy Kerr <jk@codeconstruct.com.au>

+       /* ndev needs to be freed before the iomemory (mapped above) gets
+        * unmapped,  devm resources get freed in reverse to the order they
+        * are added.
+        */
+       rc = mctp_register_netdev(ndev, &mctp_netdev_ops, MCTP_PHYS_BINDING_PCC);
+       if (rc)
+               goto cleanup_netdev;
+       return  devm_add_action_or_reset(dev, mctp_cleanup_netdev, ndev);

You seem to have a trigger-happy spacebar; there is another double-space
creeping in here, after the 'return'.

Cheers,


Jeremy