Message ID | 20230824105703.19612-7-tychang@realtek.com |
---|---|
State | Changes Requested, archived |
Headers | show |
Series | Add pinctrl driver support for Realtek DHC SoCs | expand |
Context | Check | Description |
---|---|---|
robh/checkpatch | success | |
robh/patch-applied | success | |
robh/dtbs-check | warning | build log |
robh/dt-meta-schema | success |
On Thu, Aug 24, 2023 at 06:57:02PM +0800, Tzuyi Chang wrote: > Add device tree bindings for RTD1319D. > > Signed-off-by: Tzuyi Chang <tychang@realtek.com> > --- > v1 to v2 change: > 1. Add a description for RTD1319D. > 2. Rename realtek,pdrive, realtekmndrive and realtek,dcycle. > 3. Add a description for PMOS and NMOS driving strength. > 4. Remove the wildcard in the compatible strings. > 5. Use '-pins$' to be node name pattern. > --- > .../pinctrl/realtek,rtd1319d-pinctrl.yaml | 189 ++++++++++++++++++ > 1 file changed, 189 insertions(+) > create mode 100644 Documentation/devicetree/bindings/pinctrl/realtek,rtd1319d-pinctrl.yaml > > diff --git a/Documentation/devicetree/bindings/pinctrl/realtek,rtd1319d-pinctrl.yaml b/Documentation/devicetree/bindings/pinctrl/realtek,rtd1319d-pinctrl.yaml > new file mode 100644 > index 000000000000..8653d42ac1f3 > --- /dev/null > +++ b/Documentation/devicetree/bindings/pinctrl/realtek,rtd1319d-pinctrl.yaml > @@ -0,0 +1,189 @@ > +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause) > +# Copyright 2023 Realtek Semiconductor Corporation > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/pinctrl/realtek,rtd1319d-pinctrl.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: Realtek DHC RTD1319D Pin Controller > + > +maintainers: > + - TY Chang <tychang@realtek.com> > + > +description: > + The Realtek DHC RTD1319D is a high-definition media processor SoC. The > + RTD1319D pin controller is used to control pin function, pull up/down > + resistor, drive strength, schmitt trigger and power source. > + > +properties: > + compatible: > + const: realtek,rtd1319d-pinctrl > + > + reg: > + maxItems: 1 > + > +patternProperties: > + '^.*$': In case it was not obvious, my comment to use '-pins' applies here and to the next patch. > + if: As does dropping if/then. > + type: object > + then: > + allOf: > + - $ref: pincfg-node.yaml# > + - $ref: pinmux-node.yaml# > + > + properties: > + pins: > + items: > + enum: [ gpio_0, gpio_1, gpio_2, gpio_3, gpio_4, gpio_5, gpio_6, gpio_7, > + gpio_8, gpio_9, gpio_10, gpio_11, gpio_12, gpio_13, gpio_14, > + gpio_15, gpio_16, gpio_17, gpio_18, gpio_19, gpio_20, gpio_21, > + gpio_22, gpio_23, usb_cc2, gpio_25, gpio_26, gpio_27, gpio_28, > + gpio_29, gpio_30, gpio_31, gpio_32, gpio_33, gpio_34, gpio_35, > + hif_data, hif_en, hif_rdy, hif_clk, gpio_40, gpio_41, gpio_42, > + gpio_43, gpio_44, gpio_45, gpio_46, gpio_47, gpio_48, gpio_49, > + gpio_50, usb_cc1, gpio_52, gpio_53, ir_rx, ur0_rx, ur0_tx, > + gpio_57, gpio_58, gpio_59, gpio_60, gpio_61, gpio_62, gpio_63, > + gpio_64, emmc_rst_n, emmc_dd_sb, emmc_clk, emmc_cmd, emmc_data_0, > + emmc_data_1, emmc_data_2, emmc_data_3, emmc_data_4, emmc_data_5, > + emmc_data_6, emmc_data_7, dummy, gpio_78, gpio_79, gpio_80, > + gpio_81, ur2_loc, gspi_loc, hi_width, sf_en, arm_trace_dbg_en, > + ejtag_aucpu_loc, ejtag_acpu_loc, ejtag_vcpu_loc, ejtag_scpu_loc, > + dmic_loc, ejtag_secpu_loc, vtc_dmic_loc, vtc_tdm_loc, vtc_i2si_loc, > + tdm_ai_loc, ai_loc, spdif_loc, hif_en_loc, sc0_loc, sc1_loc, > + scan_switch, wd_rset, boot_sel, reset_n, testmode ] > + > + function: > + enum: [ gpio, nf, emmc, tp0, tp1, sc0, sc0_data0, sc0_data1, sc0_data2, > + sc1, sc1_data0, sc1_data1, sc1_data2, ao, gspi_loc0, gspi_loc1, > + uart0, uart1, uart2_loc0, uart2_loc1, i2c0, i2c1, i2c3, i2c4, > + i2c5, pcie1, sdio, etn_led, etn_phy, spi, pwm0_loc0, pwm0_loc1, > + pwm1_loc0, pwm1_loc1, pwm2_loc0, pwm2_loc1, pwm3_loc0, pwm3_loc1, > + qam_agc_if0, qam_agc_if1, spdif_optical_loc0, spdif_optical_loc1, > + usb_cc1, usb_cc2, vfd, sd, dmic_loc0, dmic_loc1, ai_loc0, ai_loc1, > + tdm_ai_loc0, tdm_ai_loc1, hi_loc0, hi_m, vtc_i2so, vtc_i2si_loc0, > + vtc_i2si_loc1, vtc_dmic_loc0, vtc_dmic_loc1, vtc_tdm_loc0, > + vtc_tdm_loc1, dc_fan, pll_test_loc0, pll_test_loc1, ir_rx, > + uart2_disable, gspi_disable, hi_width_disable, hi_width_1bit, > + sf_disable, sf_enable, scpu_ejtag_loc0, scpu_ejtag_loc1, > + scpu_ejtag_loc2, acpu_ejtag_loc0, acpu_ejtag_loc1, acpu_ejtag_loc2, > + vcpu_ejtag_loc0, vcpu_ejtag_loc1, vcpu_ejtag_loc2, secpu_ejtag_loc0, > + secpu_ejtag_loc1, secpu_ejtag_loc2, aucpu_ejtag_loc0, aucpu_ejtag_loc1, > + aucpu_ejtag_loc2, iso_tristate, dbg_out0, dbg_out1, standby_dbg, > + spdif, arm_trace_debug_disable, arm_trace_debug_enable, > + aucpu_ejtag_disable, acpu_ejtag_disable, vcpu_ejtag_disable, > + scpu_ejtag_disable, secpu_ejtag_disable, vtc_dmic_loc_disable, > + vtc_tdm_disable, vtc_i2si_disable, tdm_ai_disable, ai_disable, > + spdif_disable, hif_disable, hif_enable, test_loop, pmic_pwrup ] > + > + drive-strength: > + enum: [4, 8] > + > + bias-pull-down: true > + > + bias-pull-up: true > + > + bias-disable: true > + > + input-schmitt-enable: true > + > + input-schmitt-disable: true > + > + drive-push-pull: true > + > + power-source: > + description: | > + Valid arguments are described as below: > + 0: power supply of 1.8V > + 1: power supply of 3.3V > + enum: [0, 1] > + > + realtek,drive-strength-p: > + description: | > + Some of pins can be driven using the P-MOS and N-MOS transistor to > + achieve finer adjustments. The block-diagram representation is as > + follows: > + VDD > + | > + ||--+ > + +-----o|| P-MOS-FET > + | ||--+ > + IN --+ +----- out > + | ||--+ > + +------|| N-MOS-FET > + ||--+ > + | > + GND > + The driving strength of the P-MOS/N-MOS transistors impacts the > + waveform's rise/fall times. Greater driving strength results in > + shorter rise/fall times. Each P-MOS and N-MOS transistor offers > + 8 configurable levels (0 to 7), with higher values indicating > + greater driving strength, contributing to achieving the desired > + speed. > + > + The realtek,drive-strength-p is used to control the driving strength > + of the P-MOS output. > + $ref: /schemas/types.yaml#/definitions/uint32 > + minimum: 0 > + maximum: 7 > + > + realtek,drive-strength-n: > + description: | > + Similar to the realtek,drive-strength-p, the realtek,drive-strength-n > + is used to control the driving strength of the N-MOS output. > + $ref: /schemas/types.yaml#/definitions/uint32 > + minimum: 0 > + maximum: 7 > + > + realtek,duty-cycle: > + description: | > + An integer describing the level to adjust output duty cycle, controlling > + the proportion of positive and negative waveforms in nanoseconds. > + Valid arguments are described as below: > + 0: 0ns > + 2: + 0.25ns > + 3: + 0.5ns > + 4: -0.25ns > + 5: -0.5ns > + $ref: /schemas/types.yaml#/definitions/uint32 > + enum: [ 0, 2, 3, 4, 5 ] > + > + required: > + - pins > + > + additionalProperties: false > + > +required: > + - compatible > + - reg > + > +additionalProperties: false > + > +examples: > + - | > + pinctrl@4e000 { > + compatible = "realtek,rtd1319d-pinctrl"; > + reg = <0x4e000 0x130>; > + > + emmc-hs200-pins { > + pins = "emmc_clk", > + "emmc_cmd", > + "emmc_data_0", > + "emmc_data_1", > + "emmc_data_2", > + "emmc_data_3", > + "emmc_data_4", > + "emmc_data_5", > + "emmc_data_6", > + "emmc_data_7"; > + function = "emmc"; > + realtek,drive-strength-p = <0x2>; > + realtek,drive-strength-n = <0x2>; > + }; > + > + i2c-0-pins { > + pins = "gpio_12", > + "gpio_13"; > + function = "i2c0"; > + drive-strength = <4>; > + }; > + }; > -- > 2.41.0 >
Hi Rob, >On Thu, Aug 24, 2023 at 06:57:02PM +0800, Tzuyi Chang wrote: >> Add device tree bindings for RTD1319D. >> >> Signed-off-by: Tzuyi Chang <tychang@realtek.com> >> --- >> v1 to v2 change: >> 1. Add a description for RTD1319D. >> 2. Rename realtek,pdrive, realtekmndrive and realtek,dcycle. >> 3. Add a description for PMOS and NMOS driving strength. >> 4. Remove the wildcard in the compatible strings. >> 5. Use '-pins$' to be node name pattern. >> --- >> .../pinctrl/realtek,rtd1319d-pinctrl.yaml | 189 ++++++++++++++++++ >> 1 file changed, 189 insertions(+) >> create mode 100644 >> Documentation/devicetree/bindings/pinctrl/realtek,rtd1319d-pinctrl.yam >> l >> >> diff --git >> a/Documentation/devicetree/bindings/pinctrl/realtek,rtd1319d-pinctrl.y >> aml >> b/Documentation/devicetree/bindings/pinctrl/realtek,rtd1319d-pinctrl.y >> aml >> new file mode 100644 >> index 000000000000..8653d42ac1f3 >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/pinctrl/realtek,rtd1319d-pinct >> +++ rl.yaml >> @@ -0,0 +1,189 @@ >> +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause) # Copyright 2023 >> +Realtek Semiconductor Corporation %YAML 1.2 >> +--- >> +$id: >> +http://devicetree.org/schemas/pinctrl/realtek,rtd1319d-pinctrl.yaml# >> +$schema: http://devicetree.org/meta-schemas/core.yaml# >> + >> +title: Realtek DHC RTD1319D Pin Controller >> + >> +maintainers: >> + - TY Chang <tychang@realtek.com> >> + >> +description: >> + The Realtek DHC RTD1319D is a high-definition media processor SoC. >> +The >> + RTD1319D pin controller is used to control pin function, pull >> +up/down >> + resistor, drive strength, schmitt trigger and power source. >> + >> +properties: >> + compatible: >> + const: realtek,rtd1319d-pinctrl >> + >> + reg: >> + maxItems: 1 >> + >> +patternProperties: >> + '^.*$': > >In case it was not obvious, my comment to use '-pins' applies here and to the next >patch. > >> + if: > >As does dropping if/then. > I will fix it in the next version. >> + type: object >> + then: >> + allOf: >> + - $ref: pincfg-node.yaml# >> + - $ref: pinmux-node.yaml# >> + >> + properties: >> + pins: >> + items: >> + enum: [ gpio_0, gpio_1, gpio_2, gpio_3, gpio_4, gpio_5, gpio_6, >gpio_7, >> + gpio_8, gpio_9, gpio_10, gpio_11, gpio_12, gpio_13, >gpio_14, >> + gpio_15, gpio_16, gpio_17, gpio_18, gpio_19, gpio_20, >gpio_21, >> + gpio_22, gpio_23, usb_cc2, gpio_25, gpio_26, gpio_27, >gpio_28, >> + gpio_29, gpio_30, gpio_31, gpio_32, gpio_33, gpio_34, >gpio_35, >> + hif_data, hif_en, hif_rdy, hif_clk, gpio_40, gpio_41, >gpio_42, >> + gpio_43, gpio_44, gpio_45, gpio_46, gpio_47, gpio_48, >gpio_49, >> + gpio_50, usb_cc1, gpio_52, gpio_53, ir_rx, ur0_rx, >ur0_tx, >> + gpio_57, gpio_58, gpio_59, gpio_60, gpio_61, gpio_62, >gpio_63, >> + gpio_64, emmc_rst_n, emmc_dd_sb, emmc_clk, >emmc_cmd, emmc_data_0, >> + emmc_data_1, emmc_data_2, emmc_data_3, >emmc_data_4, emmc_data_5, >> + emmc_data_6, emmc_data_7, dummy, gpio_78, >gpio_79, gpio_80, >> + gpio_81, ur2_loc, gspi_loc, hi_width, sf_en, >arm_trace_dbg_en, >> + ejtag_aucpu_loc, ejtag_acpu_loc, ejtag_vcpu_loc, >ejtag_scpu_loc, >> + dmic_loc, ejtag_secpu_loc, vtc_dmic_loc, vtc_tdm_loc, >vtc_i2si_loc, >> + tdm_ai_loc, ai_loc, spdif_loc, hif_en_loc, sc0_loc, >sc1_loc, >> + scan_switch, wd_rset, boot_sel, reset_n, testmode >> + ] >> + >> + function: >> + enum: [ gpio, nf, emmc, tp0, tp1, sc0, sc0_data0, sc0_data1, >sc0_data2, >> + sc1, sc1_data0, sc1_data1, sc1_data2, ao, gspi_loc0, >gspi_loc1, >> + uart0, uart1, uart2_loc0, uart2_loc1, i2c0, i2c1, i2c3, i2c4, >> + i2c5, pcie1, sdio, etn_led, etn_phy, spi, pwm0_loc0, >pwm0_loc1, >> + pwm1_loc0, pwm1_loc1, pwm2_loc0, pwm2_loc1, >pwm3_loc0, pwm3_loc1, >> + qam_agc_if0, qam_agc_if1, spdif_optical_loc0, >spdif_optical_loc1, >> + usb_cc1, usb_cc2, vfd, sd, dmic_loc0, dmic_loc1, ai_loc0, >ai_loc1, >> + tdm_ai_loc0, tdm_ai_loc1, hi_loc0, hi_m, vtc_i2so, >vtc_i2si_loc0, >> + vtc_i2si_loc1, vtc_dmic_loc0, vtc_dmic_loc1, >vtc_tdm_loc0, >> + vtc_tdm_loc1, dc_fan, pll_test_loc0, pll_test_loc1, ir_rx, >> + uart2_disable, gspi_disable, hi_width_disable, >hi_width_1bit, >> + sf_disable, sf_enable, scpu_ejtag_loc0, scpu_ejtag_loc1, >> + scpu_ejtag_loc2, acpu_ejtag_loc0, acpu_ejtag_loc1, >acpu_ejtag_loc2, >> + vcpu_ejtag_loc0, vcpu_ejtag_loc1, vcpu_ejtag_loc2, >secpu_ejtag_loc0, >> + secpu_ejtag_loc1, secpu_ejtag_loc2, aucpu_ejtag_loc0, >aucpu_ejtag_loc1, >> + aucpu_ejtag_loc2, iso_tristate, dbg_out0, dbg_out1, >standby_dbg, >> + spdif, arm_trace_debug_disable, >arm_trace_debug_enable, >> + aucpu_ejtag_disable, acpu_ejtag_disable, >vcpu_ejtag_disable, >> + scpu_ejtag_disable, secpu_ejtag_disable, >vtc_dmic_loc_disable, >> + vtc_tdm_disable, vtc_i2si_disable, tdm_ai_disable, >ai_disable, >> + spdif_disable, hif_disable, hif_enable, test_loop, >> + pmic_pwrup ] >> + >> + drive-strength: >> + enum: [4, 8] >> + >> + bias-pull-down: true >> + >> + bias-pull-up: true >> + >> + bias-disable: true >> + >> + input-schmitt-enable: true >> + >> + input-schmitt-disable: true >> + >> + drive-push-pull: true >> + >> + power-source: >> + description: | >> + Valid arguments are described as below: >> + 0: power supply of 1.8V >> + 1: power supply of 3.3V >> + enum: [0, 1] >> + >> + realtek,drive-strength-p: >> + description: | >> + Some of pins can be driven using the P-MOS and N-MOS >transistor to >> + achieve finer adjustments. The block-diagram representation is >as >> + follows: >> + VDD >> + | >> + ||--+ >> + +-----o|| P-MOS-FET >> + | ||--+ >> + IN --+ +----- out >> + | ||--+ >> + +------|| N-MOS-FET >> + ||--+ >> + | >> + GND >> + The driving strength of the P-MOS/N-MOS transistors impacts the >> + waveform's rise/fall times. Greater driving strength results in >> + shorter rise/fall times. Each P-MOS and N-MOS transistor offers >> + 8 configurable levels (0 to 7), with higher values indicating >> + greater driving strength, contributing to achieving the desired >> + speed. >> + >> + The realtek,drive-strength-p is used to control the driving >strength >> + of the P-MOS output. >> + $ref: /schemas/types.yaml#/definitions/uint32 >> + minimum: 0 >> + maximum: 7 >> + >> + realtek,drive-strength-n: >> + description: | >> + Similar to the realtek,drive-strength-p, the >realtek,drive-strength-n >> + is used to control the driving strength of the N-MOS output. >> + $ref: /schemas/types.yaml#/definitions/uint32 >> + minimum: 0 >> + maximum: 7 >> + >> + realtek,duty-cycle: >> + description: | >> + An integer describing the level to adjust output duty cycle, >controlling >> + the proportion of positive and negative waveforms in >nanoseconds. >> + Valid arguments are described as below: >> + 0: 0ns >> + 2: + 0.25ns >> + 3: + 0.5ns >> + 4: -0.25ns >> + 5: -0.5ns >> + $ref: /schemas/types.yaml#/definitions/uint32 >> + enum: [ 0, 2, 3, 4, 5 ] >> + >> + required: >> + - pins >> + >> + additionalProperties: false >> + >> +required: >> + - compatible >> + - reg >> + >> +additionalProperties: false >> + >> +examples: >> + - | >> + pinctrl@4e000 { >> + compatible = "realtek,rtd1319d-pinctrl"; >> + reg = <0x4e000 0x130>; >> + >> + emmc-hs200-pins { >> + pins = "emmc_clk", >> + "emmc_cmd", >> + "emmc_data_0", >> + "emmc_data_1", >> + "emmc_data_2", >> + "emmc_data_3", >> + "emmc_data_4", >> + "emmc_data_5", >> + "emmc_data_6", >> + "emmc_data_7"; >> + function = "emmc"; >> + realtek,drive-strength-p = <0x2>; >> + realtek,drive-strength-n = <0x2>; >> + }; >> + >> + i2c-0-pins { >> + pins = "gpio_12", >> + "gpio_13"; >> + function = "i2c0"; >> + drive-strength = <4>; >> + }; >> + }; >> -- >> 2.41.0 >> Thanks, Tzuyi Chang
On Thu, Aug 24, 2023 at 12:57 PM Tzuyi Chang <tychang@realtek.com> wrote: > Add device tree bindings for RTD1319D. > > Signed-off-by: Tzuyi Chang <tychang@realtek.com> > --- > v1 to v2 change: My comments are addressed so: Reviewed-by: Linus Walleij <linus.walleij@linaro.org> (Then you need to also fix Rob's comments.) Yours, Linus Walleij
diff --git a/Documentation/devicetree/bindings/pinctrl/realtek,rtd1319d-pinctrl.yaml b/Documentation/devicetree/bindings/pinctrl/realtek,rtd1319d-pinctrl.yaml new file mode 100644 index 000000000000..8653d42ac1f3 --- /dev/null +++ b/Documentation/devicetree/bindings/pinctrl/realtek,rtd1319d-pinctrl.yaml @@ -0,0 +1,189 @@ +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause) +# Copyright 2023 Realtek Semiconductor Corporation +%YAML 1.2 +--- +$id: http://devicetree.org/schemas/pinctrl/realtek,rtd1319d-pinctrl.yaml# +$schema: http://devicetree.org/meta-schemas/core.yaml# + +title: Realtek DHC RTD1319D Pin Controller + +maintainers: + - TY Chang <tychang@realtek.com> + +description: + The Realtek DHC RTD1319D is a high-definition media processor SoC. The + RTD1319D pin controller is used to control pin function, pull up/down + resistor, drive strength, schmitt trigger and power source. + +properties: + compatible: + const: realtek,rtd1319d-pinctrl + + reg: + maxItems: 1 + +patternProperties: + '^.*$': + if: + type: object + then: + allOf: + - $ref: pincfg-node.yaml# + - $ref: pinmux-node.yaml# + + properties: + pins: + items: + enum: [ gpio_0, gpio_1, gpio_2, gpio_3, gpio_4, gpio_5, gpio_6, gpio_7, + gpio_8, gpio_9, gpio_10, gpio_11, gpio_12, gpio_13, gpio_14, + gpio_15, gpio_16, gpio_17, gpio_18, gpio_19, gpio_20, gpio_21, + gpio_22, gpio_23, usb_cc2, gpio_25, gpio_26, gpio_27, gpio_28, + gpio_29, gpio_30, gpio_31, gpio_32, gpio_33, gpio_34, gpio_35, + hif_data, hif_en, hif_rdy, hif_clk, gpio_40, gpio_41, gpio_42, + gpio_43, gpio_44, gpio_45, gpio_46, gpio_47, gpio_48, gpio_49, + gpio_50, usb_cc1, gpio_52, gpio_53, ir_rx, ur0_rx, ur0_tx, + gpio_57, gpio_58, gpio_59, gpio_60, gpio_61, gpio_62, gpio_63, + gpio_64, emmc_rst_n, emmc_dd_sb, emmc_clk, emmc_cmd, emmc_data_0, + emmc_data_1, emmc_data_2, emmc_data_3, emmc_data_4, emmc_data_5, + emmc_data_6, emmc_data_7, dummy, gpio_78, gpio_79, gpio_80, + gpio_81, ur2_loc, gspi_loc, hi_width, sf_en, arm_trace_dbg_en, + ejtag_aucpu_loc, ejtag_acpu_loc, ejtag_vcpu_loc, ejtag_scpu_loc, + dmic_loc, ejtag_secpu_loc, vtc_dmic_loc, vtc_tdm_loc, vtc_i2si_loc, + tdm_ai_loc, ai_loc, spdif_loc, hif_en_loc, sc0_loc, sc1_loc, + scan_switch, wd_rset, boot_sel, reset_n, testmode ] + + function: + enum: [ gpio, nf, emmc, tp0, tp1, sc0, sc0_data0, sc0_data1, sc0_data2, + sc1, sc1_data0, sc1_data1, sc1_data2, ao, gspi_loc0, gspi_loc1, + uart0, uart1, uart2_loc0, uart2_loc1, i2c0, i2c1, i2c3, i2c4, + i2c5, pcie1, sdio, etn_led, etn_phy, spi, pwm0_loc0, pwm0_loc1, + pwm1_loc0, pwm1_loc1, pwm2_loc0, pwm2_loc1, pwm3_loc0, pwm3_loc1, + qam_agc_if0, qam_agc_if1, spdif_optical_loc0, spdif_optical_loc1, + usb_cc1, usb_cc2, vfd, sd, dmic_loc0, dmic_loc1, ai_loc0, ai_loc1, + tdm_ai_loc0, tdm_ai_loc1, hi_loc0, hi_m, vtc_i2so, vtc_i2si_loc0, + vtc_i2si_loc1, vtc_dmic_loc0, vtc_dmic_loc1, vtc_tdm_loc0, + vtc_tdm_loc1, dc_fan, pll_test_loc0, pll_test_loc1, ir_rx, + uart2_disable, gspi_disable, hi_width_disable, hi_width_1bit, + sf_disable, sf_enable, scpu_ejtag_loc0, scpu_ejtag_loc1, + scpu_ejtag_loc2, acpu_ejtag_loc0, acpu_ejtag_loc1, acpu_ejtag_loc2, + vcpu_ejtag_loc0, vcpu_ejtag_loc1, vcpu_ejtag_loc2, secpu_ejtag_loc0, + secpu_ejtag_loc1, secpu_ejtag_loc2, aucpu_ejtag_loc0, aucpu_ejtag_loc1, + aucpu_ejtag_loc2, iso_tristate, dbg_out0, dbg_out1, standby_dbg, + spdif, arm_trace_debug_disable, arm_trace_debug_enable, + aucpu_ejtag_disable, acpu_ejtag_disable, vcpu_ejtag_disable, + scpu_ejtag_disable, secpu_ejtag_disable, vtc_dmic_loc_disable, + vtc_tdm_disable, vtc_i2si_disable, tdm_ai_disable, ai_disable, + spdif_disable, hif_disable, hif_enable, test_loop, pmic_pwrup ] + + drive-strength: + enum: [4, 8] + + bias-pull-down: true + + bias-pull-up: true + + bias-disable: true + + input-schmitt-enable: true + + input-schmitt-disable: true + + drive-push-pull: true + + power-source: + description: | + Valid arguments are described as below: + 0: power supply of 1.8V + 1: power supply of 3.3V + enum: [0, 1] + + realtek,drive-strength-p: + description: | + Some of pins can be driven using the P-MOS and N-MOS transistor to + achieve finer adjustments. The block-diagram representation is as + follows: + VDD + | + ||--+ + +-----o|| P-MOS-FET + | ||--+ + IN --+ +----- out + | ||--+ + +------|| N-MOS-FET + ||--+ + | + GND + The driving strength of the P-MOS/N-MOS transistors impacts the + waveform's rise/fall times. Greater driving strength results in + shorter rise/fall times. Each P-MOS and N-MOS transistor offers + 8 configurable levels (0 to 7), with higher values indicating + greater driving strength, contributing to achieving the desired + speed. + + The realtek,drive-strength-p is used to control the driving strength + of the P-MOS output. + $ref: /schemas/types.yaml#/definitions/uint32 + minimum: 0 + maximum: 7 + + realtek,drive-strength-n: + description: | + Similar to the realtek,drive-strength-p, the realtek,drive-strength-n + is used to control the driving strength of the N-MOS output. + $ref: /schemas/types.yaml#/definitions/uint32 + minimum: 0 + maximum: 7 + + realtek,duty-cycle: + description: | + An integer describing the level to adjust output duty cycle, controlling + the proportion of positive and negative waveforms in nanoseconds. + Valid arguments are described as below: + 0: 0ns + 2: + 0.25ns + 3: + 0.5ns + 4: -0.25ns + 5: -0.5ns + $ref: /schemas/types.yaml#/definitions/uint32 + enum: [ 0, 2, 3, 4, 5 ] + + required: + - pins + + additionalProperties: false + +required: + - compatible + - reg + +additionalProperties: false + +examples: + - | + pinctrl@4e000 { + compatible = "realtek,rtd1319d-pinctrl"; + reg = <0x4e000 0x130>; + + emmc-hs200-pins { + pins = "emmc_clk", + "emmc_cmd", + "emmc_data_0", + "emmc_data_1", + "emmc_data_2", + "emmc_data_3", + "emmc_data_4", + "emmc_data_5", + "emmc_data_6", + "emmc_data_7"; + function = "emmc"; + realtek,drive-strength-p = <0x2>; + realtek,drive-strength-n = <0x2>; + }; + + i2c-0-pins { + pins = "gpio_12", + "gpio_13"; + function = "i2c0"; + drive-strength = <4>; + }; + };
Add device tree bindings for RTD1319D. Signed-off-by: Tzuyi Chang <tychang@realtek.com> --- v1 to v2 change: 1. Add a description for RTD1319D. 2. Rename realtek,pdrive, realtekmndrive and realtek,dcycle. 3. Add a description for PMOS and NMOS driving strength. 4. Remove the wildcard in the compatible strings. 5. Use '-pins$' to be node name pattern. --- .../pinctrl/realtek,rtd1319d-pinctrl.yaml | 189 ++++++++++++++++++ 1 file changed, 189 insertions(+) create mode 100644 Documentation/devicetree/bindings/pinctrl/realtek,rtd1319d-pinctrl.yaml