[PATCH v1 0/7] Add Starfive JH7110 Cadence USB driver
Minda Chen
minda.chen at starfivetech.com
Mon May 20 08:20:55 CEST 2024
> -----邮件原件-----
> 发件人: E Shattow <lucent at gmail.com>
> 发送时间: 2024年5月20日 13:06
> 收件人: Minda Chen <minda.chen at starfivetech.com>
> 抄送: Marek Vasut <marex at denx.de>; Tom Rini <trini at konsulko.com>; Roger
> Quadros <rogerq at kernel.org>; Neil Armstrong <neil.armstrong at linaro.org>;
> Alexey Romanov <avromanov at salutedevices.com>; Sumit Garg
> <sumit.garg at linaro.org>; Mark Kettenis <kettenis at openbsd.org>; Nishanth
> Menon <nm at ti.com>; Rick Chen <rick at andestech.com>; Leo Yu-Chi Liang
> <ycliang at andestech.com>; u-boot at lists.denx.de; Heinrich Schuchardt
> <xypron.glpk at gmx.de>; Simon Glass <sjg at chromium.org>
> 主题: Re: [PATCH v1 0/7] Add Starfive JH7110 Cadence USB driver
>
> Hi, there is a compile warning. I don't know why.
>
> On Sat, May 4, 2024 at 8:04 AM Minda Chen <minda.chen at starfivetech.com>
> wrote:
> >
> > Add Starfive JH7110 Cadence USB driver and related PHY driver.
> > So the codes can be used in visionfive2 and milkv 7110 board.
> >
> > The driver is almost the same with kernel driver.
> >
> > patch1: Add set phy mode function in cdns3 core driver
> > which is used by Starfive.
> >
> > patch2-3: USB and PCIe 2.0 (usb 3.0) PHY drivier
> > patch4: Cadence USB wrapper driver.
> > patch5-7 dts, config and maintainers update.
> >
> > Minda Chen (7):
> > usb: cdns3: Set USB PHY mode in cdns3_probe()
> > phy: starfive: Add Starfive JH7110 USB 2.0 PHY driver
> > phy: starfive: Add Starfive JH7110 PCIe 2.0 PHY driver
> > usb: cdns: starfive: Add cdns USB driver
> > configs: starfive: Add visionfive2 cadence USB configuration
> > dts: starfive: Add JH7110 Cadence USB dts node
> > MAINTAINERS: Update Starfive visionfive2 maintain files.
> >
> > .../dts/jh7110-starfive-visionfive-2.dtsi | 5 +
> > arch/riscv/dts/jh7110.dtsi | 52 +++++
> > board/starfive/visionfive2/MAINTAINERS | 2 +
> > configs/starfive_visionfive2_defconfig | 9 +
> > drivers/phy/Kconfig | 1 +
> > drivers/phy/Makefile | 1 +
> > drivers/phy/starfive/Kconfig | 19 ++
> > drivers/phy/starfive/Makefile | 7 +
> > drivers/phy/starfive/phy-jh7110-pcie.c | 211
> ++++++++++++++++++
> > drivers/phy/starfive/phy-jh7110-usb2.c | 135 +++++++++++
> > drivers/usb/cdns3/Kconfig | 7 +
> > drivers/usb/cdns3/Makefile | 2 +
> > drivers/usb/cdns3/cdns3-starfive.c | 184 +++++++++++++++
> > drivers/usb/cdns3/core.c | 17 ++
> > 14 files changed, 652 insertions(+)
> > create mode 100644 drivers/phy/starfive/Kconfig create mode 100644
> > drivers/phy/starfive/Makefile create mode 100644
> > drivers/phy/starfive/phy-jh7110-pcie.c
> > create mode 100644 drivers/phy/starfive/phy-jh7110-usb2.c
> > create mode 100644 drivers/usb/cdns3/cdns3-starfive.c
> >
> >
> > base-commit: 174ac987655c888017c82df1883c0c2ea0dc2495
> > --
> > 2.17.1
> >
>
> The compile warning as follows:
>
> In file included from
> /home/user/source/u-boot.git/drivers/usb/cdns3/gadget.c:70:
> /home/user/source/u-boot.git/include/linux/bitmap.h: In function
> ‘bitmap_find_next_zero_area’:
> /home/user/source/u-boot.git/include/linux/bitmap.h:170:17: warning:
> implicit declaration of function ‘find_next_zero_bit’; did you mean
> ‘find_next_bit’? [-Wimplicit-function-declaration]
> 170 | index = find_next_zero_bit(map, size, start);
> | ^~~~~~~~~~~~~~~~~~
> | find_next_bit
> CC drivers/usb/cdns3/ep0.o
> In file included from
> /home/user/source/u-boot.git/include/linux/usb/composite.h:26,
> from
> /home/user/source/u-boot.git/drivers/usb/cdns3/ep0.c:19:
> /home/user/source/u-boot.git/include/linux/bitmap.h: In function
> ‘bitmap_find_next_zero_area’:
> /home/user/source/u-boot.git/include/linux/bitmap.h:170:17: warning:
> implicit declaration of function ‘find_next_zero_bit’; did you mean
> ‘find_next_bit’? [-Wimplicit-function-declaration]
> 170 | index = find_next_zero_bit(map, size, start);
> | ^~~~~~~~~~~~~~~~~~
> | find_next_bit
>
>
> Is this something missing in the patch series?
>
> -E
I have not noticed this. I just check this it is risc-v code do not contain "find_next_zero_bit" macro define.
More information about the U-Boot
mailing list