[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