On Sat, Aug 6, 2022 at 5:07 PM Ramon Fried <rfried....@gmail.com> wrote:
>
> On Thu, Jun 23, 2022 at 3:43 PM Vladimir Oltean <vladimir.olt...@nxp.com>
> wrote:
> >
> > On Tue, Jun 21, 2022 at 08:11:06AM -0700, Tim Harvey wrote:
> > > On Tue, Jun 21, 2022 at 12:21 AM Vladimir Oltean
> > > <vladimir.olt...@nxp.com> wrote:
> > > >
> > > > On Mon, Jun 20, 2022 at 04:37:45PM -0700, Tim Harvey wrote:
> > > > > On Mon, Jun 20, 2022 at 4:58 AM Vladimir Oltean
> > > > > <vladimir.olt...@nxp.com> wrote:
> > > > > >
> > > > > > On Mon, May 23, 2022 at 11:25:48AM -0700, Tim Harvey wrote:
> > > > > > > +/* bind and probe the switch mdios */
> > > > > > > +static int mv88e61xx_dsa_probe_mdio(struct udevice *dev)
> > > > > > > +{
> > > > > > > + struct udevice *pdev;
> > > > > > > + ofnode node, mdios;
> > > > > > > + const char *name;
> > > > > > > + int ret;
> > > > > > > +
> > > > > > > + /* bind phy ports of mdios child node to mv88e61xx_mdio
> > > > > > > device */
> > > > > > > + mdios = dev_read_subnode(dev, "mdios");
> > > > > > > + if (ofnode_valid(mdios)) {
> > > > > > > + ofnode_for_each_subnode(node, mdios) {
> > > > > > > + name = ofnode_get_name(node);
> > > > > > > + ret = device_bind_driver_to_node(dev,
> > > > > > > +
> > > > > > > "mv88e61xx_mdio",
> > > > > > > + name,
> > > > > > > node, &pdev);
> > > > > > > + if (ret) {
> > > > > > > + dev_err(dev, "failed to bind %s:
> > > > > > > %d\n", name, ret);
> > > > > > > + continue;
> > > > > > > + }
> > > > > > > +
> > > > > > > + /* need to probe it as there is no
> > > > > > > compatible to do so */
> > > > > > > + ret =
> > > > > > > uclass_get_device_by_ofnode(UCLASS_MDIO, node, &pdev);
> > > > > > > + if (ret) {
> > > > > > > + dev_err(dev, "failed to probe %s:
> > > > > > > %d\n", name, ret);
> > > > > > > + continue;
> > > > > > > + }
> > > > > >
> > > > > > What do you do with this pdev once you get it? Are you missing a
> > > > > > device_probe() call?
> > > > > > Also, why "pdev" and not "dev"? What does the "p" stand for?
> > > > >
> > > > > struct udevice *dev is passed into the function so I use pdev to
> > > > > iterate over the ports in the mdios node so 'pdev' means 'port' here.
> > > >
> > > > Yes, but those under the mdios node aren't ports, they're MDIO
> > > > controllers, hence my comment.
> > >
> > > how about devp (dev pointer) or subdev or mdio?
> >
> > The terminology problem I have with "mdio" is that it would be a
> > struct udevice *, whereas the plural variable, "mdios", is an ofnode.
> > How about mdev, for mdio device?
> Tim ?
Hi Ramon,
I haven't had time to get back to this yet. I hope to do so in the
next few weeks.
Best Regards,
Tim
