pinctrl: qcom: Add multiple copy support
Subsystems (5)
cover pinctrl: qcom: Add multiple copy support
These patches add multiple copy support for functions which require additional mux configurations on specific Qualcomm processor pincontrol blocks. Functions which may have multiple copies are slimbus, mi2s, pdm, pcie, and GSBI.
Andy Gross (4):
pinctrl: qcom: Add multiple copy base support
pinctrl: qcom: ipq8064: Add multi copy support
pinctrl: qcom: apq8064: Add multi copy support
pinctrl: qcom: msm8960: Add multi copy support
.../bindings/pinctrl/qcom,apq8064-pinctrl.txt | 3 +- .../bindings/pinctrl/qcom,ipq8064-pinctrl.txt | 16 +- .../bindings/pinctrl/qcom,msm8960-pinctrl.txt | 19 +- drivers/pinctrl/qcom/pinctrl-apq8064.c | 29 ++- drivers/pinctrl/qcom/pinctrl-ipq8064.c | 244 ++++++++++++++------ drivers/pinctrl/qcom/pinctrl-msm.c | 10 + drivers/pinctrl/qcom/pinctrl-msm.h | 4 + drivers/pinctrl/qcom/pinctrl-msm8960.c | 32 ++- 8 files changed, 257 insertions(+), 100 deletions(-)
Signature · 3 lines
Qualcomm Innovation Center, Inc The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum, a Linux Foundation Collaborative Project
PATCH 1/4 pinctrl: qcom: Add multiple copy base support
Qualcomm pinctrl devices support functions that can be routed to multiple pins. In some cases, there are additional mux registers that must be set for the pins to work correctly.
--- drivers/pinctrl/qcom/pinctrl-msm.c | 10 ++++++++++ drivers/pinctrl/qcom/pinctrl-msm.h | 4 ++++ 2 files changed, 14 insertions(+)
drivers/pinctrl/qcom/pinctrl-msm.c +10 -0
@@ -141,11 +141,13 @@ static int msm_pinmux_set_mux(struct pinctrl_dev *pctldev, {struct msm_pinctrl *pctrl =pinctrl_dev_get_drvdata (pctldev);const struct msm_pingroup *g; +const struct msm_function *f;unsigned long flags;u32 val;int i; g = &pctrl->soc->groups[group]; + f = &pctrl->soc->functions[function];for (i =0 ; i < g->nfuncs; i++) {if (g->funcs[i] == function)
@@ -162,6 +164,14 @@ static int msm_pinmux_set_mux(struct pinctrl_dev *pctldev, val |= i << g->mux_bit;writel (val, pctrl->regs + g->ctl_reg); +/* +* if an alternate copy configuration is required, configure the pins to +* steer the function to the correct set of pins. This is used in cases +* where we have more than one copy of the pins for a function +*/ +if (f->requires_copy_select) +writel (f->copy_select_value, pctrl->regs + f->copy_select_reg); +spin_unlock_irqrestore (&pctrl->lock, flags);return 0 ;
drivers/pinctrl/qcom/pinctrl-msm.h +4 -0
@@ -25,6 +25,10 @@ struct msm_function {const char *name;const char *const *groups;unsigned ngroups; + +unsigned requires_copy_select; +unsigned copy_select_reg; +unsigned copy_select_value; };/**
Signature · 3 lines
Qualcomm Innovation Center, Inc The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum, a Linux Foundation Collaborative Project
Re: [PATCH 1/4] pinctrl: qcom: Add multiple copy base support
Quoted 4 lines · Andy Gross
Qualcomm pinctrl devices support functions that can be routed to multiple pins. In some cases, there are additional mux registers that must be set for the pins to work correctly.
I've described it as "second level muxing", but your description works too...
[..]
drivers/pinctrl/qcom/pinctrl-msm.c static int msm_pinmux_set_mux(struct pinctrl_dev *pctldev, · line 173
Quoted 7 lines
drivers/pinctrl/qcom/pinctrl-msm.c +7
-0+/* +* if an alternate copy configuration is required, configure the pins to +* steer the function to the correct set of pins. This is used in cases +* where we have more than one copy of the pins for a function +*/ +if (f->requires_copy_select) +writel (f->copy_select_value, pctrl->regs + f->copy_select_reg);
I'm not sure if this is sufficient.
In the APQ8064 case (patch 3) you use this to write 0 or 1 to $2074, but if I read the documentation correctly you should also write to $207c and $2080 to enable/disable slew rate control of the individual paths.
On 8974 we don't have the muxing, but the documentation states that we should set bit 0 of $2030 depending on slimbus being muxed or not. (not sure what to do about bit 1 though)
I looked at assigning an optional function pointer to the function, that way we could easily express the platform specific tweaks in the individual drivers.
However, as the muxing is deselected we need to make sure the slew rate is disabled and the only sane way I can think of then would be to tie this to the pingroup, as selecting any other entry from the pingroup should trigger the reset.
Regards,
Bjorn
PATCH 2/4 pinctrl: qcom: ipq8064: Add multi copy support
In reply to Andy GrossThis patch adds multiple copy support for functions that can be mapped to more than one pin and that also require an additional mux configuration setting to work properly.
--- .../bindings/pinctrl/qcom,ipq8064-pinctrl.txt | 16 +- drivers/pinctrl/qcom/pinctrl-ipq8064.c | 244 ++++++++++++++------ 2 files changed, 185 insertions(+), 75 deletions(-)
Documentation/devicetree/bindings/pinctrl/qcom,ipq8064-pinctrl.txt +9 -7
@@ -51,13 +51,15 @@ Valid values for qcom,pins are: Valid values for function are: - mdio, mi2s, pdm, ssbi, spmi, audio_pcm, gpio, gsbi1, gsbi2, gsbi4, gsbi5, - gsbi5_spi_cs1, gsbi5_spi_cs2, gsbi5_spi_cs3, gsbi6, gsbi7, nss_spi, sdc1, - spdif, nand, tsif1, tsif2, usb_fs_n, usb_fs, usb2_hsic, rgmii2, sata, - pcie1_rst, pcie1_prsnt, pcie1_pwren_n, pcie1_pwren, pcie1_pwrflt, - pcie1_clk_req, pcie2_rst, pcie2_prsnt, pcie2_pwren_n, pcie2_pwren, - pcie2_pwrflt, pcie2_clk_req, pcie3_rst, pcie3_prsnt, pcie3_pwren_n, - pcie3_pwren, pcie3_pwrflt, pcie3_clk_req, ps_hold + mdio, mi2s_a, mi2s_b, mi2s, pdm0_a, pdm0_b, pdm1_a, pdm1_b, pdm2_a, pdm2_b, + ssbi, spmi, audio_pcm, gpio, gsbi1, gsbi2, gsbi4, gsbi5, gsbi5_spi_cs1, + gsbi5_spi_cs2, gsbi5_spi_cs3, gsbi6_a, gsbi6_b, gsbi7, nss_spi, sdc1, spdif, + nand, tsif1, tsif2, usb_fs_n, usb_fs, usb2_hsic, rgmii2, sata, pcie1_rst, + pcie1_prsnt_a, pcie1_prsnt_b, pcie1_pwren_n_a, pcie1_pwren_n_b, pcie1_pwren_a, + pcie1_pwren_b, pcie1_pwrflt_a, pcie1_pwrflt_b, pcie1_clk_req, pcie2_rst, + pcie2_prsnt_a, pcie2_prsnt_b, pcie2_pwren_n_a, pcie2_pwren_n_b, pcie2_pwren_a, + pcie2_pwren_b, pcie2_pwrflt_a, pcie2_pwrflt_b, pcie2_clk_req, pcie3_rst, + pcie3_prsnt, pcie3_pwren_n, pcie3_pwren, pcie3_pwrflt, pcie3_clk_req, ps_hold Example:
drivers/pinctrl/qcom/pinctrl-ipq8064.c +176 -68
362 lines not on this page · raw message
Signature · 3 lines
Qualcomm Innovation Center, Inc The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum, a Linux Foundation Collaborative Project
PATCH 3/4 pinctrl: qcom: apq8064: Add multi copy support
In reply to Andy GrossThis patch adds multiple copy support for functions that can be mapped to more than one pin and that also require an additional mux configuration setting to work properly.
--- .../bindings/pinctrl/qcom,apq8064-pinctrl.txt | 3 +- drivers/pinctrl/qcom/pinctrl-apq8064.c | 29 +++++++++++++++----- 2 files changed, 24 insertions(+), 8 deletions(-)
Documentation/devicetree/bindings/pinctrl/qcom,apq8064-pinctrl.txt +2 -1
@@ -50,7 +50,8 @@ Valid values for function are: gsbi4_cam_i2c, gsbi5, gsbi5_spi_cs1, gsbi5_spi_cs2, gsbi5_spi_cs3, gsbi6, gsbi6_spi_cs1, gsbi6_spi_cs2, gsbi6_spi_cs3, gsbi7, gsbi7_spi_cs1, gsbi7_spi_cs2, gsbi7_spi_cs3, gsbi_cam_i2c, hdmi, mi2s, riva_bt, riva_fm, - riva_wlan, sdc2, sdc4, slimbus, spkr_i2s, tsif1, tsif2, usb2_hsic, ps_hold + riva_wlan, sdc2, sdc4, slimbus_a, slimbus_b, spkr_i2s, tsif1, tsif2, + usb2_hsic, ps_hold Example:
drivers/pinctrl/qcom/pinctrl-apq8064.c +22 -7
@@ -224,6 +224,16 @@ static const unsigned int sdc3_data_pins[] = { 95 }; .ngroups = ARRAY_SIZE(fname##_groups), \ } +#define FUNCTION_MULTI_COPY (fname, reg, value) \ + [APQ_MUX_##fname] = { \ + .name = #fname, \ + .groups = fname##_groups, \ + .ngroups = ARRAY_SIZE(fname##_groups), \ + .requires_copy_select = 1, \ + .copy_select_reg = reg, \ + .copy_select_value = value, \ + } +#define PINGROUP (id, f1, f2, f3, f4, f5, f6, f7, f8, f9, f10) \ { \ .name = "gpio" #id, \
@@ -321,7 +331,8 @@ enum apq8064_functions { APQ_MUX_riva_wlan, APQ_MUX_sdc2, APQ_MUX_sdc4, - APQ_MUX_slimbus, + APQ_MUX_slimbus_a, + APQ_MUX_slimbus_b, APQ_MUX_spkr_i2s, APQ_MUX_tsif1, APQ_MUX_tsif2,
@@ -432,9 +443,12 @@ static const char * const sdc2_groups[] = {static const char *const sdc4_groups[] = {"gpio63" ,"gpio64" ,"gpio65" ,"gpio66" ,"gpio67" ,"gpio68" }; -static const char *const slimbus_groups[] = { +static const char *const slimbus_a_groups[] = {"gpio40" ,"gpio41" }; +static const char *const slimbus_b_groups[] = { +"gpio30" ,"gpio31" +};static const char *const spkr_i2s_groups[] = {"gpio47" ,"gpio48" ,"gpio49" ,"gpio50" };
@@ -478,7 +492,8 @@ static const struct msm_function apq8064_functions[] = {FUNCTION (riva_wlan),FUNCTION (sdc2),FUNCTION (sdc4), -FUNCTION (slimbus), +FUNCTION_MULTI_COPY (slimbus_a,0x2074 ,0 ), +FUNCTION_MULTI_COPY (slimbus_b,0x2074 ,1 ),FUNCTION (spkr_i2s),FUNCTION (tsif1), FUNCTION(tsif2),
@@ -517,8 +532,8 @@ static const struct msm_pingroup apq8064_groups[] = {PINGROUP (27 , mi2s,NA ,NA ,NA ,NA ,NA ,NA ,NA ,NA ,NA ),PINGROUP (28 , mi2s,NA ,NA ,NA ,NA ,NA ,NA ,NA ,NA ,NA ),PINGROUP (29 , mi2s,NA ,NA ,NA ,NA ,NA ,NA ,NA ,NA ,NA ), -PINGROUP (30 , mi2s,NA ,NA ,NA ,NA ,NA ,NA ,NA ,NA ,NA ), -PINGROUP (31 , mi2s,NA , gsbi5_spi_cs2, gsbi6_spi_cs2, gsbi7_spi_cs2,NA ,NA ,NA ,NA ,NA ), +PINGROUP (30 , mi2s, slimbus_b,NA ,NA ,NA ,NA ,NA ,NA ,NA ,NA ), +PINGROUP (31 , mi2s, slimbus_b, gsbi5_spi_cs2, gsbi6_spi_cs2, gsbi7_spi_cs2,NA ,NA ,NA ,NA ,NA ),PINGROUP (32 , mi2s,NA ,NA ,NA ,NA , gsbi5_spi_cs3, gsbi6_spi_cs3, gsbi7_spi_cs3,NA ,NA ),PINGROUP (33 , mi2s,NA ,NA ,NA ,NA ,NA ,NA ,NA ,NA ,NA ), PINGROUP(34, codec_mic_i2s, NA, NA, NA, NA, NA, NA, NA, NA, NA),
@@ -527,8 +542,8 @@ static const struct msm_pingroup apq8064_groups[] = {PINGROUP (37 , codec_mic_i2s,NA ,NA ,NA ,NA ,NA ,NA ,NA ,NA ,NA ),PINGROUP (38 , codec_mic_i2s,NA ,NA ,NA ,NA ,NA ,NA ,NA ,NA ,NA ),PINGROUP (39 , codec_spkr_i2s,NA ,NA ,NA ,NA ,NA ,NA ,NA ,NA ,NA ), -PINGROUP (40 , slimbus, codec_spkr_i2s,NA ,NA ,NA ,NA ,NA ,NA ,NA ,NA ), -PINGROUP (41 , slimbus, codec_spkr_i2s,NA ,NA ,NA ,NA ,NA ,NA ,NA ,NA ), +PINGROUP (40 , slimbus_a, codec_spkr_i2s,NA ,NA ,NA ,NA ,NA ,NA ,NA ,NA ), +PINGROUP (41 , slimbus_a, codec_spkr_i2s,NA ,NA ,NA ,NA ,NA ,NA ,NA ,NA ),PINGROUP (42 , codec_spkr_i2s,NA ,NA ,NA ,NA ,NA ,NA ,NA ,NA ,NA ),PINGROUP (43 ,NA ,NA ,NA ,NA ,NA ,NA ,NA ,NA ,NA ,NA ), PINGROUP(44, NA, NA, NA, NA, NA, NA, NA, NA, NA, NA),
Signature · 3 lines
Qualcomm Innovation Center, Inc The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum, a Linux Foundation Collaborative Project
PATCH 4/4 pinctrl: qcom: msm8960: Add multi copy support
In reply to Andy GrossThis patch adds multiple copy support for functions that can be mapped to more than one pin and that also require an additional mux configuration setting to work properly.
--- .../bindings/pinctrl/qcom,msm8960-pinctrl.txt | 19 ++++++------ drivers/pinctrl/qcom/pinctrl-msm8960.c | 32 +++++++++++++++----- 2 files changed, 34 insertions(+), 17 deletions(-)
Documentation/devicetree/bindings/pinctrl/qcom,msm8960-pinctrl.txt +10 -9
@@ -103,15 +103,16 @@ to specify in a pin configuration subnode: gsbi12, hdmi_cec, hdmi_ddc_clock, hdmi_ddc_data, hdmi_hot_plug_detect, hsic, mdp_vsync, mi2s, mic_i2s, pmb_clk, pmb_ext_ctrl, ps_hold, rpm_wdog, sdc2, sdc4, sdc5, - slimbus1, slimbus2, spkr_i2s, ssbi1, ssbi2, ssbi_ext_gps, - ssbi_pmic2, ssbi_qpa1, ssbi_ts, tsif1, tsif2, ts_eoc, - usb_fs1, usb_fs1_oe, usb_fs1_oe_n, usb_fs2, usb_fs2_oe, - usb_fs2_oe_n, vfe_camif_timer1_a, vfe_camif_timer1_b, - vfe_camif_timer2, vfe_camif_timer3_a, vfe_camif_timer3_b, - vfe_camif_timer4_a, vfe_camif_timer4_b, vfe_camif_timer4_c, - vfe_camif_timer5_a, vfe_camif_timer5_b, vfe_camif_timer6_a, - vfe_camif_timer6_b, vfe_camif_timer6_c, vfe_camif_timer7_a, - vfe_camif_timer7_b, vfe_camif_timer7_c, wlan + slimbus1_a, slimbus1_b, slimbus2, spkr_i2s, ssbi1, ssbi2, + ssbi_ext_gps, ssbi_pmic2, ssbi_qpa1, ssbi_ts, tsif1, tsif2, + ts_eoc, usb_fs1, usb_fs1_oe, usb_fs1_oe_n, usb_fs2, + usb_fs2_oe, usb_fs2_oe_n, vfe_camif_timer1_a, + vfe_camif_timer1_b, vfe_camif_timer2, vfe_camif_timer3_a, + vfe_camif_timer3_b, vfe_camif_timer4_a, vfe_camif_timer4_b, + vfe_camif_timer4_c, vfe_camif_timer5_a, vfe_camif_timer5_b, + vfe_camif_timer6_a, vfe_camif_timer6_b, vfe_camif_timer6_c, + vfe_camif_timer7_a, vfe_camif_timer7_b, vfe_camif_timer7_c, + wlan - bias-disable: Usage: optional
drivers/pinctrl/qcom/pinctrl-msm8960.c +24 -8
@@ -349,6 +349,16 @@ static const unsigned int sdc3_data_pins[] = { 157 }; .ngroups = ARRAY_SIZE(fname##_groups), \ } +#define FUNCTION_MULTI_COPY (fname, reg, value) \ + [MSM_MUX_##fname] = { \ + .name = #fname, \ + .groups = fname##_groups, \ + .ngroups = ARRAY_SIZE(fname##_groups), \ + .requires_copy_select = 1, \ + .copy_select_reg = reg, \ + .copy_select_value = value, \ + } +#define PINGROUP (id, f1, f2, f3, f4, f5, f6, f7, f8, f9, f10, f11) \ { \ .name = "gpio" #id, \
@@ -487,7 +497,8 @@ enum msm8960_functions { MSM_MUX_sdc2, MSM_MUX_sdc4, MSM_MUX_sdc5, - MSM_MUX_slimbus1, + MSM_MUX_slimbus1_a, + MSM_MUX_slimbus1_b, MSM_MUX_slimbus2, MSM_MUX_spkr_i2s, MSM_MUX_ssbi1,
@@ -824,8 +835,12 @@ static const char * const sdc5_groups[] = {"gpio77" ,"gpio78" ,"gpio79" ,"gpio80" ,"gpio81" ,"gpio82" }; -static const char *const slimbus1_groups[] = { -"gpio50" ,"gpio51" ,"gpio60" ,"gpio61" +static const char *const slimbus1_a_groups[] = { +"gpio60" ,"gpio61" +}; + +static const char *const slimbus1_b_groups[] = { +"gpio50" ,"gpio51" , }; static const char * const slimbus2_groups[] = {
@@ -1034,7 +1049,8 @@ static const struct msm_function msm8960_functions[] = {FUNCTION (sdc2),FUNCTION (sdc4),FUNCTION (sdc5), -FUNCTION (slimbus1), +FUNCTION_MULTI_COPY (slimbus1_a,0x2074 ,0 ), +FUNCTION_MULTI_COPY (slimbus1_b,0x2074 ,1 ),FUNCTION (slimbus2),FUNCTION (spkr_i2s), FUNCTION(ssbi1),
@@ -1122,8 +1138,8 @@ static const struct msm_pingroup msm8960_groups[] = {PINGROUP (47 , mi2s,NA ,NA ,NA ,NA ,NA ,NA ,NA ,NA ,NA ,NA ),PINGROUP (48 , mi2s,NA ,NA ,NA ,NA ,NA ,NA ,NA ,NA ,NA ,NA ),PINGROUP (49 , mi2s,NA ,NA ,NA ,NA ,NA ,NA ,NA ,NA ,NA ,NA ), -PINGROUP (50 , mi2s, slimbus1,NA ,NA ,NA ,NA ,NA ,NA ,NA ,NA ,NA ), -PINGROUP (51 , mi2s, slimbus1,NA ,NA ,NA ,NA ,NA ,NA ,NA ,NA ,NA ), +PINGROUP (50 , mi2s, slimbus1_b,NA ,NA ,NA ,NA ,NA ,NA ,NA ,NA ,NA ), +PINGROUP (51 , mi2s, slimbus1_b,NA ,NA ,NA ,NA ,NA ,NA ,NA ,NA ,NA ),PINGROUP (52 , mi2s, gp_clk_2a, gsbi2_spi_cs1_n,NA ,NA ,NA ,NA ,NA ,NA ,NA ,NA ),PINGROUP (53 , mi2s, gp_pdm_2b,NA ,NA ,NA ,NA ,NA ,NA ,NA ,NA ,NA ), PINGROUP(54, codec_mic_i2s, gp_clk_0b, NA, NA, NA, NA, NA, NA, NA, NA, NA),
@@ -1132,8 +1148,8 @@ static const struct msm_pingroup msm8960_groups[] = {PINGROUP (57 , codec_mic_i2s,NA ,NA ,NA ,NA ,NA ,NA ,NA ,NA ,NA ,NA ),PINGROUP (58 , codec_mic_i2s, gp_pdm_0a,NA ,NA ,NA ,NA ,NA ,NA ,NA ,NA ,NA ),PINGROUP (59 , codec_spkr_i2s,NA ,NA ,NA ,NA ,NA ,NA ,NA ,NA ,NA ,NA ), -PINGROUP (60 , slimbus1, codec_spkr_i2s,NA ,NA ,NA ,NA ,NA ,NA ,NA ,NA ,NA ), -PINGROUP (61 , slimbus1, codec_spkr_i2s,NA ,NA ,NA ,NA ,NA ,NA ,NA ,NA ,NA ), +PINGROUP (60 , slimbus1_a, codec_spkr_i2s,NA ,NA ,NA ,NA ,NA ,NA ,NA ,NA ,NA ), +PINGROUP (61 , slimbus1_a, codec_spkr_i2s,NA ,NA ,NA ,NA ,NA ,NA ,NA ,NA ,NA ),PINGROUP (62 , codec_spkr_i2s,NA ,NA ,NA ,NA ,NA ,NA ,NA ,NA ,NA ,NA ),PINGROUP (63 , audio_pcm,NA ,NA ,NA ,NA ,NA ,NA ,NA ,NA ,NA ,NA ), PINGROUP(64, audio_pcm, gp_pdm_1b, NA, NA, NA, NA, NA, NA, NA, NA, NA),
Signature · 3 lines
Qualcomm Innovation Center, Inc The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum, a Linux Foundation Collaborative Project
cover pinctrl: qcom: Add multiple copy support
In reply to Andy GrossQuoted 3 lines · Andy Gross
These patches add multiple copy support for functions which require additional mux configurations on specific Qualcomm processor pincontrol blocks. Functions which may have multiple copies are slimbus, mi2s, pdm, pcie, and GSBI.
Bj?rn, can you look at this?
Yours,
Linus Walleij