[PATCH v1] net: nuvoton: fix build broken for use phy_get_interface_by_name

Tom Rini trini at konsulko.com
Tue Dec 13 04:01:19 CET 2022


On Tue, Dec 13, 2022 at 10:56:49AM +0800, Jim Liu wrote:
> Hi Tom
> 
> Thanks for your quick response.
> 
> I add defconfig patch and found this build error and i2c naming issue.
> https://patchwork.ozlabs.org/project/uboot/patch/20220712092407.16567-2-JJLIU0@nuvoton.com/
> 
> So I add patch to fix this build error.
> -> https://patchwork.ozlabs.org/project/uboot/patch/20220913062521.6067-1-JJLIU0@nuvoton.com/
> -> https://patchwork.ozlabs.org/project/uboot/patch/20221121085932.29686-1-JJLIU0@nuvoton.com/
> 
> i2c patch is already merged.

OK, I'll put it in my list of things to not forget for the next release
as well.

> 
> Best regards,
> Jim
> 
> On Tue, Dec 13, 2022 at 10:02 AM Tom Rini <trini at konsulko.com> wrote:
> >
> > On Tue, Dec 13, 2022 at 09:48:12AM +0800, Jim Liu wrote:
> > > Hi Tom
> > >
> > > My upstream topic status is Awaiting Upstream.
> > > What could I do to make it merge fast?
> > > Your comments are welcome.
> >
> > It's in -next now, for v2023.04.  Does this fix some regression already
> > present in-tree?
> >
> > >
> > > Best regards,
> > > Jim
> > >
> > > On Mon, Nov 21, 2022 at 3:41 PM Jim Liu <jim.t90615 at gmail.com> wrote:
> > > >
> > > > Hi  Ramon
> > > >
> > > > My upstream topic status is Awaiting Upstream.
> > > > What could I do to make it merge fast?
> > > >
> > > > Best regards,
> > > > Jim
> > > >
> > > > On Sun, Sep 18, 2022 at 2:07 PM Ramon Fried <rfried.dev at gmail.com> wrote:
> > > > >
> > > > > On Tue, Sep 13, 2022 at 9:25 AM Jim Liu <jim.t90615 at gmail.com> wrote:
> > > > > >
> > > > > > The original patch is use phy_get_interface_by_name to set interface.
> > > > > > The new patch is use dev_read_phy_mode to replace it.
> > > > > >
> > > > > > Signed-off-by: Jim Liu <JJLIU0 at nuvoton.com>
> > > > > > ---
> > > > > >  drivers/net/npcm750_eth.c | 8 ++++----
> > > > > >  1 file changed, 4 insertions(+), 4 deletions(-)
> > > > > >
> > > > > > diff --git a/drivers/net/npcm750_eth.c b/drivers/net/npcm750_eth.c
> > > > > > index 409d5cce4a..bd29a10def 100644
> > > > > > --- a/drivers/net/npcm750_eth.c
> > > > > > +++ b/drivers/net/npcm750_eth.c
> > > > > > @@ -710,12 +710,12 @@ static int npcm750_eth_ofdata_to_platdata(struct udevice *dev)
> > > > > >
> > > > > >         pdata->phy_interface = -1;
> > > > > >         phy_mode = fdt_getprop(gd->fdt_blob, dev_of_offset(dev), "phy-mode", NULL);
> > > > > > +
> > > > > >         if (phy_mode)
> > > > > > -               pdata->phy_interface = phy_get_interface_by_name(phy_mode);
> > > > > > -       if (pdata->phy_interface == -1) {
> > > > > > -               printf("%s: Invalid PHY interface '%s'\n", __func__, phy_mode);
> > > > > > +               pdata->phy_interface = dev_read_phy_mode(dev);
> > > > > > +
> > > > > > +       if (pdata->phy_interface == PHY_INTERFACE_MODE_NA)
> > > > > >                 return -EINVAL;
> > > > > > -       }
> > > > > >
> > > > > >         pdata->max_speed = 0;
> > > > > >         cell = fdt_getprop(gd->fdt_blob, dev_of_offset(dev), "max-speed", NULL);
> > > > > > --
> > > > > > 2.17.1
> > > > > >
> > > > > Reviewed-by: Ramon Fried <rfried.dev at gmail.com>
> >
> > --
> > Tom

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20221212/74833b9f/attachment.sig>


More information about the U-Boot mailing list