From: Paul Cercueil <paul@crapouillou.net>
To: Zhou Yanjie <zhouyanjie@zoho.com>
Cc: linux-mips@vger.kernel.org, linux-kernel@vger.kernel.org,
linux-gpio@vger.kernel.org, devicetree@vger.kernel.org,
robh+dt@kernel.org, paul.burton@mips.com, paulburton@kernel.org,
mark.rutland@arm.com, syq@debian.org
Subject: Re: [PATCH v5 2/4] pinctrl: Ingenic: Add missing parts for X1000 and X1500.
Date: Mon, 25 Nov 2019 14:31:21 +0100 [thread overview]
Message-ID: <1574688681.3.0@crapouillou.net> (raw)
In-Reply-To: <1574682283-87655-3-git-send-email-zhouyanjie@zoho.com>
Hi Zhou,
Le lun., nov. 25, 2019 at 19:44, Zhou Yanjie <zhouyanjie@zoho.com> a
écrit :
> 1.Add pinctrl drivers for the SPI flash controller (SFC) of
> X1000 and X1500.
> 2.Add pinctrl driver for the synchronous serial interface (SSI)
> of X1000.
>
> Signed-off-by: Zhou Yanjie <zhouyanjie@zoho.com>
> ---
>
> Notes:
> v2:
> New patch.
>
> v2->v3:
> Fix typo.
>
> v3->v4:
> No change.
>
> v4->v5:
> No change.
>
> drivers/pinctrl/pinctrl-ingenic.c | 64
> +++++++++++++++++++++++++++++++++++++++
> 1 file changed, 64 insertions(+)
>
> diff --git a/drivers/pinctrl/pinctrl-ingenic.c
> b/drivers/pinctrl/pinctrl-ingenic.c
> index 059e39a..1a99715 100644
> --- a/drivers/pinctrl/pinctrl-ingenic.c
> +++ b/drivers/pinctrl/pinctrl-ingenic.c
> @@ -1031,6 +1031,23 @@ static int x1000_uart1_data_d_pins[] = { 0x62,
> 0x63, };
> static int x1000_uart1_hwflow_pins[] = { 0x64, 0x65, };
> static int x1000_uart2_data_a_pins[] = { 0x02, 0x03, };
> static int x1000_uart2_data_d_pins[] = { 0x65, 0x64, };
> +static int x1000_sfc_pins[] = { 0x1d, 0x1c, 0x1e, 0x1f, 0x1a, 0x1b,
> };
> +static int x1000_ssi_dt_a_22_pins[] = { 0x16, };
> +static int x1000_ssi_dt_a_29_pins[] = { 0x1d, };
> +static int x1000_ssi_dt_d_pins[] = { 0x62, };
> +static int x1000_ssi_dr_a_23_pins[] = { 0x17, };
> +static int x1000_ssi_dr_a_28_pins[] = { 0x1c, };
> +static int x1000_ssi_dr_d_pins[] = { 0x63, };
> +static int x1000_ssi_clk_a_24_pins[] = { 0x18, };
> +static int x1000_ssi_clk_a_26_pins[] = { 0x1a, };
> +static int x1000_ssi_clk_d_pins[] = { 0x60, };
> +static int x1000_ssi_gpc_a_20_pins[] = { 0x14, };
> +static int x1000_ssi_gpc_a_31_pins[] = { 0x1f, };
> +static int x1000_ssi_ce0_a_25_pins[] = { 0x19, };
> +static int x1000_ssi_ce0_a_27_pins[] = { 0x1b, };
> +static int x1000_ssi_ce0_d_pins[] = { 0x61, };
> +static int x1000_ssi_ce1_a_21_pins[] = { 0x15, };
> +static int x1000_ssi_ce1_a_30_pins[] = { 0x1e, };
> static int x1000_mmc0_1bit_pins[] = { 0x18, 0x19, 0x17, };
> static int x1000_mmc0_4bit_pins[] = { 0x16, 0x15, 0x14, };
> static int x1000_mmc0_8bit_pins[] = { 0x13, 0x12, 0x11, 0x10, };
> @@ -1081,6 +1098,23 @@ static int x1000_uart1_data_d_funcs[] = { 1,
> 1, };
> static int x1000_uart1_hwflow_funcs[] = { 1, 1, };
> static int x1000_uart2_data_a_funcs[] = { 2, 2, };
> static int x1000_uart2_data_d_funcs[] = { 0, 0, };
> +static int x1000_sfc_funcs[] = { 1, 1, 1, 1, 1, 1, };
> +static int x1000_ssi_dt_a_22_funcs[] = { 2, };
> +static int x1000_ssi_dt_a_29_funcs[] = { 2, };
> +static int x1000_ssi_dt_d_funcs[] = { 0, };
> +static int x1000_ssi_dr_a_23_funcs[] = { 2, };
> +static int x1000_ssi_dr_a_28_funcs[] = { 2, };
> +static int x1000_ssi_dr_d_funcs[] = { 0, };
> +static int x1000_ssi_clk_a_24_funcs[] = { 2, };
> +static int x1000_ssi_clk_a_26_funcs[] = { 2, };
> +static int x1000_ssi_clk_d_funcs[] = { 0, };
> +static int x1000_ssi_gpc_a_20_funcs[] = { 2, };
> +static int x1000_ssi_gpc_a_31_funcs[] = { 2, };
> +static int x1000_ssi_ce0_a_25_funcs[] = { 2, };
> +static int x1000_ssi_ce0_a_27_funcs[] = { 2, };
> +static int x1000_ssi_ce0_d_funcs[] = { 0, };
> +static int x1000_ssi_ce1_a_21_funcs[] = { 2, };
> +static int x1000_ssi_ce1_a_30_funcs[] = { 2, };
> static int x1000_mmc0_1bit_funcs[] = { 1, 1, 1, };
> static int x1000_mmc0_4bit_funcs[] = { 1, 1, 1, };
> static int x1000_mmc0_8bit_funcs[] = { 1, 1, 1, 1, 1, };
> @@ -1119,6 +1153,23 @@ static const struct group_desc x1000_groups[]
> = {
> INGENIC_PIN_GROUP("uart1-hwflow", x1000_uart1_hwflow),
> INGENIC_PIN_GROUP("uart2-data-a", x1000_uart2_data_a),
> INGENIC_PIN_GROUP("uart2-data-d", x1000_uart2_data_d),
> + INGENIC_PIN_GROUP("sfc", x1000_sfc),
> + INGENIC_PIN_GROUP("ssi-dt-a-22", x1000_ssi_dt_a_22),
> + INGENIC_PIN_GROUP("ssi-dt-a-29", x1000_ssi_dt_a_29),
> + INGENIC_PIN_GROUP("ssi-dt-d", x1000_ssi_dt_d),
> + INGENIC_PIN_GROUP("ssi-dr-a-23", x1000_ssi_dr_a_23),
> + INGENIC_PIN_GROUP("ssi-dr-a-28", x1000_ssi_dr_a_28),
> + INGENIC_PIN_GROUP("ssi-dr-d", x1000_ssi_dr_d),
> + INGENIC_PIN_GROUP("ssi-clk-a-24", x1000_ssi_clk_a_24),
> + INGENIC_PIN_GROUP("ssi-clk-a-26", x1000_ssi_clk_a_26),
> + INGENIC_PIN_GROUP("ssi-clk-d", x1000_ssi_clk_d),
> + INGENIC_PIN_GROUP("ssi-gpc-a-20", x1000_ssi_gpc_a_20),
> + INGENIC_PIN_GROUP("ssi-gpc-a-31", x1000_ssi_gpc_a_31),
> + INGENIC_PIN_GROUP("ssi-ce0-a-25", x1000_ssi_ce0_a_25),
> + INGENIC_PIN_GROUP("ssi-ce0-a-27", x1000_ssi_ce0_a_27),
> + INGENIC_PIN_GROUP("ssi-ce0-d", x1000_ssi_ce0_d),
> + INGENIC_PIN_GROUP("ssi-ce1-a-21", x1000_ssi_ce1_a_21),
> + INGENIC_PIN_GROUP("ssi-ce1-a-30", x1000_ssi_ce1_a_30),
> INGENIC_PIN_GROUP("mmc0-1bit", x1000_mmc0_1bit),
> INGENIC_PIN_GROUP("mmc0-4bit", x1000_mmc0_4bit),
> INGENIC_PIN_GROUP("mmc0-8bit", x1000_mmc0_8bit),
> @@ -1152,6 +1203,15 @@ static const char *x1000_uart1_groups[] = {
> "uart1-data-a", "uart1-data-d", "uart1-hwflow",
> };
> static const char *x1000_uart2_groups[] = { "uart2-data-a",
> "uart2-data-d", };
> +static const char *x1000_sfc_groups[] = { "sfc", };
> +static const char *x1000_ssi_groups[] = {
> + "ssi-dt-a-22", "ssi-dt-a-29", "ssi-dt-d",
> + "ssi-dr-a-23", "ssi-dr-a-28", "ssi-dr-d",
> + "ssi-clk-a-24", "ssi-clk-a-26", "ssi-clk-d",
> + "ssi-gpc-a-20", "ssi-gpc-a-31",
> + "ssi-ce0-a-25", "ssi-ce0-a-27", "ssi-ce0-d",
> + "ssi-ce1-a-21", "ssi-ce1-a-30", "ssi-ce0-d",
"ssi-ce0-d" is set twice.
> +};
> static const char *x1000_mmc0_groups[] = {
> "mmc0-1bit", "mmc0-4bit", "mmc0-8bit",
> };
> @@ -1182,6 +1242,8 @@ static const struct function_desc
> x1000_functions[] = {
> { "uart0", x1000_uart0_groups, ARRAY_SIZE(x1000_uart0_groups), },
> { "uart1", x1000_uart1_groups, ARRAY_SIZE(x1000_uart1_groups), },
> { "uart2", x1000_uart2_groups, ARRAY_SIZE(x1000_uart2_groups), },
> + { "sfc", x1000_sfc_groups, ARRAY_SIZE(x1000_sfc_groups), },
> + { "ssi", x1000_ssi_groups, ARRAY_SIZE(x1000_ssi_groups), },
> { "mmc0", x1000_mmc0_groups, ARRAY_SIZE(x1000_mmc0_groups), },
> { "mmc1", x1000_mmc1_groups, ARRAY_SIZE(x1000_mmc1_groups), },
> { "emc", x1000_emc_groups, ARRAY_SIZE(x1000_emc_groups), },
> @@ -1271,6 +1333,7 @@ static const struct group_desc x1500_groups[] =
> {
> INGENIC_PIN_GROUP("uart1-hwflow", x1500_uart1_hwflow),
> INGENIC_PIN_GROUP("uart2-data-a", x1500_uart2_data_a),
> INGENIC_PIN_GROUP("uart2-data-d", x1500_uart2_data_d),
> + INGENIC_PIN_GROUP("sfc", x1000_sfc),
> INGENIC_PIN_GROUP("mmc-1bit", x1500_mmc_1bit),
> INGENIC_PIN_GROUP("mmc-4bit", x1500_mmc_4bit),
> INGENIC_PIN_GROUP("i2c0-data", x1500_i2c0),
> @@ -1307,6 +1370,7 @@ static const struct function_desc
> x1500_functions[] = {
> { "uart0", x1500_uart0_groups, ARRAY_SIZE(x1500_uart0_groups), },
> { "uart1", x1500_uart1_groups, ARRAY_SIZE(x1500_uart1_groups), },
> { "uart2", x1500_uart2_groups, ARRAY_SIZE(x1500_uart2_groups), },
> + { "sfc", x1000_sfc_groups, ARRAY_SIZE(x1000_sfc_groups), },
> { "mmc", x1500_mmc_groups, ARRAY_SIZE(x1500_mmc_groups), },
> { "i2c0", x1500_i2c0_groups, ARRAY_SIZE(x1500_i2c0_groups), },
> { "i2c1", x1500_i2c1_groups, ARRAY_SIZE(x1500_i2c1_groups), },
> --
> 2.7.4
>
>
next prev parent reply other threads:[~2019-11-25 13:31 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-11-25 11:44 Fix bugs in X1000/X1500 and add X1830 pinctrl driver v5 Zhou Yanjie
2019-11-25 11:44 ` [PATCH v5 1/4] pinctrl: Ingenic: Fix bugs in X1000 and X1500 Zhou Yanjie
2019-11-25 11:44 ` [PATCH v5 2/4] pinctrl: Ingenic: Add missing parts for " Zhou Yanjie
2019-11-25 13:31 ` Paul Cercueil [this message]
2019-11-25 13:46 ` Zhou Yanjie
2019-11-25 11:44 ` [PATCH v5 3/4] dt-bindings: pinctrl: Add bindings for Ingenic X1830 Zhou Yanjie
2019-11-25 11:44 ` [PATCH v5 4/4] pinctrl: Ingenic: Add pinctrl driver for X1830 Zhou Yanjie
2019-11-25 13:54 ` Paul Cercueil
2019-11-25 14:12 ` Zhou Yanjie
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1574688681.3.0@crapouillou.net \
--to=paul@crapouillou.net \
--cc=devicetree@vger.kernel.org \
--cc=linux-gpio@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mips@vger.kernel.org \
--cc=mark.rutland@arm.com \
--cc=paul.burton@mips.com \
--cc=paulburton@kernel.org \
--cc=robh+dt@kernel.org \
--cc=syq@debian.org \
--cc=zhouyanjie@zoho.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).