[PATCH v4 1/4] dt-bindings: mmc: Add StarFive MMC module

[Shengyu Qu]

Hello William,

Thanks for your reply. So there's v5 series? Btw, please fix maintainer 
information:

https://patchwork.kernel.org/project/linux-riscv/patch/20230215080203.27445-1-lukas.bulwahn@gmail.com/

Best regards,

Shengyu

>
> On 2023/2/15 19:59, Shengyu Qu wrote:
>> Hello William,
>>
>> Are you sure changing driver is better than changing yaml bindings? All
>>
>> previous version sent was syscon and sysreg seems not consistent with
>>
>> other codes.
>>
>> Best regards,
>>
>> Shengyu
>>
> Hi Shengyu,
>
> After discussing with colleagues, we decided to restore the lable name to
> sys_syscon, and sysreg was just a unique name for the functionality of MMC,
> which will be used in all future versions.
>
> Thanks for taking time reviewing this patch series.
>
> Best Regards
> William
>
>>> Add documentation to describe StarFive designware mobile storage
>>> host controller driver.
>>>
>>> Signed-off-by: William Qiu <william.qiu at starfivetech.com>
>>> Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski at linaro.org>
>>> ---
>>>    .../bindings/mmc/starfive,jh7110-mmc.yaml     | 77 +++++++++++++++++++
>>>    1 file changed, 77 insertions(+)
>>>    create mode 100644 Documentation/devicetree/bindings/mmc/starfive,jh7110-mmc.yaml
>>>
>>> diff --git a/Documentation/devicetree/bindings/mmc/starfive,jh7110-mmc.yaml b/Documentation/devicetree/bindings/mmc/starfive,jh7110-mmc.yaml
>>> new file mode 100644
>>> index 000000000000..51e1b04e799f
>>> --- /dev/null
>>> +++ b/Documentation/devicetree/bindings/mmc/starfive,jh7110-mmc.yaml
>>> @@ -0,0 +1,77 @@
>>> +# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause
>>> +%YAML 1.2
>>> +---
>>> +$id: http://devicetree.org/schemas/mmc/starfive,jh7110-mmc.yaml#
>>> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>>> +
>>> +title: StarFive Designware Mobile Storage Host Controller
>>> +
>>> +description:
>>> +  StarFive uses the Synopsys designware mobile storage host controller
>>> +  to interface a SoC with storage medium such as eMMC or SD/MMC cards.
>>> +
>>> +allOf:
>>> +  - $ref: synopsys-dw-mshc-common.yaml#
>>> +
>>> +maintainers:
>>> +  - William Qiu <william.qiu at starfivetech.com>
>>> +
>>> +properties:
>>> +  compatible:
>>> +    const: starfive,jh7110-mmc
>>> +
>>> +  reg:
>>> +    maxItems: 1
>>> +
>>> +  clocks:
>>> +    items:
>>> +      - description: biu clock
>>> +      - description: ciu clock
>>> +
>>> +  clock-names:
>>> +    items:
>>> +      - const: biu
>>> +      - const: ciu
>>> +
>>> +  interrupts:
>>> +    maxItems: 1
>>> +
>>> +  starfive,sysreg:
>>> +    $ref: /schemas/types.yaml#/definitions/phandle-array
>>> +    items:
>>> +      - items:
>>> +          - description: phandle to System Register Controller syscon node
>>> +          - description: offset of SYS_SYSCONSAIF__SYSCFG register for MMC controller
>>> +          - description: shift of SYS_SYSCONSAIF__SYSCFG register for MMC controller
>>> +          - description: mask of SYS_SYSCONSAIF__SYSCFG register for MMC controller
>>> +    description:
>>> +      Should be four parameters, the phandle to System Register Controller
>>> +      syscon node and the offset/shift/mask of SYS_SYSCONSAIF__SYSCFG register
>>> +      for MMC controller.
>>> +
>>> +required:
>>> +  - compatible
>>> +  - reg
>>> +  - clocks
>>> +  - clock-names
>>> +  - interrupts
>>> +  - starfive,sysreg
>>> +
>>> +unevaluatedProperties: false
>>> +
>>> +examples:
>>> +  - |
>>> +    mmc at 16010000 {
>>> +        compatible = "starfive,jh7110-mmc";
>>> +        reg = <0x16010000 0x10000>;
>>> +        clocks = <&syscrg 91>,
>>> +                 <&syscrg 93>;
>>> +        clock-names = "biu","ciu";
>>> +        resets = <&syscrg 64>;
>>> +        reset-names = "reset";
>>> +        interrupts = <74>;
>>> +        fifo-depth = <32>;
>>> +        fifo-watermark-aligned;
>>> +        data-addr = <0>;
>>> +        starfive,sysreg = <&sys_syscon 0x14 0x1a 0x7c000000>;
>>> +    };
-------------- next part --------------
A non-text attachment was scrubbed...
Name: OpenPGP_0xE3520CC91929C8E7.asc
Type: application/pgp-keys
Size: 6868 bytes
Desc: OpenPGP public key
URL: <http://lists.infradead.org/pipermail/linux-riscv/attachments/20230216/0ac2d573/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: OpenPGP_signature
Type: application/pgp-signature
Size: 833 bytes
Desc: OpenPGP digital signature
URL: <http://lists.infradead.org/pipermail/linux-riscv/attachments/20230216/0ac2d573/attachment.sig>