Re: [PATCH 2/2] powerpc/85xx: add Fman MDIO muxing support to the P4080DS

On 08/14/2012 04:48 PM, Timur Tabi wrote:

Kumar Gala wrote:

quoted

quoted

quoted

+	{
+		/*
+		 * Warning: this entry might need to be located before those
+		 * for the Fman Ethernet nodes.
+		 */
+		.compatible	= "mdio-mux",
+	},
	{}
};

Under what condition would that be the case?

We had this discussion already.

I think that was internally, and not on this specific comment wording.
I don't think that code comment adequately explains things.

otherwise, the mdio-mux code would not prepare the mdio mus in time, and
there would be initialization failures.  Now maybe this goes away with
-EPROBE_DEFER, or maybe it doesn't.  But until we push the DPAA drivers
upstream, we won't know.

Do you know if it's theoretically supposed to be fixed and just can't
test it, or are you unsure of whether it's even supposed to work?

I don't think we should be relying on the order of this list to
determine probe order.  For one thing, it won't work if the drivers
register after you create the platform devices (e.g. they're modules).

-Scott