Re: [PATCH v1 1/3] scsi: ufs: dt-bindings: starfive: Add UFS Host Controller for JHB100 soc

[Minda Chen]

> 
> On Tue, Apr 21, 2026 at 05:12:13PM +0800, Minda Chen wrote:
> > Add devicetree document for UFS Host Controller StarFive JHB100 SoC.
> > The UFS controller is based on the Synopsys DesignWare UFS controller.
> >
> > Signed-off-by: Minda Chen <minda.chen@xxxxxxxxxxxxxxxx>
> > ---
> >  .../devicetree/bindings/ufs/starfive,ufs.yaml | 76 +++++++++++++++++++
> >  MAINTAINERS                                   |  5 ++
> >  2 files changed, 81 insertions(+)
> >  create mode 100644
> > Documentation/devicetree/bindings/ufs/starfive,ufs.yaml
> >
> > diff --git a/Documentation/devicetree/bindings/ufs/starfive,ufs.yaml
> > b/Documentation/devicetree/bindings/ufs/starfive,ufs.yaml
> > new file mode 100644
> > index 000000000000..c408973dd0ce
> > --- /dev/null
> > +++ b/Documentation/devicetree/bindings/ufs/starfive,ufs.yaml
> 
> Filename should be starfive,jhb100-ufs.
> 
Thanks. Conor. I see some vendors's dt-binding doc without IC name
because ufs host controller registers are standard. So I think if we(StarFive) change
IP vendor will still using the same driver files and dt doc.  
If the scsi UFS maintainer no comments to this I will change this.

> > @@ -0,0 +1,76 @@
> > +# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause %YAML 1.2
> > +---
> > +$id: http://devicetree.org/schemas/ufs/starfive,ufs.yaml#
> > +$schema: http://devicetree.org/meta-schemas/core.yaml#
> > +
> > +title: Starfive Universal Flash Storage (UFS) Controller
> > +
> > +maintainers:
> > +  - Minda Chen <minda.chen@xxxxxxxxxxxxxxxx>
> > +
> > +allOf:
> > +  - $ref: ufs-common.yaml
> > +
> > +properties:
> > +  compatible:
> > +    const: starfive,jhb100-ufs
> > +
> > +  reg:
> > +    maxItems: 1
> > +
> > +  clocks:
> > +    items:
> > +      - description: UFS reference clock
> > +      - description: UFS main enable clock
> > +
> > +  clock-names:
> > +    items:
> > +      - const: ref_clk
> 
> Think "ref" suffices here.
> 
> > +      - const: ufs
> > +
> > +  resets:
> > +    items:
> > +      - description: UFS main reset
> > +      - description: UFS PHY reset
> > +
> > +  reset-names:
> > +    items:
> > +      - const: main
> > +      - const: phy
> > +
> > +  interrupts:
> > +    maxItems: 1
> > +
> > +  starfive,syscon:
> > +    $ref: /schemas/types.yaml#/definitions/phandle-array
> > +    description:
> > +      The phandle to System Register Controller syscon node.
> > +
> > +required:
> > +  - compatible
> > +  - reg
> > +  - clocks
> > +  - resets
> > +  - reset-names
> > +  - interrupts
> > +  - starfive,syscon
> > +
> > +unevaluatedProperties: false
> > +
> > +examples:
> > +  - |
> > +    ufs@11b10000 {
> > +        compatible = "starfive,jhb100-ufs";
> > +        reg = <0x11b10000 0x20000>;
> > +        interrupts = <105>;
> > +        clocks = <&syscrg 4>,
> > +                 <&syscrg 5>;
> > +        clock-names = "ref_clk", "ufs";
> > +        freq-table-hz = <26000000 26000000>,
> > +                        <100000000 100000000>;
> > +        resets = <&syscrg 10>,
> > +                 <&syscrg 7>;
> > +        reset-names = "main", "phy";
> > +        starfive,syscon = <&syscon>;
> > +    };
> > diff --git a/MAINTAINERS b/MAINTAINERS index
> > 32bd94a0b94c..3792c51da63c 100644
> > --- a/MAINTAINERS
> > +++ b/MAINTAINERS
> > @@ -27190,6 +27190,11 @@ L:	linux-scsi@xxxxxxxxxxxxxxx
> >  S:	Maintained
> >  F:	drivers/ufs/host/ufs-renesas.c
> >
> > +UNIVERSAL FLASH STORAGE HOST CONTROLLER DRIVER STARFIVE
> > +M:	Minda Chen <minda.cheb@xxxxxxxxxxxxxxxx>
> 
> Typo in your email address here.
> 
> pw-bot: changes-requested
> 
> Thanks,
> Conor.
> 
> > +S:	Maintained
> > +F:	Documentation/devicetree/bindings/ufs/starfive,ufs.yaml
> > +
> >  UNIWILL LAPTOP DRIVER
> >  M:	Armin Wolf <W_Armin@xxxxxx>
> >  L:	platform-driver-x86@xxxxxxxxxxxxxxx
> > --
> > 2.17.1
> >
> >