Re: [PATCH v1 0/7] Add Starfive JH7110 Cadence USB driver
> > Hi, > > On Sun, Jun 23, 2024 at 6:28 PM Minda Chen > wrote: > > > > > > > > > > > > Minda, can you test USB Host function on VisionFive2? I guess that > > > it is connected to the USB-C port. For the boards with JH7110 and > > > not any > > > VL805 USB controller this Cadence USB is the only way to have host > > > USB. It is very much wanted to have host USB. Thanks! -E > > > > > > > In VF2, PCIe0 connect with VL805 USB 3.0 host controller. Now PCIe > > driver have commit to Uboot upstream code. USB 3.0 can be used in uboot > upstream code. > > Milk-v mars also connect VL805 and can use USB 3.0 host too. > > No no I am asking about Cadence USB of JH7110 CPU. This VL805 is not the > question, sorry that my question was not easy to understand before. > > > > > You can use "pci e" command to active USB 3.0 host controller and then > > "usb reset" to scan usb devices. If you have any issue about this. Also > > reply it > in this. Thanks. > > Can you show that Host USB is functioning on VF2 with the JH7110 CPU Cadence > USB, not the VL805 controller? > VF2 cadence USB is dr mode device.I cant test is with host. > This is needed so Milk-V Mars CM and Pine64 Star64 can have USB Host. > There is no use of VL805 and we need JH7110 Cadence USB then. > > Thanks! > > -E > Okay. I will test cadence USB host in star64 board. > > > > > On Sun, May 19, 2024 at 11:20 PM Minda Chen > > > > > > wrote: > > > > > > > > > > > > > > > > > > > > > > Hi, there is a compile warning. I don't know why. > > > > > > > > > > On Sat, May 4, 2024 at 8:04 AM Minda Chen > > > > > > > > > > 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, >
Re: [PATCH v1 0/7] Add Starfive JH7110 Cadence USB driver
Hi, On Sun, Jun 23, 2024 at 6:28 PM Minda Chen wrote: > > > > > > > Minda, can you test USB Host function on VisionFive2? I guess that it is > > connected to the USB-C port. For the boards with JH7110 and not any > > VL805 USB controller this Cadence USB is the only way to have host USB. It > > is > > very much wanted to have host USB. Thanks! -E > > > > In VF2, PCIe0 connect with VL805 USB 3.0 host controller. Now PCIe driver have > commit to Uboot upstream code. USB 3.0 can be used in uboot upstream code. > Milk-v mars also connect VL805 and can use USB 3.0 host too. No no I am asking about Cadence USB of JH7110 CPU. This VL805 is not the question, sorry that my question was not easy to understand before. > > You can use "pci e" command to active USB 3.0 host controller and then "usb > reset" to > scan usb devices. If you have any issue about this. Also reply it in this. > Thanks. Can you show that Host USB is functioning on VF2 with the JH7110 CPU Cadence USB, not the VL805 controller? This is needed so Milk-V Mars CM and Pine64 Star64 can have USB Host. There is no use of VL805 and we need JH7110 Cadence USB then. Thanks! -E > > > On Sun, May 19, 2024 at 11:20 PM Minda Chen > > wrote: > > > > > > > > > > > > > > > > > Hi, there is a compile warning. I don't know why. > > > > > > > > On Sat, May 4, 2024 at 8:04 AM Minda Chen > > > > > > > > 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 me
Re: [PATCH v1 0/7] Add Starfive JH7110 Cadence USB driver
> > Minda, can you test USB Host function on VisionFive2? I guess that it is > connected to the USB-C port. For the boards with JH7110 and not any > VL805 USB controller this Cadence USB is the only way to have host USB. It is > very much wanted to have host USB. Thanks! -E > In VF2, PCIe0 connect with VL805 USB 3.0 host controller. Now PCIe driver have commit to Uboot upstream code. USB 3.0 can be used in uboot upstream code. Milk-v mars also connect VL805 and can use USB 3.0 host too. You can use "pci e" command to active USB 3.0 host controller and then "usb reset" to scan usb devices. If you have any issue about this. Also reply it in this. Thanks. > On Sun, May 19, 2024 at 11:20 PM Minda Chen > wrote: > > > > > > > > > > > > Hi, there is a compile warning. I don't know why. > > > > > > On Sat, May 4, 2024 at 8:04 AM Minda Chen > > > > > > 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.
Re: [PATCH v1 0/7] Add Starfive JH7110 Cadence USB driver
Minda, can you test USB Host function on VisionFive2? I guess that it is connected to the USB-C port. For the boards with JH7110 and not any VL805 USB controller this Cadence USB is the only way to have host USB. It is very much wanted to have host USB. Thanks! -E On Sun, May 19, 2024 at 11:20 PM Minda Chen wrote: > > > > > -邮件原件- > > 发件人: E Shattow > > 发送时间: 2024年5月20日 13:06 > > 收件人: Minda Chen > > 抄送: Marek Vasut ; Tom Rini ; Roger > > Quadros ; Neil Armstrong ; > > Alexey Romanov ; Sumit Garg > > ; Mark Kettenis ; Nishanth > > Menon ; Rick Chen ; Leo Yu-Chi Liang > > ; u-boot@lists.denx.de; Heinrich Schuchardt > > ; Simon Glass > > 主题: 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 > > 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.
Re: [PATCH v1 0/7] Add Starfive JH7110 Cadence USB driver
> -邮件原件- > 发件人: E Shattow > 发送时间: 2024年5月20日 13:06 > 收件人: Minda Chen > 抄送: Marek Vasut ; Tom Rini ; Roger > Quadros ; Neil Armstrong ; > Alexey Romanov ; Sumit Garg > ; Mark Kettenis ; Nishanth > Menon ; Rick Chen ; Leo Yu-Chi Liang > ; u-boot@lists.denx.de; Heinrich Schuchardt > ; Simon Glass > 主题: 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 > 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.
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 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
[PATCH v1 0/7] Add Starfive JH7110 Cadence USB driver
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