> -----Original Message----- > From: netdev-ow...@vger.kernel.org [mailto:netdev-ow...@vger.kernel.org] > On Behalf Of Madalin-cristian Bucur > Sent: Wednesday, October 04, 2017 12:54 PM > To: David Miller <da...@davemloft.net> > Subject: RE: [PATCH] fsl/fman: remove of_node > > > -----Original Message----- > > From: David Miller [mailto:da...@davemloft.net] > > Sent: Wednesday, October 04, 2017 7:44 AM > > To: Madalin-cristian Bucur <madalin.bu...@nxp.com> > > Cc: netdev@vger.kernel.org; and...@lunn.ch; f.faine...@gmail.com; linux- > > ker...@vger.kernel.org > > Subject: Re: [PATCH] fsl/fman: remove of_node > > > > From: Madalin-cristian Bucur <madalin.bu...@nxp.com> > > Date: Tue, 3 Oct 2017 08:49:31 +0000 > > > > > My patch removes the of_node that was set to a device that was not an > > > of_device, preventing duplicated probing of both the real of_device > > > and the "fake" one created through this assignment. > > > > > > I understand that the DSA issue that triggered the initial change > > > was related to DSA finding the network devices using > > > of_find_net_device_by_node(), something that will not work for the > > > DPAA case where the netdevice does not have an of_node. I do not know > > > enough about DSA to come up with a solution for this problem now. > > > Andrew, Florian, can you please comment on this? > > > > It sounds like you're knowingly breaking DSA. > > It never worked, even with the change I'm reverting.
I'll resend this change as part of a patch set that changes the device used as net_dev dev to ensure DSA will find a of_device there. To make that work some changes to adjust link (that also make it cleaner) were needed. Also, to keep the old udev rules happy, I've changed the names of the FMan kernel modules from fsl_fman_* to fsl_dpaa_fman*. I do not have a DSA setup to test so I just tested the part related to of_find_net_device_by_node() being able to determine the net_device based on a device tree handle using an artificial device tree and code construct. I hope that will help the initial reporter of the DSA issue on DPAA (Junote Cai). Madalin
