[PATCH v2 1/2] dt-bindings: serial: update bindings of ultrarisc dp1000 uart
Jia Wang
wangjia at ultrarisc.com
Mon Mar 16 18:29:22 PDT 2026
On 2026-03-16 09:11 +0100, Krzysztof Kozlowski wrote:
> On Mon, Mar 16, 2026 at 02:33:22PM +0800, Jia Wang wrote:
> > Add DP1000 UART compatible information
>
> We see this from the diff. Why this is not compatible with
> snps,dw-apb-uart? What are the differences. You have entire commit msg
> to explain the hardware.
>
The UART controller in DP1000 is based on the Synopsys
DesignWare APB UART IP, but the hardware does not support
automatic FIFO size detection used by the generic driver.
I will update the commit message to clarify this.
> >
> > Signed-off-by: Jia Wang <wangjia at ultrarisc.com>
> > Reviewed-by: Yao Zi <me at ziyao.cc>
> > ---
> > Documentation/devicetree/bindings/serial/snps-dw-apb-uart.yaml | 2 ++
> > 1 file changed, 2 insertions(+)
> >
> > diff --git a/Documentation/devicetree/bindings/serial/snps-dw-apb-uart.yaml b/Documentation/devicetree/bindings/serial/snps-dw-apb-uart.yaml
> > index 6efe43089a74..0040ed28e35b 100644
> > --- a/Documentation/devicetree/bindings/serial/snps-dw-apb-uart.yaml
> > +++ b/Documentation/devicetree/bindings/serial/snps-dw-apb-uart.yaml
> > @@ -78,6 +78,8 @@ properties:
> > - starfive,jh7100-uart
> > - starfive,jh7110-uart
> > - const: snps,dw-apb-uart
> > + - items:
>
> Drop, not needed.
>
> > + - const: ultrarisc,dp1000-uart
> > - const: snps,dw-apb-uart
>
> So just enum with the snps one.
Understood. I will drop the items block and add
"ultrarisc,dp1000-uart" directly to the enum list in v3.
Best regards,
Jia
>
> Best regards,
> Krzysztof
>
>
More information about the linux-riscv
mailing list