From: Calvin Johnson <linux.cj@gmail.com>
To: Florian Fainelli <f.fainelli@gmail.com>
Cc: Calvin Johnson <calvin.johnson@nxp.com>,
Jon Nettleton <jon@solid-run.com>,
linux@armlinux.org.uk, Makarand Pawagi <makarand.pawagi@nxp.com>,
cristian.sovaiala@nxp.com, laurentiu.tudor@nxp.com,
ioana.ciornei@nxp.com, V.Sethi@nxp.com, pankaj.bansal@nxp.com,
"Rajesh V . Bikkina" <rajesh.bikkina@nxp.com>,
Calvin Johnson <calvin.johnson@oss.nxp.com>,
"David S. Miller" <davem@davemloft.net>,
Madalin Bucur <madalin.bucur@oss.nxp.com>,
linux-kernel@vger.kernel.org, netdev@vger.kernel.org
Subject: Re: [PATCH v1 3/7] net/fsl: add ACPI support for mdio bus
Date: Wed, 5 Feb 2020 00:16:39 +0530 [thread overview]
Message-ID: <CAEhpT-VfWhha3B6qS6wFu-PQP9+zb4gxt7=nsrA58cecX21Dog@mail.gmail.com> (raw)
In-Reply-To: <6501a53b-40aa-5374-3c4a-6b21824f82fd@gmail.com>
On Sun, Feb 02, 2020 at 07:44:40PM -0800, Florian Fainelli wrote:
>
>
> On 1/31/2020 7:34 AM, Calvin Johnson wrote:
> > From: Calvin Johnson <calvin.johnson@oss.nxp.com>
> >
> > Add ACPI support for MDIO bus registration while maintaining
> > the existing DT support.
> >
> > Signed-off-by: Calvin Johnson <calvin.johnson@oss.nxp.com>
> > ---
>
> [snip]
>
> > bus = mdiobus_alloc_size(sizeof(struct mdio_fsl_priv));
> > @@ -263,25 +265,41 @@ static int xgmac_mdio_probe(struct platform_device *pdev)
> > bus->read = xgmac_mdio_read;
> > bus->write = xgmac_mdio_write;
> > bus->parent = &pdev->dev;
> > - snprintf(bus->id, MII_BUS_ID_SIZE, "%llx", (unsigned long long)res.start);
> > + snprintf(bus->id, MII_BUS_ID_SIZE, "%llx",
> > + (unsigned long long)res->start);
>
> You could omit this clean up change.
Sure, will avoid split to newline.
> >
> > /* Set the PHY base address */
> > priv = bus->priv;
> > - priv->mdio_base = of_iomap(np, 0);
> > + priv->mdio_base = devm_ioremap_resource(&pdev->dev, res);
> > if (!priv->mdio_base) {
>
> This probably needs to become IS_ERR() instead of a plain NULL check
Ok. Will take care in v2.
>
> > ret = -ENOMEM;
> > goto err_ioremap;
> > }
> >
> > - priv->is_little_endian = of_property_read_bool(pdev->dev.of_node,
> > - "little-endian");
> > -
> > - priv->has_a011043 = of_property_read_bool(pdev->dev.of_node,
> > - "fsl,erratum-a011043");
> > -
> > - ret = of_mdiobus_register(bus, np);
> > - if (ret) {
> > - dev_err(&pdev->dev, "cannot register MDIO bus\n");
> > + if (is_of_node(pdev->dev.fwnode)) {
> > + priv->is_little_endian = of_property_read_bool(pdev->dev.of_node,
> > + "little-endian");
> > +
> > + priv->has_a011043 = of_property_read_bool(pdev->dev.of_node,
> > + "fsl,erratum-a011043");
> > +
> > + ret = of_mdiobus_register(bus, np);
> > + if (ret) {
> > + dev_err(&pdev->dev, "cannot register MDIO bus\n");
> > + goto err_registration;
> > + }
> > + } else if (is_acpi_node(pdev->dev.fwnode)) {
> > + priv->is_little_endian =
> > + fwnode_property_read_bool(pdev->dev.fwnode,
> > + "little-endian");
> > + ret = fwnode_mdiobus_register(bus, pdev->dev.fwnode);
> > + if (ret) {
> > + dev_err(&pdev->dev, "cannot register MDIO bus\n");
> > + goto err_registration;
> > + }
>
> The little-endian property read can be moved out of the DT/ACPI paths
> and you can just use device_property_read_bool() for that purpose.
> Having both fwnode_mdiobus_register() and of_mdiobus_register() looks
> fairly redundant, you could quite easily introduce a wrapper:
> device_mdiobus_register() which internally takes the appropriate DT/ACPI
> paths as needed.
Had some difficulty with DT while using fwnode APIs. Will resolve them
and provide better integrated code.
Thanks
Calvin
next prev parent reply other threads:[~2020-02-04 18:46 UTC|newest]
Thread overview: 34+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-01-31 15:34 [PATCH v1 0/7] ACPI support for xgmac_mdio and dpaa2-mac drivers Calvin Johnson
2020-01-31 15:34 ` [PATCH v1 1/7] mdio_bus: Introduce fwnode MDIO helpers Calvin Johnson
2020-01-31 16:28 ` Andrew Lunn
2020-02-05 7:11 ` [EXT] " Calvin Johnson (OSS)
2020-02-03 9:49 ` kbuild test robot
2020-02-05 14:17 ` Jeremy Linton
2020-02-07 9:42 ` [EXT] " Calvin Johnson (OSS)
2020-03-17 11:36 ` Calvin Johnson
2020-03-17 14:04 ` Andrew Lunn
2020-03-18 6:03 ` Calvin Johnson
2020-01-31 15:34 ` [PATCH v1 2/7] mdio_bus: modify fwnode phy related functions Calvin Johnson
2020-01-31 15:34 ` [PATCH v1 3/7] net/fsl: add ACPI support for mdio bus Calvin Johnson
2020-01-31 16:08 ` Andy Shevchenko
2020-02-04 7:18 ` Calvin Johnson (OSS)
2020-02-04 11:17 ` Andy Shevchenko
2020-02-03 3:44 ` Florian Fainelli
2020-02-04 18:46 ` Calvin Johnson [this message]
2020-01-31 15:34 ` [PATCH v1 4/7] device property: fwnode_get_phy_mode: Change API to solve int/unit warnings Calvin Johnson
2020-01-31 15:55 ` Andy Shevchenko
2020-02-03 9:13 ` Calvin Johnson (OSS)
2020-02-03 9:22 ` Andy Shevchenko
2020-02-03 2:32 ` kbuild test robot
2020-02-03 8:41 ` kbuild test robot
2020-01-31 15:34 ` [PATCH v1 5/7] device property: Introduce fwnode_phy_is_fixed_link() Calvin Johnson
2020-01-31 15:57 ` Andy Shevchenko
2020-02-03 9:21 ` Calvin Johnson (OSS)
2020-01-31 15:34 ` [PATCH v1 6/7] net: phylink: Introduce phylink_fwnode_phy_connect() Calvin Johnson
2020-02-03 18:21 ` kbuild test robot
2020-02-03 18:41 ` Russell King - ARM Linux admin
2020-02-03 18:43 ` Russell King - ARM Linux admin
2020-02-05 11:33 ` [EXT] " Calvin Johnson (OSS)
2020-01-31 15:34 ` [PATCH v1 7/7] dpaa2-eth: Add ACPI support for DPAA2 MAC driver Calvin Johnson
2020-02-03 18:02 ` [PATCH v1 0/7] ACPI support for xgmac_mdio and dpaa2-mac drivers Florian Fainelli
2020-02-05 8:31 ` [EXT] " Calvin Johnson (OSS)
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to='CAEhpT-VfWhha3B6qS6wFu-PQP9+zb4gxt7=nsrA58cecX21Dog@mail.gmail.com' \
--to=linux.cj@gmail.com \
--cc=V.Sethi@nxp.com \
--cc=calvin.johnson@nxp.com \
--cc=calvin.johnson@oss.nxp.com \
--cc=cristian.sovaiala@nxp.com \
--cc=davem@davemloft.net \
--cc=f.fainelli@gmail.com \
--cc=ioana.ciornei@nxp.com \
--cc=jon@solid-run.com \
--cc=laurentiu.tudor@nxp.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux@armlinux.org.uk \
--cc=madalin.bucur@oss.nxp.com \
--cc=makarand.pawagi@nxp.com \
--cc=netdev@vger.kernel.org \
--cc=pankaj.bansal@nxp.com \
--cc=rajesh.bikkina@nxp.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).