Skip to content

[PW_SID:986542] dt-bindings: serial: 8250: allow "main" and "uart" as clock names - #711

Closed
linux-riscv-bot wants to merge 2 commits into
workflowfrom
pw986542
Closed

[PW_SID:986542] dt-bindings: serial: 8250: allow "main" and "uart" as clock names#711
linux-riscv-bot wants to merge 2 commits into
workflowfrom
pw986542

Conversation

@linux-riscv-bot

@linux-riscv-bot linux-riscv-bot commented Jul 29, 2025

Copy link
Copy Markdown

PR for series 986542 applied to workflow

Name: dt-bindings: serial: 8250: allow "main" and "uart" as clock names
URL: https://patchwork.kernel.org/project/linux-riscv/list/?series=986542
Version: 1

Linux RISC-V bot and others added 2 commits July 29, 2025 19:01
Linux RISC-V bot
Adding CI files
ea672f4
@alexelder Linux RISC-V bot
dt-bindings: serial: 8250: allow "main" and "uart" as clock names
abbf3b1 
There are two compatible strings defined in "8250.yaml" that require
two clocks to be specified, along with their names:
  - "spacemit,k1-uart", used in "spacemit/k1.dtsi"
  - "nxp,lpc1850-uart", used in "lpc/lpc18xx.dtsi"

When only one clock is used, the name is not required.  However there
are two places that do specify a name:
  - In "mediatek/mt7623.dtsi", the clock for the "mediatek,mtk-btif"
    compatible serial device is named "main"
  - In "qca/ar9132.dtsi", the clock for the "ns8250" compatible
    serial device is named "uart"

In commit d2db0d7 ("dt-bindings: serial: 8250: allow clock 'uartclk'
and 'reg' for nxp,lpc1850-uart"), Frank Li added the restriction that two
named clocks be used for the NXP platform mentioned above.  Extend that
so that the two named clocks used by the SpacemiT platform are similarly
restricted.

Add "main" and "uart" as allowed names when a single clock is specified.

Fixes: 2c0594f ("dt-bindings: serial: 8250: support an optional second clock")
Reported-by: kernel test robot <lkp@intel.com>
Closes: https://lore.kernel.org/oe-kbuild-all/202507160314.wrC51lXX-lkp@intel.com/
Signed-off-by: Alex Elder <elder@riscstar.com>
Signed-off-by: Linux RISC-V bot <linux.riscv.bot@gmail.com>
@linux-riscv-bot

linux-riscv-bot commented Jul 29, 2025

Copy link
Copy Markdown
Author

Patch 1: "dt-bindings: serial: 8250: allow "main" and "uart" as clock names"
build-rv32-defconfig
Desc: Builds riscv32 defconfig
Duration: 101.86 seconds
Result: PASS

@linux-riscv-bot

linux-riscv-bot commented Jul 29, 2025

Copy link
Copy Markdown
Author

Patch 1: "dt-bindings: serial: 8250: allow "main" and "uart" as clock names"
build-rv64-clang-allmodconfig
Desc: Builds riscv64 allmodconfig with Clang, and checks for errors and added warnings
Duration: 932.80 seconds
Result: PASS

@linux-riscv-bot

linux-riscv-bot commented Jul 29, 2025

Copy link
Copy Markdown
Author

Patch 1: "dt-bindings: serial: 8250: allow "main" and "uart" as clock names"
build-rv64-gcc-allmodconfig
Desc: Builds riscv64 allmodconfig with GCC, and checks for errors and added warnings
Duration: 1250.97 seconds
Result: PASS

@linux-riscv-bot

linux-riscv-bot commented Jul 29, 2025

Copy link
Copy Markdown
Author

Patch 1: "dt-bindings: serial: 8250: allow "main" and "uart" as clock names"
build-rv64-nommu-k210-defconfig
Desc: Builds riscv64 defconfig with NOMMU for K210
Duration: 21.22 seconds
Result: PASS

@linux-riscv-bot

linux-riscv-bot commented Jul 29, 2025

Copy link
Copy Markdown
Author

Patch 1: "dt-bindings: serial: 8250: allow "main" and "uart" as clock names"
build-rv64-nommu-k210-virt
Desc: Builds riscv64 defconfig with NOMMU for the virt platform
Duration: 22.33 seconds
Result: PASS

@linux-riscv-bot

linux-riscv-bot commented Jul 29, 2025

Copy link
Copy Markdown
Author

Patch 1: "dt-bindings: serial: 8250: allow "main" and "uart" as clock names"
checkpatch
Desc: Runs checkpatch.pl on the patch
Duration: 0.76 seconds
Result: PASS

@linux-riscv-bot

linux-riscv-bot commented Jul 29, 2025

Copy link
Copy Markdown
Author

Patch 1: "dt-bindings: serial: 8250: allow "main" and "uart" as clock names"
dtb-warn-rv64
Desc: Checks for Device Tree warnings/errors
Duration: 72.79 seconds
Result: PASS

@linux-riscv-bot

linux-riscv-bot commented Jul 29, 2025

Copy link
Copy Markdown
Author

Patch 1: "dt-bindings: serial: 8250: allow "main" and "uart" as clock names"
header-inline
Desc: Detects static functions without inline keyword in header files
Duration: 0.24 seconds
Result: PASS

@linux-riscv-bot

linux-riscv-bot commented Jul 29, 2025

Copy link
Copy Markdown
Author

Patch 1: "dt-bindings: serial: 8250: allow "main" and "uart" as clock names"
kdoc
Desc: Detects for kdoc errors
Duration: 0.81 seconds
Result: PASS

@linux-riscv-bot

linux-riscv-bot commented Jul 29, 2025

Copy link
Copy Markdown
Author

Patch 1: "dt-bindings: serial: 8250: allow "main" and "uart" as clock names"
module-param
Desc: Detect module_param changes
Duration: 0.25 seconds
Result: PASS

@linux-riscv-bot

linux-riscv-bot commented Jul 29, 2025

Copy link
Copy Markdown
Author

Patch 1: "dt-bindings: serial: 8250: allow "main" and "uart" as clock names"
verify-fixes
Desc: Verifies that the Fixes: tags exist
Duration: 0.28 seconds
Result: PASS

@linux-riscv-bot

linux-riscv-bot commented Jul 29, 2025

Copy link
Copy Markdown
Author

Patch 1: "dt-bindings: serial: 8250: allow "main" and "uart" as clock names"
verify-signedoff
Desc: Verifies that Signed-off-by: tags are correct
Duration: 0.30 seconds
Result: PASS

@linux-riscv-bot
linux-riscv-bot force-pushed the workflow branch 16 times, most recently from e93973b to b4c50e7 Compare July 31, 2025 23:47
@linux-riscv-bot
linux-riscv-bot force-pushed the workflow branch 15 times, most recently from adb1c84 to a009b55 Compare August 5, 2025 01:11
@linux-riscv-bot
linux-riscv-bot deleted the pw986542 branch August 5, 2025 01:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@linux-riscv-bot @alexelder