linux-kernel - Re: [PATCH v2 2/2] devicetree bindings mmc Add bindings doc for Sunplus SP7021

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [day] [month] [year] [list]

Date:   Mon, 29 Nov 2021 10:29:53 -0600
From:   Rob Herring <robh@...nel.org>
To:     Lh Kuo 郭力豪 <lh.Kuo@...plus.com>
Cc:     "LH.Kuo" <lhjeff911@...il.com>,
        "p.zabel@...gutronix.de" <p.zabel@...gutronix.de>,
        "daniel.thompson@...aro.org" <daniel.thompson@...aro.org>,
        "lee.jones@...aro.org" <lee.jones@...aro.org>,
        "u.kleine-koenig@...gutronix.de" <u.kleine-koenig@...gutronix.de>,
        "ulf.hansson@...aro.org" <ulf.hansson@...aro.org>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
        "linux-mmc@...r.kernel.org" <linux-mmc@...r.kernel.org>,
        "devicetree@...r.kernel.org" <devicetree@...r.kernel.org>,
        "qinjian@...lus1.com" <qinjian@...lus1.com>,
        Wells Lu 呂芳騰 <wells.lu@...plus.com>
Subject: Re: [PATCH v2 2/2] devicetree bindings mmc Add bindings doc for
 Sunplus SP7021

On Sun, Nov 28, 2021 at 11:30 PM Lh Kuo 郭力豪 <lh.Kuo@...plus.com> wrote:
>
> > > +
> > > +properties:
> > > +  compatible:
> > > +    enum:
> > > +      - sunplus,sp7021-card1
> > > +      - sunplus,sp7021-sdio
> >
> > What's the difference between these 2 blocks?
> >
>
> One for SD card One for SDIO

If the programming model is the same, then it should be the same
compatible string. We have various properties to handle differences
like bus width, card detect or not, etc.

> > > +  reg:
> > > +    items:
> > > +      - description: Base address and length of the SD/SDIO registers
> >
> > Just 'maxItems: 1' is sufficient.
> >
> > > +
> > > +  interrupts:
> > > +    maxItems: 1
> > > +
> > > +  clocks:
> > > +    maxItems: 1
> > > +
> > > +  resets:
> > > +    maxItems: 1
> > > +
> > > +  pinctrl-names:
> > > +    description:
> > > +      A pinctrl state named "default" must be defined.
> > > +    const: default
> > > +
> > > +  pinctrl-0:
> > > +    description:
> > > +      A phandle to the default pinctrl state.
> > > +
> > > +  max-frequency: true
> > > +
> > > +allOf:
> > > +  - $ref: "mmc-controller.yaml"
> > > +
> > > +required:
> > > +  - compatible
> > > +  - reg
> > > +  - interrupts
> > > +  - clocks
> > > +  - resets
> > > +  - pinctrl-names
> > > +  - pinctrl-0
> > > +
> > > +additionalProperties: false
> > > +
> > > +examples:
> > > +  - |
> > > +    #include <dt-bindings/clock/sp-sp7021.h>
> > > +    #include <dt-bindings/reset/sp-sp7021.h>
> > > +    #include <dt-bindings/interrupt-controller/irq.h>
> > > +    mmc1: mmc@...03e80 {
> >
> > Use lower case hex.
>
> Do you mean as follows? ?
>
> mmc1: mmc@...0 {

No, 'mmc@...03e80 {'

You also don't need 'mmc1'.

> > > +       compatible = "sunplus,sp7021-card1";
> > > +       reg = <0x9c003e80 0x280>;
> > > +       interrupts = <21 IRQ_TYPE_LEVEL_HIGH>;
> > > +       clocks = <&clkc CARD_CTL1>;
> > > +       resets = <&rstc RST_CARD_CTL1>;
> > > +       pinctrl-names = "default";
> > > +       pinctrl-0 = <&mmc1_mux &mmc1_mux_cd>;
> > > +       max-frequency = <52000000>;
> > > +    };
> > > +    sdio: mmc@...08400 {
> >
>
> Do you mean as follows? ?
>
> mmc1: mmc@...0 {
>
>
> > Use lower case hex.
> >
> > > +       compatible = "sunplus,sp7021-sdio";
>