Re: Add USB2.0 PHY and USB3.0 PHY support for SpacemiT K1

1 view
Skip to first unread message

linux.r...@gmail.com

unread,
May 17, 2025, 1:25:17 PMMay 17
to linux-ris...@googlegroups.com, hua...@whut.edu.cn
This is automated email and please do not reply to this email!

Dear submitter,

Thank you for submitting the patches to the Linux RISC-V mailing list.
This is a CI test results with your patch series:
PW Link:https://patchwork.kernel.org/project/linux-riscv/list/?series=963855

---Test result---
Test Summary:
Patch 1: "[v3,1/4] dt-bindings: phy: spacemit: add K1 USB2 PHY" build-rv32-defconfig PASS 105.30 seconds
Patch 1: "[v3,1/4] dt-bindings: phy: spacemit: add K1 USB2 PHY" build-rv64-clang-allmodconfig PASS 881.12 seconds
Patch 1: "[v3,1/4] dt-bindings: phy: spacemit: add K1 USB2 PHY" build-rv64-gcc-allmodconfig PASS 1143.45 seconds
Patch 1: "[v3,1/4] dt-bindings: phy: spacemit: add K1 USB2 PHY" build-rv64-nommu-k210-defconfig PASS 20.05 seconds
Patch 1: "[v3,1/4] dt-bindings: phy: spacemit: add K1 USB2 PHY" build-rv64-nommu-k210-virt PASS 21.38 seconds
Patch 1: "[v3,1/4] dt-bindings: phy: spacemit: add K1 USB2 PHY" checkpatch WARNING 1.48 seconds
Patch 1: "[v3,1/4] dt-bindings: phy: spacemit: add K1 USB2 PHY" dtb-warn-rv64 PASS 68.23 seconds
Patch 1: "[v3,1/4] dt-bindings: phy: spacemit: add K1 USB2 PHY" header-inline PASS 0.24 seconds
Patch 1: "[v3,1/4] dt-bindings: phy: spacemit: add K1 USB2 PHY" kdoc PASS 0.89 seconds
Patch 1: "[v3,1/4] dt-bindings: phy: spacemit: add K1 USB2 PHY" module-param PASS 0.25 seconds
Patch 1: "[v3,1/4] dt-bindings: phy: spacemit: add K1 USB2 PHY" verify-fixes PASS 0.23 seconds
Patch 1: "[v3,1/4] dt-bindings: phy: spacemit: add K1 USB2 PHY" verify-signedoff PASS 0.30 seconds
Patch 2: "[v3,2/4] dt-bindings: phy: spacemit: add K1 PCIe/USB3 combo PHY" build-rv32-defconfig PASS 105.30 seconds
Patch 2: "[v3,2/4] dt-bindings: phy: spacemit: add K1 PCIe/USB3 combo PHY" build-rv64-clang-allmodconfig PASS 883.15 seconds
Patch 2: "[v3,2/4] dt-bindings: phy: spacemit: add K1 PCIe/USB3 combo PHY" build-rv64-gcc-allmodconfig PASS 1144.45 seconds
Patch 2: "[v3,2/4] dt-bindings: phy: spacemit: add K1 PCIe/USB3 combo PHY" build-rv64-nommu-k210-defconfig PASS 20.69 seconds
Patch 2: "[v3,2/4] dt-bindings: phy: spacemit: add K1 PCIe/USB3 combo PHY" build-rv64-nommu-k210-virt PASS 21.41 seconds
Patch 2: "[v3,2/4] dt-bindings: phy: spacemit: add K1 PCIe/USB3 combo PHY" checkpatch WARNING 1.57 seconds
Patch 2: "[v3,2/4] dt-bindings: phy: spacemit: add K1 PCIe/USB3 combo PHY" dtb-warn-rv64 PASS 68.94 seconds
Patch 2: "[v3,2/4] dt-bindings: phy: spacemit: add K1 PCIe/USB3 combo PHY" header-inline PASS 0.24 seconds
Patch 2: "[v3,2/4] dt-bindings: phy: spacemit: add K1 PCIe/USB3 combo PHY" kdoc PASS 0.80 seconds
Patch 2: "[v3,2/4] dt-bindings: phy: spacemit: add K1 PCIe/USB3 combo PHY" module-param PASS 0.24 seconds
Patch 2: "[v3,2/4] dt-bindings: phy: spacemit: add K1 PCIe/USB3 combo PHY" verify-fixes PASS 0.22 seconds
Patch 2: "[v3,2/4] dt-bindings: phy: spacemit: add K1 PCIe/USB3 combo PHY" verify-signedoff PASS 0.30 seconds
Patch 3: "[v3,3/4] phy: spacemit: support K1 USB2.0 PHY controller" build-rv32-defconfig PASS 104.82 seconds
Patch 3: "[v3,3/4] phy: spacemit: support K1 USB2.0 PHY controller" build-rv64-clang-allmodconfig PASS 972.17 seconds
Patch 3: "[v3,3/4] phy: spacemit: support K1 USB2.0 PHY controller" build-rv64-gcc-allmodconfig PASS 1258.55 seconds
Patch 3: "[v3,3/4] phy: spacemit: support K1 USB2.0 PHY controller" build-rv64-nommu-k210-defconfig PASS 20.29 seconds
Patch 3: "[v3,3/4] phy: spacemit: support K1 USB2.0 PHY controller" build-rv64-nommu-k210-virt PASS 21.60 seconds
Patch 3: "[v3,3/4] phy: spacemit: support K1 USB2.0 PHY controller" checkpatch WARNING 2.58 seconds
Patch 3: "[v3,3/4] phy: spacemit: support K1 USB2.0 PHY controller" dtb-warn-rv64 PASS 67.26 seconds
Patch 3: "[v3,3/4] phy: spacemit: support K1 USB2.0 PHY controller" header-inline PASS 0.25 seconds
Patch 3: "[v3,3/4] phy: spacemit: support K1 USB2.0 PHY controller" kdoc PASS 1.24 seconds
Patch 3: "[v3,3/4] phy: spacemit: support K1 USB2.0 PHY controller" module-param PASS 0.26 seconds
Patch 3: "[v3,3/4] phy: spacemit: support K1 USB2.0 PHY controller" verify-fixes PASS 0.22 seconds
Patch 3: "[v3,3/4] phy: spacemit: support K1 USB2.0 PHY controller" verify-signedoff PASS 0.32 seconds
Patch 4: "[v3,4/4] phy: spacemit: add USB3 support for K1 PCIe/USB3 combo PHY" build-rv32-defconfig PASS 104.93 seconds
Patch 4: "[v3,4/4] phy: spacemit: add USB3 support for K1 PCIe/USB3 combo PHY" build-rv64-clang-allmodconfig PASS 917.59 seconds
Patch 4: "[v3,4/4] phy: spacemit: add USB3 support for K1 PCIe/USB3 combo PHY" build-rv64-gcc-allmodconfig PASS 1188.98 seconds
Patch 4: "[v3,4/4] phy: spacemit: add USB3 support for K1 PCIe/USB3 combo PHY" build-rv64-nommu-k210-defconfig PASS 20.17 seconds
Patch 4: "[v3,4/4] phy: spacemit: add USB3 support for K1 PCIe/USB3 combo PHY" build-rv64-nommu-k210-virt PASS 21.58 seconds
Patch 4: "[v3,4/4] phy: spacemit: add USB3 support for K1 PCIe/USB3 combo PHY" checkpatch WARNING 2.43 seconds
Patch 4: "[v3,4/4] phy: spacemit: add USB3 support for K1 PCIe/USB3 combo PHY" dtb-warn-rv64 PASS 69.19 seconds
Patch 4: "[v3,4/4] phy: spacemit: add USB3 support for K1 PCIe/USB3 combo PHY" header-inline PASS 0.90 seconds
Patch 4: "[v3,4/4] phy: spacemit: add USB3 support for K1 PCIe/USB3 combo PHY" kdoc PASS 0.87 seconds
Patch 4: "[v3,4/4] phy: spacemit: add USB3 support for K1 PCIe/USB3 combo PHY" module-param PASS 0.25 seconds
Patch 4: "[v3,4/4] phy: spacemit: add USB3 support for K1 PCIe/USB3 combo PHY" verify-fixes PASS 0.55 seconds
Patch 4: "[v3,4/4] phy: spacemit: add USB3 support for K1 PCIe/USB3 combo PHY" verify-signedoff PASS 0.29 seconds

Details
##############################
Patch 1: "[v3,1/4] dt-bindings: phy: spacemit: add K1 USB2 PHY"
Test: checkpatch - WARNING
Desc: Runs checkpatch.pl on the patch
Output:
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#17:
new file mode 100644

total: 0 errors, 1 warnings, 0 checks, 40 lines checked

NOTE: For some of the reported defects, checkpatch may be able to
mechanically convert to the typical style using --fix or --fix-inplace.

Commit 2de38ba2941d ("dt-bindings: phy: spacemit: add K1 USB2 PHY") has style problems, please review.

NOTE: Ignored message types: ALLOC_SIZEOF_STRUCT CAMELCASE COMMIT_LOG_LONG_LINE GIT_COMMIT_ID MACRO_ARG_REUSE NO_AUTHOR_SIGN_OFF

NOTE: If any of the errors are false positives, please report
them to the maintainer, see CHECKPATCH in MAINTAINERS.
total: 0 errors, 1 warnings, 0 checks, 40 lines checked
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?


##############################
Patch 2: "[v3,2/4] dt-bindings: phy: spacemit: add K1 PCIe/USB3 combo PHY"
Test: checkpatch - WARNING
Desc: Runs checkpatch.pl on the patch
Output:
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#22:
new file mode 100644

total: 0 errors, 1 warnings, 0 checks, 72 lines checked

NOTE: For some of the reported defects, checkpatch may be able to
mechanically convert to the typical style using --fix or --fix-inplace.

Commit fecc415b3df1 ("dt-bindings: phy: spacemit: add K1 PCIe/USB3 combo PHY") has style problems, please review.

NOTE: Ignored message types: ALLOC_SIZEOF_STRUCT CAMELCASE COMMIT_LOG_LONG_LINE GIT_COMMIT_ID MACRO_ARG_REUSE NO_AUTHOR_SIGN_OFF

NOTE: If any of the errors are false positives, please report
them to the maintainer, see CHECKPATCH in MAINTAINERS.
total: 0 errors, 1 warnings, 0 checks, 72 lines checked
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?


##############################
Patch 3: "[v3,3/4] phy: spacemit: support K1 USB2.0 PHY controller"
Test: checkpatch - WARNING
Desc: Runs checkpatch.pl on the patch
Output:
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#57:
new file mode 100644

WARNING: please write a help paragraph that fully describes the config symbol with at least 4 lines
#66: FILE: drivers/phy/spacemit/Kconfig:5:
+config PHY_SPACEMIT_K1_USB2
+ tristate "SpacemiT K1 USB 2.0 PHY support"
+ depends on (ARCH_SPACEMIT || COMPILE_TEST) && OF
+ depends on COMMON_CLK
+ depends on USB_COMMON
+ select GENERIC_PHY
+ help
+ Enable this to support K1 USB 2.0 PHY driver. This driver takes care of
+ enabling and clock setup and will be used by K1 udc/ehci/otg/xhci driver.

total: 0 errors, 2 warnings, 0 checks, 160 lines checked

NOTE: For some of the reported defects, checkpatch may be able to
mechanically convert to the typical style using --fix or --fix-inplace.

Commit 00810e3a49b0 ("phy: spacemit: support K1 USB2.0 PHY controller") has style problems, please review.

NOTE: Ignored message types: ALLOC_SIZEOF_STRUCT CAMELCASE COMMIT_LOG_LONG_LINE GIT_COMMIT_ID MACRO_ARG_REUSE NO_AUTHOR_SIGN_OFF

NOTE: If any of the errors are false positives, please report
them to the maintainer, see CHECKPATCH in MAINTAINERS.
total: 0 errors, 2 warnings, 0 checks, 160 lines checked
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
WARNING: please write a help paragraph that fully describes the config symbol with at least 4 lines


##############################
Patch 4: "[v3,4/4] phy: spacemit: add USB3 support for K1 PCIe/USB3 combo PHY"
Test: checkpatch - WARNING
Desc: Runs checkpatch.pl on the patch
Output:
WARNING: please write a help paragraph that fully describes the config symbol with at least 4 lines
#28: FILE: drivers/phy/spacemit/Kconfig:15:
+config PHY_SPACEMIT_K1_COMBPHY
+ tristate "SpacemiT K1 PCIe/USB3 combo PHY support"
+ depends on (ARCH_SPACEMIT || COMPILE_TEST) && OF
+ depends on COMMON_CLK
+ select GENERIC_PHY
+ help
+ USB3/PCIe Combo PHY Support for SpacemiT K1 SoC

WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#44:
new file mode 100644

total: 0 errors, 2 warnings, 0 checks, 262 lines checked

NOTE: For some of the reported defects, checkpatch may be able to
mechanically convert to the typical style using --fix or --fix-inplace.

Commit cdff544740b8 ("phy: spacemit: add USB3 support for K1 PCIe/USB3 combo PHY") has style problems, please review.

NOTE: Ignored message types: ALLOC_SIZEOF_STRUCT CAMELCASE COMMIT_LOG_LONG_LINE GIT_COMMIT_ID MACRO_ARG_REUSE NO_AUTHOR_SIGN_OFF

NOTE: If any of the errors are false positives, please report
them to the maintainer, see CHECKPATCH in MAINTAINERS.
total: 0 errors, 2 warnings, 0 checks, 262 lines checked
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
WARNING: please write a help paragraph that fully describes the config symbol with at least 4 lines




---
Regards,
Linux RISC-V bot

Reply all
Reply to author
Forward
0 new messages