Re: [PATCH v8 3/8] dt-bindings: riscv: Add UltraRISC DP1000 board compatibles

[sashiko-bot@kernel.org]

Thank you for your contribution! Sashiko AI review found 1 potential
issue to consider:
- [Low] dt-bindings: riscv: ultrarisc lists departed maintainer and
  stale git tree
--

commit 12ea55f6c9205d92a219a92f47c317d3c2daaad7
Author: Jia Wang <wangjia@ultrarisc.com>

dt-bindings: riscv: Add UltraRISC DP1000 board compatibles

This commit adds the device tree binding schema and MAINTAINERS entry for
UltraRISC DP1000 SoC-based boards (Shenzhen Rongda M0 and Milk-V Titan).

> diff --git a/Documentation/devicetree/bindings/riscv/ultrarisc.yaml b/Documentation/devicetree/bindings/riscv/ultrarisc.yaml
> new file mode 100644
> index 0000000000000..a51e63dbeca33
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/riscv/ultrarisc.yaml
> @@ -0,0 +1,27 @@
> +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
> +%YAML 1.2
> +---
> +$id: http://devicetree.org/schemas/riscv/ultrarisc.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
> +
> +title: UltraRISC SoC-based boards
> +
> +maintainers:
> +  - Jia Wang <wangjia@ultrarisc.com>

[ ... ]

> diff --git a/MAINTAINERS b/MAINTAINERS
> index ff1c1aea9a989..7eb1b2971bead 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -23639,6 +23639,13 @@ F:	include/dt-bindings/power/thead,th1520-power.h
>  F:	include/dt-bindings/reset/thead,th1520-reset.h
>  F:	include/linux/firmware/thead/thead,th1520-aon.h
>  
> +RISC-V ULTRARISC SoC SUPPORT
> +M:	Jia Wang <wangjia@ultrarisc.com>
> +L:	linux-riscv@lists.infradead.org
> +S:	Supported
> +T:	git https://github.com/ultrarisc/linux.git
> +F:	Documentation/devicetree/bindings/riscv/ultrarisc.yaml

[Severity: Low]
As reported by Conor Dooley on v7 [1], should both the new binding schema and
the MAINTAINERS entry list Xincheng Ji as maintainer and
https://git.kernel.org/pub/scm/linux/kernel/git/conor/linux.git/ as the git
tree instead of Jia Wang and the github.com/ultrarisc/linux.git tree?

Later in the series, commit "MAINTAINERS: Update UltraRISC platform
maintainer and tree" overwrites the MAINTAINERS entry added here, while
Documentation/devicetree/bindings/riscv/ultrarisc.yaml is left listing Jia
Wang, who has left UltraRISC.

[1] https://lore.kernel.org/r/20261110-amused-strobe-d6cf2bf24861@spud

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20261112-ultrarisc-dts-v8-0-14fa0834cac4@ultrarisc.com?part=3