Re: [PATCH 1/4] dt-bindings: pinctrl: tegra: Convert to json-schema

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Jul 01, 2022 at 03:17:08PM -0600, Rob Herring wrote:
> On Fri, Jul 01, 2022 at 05:04:58PM +0200, Thierry Reding wrote:
> > From: Thierry Reding <treding@xxxxxxxxxx>
> > 
> > Convert the NVIDIA Tegra pinmux controller bindings from the free-form
> > text format to json-schema.
> > 
> > Signed-off-by: Thierry Reding <treding@xxxxxxxxxx>
> > ---
> >  .../bindings/clock/nvidia,tegra124-dfll.yaml  |   2 +-
> >  .../pinctrl/nvidia,tegra114-pinmux.txt        | 131 --------
> >  .../pinctrl/nvidia,tegra114-pinmux.yaml       | 264 ++++++++++++++++
> >  .../pinctrl/nvidia,tegra124-pinmux.txt        | 153 ----------
> >  .../pinctrl/nvidia,tegra124-pinmux.yaml       | 287 ++++++++++++++++++
> >  .../pinctrl/nvidia,tegra194-pinmux.txt        | 107 -------
> >  .../pinctrl/nvidia,tegra194-pinmux.yaml       | 176 +++++++++++
> >  .../pinctrl/nvidia,tegra20-pinmux.txt         | 143 ---------
> >  .../pinctrl/nvidia,tegra20-pinmux.yaml        | 203 +++++++++++++
> >  .../pinctrl/nvidia,tegra210-pinmux.txt        | 166 ----------
> >  .../pinctrl/nvidia,tegra210-pinmux.yaml       | 232 ++++++++++++++
> >  .../pinctrl/nvidia,tegra30-pinmux.txt         | 144 ---------
> >  .../pinctrl/nvidia,tegra30-pinmux.yaml        | 268 ++++++++++++++++
> >  13 files changed, 1431 insertions(+), 845 deletions(-)
> >  delete mode 100644 Documentation/devicetree/bindings/pinctrl/nvidia,tegra114-pinmux.txt
> >  create mode 100644 Documentation/devicetree/bindings/pinctrl/nvidia,tegra114-pinmux.yaml
> >  delete mode 100644 Documentation/devicetree/bindings/pinctrl/nvidia,tegra124-pinmux.txt
> >  create mode 100644 Documentation/devicetree/bindings/pinctrl/nvidia,tegra124-pinmux.yaml
> >  delete mode 100644 Documentation/devicetree/bindings/pinctrl/nvidia,tegra194-pinmux.txt
> >  create mode 100644 Documentation/devicetree/bindings/pinctrl/nvidia,tegra194-pinmux.yaml
> >  delete mode 100644 Documentation/devicetree/bindings/pinctrl/nvidia,tegra20-pinmux.txt
> >  create mode 100644 Documentation/devicetree/bindings/pinctrl/nvidia,tegra20-pinmux.yaml
> >  delete mode 100644 Documentation/devicetree/bindings/pinctrl/nvidia,tegra210-pinmux.txt
> >  create mode 100644 Documentation/devicetree/bindings/pinctrl/nvidia,tegra210-pinmux.yaml
> >  delete mode 100644 Documentation/devicetree/bindings/pinctrl/nvidia,tegra30-pinmux.txt
> >  create mode 100644 Documentation/devicetree/bindings/pinctrl/nvidia,tegra30-pinmux.yaml
> > 
> > diff --git a/Documentation/devicetree/bindings/clock/nvidia,tegra124-dfll.yaml b/Documentation/devicetree/bindings/clock/nvidia,tegra124-dfll.yaml
> > index 85234a48b590..96c54c215f77 100644
> > --- a/Documentation/devicetree/bindings/clock/nvidia,tegra124-dfll.yaml
> > +++ b/Documentation/devicetree/bindings/clock/nvidia,tegra124-dfll.yaml
> > @@ -219,7 +219,7 @@ examples:
> >  
> >      /*
> >       * pinmux nodes added for completeness. Binding doc can be found in:
> > -     * Documentation/devicetree/bindings/pinctrl/nvidia,tegra210-pinmux.txt
> > +     * Documentation/devicetree/bindings/pinctrl/nvidia,tegra210-pinmux.yaml
> >       */
> >  
> >      pinmux: pinmux@700008d4 {
> 
> 
> > diff --git a/Documentation/devicetree/bindings/pinctrl/nvidia,tegra114-pinmux.yaml b/Documentation/devicetree/bindings/pinctrl/nvidia,tegra114-pinmux.yaml
> > new file mode 100644
> > index 000000000000..fa95d0bd6fe4
> > --- /dev/null
> > +++ b/Documentation/devicetree/bindings/pinctrl/nvidia,tegra114-pinmux.yaml
> > @@ -0,0 +1,264 @@
> > +# SPDX-License-Identifier: (GPL-2.0+ OR BSD-2-Clause)
> 
> GPL-2.0-only rather than GPL-2.0+
> 
> 
> > +%YAML 1.2
> > +---
> > +$id: http://devicetree.org/schemas/pinctrl/nvidia,tegra114-pinmux.yaml#
> > +$schema: http://devicetree.org/meta-schemas/core.yaml#
> > +
> > +title: NVIDIA Tegra114 pinmux Controller
> > +
> > +maintainers:
> > +  - Thierry Reding <thierry.reding@xxxxxxxxx>
> > +  - Jon Hunter <jonathanh@xxxxxxxxxx>
> > +
> > +properties:
> > +  compatible:
> > +    const: nvidia,tegra114-pinmux
> > +
> > +  reg:
> > +    items:
> > +      - description: pad control registers
> > +      - description: mux registers
> > +
> > +additionalProperties:
> > +  type: object
> > +  description: |
> > +    Please refer to pinctrl-bindings.txt in this directory for details of the common pinctrl
> > +    bindings used by client devices, including the meaning of the phrase "pin configuration
> > +    node".
> > +
> > +    Tegra's pin configuration nodes act as a container for an arbitrary number of subnodes. Each
> > +    of these subnodes represents some desired configuration for a pin, a group, or a list of
> > +    pins or groups. This configuration can include the mux function to select on those pin(s)/
> > +    group(s), and various pin configuration parameters, such as pull-up, tristate, drive
> > +    strength, etc.
> > +
> > +    The name of each subnode is not important; all subnodes should be enumerated and processed
> > +    purely based on their content.
> > +
> > +    Each subnode only affects those parameters that are explicitly listed. In other words, a
> > +    subnode that lists a mux function but no pin configuration parameters implies no information
> > +    about any pin configuration parameters.
> > +
> > +    Similarly, a pin subnode that describes a pullup parameter implies no information about e.g.
> > +    the mux function or tristate parameter. For this reason, even seemingly boolean values are
> > +    actually tristates in this binding: unspecified, off, or on. Unspecified is represented as
> > +    an absent property, and off/on are represented as integer values 0 and 1.
> > +
> > +    Note that many of these properties are only valid for certain specific pins or groups. See
> > +    the Tegra TRM and various pinmux spreadsheets for complete details regarding which groups
> > +    support which functionality. The Linux pinctrl driver may also be a useful reference, since
> > +    it consolidates, disambiguates, and corrects data from all those sources.
> 
> Wrap at 80 unless there's some reason not to for certain lines. 
> 
> > +
> > +  patternProperties:
> > +    "^pinmux(-[a-z0-9-_]+)?$":
> > +      type: object
> > +      properties:
> > +        nvidia,pins:
> > +          $ref: /schemas/types.yaml#/definitions/string-array
> > +          description: An array of strings. Each string contains the name of a pin or group.
> > +            Valid values for these names are listed below.
> > +          items:
> > +            enum: [ ulpi_data0_po1, ulpi_data1_po2, ulpi_data2_po3, ulpi_data3_po4, ulpi_data4_po5,
> > +                    ulpi_data5_po6, ulpi_data6_po7, ulpi_data7_po0, ulpi_clk_py0, ulpi_dir_py1,
> > +                    ulpi_nxt_py2, ulpi_stp_py3, dap3_fs_pp0, dap3_din_pp1, dap3_dout_pp2,
> > +                    dap3_sclk_pp3, pv0, pv1, sdmmc1_clk_pz0, sdmmc1_cmd_pz1, sdmmc1_dat3_py4,
> > +                    sdmmc1_dat2_py5, sdmmc1_dat1_py6, sdmmc1_dat0_py7, clk2_out_pw5, clk2_req_pcc5,
> > +                    hdmi_int_pn7, ddc_scl_pv4, ddc_sda_pv5, uart2_rxd_pc3, uart2_txd_pc2,
> > +                    uart2_rts_n_pj6, uart2_cts_n_pj5, uart3_txd_pw6, uart3_rxd_pw7,
> > +                    uart3_cts_n_pa1, uart3_rts_n_pc0, pu0, pu1, pu2, pu3, pu4, pu5, pu6,
> > +                    gen1_i2c_sda_pc5, gen1_i2c_scl_pc4, dap4_fs_pp4, dap4_din_pp5, dap4_dout_pp6,
> > +                    dap4_sclk_pp7, clk3_out_pee0, clk3_req_pee1, gmi_wp_n_pc7, gmi_iordy_pi5,
> > +                    gmi_wait_pi7, gmi_adv_n_pk0, gmi_clk_pk1, gmi_cs0_n_pj0, gmi_cs1_n_pj2,
> > +                    gmi_cs2_n_pk3, gmi_cs3_n_pk4, gmi_cs4_n_pk2, gmi_cs6_n_pi3, gmi_cs7_n_pi6,
> > +                    gmi_ad0_pg0, gmi_ad1_pg1, gmi_ad2_pg2, gmi_ad3_pg3, gmi_ad4_pg4, gmi_ad5_pg5,
> > +                    gmi_ad6_pg6, gmi_ad7_pg7, gmi_ad8_ph0, gmi_ad9_ph1, gmi_ad10_ph2, gmi_ad11_ph3,
> > +                    gmi_ad12_ph4, gmi_ad13_ph5, gmi_ad14_ph6, gmi_ad15_ph7, gmi_a16_pj7,
> > +                    gmi_a17_pb0, gmi_a18_pb1, gmi_a19_pk7, gmi_wr_n_pi0, gmi_oe_n_pi1,
> > +                    gmi_dqs_p_pj3, gmi_rst_n_pi4, gen2_i2c_scl_pt5, gen2_i2c_sda_pt6,
> > +                    sdmmc4_clk_pcc4, sdmmc4_cmd_pt7, sdmmc4_dat0_paa0, sdmmc4_dat1_paa1,
> > +                    sdmmc4_dat2_paa2, sdmmc4_dat3_paa3, sdmmc4_dat4_paa4, sdmmc4_dat5_paa5,
> > +                    sdmmc4_dat6_paa6, sdmmc4_dat7_paa7, cam_mclk_pcc0, pcc1, pbb0,
> > +                    cam_i2c_scl_pbb1, cam_i2c_sda_pbb2, pbb3, pbb4, pbb5, pbb6, pbb7, pcc2,
> > +                    pwr_i2c_scl_pz6, pwr_i2c_sda_pz7, kb_row0_pr0, kb_row1_pr1, kb_row2_pr2,
> > +                    kb_row3_pr3, kb_row4_pr4, kb_row5_pr5, kb_row6_pr6, kb_row7_pr7, kb_row8_ps0,
> > +                    kb_row9_ps1, kb_row10_ps2, kb_col0_pq0, kb_col1_pq1, kb_col2_pq2, kb_col3_pq3,
> > +                    kb_col4_pq4, kb_col5_pq5, kb_col6_pq6, kb_col7_pq7, clk_32k_out_pa0,
> > +                    sys_clk_req_pz5, core_pwr_req, cpu_pwr_req, pwr_int_n, owr, dap1_fs_pn0,
> > +                    dap1_din_pn1, dap1_dout_pn2, dap1_sclk_pn3, clk1_req_pee2, clk1_out_pw4,
> > +                    spdif_in_pk6, spdif_out_pk5, dap2_fs_pa2, dap2_din_pa4, dap2_dout_pa5,
> > +                    dap2_sclk_pa3, dvfs_pwm_px0, gpio_x1_aud_px1, gpio_x3_aud_px3, dvfs_clk_px2,
> > +                    gpio_x4_aud_px4, gpio_x5_aud_px5, gpio_x6_aud_px6, gpio_x7_aud_px7,
> > +                    sdmmc3_clk_pa6, sdmmc3_cmd_pa7, sdmmc3_dat0_pb7, sdmmc3_dat1_pb6,
> > +                    sdmmc3_dat2_pb5, sdmmc3_dat3_pb4, hdmi_cec_pee3, sdmmc1_wp_n_pv3,
> > +                    sdmmc3_cd_n_pv2, gpio_w2_aud_pw2, gpio_w3_aud_pw3, usb_vbus_en0_pn4,
> > +                    usb_vbus_en1_pn5, sdmmc3_clk_lb_in_pee5, sdmmc3_clk_lb_out_pee4, reset_out_n,
> > +                    # drive groups
> > +                    drive_ao1, drive_ao2, drive_at1, drive_at2, drive_at3, drive_at4, drive_at5,
> > +                    drive_cdev1, drive_cdev2, drive_dap1, drive_dap2, drive_dap3, drive_dap4,
> > +                    drive_dbg, drive_sdio3, drive_spi, drive_uaa, drive_uab, drive_uart2,
> > +                    drive_uart3, drive_sdio1, drive_ddc, drive_gma, drive_gme, drive_gmf,
> > +                    drive_gmg, drive_gmh, drive_owr, drive_uda ]
> > +
> > +        nvidia,function:
> > +          $ref: /schemas/types.yaml#/definitions/string
> > +          description: A string containing the name of the function to mux to the pin or group.
> > +            Valid values for function names are listed below. See the Tegra TRM to determine
> > +            which are valid for each pin or group.
> > +          enum: [ blink, cec, cldvfs, clk12, cpu, dap, dap1, dap2, dev3, displaya, displaya_alt,
> > +                  displayb, dtv, emc_dll, extperiph1, extperiph2, extperiph3, gmi, gmi_alt, hda,
> > +                  hsi, i2c1, i2c2, i2c3, i2c4, i2cpwr, i2s0, i2s1, i2s2, i2s3, i2s4, irda, kbc,
> > +                  nand, nand_alt, owr, pmi, pwm0, pwm1, pwm2, pwm3, pwron, reset_out_n, rsvd1,
> > +                  rsvd2, rsvd3, rsvd4, sdmmc1, sdmmc2, sdmmc3, sdmmc4, soc, spdif, spi1, spi2,
> > +                  spi3, spi4, spi5, spi6, sysclk, trace, uarta, uartb, uartc, uartd, ulpi, usb,
> > +                  vgp1, vgp2, vgp3, vgp4, vgp5, vgp6, vi, vi_alt1, vi_alt3 ]
> > +
> > +        nvidia,pull:
> > +          description: Pull-down/up setting to apply to the pin.
> > +          $ref: /schemas/types.yaml#/definitions/uint32
> > +          oneOf:
> > +            - description: none
> > +              const: 0
> > +            - description: down
> > +              const: 1
> > +            - description: up
> > +              const: 2
> > +
> > +        nvidia,tristate:
> > +          description: Tristate setting to apply to the pin.
> > +          $ref: /schemas/types.yaml#/definitions/uint32
> > +          oneOf:
> > +            - description: drive
> > +              const: 0
> > +            - description: tristate
> > +              const: 1
> > +
> > +        nvidia,enable-input:
> > +          description: Enable the pin's input path.
> > +          $ref: /schemas/types.yaml#/definitions/uint32
> > +          oneOf:
> > +            - description: disable input (i.e. output only)
> > +              const: 0
> > +            - description: enable input
> > +              const: 1
> > +
> > +        nvidia,open-drain:
> > +          description: Open-drain configuration for the pin.
> > +          $ref: /schemas/types.yaml#/definitions/uint32
> > +          oneOf:
> > +            - description: disable open-drain
> > +              const: 0
> > +            - description: enable open-drain
> > +              const: 1
> > +
> > +        nvidia,lock:
> > +          description: Lock the pin configuration against further changes until reset.
> > +          $ref: /schemas/types.yaml#/definitions/uint32
> > +          oneOf:
> > +            - description: disable pin configuration lock
> > +              const: 0
> > +            - description: enable pin configuration lock
> > +              const: 1
> > +
> > +        nvidia,io-reset:
> > +          description: reset the I/O path
> > +          $ref: /schemas/types.yaml#/definitions/uint32
> > +          enum: [ 0, 1 ]
> > +
> > +        nvidia,rcv-sel:
> > +          description: select VIL/VIH receivers
> > +          $ref: /schemas/types.yaml#/definitions/uint32
> > +          oneOf:
> > +            - description: normal receivers
> > +              const: 0
> > +            - description: high-voltage receivers
> > +              const: 1
> > +
> > +        nvidia,high-speed-mode:
> > +          description: Enable high speed mode the pins.
> > +          $ref: /schemas/types.yaml#/definitions/uint32
> > +          oneOf:
> > +            - description: normal speed mode
> > +              const: 0
> > +            - description: high speed mode
> > +              const: 1
> > +
> > +        nvidia,schmitt:
> > +          description: Enable Schmitt trigger on the input.
> > +          $ref: /schemas/types.yaml#/definitions/uint32
> > +          oneOf:
> > +            - description: disable Schmitt trigger on the input
> > +              const: 0
> > +            - description: enable Schmitt trigger on the input
> > +              const: 1
> > +
> > +        nvidia,low-power-mode:
> > +          description: Controls the drive power or current. Valid values are from 0 through 3,
> > +            where 0 specifies the least power and 3 specifies the most power. See "Low Power
> > +            Mode" or "LPMD1" and "LPMD0" in the Tegra TRM.
> > +          $ref: /schemas/types.yaml#/definitions/uint32
> > +          enum: [ 0, 1, 2, 3 ]
> > +
> > +        nvidia,drive-type:
> > +          description: Drive type to configure for the pin.
> > +          $ref: /schemas/types.yaml#/definitions/uint32
> > +          enum: [ 0, 1, 2, 3 ]
> > +
> > +        nvidia,pull-down-strength:
> > +          description: Controls drive strength. 0 is weakest. The range of valid values depends
> > +            on the pingroup. See "CAL_DRVDN" in the Tegra TRM.
> > +          $ref: /schemas/types.yaml#/definitions/uint32
> > +
> > +        nvidia,pull-up-strength:
> > +          description: Controls drive strength. 0 is weakest. The range of valid values depends
> > +            on the pingroup. See "CAL_DRVUP" in the Tegra TRM.
> > +          $ref: /schemas/types.yaml#/definitions/uint32
> > +
> > +        nvidia,slew-rate-rising:
> > +          description: Controls rising signal slew rate. 0 is fastest. The range of valid values
> > +            depends on the pingroup. See "DRVDN_SLWR" in the Tegra TRM.
> > +          $ref: /schemas/types.yaml#/definitions/uint32
> > +
> > +        nvidia,slew-rate-falling:
> > +          description: Controls falling signal slew rate. 0 is fastest. The range of valid
> > +            values depends on the pingroup. See "DRVUP_SLWF" in the Tegra TRM.
> > +          $ref: /schemas/types.yaml#/definitions/uint32
> 
> Should all or some of these go in a shared schema file?

I've put the truly common ones into a shared schema file that can be
inherited. I then also added definitions for the other properties to the
shared schema so that they can be individually referenced as needed. Not
all of those properties are common across SoC generations,
unfortunately. Let me know if the result in v2 looks good to you.

> 
> > +
> > +      additionalProperties: false
> 
> For indented cases, it's more readable to put this at the beginning 
> above 'properties'.

Done.

> > +
> > +      required:
> > +        - nvidia,pins
> > +
> > +required:
> > +  - compatible
> > +  - reg
> > +
> > +examples:
> > +  - |
> > +    pinmux@70000868 {
> > +      compatible = "nvidia,tegra114-pinmux";
> > +      reg = <0x70000868 0x148>, /* Pad control registers */
> > +            <0x70003000 0x40c>; /* PinMux registers */
> > +
> > +      pinmux {
> > +        sdmmc4_clk_pcc4 {
> > +          nvidia,pins = "sdmmc4_clk_pcc4";
> > +          nvidia,function = "sdmmc4";
> > +          nvidia,pull = <0>;
> > +          nvidia,tristate = <0>;
> > +        };
> > +
> > +        sdmmc4_dat0_paa0 {
> > +          nvidia,pins = "sdmmc4_dat0_paa0",
> > +                        "sdmmc4_dat1_paa1",
> > +                        "sdmmc4_dat2_paa2",
> > +                        "sdmmc4_dat3_paa3",
> > +                        "sdmmc4_dat4_paa4",
> > +                        "sdmmc4_dat5_paa5",
> > +                        "sdmmc4_dat6_paa6",
> > +                        "sdmmc4_dat7_paa7";
> > +          nvidia,function = "sdmmc4";
> > +          nvidia,pull = <2>;
> > +          nvidia,tristate = <0>;
> > +        };
> > +      };
> > +    };
> 
> 
> > diff --git a/Documentation/devicetree/bindings/pinctrl/nvidia,tegra124-pinmux.yaml b/Documentation/devicetree/bindings/pinctrl/nvidia,tegra124-pinmux.yaml
> > new file mode 100644
> > index 000000000000..2702a80104cc
> > --- /dev/null
> > +++ b/Documentation/devicetree/bindings/pinctrl/nvidia,tegra124-pinmux.yaml
> > @@ -0,0 +1,287 @@
> > +# SPDX-License-Identifier: (GPL-2.0+ OR BSD-2-Clause)
> > +%YAML 1.2
> > +---
> > +$id: http://devicetree.org/schemas/pinctrl/nvidia,tegra124-pinmux.yaml#
> > +$schema: http://devicetree.org/meta-schemas/core.yaml#
> > +
> > +title: NVIDIA Tegra124 Pinmux Controller
> > +
> > +maintainers:
> > +  - Thierry Reding <thierry.reding@xxxxxxxxx>
> > +  - Jon Hunter <jonathanh@xxxxxxxxxx>
> > +
> > +description: The Tegra124 pinctrl binding is very similar to the Tegra20 and Tegra30 pinctrl
> > +  binding, as described in nvidia,tegra20-pinmux.yaml and nvidia,tegra30-pinmux.yaml. In fact, this
> > +  document assumes that binding as a baseline, and only documents the differences between the two
> > +  bindings.
> > +
> > +properties:
> > +  compatible:
> > +    oneOf:
> > +      - const: nvidia,tegra124-pinmux
> > +      - items:
> > +          - const: nvidia,tegra132-pinmux
> > +          - const: nvidia,tegra124-pinmux
> > +
> > +  reg:
> > +    items:
> > +      - description: driver strength and pad control registers
> > +      - description: pinmux registers
> > +      - description: MIPI_PAD_CTRL registers
> > +
> > +additionalProperties:
> > +  description: |
> > +    Tegra's pin configuration nodes act as containers for an arbitrary number of subnodes. Each of
> > +    these subnodes represents some desired configuration for a pin, a group, or a list of pins or
> > +    groups. This configuration can include the mux function to select on those pin(s)/group(s), and
> > +    various pin configuration parameters, such as pull-up, tristate, drive strength, etc.
> > +
> > +    The name of each subnode is not important; all subnodes should be enumerated and processed
> > +    purely based on their content.
> > +
> > +    Each subnode only affects those parameters that are explicitly listed. In other words, a
> > +    subnode that lists a mux function but no pin configuration parameters implies no information
> > +    about any pin configuration parameters. Similarly, a pin subnode that describes a pullup
> > +    parameter implies no information about e.g. the mux function or tristate parameter. For this
> > +    reason, even seemingly boolean values are actually tristates in this binding: unspecified, off,
> > +    or on. Unspecified is represented as an absent property, and off/on are represented as integer
> > +    values 0 and 1.
> > +
> > +    See the TRM to determine which properties and values apply to each pin/group. Macro values for
> > +    property values are defined in include/dt-binding/pinctrl/pinctrl-tegra.h.
> > +  type: object
> > +  patternProperties:
> > +    "^pinmux(-[a-z0-9-_]+)?$":
> 
>          type: object
> 
> > +      properties:
> > +        nvidia,pins:
> > +          description: An array of strings. Each string contains the name of a pin or group. Valid
> > +            values for these names are listed below.
> > +          $ref: /schemas/types.yaml#/definitions/string-array
> > +          items:
> > +            enum: [ ulpi_data0_po1, ulpi_data1_po2, ulpi_data2_po3, ulpi_data3_po4,
> > +                    ulpi_data4_po5, ulpi_data5_po6, ulpi_data6_po7, ulpi_data7_po0, ulpi_clk_py0,
> > +                    ulpi_dir_py1, ulpi_nxt_py2, ulpi_stp_py3, dap3_fs_pp0, dap3_din_pp1,
> > +                    dap3_dout_pp2, dap3_sclk_pp3, pv0, pv1, sdmmc1_clk_pz0, sdmmc1_cmd_pz1,
> > +                    sdmmc1_dat3_py4, sdmmc1_dat2_py5, sdmmc1_dat1_py6, sdmmc1_dat0_py7,
> > +                    clk2_out_pw5, clk2_req_pcc5, hdmi_int_pn7, ddc_scl_pv4, ddc_sda_pv5,
> > +                    uart2_rxd_pc3, uart2_txd_pc2, uart2_rts_n_pj6, uart2_cts_n_pj5,
> > +                    uart3_txd_pw6, uart3_rxd_pw7, uart3_cts_n_pa1, uart3_rts_n_pc0, pu0, pu1,
> > +                    pu2, pu3, pu4, pu5, pu6, gen1_i2c_scl_pc4, gen1_i2c_sda_pc5, dap4_fs_pp4,
> > +                    dap4_din_pp5, dap4_dout_pp6, dap4_sclk_pp7, clk3_out_pee0, clk3_req_pee1,
> > +                    pc7, pi5, pi7, pk0, pk1, pj0, pj2, pk3, pk4, pk2, pi3, pi6, pg0, pg1, pg2,
> > +                    pg3, pg4, pg5, pg6, pg7, ph0, ph1, ph2, ph3, ph4, ph5, ph6, ph7, pj7, pb0,
> > +                    pb1, pk7, pi0, pi1, pi2, pi4, gen2_i2c_scl_pt5, gen2_i2c_sda_pt6,
> > +                    sdmmc4_clk_pcc4, sdmmc4_cmd_pt7, sdmmc4_dat0_paa0, sdmmc4_dat1_paa1,
> > +                    sdmmc4_dat2_paa2, sdmmc4_dat3_paa3, sdmmc4_dat4_paa4, sdmmc4_dat5_paa5,
> > +                    sdmmc4_dat6_paa6, sdmmc4_dat7_paa7, cam_mclk_pcc0, pcc1, pbb0,
> > +                    cam_i2c_scl_pbb1, cam_i2c_sda_pbb2, pbb3, pbb4, pbb5, pbb6, pbb7, pcc2,
> > +                    jtag_rtck, pwr_i2c_scl_pz6, pwr_i2c_sda_pz7, kb_row0_pr0, kb_row1_pr1,
> > +                    kb_row2_pr2, kb_row3_pr3, kb_row4_pr4, kb_row5_pr5, kb_row6_pr6, kb_row7_pr7,
> > +                    kb_row8_ps0, kb_row9_ps1, kb_row10_ps2, kb_row11_ps3, kb_row12_ps4,
> > +                    kb_row13_ps5, kb_row14_ps6, kb_row15_ps7, kb_col0_pq0, kb_col1_pq1,
> > +                    kb_col2_pq2, kb_col3_pq3, kb_col4_pq4, kb_col5_pq5, kb_col6_pq6, kb_col7_pq7,
> > +                    clk_32k_out_pa0, core_pwr_req, cpu_pwr_req, pwr_int_n, clk_32k_in, owr,
> > +                    dap1_fs_pn0, dap1_din_pn1, dap1_dout_pn2, dap1_sclk_pn3, dap_mclk1_req_pee2,
> > +                    dap_mclk1_pw4, spdif_in_pk6, spdif_out_pk5, dap2_fs_pa2, dap2_din_pa4,
> > +                    dap2_dout_pa5, dap2_sclk_pa3, dvfs_pwm_px0, gpio_x1_aud_px1, gpio_x3_aud_px3,
> > +                    dvfs_clk_px2, gpio_x4_aud_px4, gpio_x5_aud_px5, gpio_x6_aud_px6,
> > +                    gpio_x7_aud_px7, sdmmc3_clk_pa6, sdmmc3_cmd_pa7, sdmmc3_dat0_pb7,
> > +                    sdmmc3_dat1_pb6, sdmmc3_dat2_pb5, sdmmc3_dat3_pb4, pex_l0_rst_n_pdd1,
> > +                    pex_l0_clkreq_n_pdd2, pex_wake_n_pdd3, pex_l1_rst_n_pdd5,
> > +                    pex_l1_clkreq_n_pdd6, hdmi_cec_pee3, sdmmc1_wp_n_pv3, sdmmc3_cd_n_pv2,
> > +                    gpio_w2_aud_pw2, gpio_w3_aud_pw3, usb_vbus_en0_pn4, usb_vbus_en1_pn5,
> > +                    sdmmc3_clk_lb_out_pee4, sdmmc3_clk_lb_in_pee5, gmi_clk_lb, reset_out_n,
> > +                    kb_row16_pt0, kb_row17_pt1, usb_vbus_en2_pff1, pff2, dp_hpd_pff0,
> > +                    # drive groups
> > +                    drive_ao1, drive_ao2, drive_at1, drive_at2, drive_at3, drive_at4, drive_at5,
> > +                    drive_cdev1, drive_cdev2, drive_dap1, drive_dap2, drive_dap3, drive_dap4,
> > +                    drive_dbg, drive_sdio3, drive_spi, drive_uaa, drive_uab, drive_uart2,
> > +                    drive_uart3, drive_sdio1, drive_ddc, drive_gma, drive_gme, drive_gmf,
> > +                    drive_gmg, drive_gmh, drive_owr, drive_uda, drive_gpv, drive_dev3, drive_cec,
> > +                    drive_usb_vbus_en, drive_ao3, drive_ao0, drive_hv0, drive_sdio4, drive_ao4,
> > +                    # MIPI pad control groups
> > +                    mipi_pad_ctrl_dsi_b ]
> > +
> > +        nvidia,function:
> > +          description: A string containing the name of the function to mux to the pin or group.
> > +          $ref: /schemas/types.yaml#/definitions/string
> > +          enum: [ blink, cec, cldvfs, clk12, cpu, dap, dap1, dap2, dev3, displaya, displaya_alt,
> > +                  displayb, dtv, extperiph1, extperiph2, extperiph3, gmi, gmi_alt, hda, hsi, i2c1,
> > +                  i2c2, i2c3, i2c4, i2cpwr, i2s0, i2s1, i2s2, i2s3, i2s4, irda, kbc, owr, pmi,
> > +                  pwm0, pwm1, pwm2, pwm3, pwron, reset_out_n, rsvd1, rsvd2, rsvd3, rsvd4, sdmmc1,
> > +                  sdmmc2, sdmmc3, sdmmc4, soc, spdif, spi1, spi2, spi3, spi4, spi5, spi6, trace,
> > +                  uarta, uartb, uartc, uartd, ulpi, usb, vgp1, vgp2, vgp3, vgp4, vgp5, vgp6, vi,
> > +                  vi_alt1, vi_alt3, vimclk2, vimclk2_alt, sata, ccla, pe0, pe, pe1, dp, rtck, sys,
> > +                  clk, tmds, csi, dsi_b ]
> > +
> > +        nvidia,pull:
> > +          description: Pull-down/up setting to apply to the pin.
> > +          $ref: /schemas/types.yaml#/definitions/uint32
> > +          oneOf:
> > +            - description: none
> > +              const: 0
> > +            - description: down
> > +              const: 1
> > +            - description: up
> > +              const: 2
> > +
> > +        nvidia,tristate:
> > +          description: Tristate setting to apply to the pin.
> > +          $ref: /schemas/types.yaml#/definitions/uint32
> > +          oneOf:
> > +            - description: drive
> > +              const: 0
> > +            - description: tristate
> > +              const: 1
> > +
> > +        nvidia,enable-input:
> > +          description: Enable the pin's input path.
> > +          $ref: /schemas/types.yaml#/definitions/uint32
> > +          oneOf:
> > +            - description: disable input (i.e. output only)
> > +              const: 0
> > +            - description: enable input
> > +              const: 1
> > +
> > +        nvidia,open-drain:
> > +          description: Open-drain configuration for the pin.
> > +          $ref: /schemas/types.yaml#/definitions/uint32
> > +          oneOf:
> > +            - description: disable open-drain
> > +              const: 0
> > +            - description: enable open-drain
> > +              const: 1
> > +
> > +        nvidia,lock:
> > +          description: Lock the pin configuration against further changes until reset.
> > +          $ref: /schemas/types.yaml#/definitions/uint32
> > +          oneOf:
> > +            - description: disable pin configuration lock
> > +              const: 0
> > +            - description: enable pin configuration lock
> > +              const: 1
> > +
> > +        nvidia,io-reset:
> > +          description: reset the I/O path
> > +          $ref: /schemas/types.yaml#/definitions/uint32
> > +          enum: [ 0, 1 ]
> > +
> > +        nvidia,rcv-sel:
> > +          description: select VIL/VIH receivers
> > +          $ref: /schemas/types.yaml#/definitions/uint32
> > +          oneOf:
> > +            - description: normal receivers
> > +              const: 0
> > +            - description: high-voltage receivers
> > +              const: 1
> > +
> > +        nvidia,high-speed-mode:
> > +          description: Enable high speed mode the pins.
> > +          $ref: /schemas/types.yaml#/definitions/uint32
> > +          oneOf:
> > +            - description: normal speed mode
> > +              const: 0
> > +            - description: high speed mode
> > +              const: 1
> > +
> > +        nvidia,schmitt:
> > +          description: Enable Schmitt trigger on the input.
> > +          $ref: /schemas/types.yaml#/definitions/uint32
> > +          oneOf:
> > +            - description: disable Schmitt trigger on the input
> > +              const: 0
> > +            - description: enable Schmitt trigger on the input
> > +              const: 1
> > +
> > +        nvidia,low-power-mode:
> > +          description: Controls the drive power or current. Valid values are from 0 through 3,
> > +            where 0 specifies the least power and 3 specifies the most power. See "Low Power Mode"
> > +            or "LPMD1" and "LPMD0" in the Tegra TRM.
> > +          $ref: /schemas/types.yaml#/definitions/uint32
> > +          enum: [ 0, 1, 2, 3 ]
> > +
> > +        nvidia,drive-type:
> > +          description: Drive type to configure for the pin.
> > +          $ref: /schemas/types.yaml#/definitions/uint32
> > +          enum: [ 0, 1, 2, 3 ]
> > +
> > +        nvidia,pull-down-strength:
> > +          description: Controls drive strength. 0 is weakest. The range of valid values depends on
> > +            the pingroup. See "CAL_DRVDN" in the Tegra TRM.
> > +          $ref: /schemas/types.yaml#/definitions/uint32
> > +
> > +        nvidia,pull-up-strength:
> > +          description: Controls drive strength. 0 is weakest. The range of valid values depends on
> > +            the pingroup. See "CAL_DRVUP" in the Tegra TRM.
> > +          $ref: /schemas/types.yaml#/definitions/uint32
> > +
> > +        nvidia,slew-rate-rising:
> > +          description: Controls rising signal slew rate. 0 is fastest. The range of valid values
> > +            depends on the pingroup. See "DRVDN_SLWR" in the Tegra TRM.
> > +          $ref: /schemas/types.yaml#/definitions/uint32
> > +
> > +        nvidia,slew-rate-falling:
> > +          description: Controls falling signal slew rate. 0 is fastest. The range of valid values
> > +            depends on the pingroup. See "DRVUP_SLWF" in the Tegra TRM.
> > +          $ref: /schemas/types.yaml#/definitions/uint32
> > +
> > +      additionalProperties: false
> > +
> > +      required:
> > +        - nvidia,pins
> > +
> > +required:
> > +  - compatible
> > +  - reg
> > +
> > +examples:
> > +  - |
> > +    #include <dt-bindings/clock/tegra124-car.h>
> > +    #include <dt-bindings/interrupt-controller/arm-gic.h>
> > +    #include <dt-bindings/pinctrl/pinctrl-tegra.h>
> > +
> > +    pinmux@70000868 {
> > +        compatible = "nvidia,tegra124-pinmux";
> > +        reg = <0x70000868 0x164>, /* Pad control registers */
> > +              <0x70003000 0x434>, /* Mux registers */
> > +              <0x70000820 0x8>;   /* MIPI pad control */
> > +
> > +        sdmmc4_default: pinmux {
> > +            sdmmc4_clk_pcc4 {
> > +                nvidia,pins = "sdmmc4_clk_pcc4";
> > +                nvidia,function = "sdmmc4";
> > +                nvidia,pull = <TEGRA_PIN_PULL_NONE>;
> > +                nvidia,tristate = <TEGRA_PIN_DISABLE>;
> > +            };
> > +
> > +            sdmmc4_dat0_paa0 {
> > +                nvidia,pins = "sdmmc4_dat0_paa0",
> > +                              "sdmmc4_dat1_paa1",
> > +                              "sdmmc4_dat2_paa2",
> > +                              "sdmmc4_dat3_paa3",
> > +                              "sdmmc4_dat4_paa4",
> > +                              "sdmmc4_dat5_paa5",
> > +                              "sdmmc4_dat6_paa6",
> > +                              "sdmmc4_dat7_paa7";
> > +                nvidia,function = "sdmmc4";
> > +                nvidia,pull = <TEGRA_PIN_PULL_UP>;
> > +                nvidia,tristate = <TEGRA_PIN_DISABLE>;
> > +            };
> > +        };
> > +    };
> > +
> > +    mmc@700b0600 {
> > +        compatible = "nvidia,tegra124-sdhci";
> > +        reg = <0x700b0600 0x200>;
> > +        interrupts = <GIC_SPI 31 IRQ_TYPE_LEVEL_HIGH>;
> > +        clocks = <&tegra_car TEGRA124_CLK_SDMMC4>;
> > +        clock-names = "sdhci";
> > +        resets = <&tegra_car 15>;
> > +        reset-names = "sdhci";
> > +
> > +        bus-width = <8>;
> > +        non-removable;
> > +
> > +        pinctrl-names = "default";
> > +        pinctrl-0 = <&sdmmc4_default>;
> > +    };
> 
> No need to show the client side.

I've removed these snippets now.

Thanks,
Thierry

Attachment: signature.asc
Description: PGP signature


[Index of Archives]     [ARM Kernel]     [Linux ARM]     [Linux ARM MSM]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux