mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
* [PATCH 0/2] upboard pinctrl support for device id INTC1055
@ 2026-06-10  8:34 GaryWang
  2026-06-10  8:34 ` [PATCH 1/2] pinctrl: tigerlake: add some pin groups and functions for INTC1055 GaryWang
                   ` (2 more replies)
  0 siblings, 3 replies; 16+ messages in thread
From: GaryWang @ 2026-06-10  8:34 UTC (permalink / raw)
  To: Mika Westerberg, Andy Shevchenko, Linus Walleij, Thomas Richard
  Cc: Daniele Cleri, JunYingLai, Louis Chen, linux-gpio, linux-kernel,
	GaryWang

Add missing groups and functions in Tigerlake's pinctrl driver for INTC1055.
Add support "UP Xtreme i12", "UP Squared Pro 7000", "UP Squared i12", "UP 7000" boards.

Signed-off-by: GaryWang <is0124@gmail.com>
---
GaryWang (2):
      pinctrl: tigerlake: add some pin groups and functions for INTC1055
      pinctrl: upboard: add device id INTC1055 based UP boards support

 drivers/pinctrl/intel/pinctrl-tigerlake.c | 32 ++++++++++++++++++++++
 drivers/pinctrl/pinctrl-upboard.c         | 45 +++++++++++++++++++++++++++++++
 2 files changed, 77 insertions(+)
---
base-commit: e43ffb69e0438cddd72aaa30898b4dc446f664f8
change-id: 20260609-upboard-pinctrl-add-upboard-intc1055-support-bd1d81311b7a

Best regards,
-- 
GaryWang <is0124@gmail.com>


^ permalink raw reply	[flat|nested] 16+ messages in thread

* [PATCH 1/2] pinctrl: tigerlake: add some pin groups and functions for INTC1055
  2026-06-10  8:34 [PATCH 0/2] upboard pinctrl support for device id INTC1055 GaryWang
@ 2026-06-10  8:34 ` GaryWang
  2026-06-10 14:24   ` Mika Westerberg
  2026-06-10  8:34 ` [PATCH 2/2] pinctrl: upboard: add device id INTC1055 based UP boards support GaryWang
  2026-06-10 14:13 ` [PATCH 0/2] upboard pinctrl support for device id INTC1055 Andy Shevchenko
  2 siblings, 1 reply; 16+ messages in thread
From: GaryWang @ 2026-06-10  8:34 UTC (permalink / raw)
  To: Mika Westerberg, Andy Shevchenko, Linus Walleij, Thomas Richard
  Cc: Daniele Cleri, JunYingLai, Louis Chen, linux-gpio, linux-kernel,
	GaryWang

Add i2c0, i2c1, pwm0, uart1, ssp2 pin groups & functions in tgllp_soc_data
 for device id INTC1055.

Signed-off-by: GaryWang <is0124@gmail.com>
---
 drivers/pinctrl/intel/pinctrl-tigerlake.c | 32 +++++++++++++++++++++++++++++++
 1 file changed, 32 insertions(+)

diff --git a/drivers/pinctrl/intel/pinctrl-tigerlake.c b/drivers/pinctrl/intel/pinctrl-tigerlake.c
index ae231f7fba49..aeb031570536 100644
--- a/drivers/pinctrl/intel/pinctrl-tigerlake.c
+++ b/drivers/pinctrl/intel/pinctrl-tigerlake.c
@@ -330,6 +330,34 @@ static const struct pinctrl_pin_desc tgllp_pins[] = {
 	PINCTRL_PIN(276, "SPI0_CLK_LOOPBK"),
 };
 
+static const unsigned int tgllp_i2c0_pins[] = { 5, 6 };
+static const unsigned int tgllp_i2c1_pins[] = { 7, 8 };
+static const unsigned int tgllp_pwm0_pins[] = { 99 };
+static const unsigned int tgllp_uart1_pins[] = { 85, 86, 87, 88 };
+static const unsigned int tgllp_ssp2_pins[] = { 108, 109, 110, 111 };
+
+static const struct intel_pingroup tgllp_groups[] = {
+	PIN_GROUP("i2c0_grp", tgllp_i2c0_pins, 2),
+	PIN_GROUP("i2c1_grp", tgllp_i2c1_pins, 2),
+	PIN_GROUP("pwm0_grp", tgllp_pwm0_pins, 1),
+	PIN_GROUP("uart1_grp", tgllp_uart1_pins, 1),
+	PIN_GROUP("ssp2_grp", tgllp_ssp2_pins, 7),
+};
+
+static const char * const tgllp_i2c0_groups[] = { "i2c0_grp" };
+static const char * const tgllp_i2c1_groups[] = { "i2c1_grp" };
+static const char * const tgllp_pwm0_groups[] = { "pwm0_grp" };
+static const char * const tgllp_uart1_groups[] = { "uart1_grp" };
+static const char * const tgllp_ssp2_groups[] = { "ssp2_grp" };
+
+static const struct intel_function tgllp_functions[] = {
+	FUNCTION("i2c0", tgllp_i2c0_groups),
+	FUNCTION("i2c1", tgllp_i2c1_groups),
+	FUNCTION("pwm0", tgllp_pwm0_groups),
+	FUNCTION("uart1", tgllp_uart1_groups),
+	FUNCTION("ssp2", tgllp_ssp2_groups),
+};
+
 static const struct intel_padgroup tgllp_community0_gpps[] = {
 	INTEL_GPP(0, 0, 25, 0),				/* GPP_B */
 	INTEL_GPP(1, 26, 41, 32),			/* GPP_T */
@@ -367,6 +395,10 @@ static const struct intel_community tgllp_communities[] = {
 static const struct intel_pinctrl_soc_data tgllp_soc_data = {
 	.pins = tgllp_pins,
 	.npins = ARRAY_SIZE(tgllp_pins),
+	.groups = tgllp_groups,
+	.ngroups = ARRAY_SIZE(tgllp_groups),
+	.functions = tgllp_functions,
+	.nfunctions = ARRAY_SIZE(tgllp_functions),
 	.communities = tgllp_communities,
 	.ncommunities = ARRAY_SIZE(tgllp_communities),
 };

-- 
2.43.0


^ permalink raw reply	[flat|nested] 16+ messages in thread

* [PATCH 2/2] pinctrl: upboard: add device id INTC1055 based UP boards support
  2026-06-10  8:34 [PATCH 0/2] upboard pinctrl support for device id INTC1055 GaryWang
  2026-06-10  8:34 ` [PATCH 1/2] pinctrl: tigerlake: add some pin groups and functions for INTC1055 GaryWang
@ 2026-06-10  8:34 ` GaryWang
  2026-06-11 12:29   ` Linus Walleij
  2026-06-10 14:13 ` [PATCH 0/2] upboard pinctrl support for device id INTC1055 Andy Shevchenko
  2 siblings, 1 reply; 16+ messages in thread
From: GaryWang @ 2026-06-10  8:34 UTC (permalink / raw)
  To: Mika Westerberg, Andy Shevchenko, Linus Walleij, Thomas Richard
  Cc: Daniele Cleri, JunYingLai, Louis Chen, linux-gpio, linux-kernel,
	GaryWang

Add support "UP Xtreme i12", "UP Squared Pro 7000", "UP Squared i12",
 "UP 7000" boards and I2C/PWM/UART/SPI pins mapping data.

Signed-off-by: GaryWang <is0124@gmail.com>
---
 drivers/pinctrl/pinctrl-upboard.c | 45 +++++++++++++++++++++++++++++++++++++++
 1 file changed, 45 insertions(+)

diff --git a/drivers/pinctrl/pinctrl-upboard.c b/drivers/pinctrl/pinctrl-upboard.c
index f8c8b9d84990..09018b665ee3 100644
--- a/drivers/pinctrl/pinctrl-upboard.c
+++ b/drivers/pinctrl/pinctrl-upboard.c
@@ -912,6 +912,19 @@ static const struct upboard_pinctrl_map upboard_pinctrl_map_apl01 = {
 	.nmaps = ARRAY_SIZE(pinctrl_map_apl01),
 };
 
+static const struct pinctrl_map pinctrl_map_adl[] = {
+	PIN_MAP_MUX_GROUP_DEFAULT("upboard-pinctrl", "INTC1055:00", "i2c0_grp", "i2c0"),
+	PIN_MAP_MUX_GROUP_DEFAULT("upboard-pinctrl", "INTC1055:00", "i2c1_grp", "i2c1"),
+	PIN_MAP_MUX_GROUP_DEFAULT("upboard-pinctrl", "INTC1055:00", "pwm0_grp", "pwm0"),
+	PIN_MAP_MUX_GROUP_DEFAULT("upboard-pinctrl", "INTC1055:00", "uart1_grp", "uart1"),
+	PIN_MAP_MUX_GROUP_DEFAULT("upboard-pinctrl", "INTC1055:00", "ssp2_grp", "ssp2"),
+};
+
+static const struct upboard_pinctrl_map upboard_pinctrl_map_adl = {
+	.maps = &pinctrl_map_adl[0],
+	.nmaps = ARRAY_SIZE(pinctrl_map_adl),
+};
+
 static const struct dmi_system_id dmi_platform_info[] = {
 	{
 		/* UP Squared */
@@ -921,6 +934,38 @@ static const struct dmi_system_id dmi_platform_info[] = {
 		},
 		.driver_data = (void *)&upboard_pinctrl_map_apl01,
 	},
+	{
+		/* UP Xtreme i12 */
+		.matches = {
+			DMI_EXACT_MATCH(DMI_SYS_VENDOR, "AAEON"),
+			DMI_EXACT_MATCH(DMI_BOARD_NAME, "UPX-ADLP01"),
+		},
+		.driver_data = (void *)&upboard_pinctrl_map_adl,
+	},
+	{
+		/* UP Squared Pro 7000 */
+		.matches = {
+			DMI_EXACT_MATCH(DMI_SYS_VENDOR, "AAEON"),
+			DMI_EXACT_MATCH(DMI_BOARD_NAME, "UPN-ADLN01"),
+		},
+		.driver_data = (void *)&upboard_pinctrl_map_adl,
+	},
+	{
+		/* UP Squared i12 */
+		.matches = {
+			DMI_EXACT_MATCH(DMI_SYS_VENDOR, "AAEON"),
+			DMI_EXACT_MATCH(DMI_BOARD_NAME, "UPS-ADLP01"),
+		},
+		.driver_data = (void *)&upboard_pinctrl_map_adl,
+	},
+	{
+		/* UP 7000 */
+		.matches = {
+			DMI_EXACT_MATCH(DMI_SYS_VENDOR, "AAEON"),
+			DMI_EXACT_MATCH(DMI_BOARD_NAME, "UP-ADLN01"),
+		},
+		.driver_data = (void *)&upboard_pinctrl_map_adl,
+	},
 	{ }
 };
 

-- 
2.43.0


^ permalink raw reply	[flat|nested] 16+ messages in thread

* Re: [PATCH 0/2] upboard pinctrl support for device id INTC1055
  2026-06-10  8:34 [PATCH 0/2] upboard pinctrl support for device id INTC1055 GaryWang
  2026-06-10  8:34 ` [PATCH 1/2] pinctrl: tigerlake: add some pin groups and functions for INTC1055 GaryWang
  2026-06-10  8:34 ` [PATCH 2/2] pinctrl: upboard: add device id INTC1055 based UP boards support GaryWang
@ 2026-06-10 14:13 ` Andy Shevchenko
  2026-06-11 12:30   ` Linus Walleij
  2 siblings, 1 reply; 16+ messages in thread
From: Andy Shevchenko @ 2026-06-10 14:13 UTC (permalink / raw)
  To: GaryWang
  Cc: Mika Westerberg, Andy Shevchenko, Linus Walleij, Thomas Richard,
	Daniele Cleri, JunYingLai, Louis Chen, linux-gpio, linux-kernel

On Wed, Jun 10, 2026 at 04:34:22PM +0800, GaryWang wrote:
> Add missing groups and functions in Tigerlake's pinctrl driver for INTC1055.
> Add support "UP Xtreme i12", "UP Squared Pro 7000", "UP Squared i12", "UP 7000" boards.

I can take both via Intel pin control tree, just waiting for Mika's and Linus' acks.
Then I can send PR to Linus to (try to) include this to v7.2-rc2 as it seems close
enough to the category of adding new HW support without breaking anything else.

-- 
With Best Regards,
Andy Shevchenko



^ permalink raw reply	[flat|nested] 16+ messages in thread

* Re: [PATCH 1/2] pinctrl: tigerlake: add some pin groups and functions for INTC1055
  2026-06-10  8:34 ` [PATCH 1/2] pinctrl: tigerlake: add some pin groups and functions for INTC1055 GaryWang
@ 2026-06-10 14:24   ` Mika Westerberg
  2026-06-10 14:37     ` Andy Shevchenko
  0 siblings, 1 reply; 16+ messages in thread
From: Mika Westerberg @ 2026-06-10 14:24 UTC (permalink / raw)
  To: GaryWang
  Cc: Andy Shevchenko, Linus Walleij, Thomas Richard, Daniele Cleri,
	JunYingLai, Louis Chen, linux-gpio, linux-kernel

On Wed, Jun 10, 2026 at 04:34:23PM +0800, GaryWang wrote:
> Add i2c0, i2c1, pwm0, uart1, ssp2 pin groups & functions in tgllp_soc_data
>  for device id INTC1055.

Okay we can see that from the code but why? Can you explain it here too?

> 
> Signed-off-by: GaryWang <is0124@gmail.com>
> ---
>  drivers/pinctrl/intel/pinctrl-tigerlake.c | 32 +++++++++++++++++++++++++++++++
>  1 file changed, 32 insertions(+)
> 
> diff --git a/drivers/pinctrl/intel/pinctrl-tigerlake.c b/drivers/pinctrl/intel/pinctrl-tigerlake.c
> index ae231f7fba49..aeb031570536 100644
> --- a/drivers/pinctrl/intel/pinctrl-tigerlake.c
> +++ b/drivers/pinctrl/intel/pinctrl-tigerlake.c
> @@ -330,6 +330,34 @@ static const struct pinctrl_pin_desc tgllp_pins[] = {
>  	PINCTRL_PIN(276, "SPI0_CLK_LOOPBK"),
>  };
>  
> +static const unsigned int tgllp_i2c0_pins[] = { 5, 6 };
> +static const unsigned int tgllp_i2c1_pins[] = { 7, 8 };
> +static const unsigned int tgllp_pwm0_pins[] = { 99 };
> +static const unsigned int tgllp_uart1_pins[] = { 85, 86, 87, 88 };
> +static const unsigned int tgllp_ssp2_pins[] = { 108, 109, 110, 111 };
> +
> +static const struct intel_pingroup tgllp_groups[] = {
> +	PIN_GROUP("i2c0_grp", tgllp_i2c0_pins, 2),
> +	PIN_GROUP("i2c1_grp", tgllp_i2c1_pins, 2),
> +	PIN_GROUP("pwm0_grp", tgllp_pwm0_pins, 1),
> +	PIN_GROUP("uart1_grp", tgllp_uart1_pins, 1),
> +	PIN_GROUP("ssp2_grp", tgllp_ssp2_pins, 7),
> +};
> +
> +static const char * const tgllp_i2c0_groups[] = { "i2c0_grp" };
> +static const char * const tgllp_i2c1_groups[] = { "i2c1_grp" };
> +static const char * const tgllp_pwm0_groups[] = { "pwm0_grp" };
> +static const char * const tgllp_uart1_groups[] = { "uart1_grp" };
> +static const char * const tgllp_ssp2_groups[] = { "ssp2_grp" };
> +
> +static const struct intel_function tgllp_functions[] = {
> +	FUNCTION("i2c0", tgllp_i2c0_groups),
> +	FUNCTION("i2c1", tgllp_i2c1_groups),
> +	FUNCTION("pwm0", tgllp_pwm0_groups),
> +	FUNCTION("uart1", tgllp_uart1_groups),
> +	FUNCTION("ssp2", tgllp_ssp2_groups),
> +};
> +
>  static const struct intel_padgroup tgllp_community0_gpps[] = {
>  	INTEL_GPP(0, 0, 25, 0),				/* GPP_B */
>  	INTEL_GPP(1, 26, 41, 32),			/* GPP_T */
> @@ -367,6 +395,10 @@ static const struct intel_community tgllp_communities[] = {
>  static const struct intel_pinctrl_soc_data tgllp_soc_data = {
>  	.pins = tgllp_pins,
>  	.npins = ARRAY_SIZE(tgllp_pins),
> +	.groups = tgllp_groups,
> +	.ngroups = ARRAY_SIZE(tgllp_groups),
> +	.functions = tgllp_functions,
> +	.nfunctions = ARRAY_SIZE(tgllp_functions),
>  	.communities = tgllp_communities,
>  	.ncommunities = ARRAY_SIZE(tgllp_communities),
>  };
> 
> -- 
> 2.43.0

^ permalink raw reply	[flat|nested] 16+ messages in thread

* Re: [PATCH 1/2] pinctrl: tigerlake: add some pin groups and functions for INTC1055
  2026-06-10 14:24   ` Mika Westerberg
@ 2026-06-10 14:37     ` Andy Shevchenko
  2026-06-11  7:17       ` GaryWang
  0 siblings, 1 reply; 16+ messages in thread
From: Andy Shevchenko @ 2026-06-10 14:37 UTC (permalink / raw)
  To: Mika Westerberg
  Cc: GaryWang, Andy Shevchenko, Linus Walleij, Thomas Richard,
	Daniele Cleri, JunYingLai, Louis Chen, linux-gpio, linux-kernel

On Wed, Jun 10, 2026 at 04:24:09PM +0200, Mika Westerberg wrote:
> On Wed, Jun 10, 2026 at 04:34:23PM +0800, GaryWang wrote:
> > Add i2c0, i2c1, pwm0, uart1, ssp2 pin groups & functions in tgllp_soc_data
> >  for device id INTC1055.
> 
> Okay we can see that from the code but why? Can you explain it here too?

Yeah, good point. I know the why? because I reviewed the pinctrl-upboard
driver, but unprepared reader doesn't know all the details.

-- 
With Best Regards,
Andy Shevchenko



^ permalink raw reply	[flat|nested] 16+ messages in thread

* Re: [PATCH 1/2] pinctrl: tigerlake: add some pin groups and functions for INTC1055
  2026-06-10 14:37     ` Andy Shevchenko
@ 2026-06-11  7:17       ` GaryWang
  2026-06-11  7:40         ` Andy Shevchenko
  0 siblings, 1 reply; 16+ messages in thread
From: GaryWang @ 2026-06-11  7:17 UTC (permalink / raw)
  To: Andy Shevchenko
  Cc: Mika Westerberg, Andy Shevchenko, Linus Walleij, Thomas Richard,
	Daniele Cleri, JunYingLai, Louis Chen, linux-gpio, linux-kernel

On Wed, 10 Jun 2026 at 22:37, Andy Shevchenko
<andriy.shevchenko@intel.com> wrote:
>
> On Wed, Jun 10, 2026 at 04:24:09PM +0200, Mika Westerberg wrote:
> > On Wed, Jun 10, 2026 at 04:34:23PM +0800, GaryWang wrote:
> > > Add i2c0, i2c1, pwm0, uart1, ssp2 pin groups & functions in tgllp_soc_data
> > >  for device id INTC1055.
> >
> > Okay we can see that from the code but why? Can you explain it here too?
>
> Yeah, good point. I know the why? because I reviewed the pinctrl-upboard
> driver, but unprepared reader doesn't know all the details.
>
To answering Mika's question,
Upboard hat pins using SOC's native function pins, and pinctrl-upboard can
map them to the correct functions through gpio-aggregator.
Thomas was involved in the implementation of the SOC pinctrl-upboard
in Apollolake,
and he is very familiar with the details.

> --
> With Best Regards,
> Andy Shevchenko
>
>

^ permalink raw reply	[flat|nested] 16+ messages in thread

* Re: [PATCH 1/2] pinctrl: tigerlake: add some pin groups and functions for INTC1055
  2026-06-11  7:17       ` GaryWang
@ 2026-06-11  7:40         ` Andy Shevchenko
  2026-06-11  8:27           ` GaryWang
  0 siblings, 1 reply; 16+ messages in thread
From: Andy Shevchenko @ 2026-06-11  7:40 UTC (permalink / raw)
  To: GaryWang
  Cc: Mika Westerberg, Andy Shevchenko, Linus Walleij, Thomas Richard,
	Daniele Cleri, JunYingLai, Louis Chen, linux-gpio, linux-kernel

On Thu, Jun 11, 2026 at 03:17:04PM +0800, GaryWang wrote:
> On Wed, 10 Jun 2026 at 22:37, Andy Shevchenko
> <andriy.shevchenko@intel.com> wrote:
> > On Wed, Jun 10, 2026 at 04:24:09PM +0200, Mika Westerberg wrote:
> > > On Wed, Jun 10, 2026 at 04:34:23PM +0800, GaryWang wrote:
> > > > Add i2c0, i2c1, pwm0, uart1, ssp2 pin groups & functions in tgllp_soc_data
> > > >  for device id INTC1055.
> > >
> > > Okay we can see that from the code but why? Can you explain it here too?
> >
> > Yeah, good point. I know the why? because I reviewed the pinctrl-upboard
> > driver, but unprepared reader doesn't know all the details.
> >
> To answering Mika's question,
> Upboard hat pins using SOC's native function pins, and pinctrl-upboard can
> map them to the correct functions through gpio-aggregator.
> Thomas was involved in the implementation of the SOC pinctrl-upboard
> in Apollolake,
> and he is very familiar with the details.

You need to make it to be the part of the commit message explaining this
clearly ("somebody who knows somebody" way of explanation is not an option).

-- 
With Best Regards,
Andy Shevchenko



^ permalink raw reply	[flat|nested] 16+ messages in thread

* Re: [PATCH 1/2] pinctrl: tigerlake: add some pin groups and functions for INTC1055
  2026-06-11  7:40         ` Andy Shevchenko
@ 2026-06-11  8:27           ` GaryWang
  0 siblings, 0 replies; 16+ messages in thread
From: GaryWang @ 2026-06-11  8:27 UTC (permalink / raw)
  To: Andy Shevchenko
  Cc: Mika Westerberg, Andy Shevchenko, Linus Walleij, Thomas Richard,
	Daniele Cleri, JunYingLai, Louis Chen, linux-gpio, linux-kernel

On Thu, 11 Jun 2026 at 15:40, Andy Shevchenko
<andriy.shevchenko@intel.com> wrote:
>
> On Thu, Jun 11, 2026 at 03:17:04PM +0800, GaryWang wrote:
> > On Wed, 10 Jun 2026 at 22:37, Andy Shevchenko
> > <andriy.shevchenko@intel.com> wrote:
> > > On Wed, Jun 10, 2026 at 04:24:09PM +0200, Mika Westerberg wrote:
> > > > On Wed, Jun 10, 2026 at 04:34:23PM +0800, GaryWang wrote:
> > > > > Add i2c0, i2c1, pwm0, uart1, ssp2 pin groups & functions in tgllp_soc_data
> > > > >  for device id INTC1055.
> > > >
> > > > Okay we can see that from the code but why? Can you explain it here too?
> > >
> > > Yeah, good point. I know the why? because I reviewed the pinctrl-upboard
> > > driver, but unprepared reader doesn't know all the details.
> > >
> > To answering Mika's question,
> > Upboard hat pins using SOC's native function pins, and pinctrl-upboard can
> > map them to the correct functions through gpio-aggregator.
> > Thomas was involved in the implementation of the SOC pinctrl-upboard
> > in Apollolake,
> > and he is very familiar with the details.
>
> You need to make it to be the part of the commit message explaining this
> clearly ("somebody who knows somebody" way of explanation is not an option).
>
got it, will detail explaining in next version.

> --
> With Best Regards,
> Andy Shevchenko
>
>

^ permalink raw reply	[flat|nested] 16+ messages in thread

* Re: [PATCH 2/2] pinctrl: upboard: add device id INTC1055 based UP boards support
  2026-06-10  8:34 ` [PATCH 2/2] pinctrl: upboard: add device id INTC1055 based UP boards support GaryWang
@ 2026-06-11 12:29   ` Linus Walleij
  2026-06-11 16:02     ` Andy Shevchenko
  0 siblings, 1 reply; 16+ messages in thread
From: Linus Walleij @ 2026-06-11 12:29 UTC (permalink / raw)
  To: GaryWang
  Cc: Mika Westerberg, Andy Shevchenko, Thomas Richard, Daniele Cleri,
	JunYingLai, Louis Chen, linux-gpio, linux-kernel

Hi Gary,

thanks for your patch!

On Wed, Jun 10, 2026 at 11:00 AM GaryWang <is0124@gmail.com> wrote:

> +static const struct pinctrl_map pinctrl_map_adl[] = {
> +       PIN_MAP_MUX_GROUP_DEFAULT("upboard-pinctrl", "INTC1055:00", "i2c0_grp", "i2c0"),
> +       PIN_MAP_MUX_GROUP_DEFAULT("upboard-pinctrl", "INTC1055:00", "i2c1_grp", "i2c1"),
> +       PIN_MAP_MUX_GROUP_DEFAULT("upboard-pinctrl", "INTC1055:00", "pwm0_grp", "pwm0"),
> +       PIN_MAP_MUX_GROUP_DEFAULT("upboard-pinctrl", "INTC1055:00", "uart1_grp", "uart1"),
> +       PIN_MAP_MUX_GROUP_DEFAULT("upboard-pinctrl", "INTC1055:00", "ssp2_grp", "ssp2"),
> +};

This looks very ACPI so I do not dare to merge it unless Andy or Mika ACKs it.

Yours,
Linus Walleij

^ permalink raw reply	[flat|nested] 16+ messages in thread

* Re: [PATCH 0/2] upboard pinctrl support for device id INTC1055
  2026-06-10 14:13 ` [PATCH 0/2] upboard pinctrl support for device id INTC1055 Andy Shevchenko
@ 2026-06-11 12:30   ` Linus Walleij
  2026-06-11 16:04     ` Andy Shevchenko
  0 siblings, 1 reply; 16+ messages in thread
From: Linus Walleij @ 2026-06-11 12:30 UTC (permalink / raw)
  To: Andy Shevchenko
  Cc: GaryWang, Mika Westerberg, Andy Shevchenko, Thomas Richard,
	Daniele Cleri, JunYingLai, Louis Chen, linux-gpio, linux-kernel

On Wed, Jun 10, 2026 at 4:13 PM Andy Shevchenko
<andriy.shevchenko@intel.com> wrote:
> On Wed, Jun 10, 2026 at 04:34:22PM +0800, GaryWang wrote:
> > Add missing groups and functions in Tigerlake's pinctrl driver for INTC1055.
> > Add support "UP Xtreme i12", "UP Squared Pro 7000", "UP Squared i12", "UP 7000" boards.
>
> I can take both via Intel pin control tree, just waiting for Mika's and Linus' acks.
> Then I can send PR to Linus to (try to) include this to v7.2-rc2 as it seems close
> enough to the category of adding new HW support without breaking anything else.

Excellent, thanks.
Acked-by: Linus Walleij< linusw@kernel.org>

Yours,
Linus Walleij

^ permalink raw reply	[flat|nested] 16+ messages in thread

* Re: [PATCH 2/2] pinctrl: upboard: add device id INTC1055 based UP boards support
  2026-06-11 12:29   ` Linus Walleij
@ 2026-06-11 16:02     ` Andy Shevchenko
  0 siblings, 0 replies; 16+ messages in thread
From: Andy Shevchenko @ 2026-06-11 16:02 UTC (permalink / raw)
  To: Linus Walleij
  Cc: GaryWang, Mika Westerberg, Andy Shevchenko, Thomas Richard,
	Daniele Cleri, JunYingLai, Louis Chen, linux-gpio, linux-kernel

On Thu, Jun 11, 2026 at 02:29:07PM +0200, Linus Walleij wrote:
> On Wed, Jun 10, 2026 at 11:00 AM GaryWang <is0124@gmail.com> wrote:

...

> > +static const struct pinctrl_map pinctrl_map_adl[] = {
> > +       PIN_MAP_MUX_GROUP_DEFAULT("upboard-pinctrl", "INTC1055:00", "i2c0_grp", "i2c0"),
> > +       PIN_MAP_MUX_GROUP_DEFAULT("upboard-pinctrl", "INTC1055:00", "i2c1_grp", "i2c1"),
> > +       PIN_MAP_MUX_GROUP_DEFAULT("upboard-pinctrl", "INTC1055:00", "pwm0_grp", "pwm0"),
> > +       PIN_MAP_MUX_GROUP_DEFAULT("upboard-pinctrl", "INTC1055:00", "uart1_grp", "uart1"),
> > +       PIN_MAP_MUX_GROUP_DEFAULT("upboard-pinctrl", "INTC1055:00", "ssp2_grp", "ssp2"),
> > +};
> 
> This looks very ACPI so I do not dare to merge it unless Andy or Mika ACKs it.

Basically I proposed to merge via Intel pin control tree, in such a case
the Ack will be implicit (as it will have my SoB). But I want you to have
a look from high level point of view of pin control subsystem.

-- 
With Best Regards,
Andy Shevchenko



^ permalink raw reply	[flat|nested] 16+ messages in thread

* Re: [PATCH 0/2] upboard pinctrl support for device id INTC1055
  2026-06-11 12:30   ` Linus Walleij
@ 2026-06-11 16:04     ` Andy Shevchenko
  2026-06-11 17:05       ` GaryWang
  0 siblings, 1 reply; 16+ messages in thread
From: Andy Shevchenko @ 2026-06-11 16:04 UTC (permalink / raw)
  To: Linus Walleij
  Cc: GaryWang, Mika Westerberg, Andy Shevchenko, Thomas Richard,
	Daniele Cleri, JunYingLai, Louis Chen, linux-gpio, linux-kernel

On Thu, Jun 11, 2026 at 02:30:00PM +0200, Linus Walleij wrote:
> On Wed, Jun 10, 2026 at 4:13 PM Andy Shevchenko
> <andriy.shevchenko@intel.com> wrote:
> > On Wed, Jun 10, 2026 at 04:34:22PM +0800, GaryWang wrote:
> > > Add missing groups and functions in Tigerlake's pinctrl driver for INTC1055.
> > > Add support "UP Xtreme i12", "UP Squared Pro 7000", "UP Squared i12", "UP 7000" boards.
> >
> > I can take both via Intel pin control tree, just waiting for Mika's and Linus' acks.
> > Then I can send PR to Linus to (try to) include this to v7.2-rc2 as it seems close
> > enough to the category of adding new HW support without breaking anything else.
> 
> Excellent, thanks.
> Acked-by: Linus Walleij< linusw@kernel.org>

Thanks! I consider that for both patches. Gary, please send a v2 with all tags
and elaborated commit messages and cover letter to address Mika's questions and
concerns.

-- 
With Best Regards,
Andy Shevchenko



^ permalink raw reply	[flat|nested] 16+ messages in thread

* Re: [PATCH 0/2] upboard pinctrl support for device id INTC1055
  2026-06-11 16:04     ` Andy Shevchenko
@ 2026-06-11 17:05       ` GaryWang
  2026-06-29 17:39         ` Andy Shevchenko
  0 siblings, 1 reply; 16+ messages in thread
From: GaryWang @ 2026-06-11 17:05 UTC (permalink / raw)
  To: Andy Shevchenko
  Cc: Linus Walleij, Mika Westerberg, Andy Shevchenko, Thomas Richard,
	Daniele Cleri, JunYingLai, Louis Chen, linux-gpio, linux-kernel

On Fri, Jun 12, 2026 at 12:04 AM Andy Shevchenko
<andriy.shevchenko@intel.com> wrote:
>
> On Thu, Jun 11, 2026 at 02:30:00PM +0200, Linus Walleij wrote:
> > On Wed, Jun 10, 2026 at 4:13 PM Andy Shevchenko
> > <andriy.shevchenko@intel.com> wrote:
> > > On Wed, Jun 10, 2026 at 04:34:22PM +0800, GaryWang wrote:
> > > > Add missing groups and functions in Tigerlake's pinctrl driver for INTC1055.
> > > > Add support "UP Xtreme i12", "UP Squared Pro 7000", "UP Squared i12", "UP 7000" boards.
> > >
> > > I can take both via Intel pin control tree, just waiting for Mika's and Linus' acks.
> > > Then I can send PR to Linus to (try to) include this to v7.2-rc2 as it seems close
> > > enough to the category of adding new HW support without breaking anything else.
> >
> > Excellent, thanks.
> > Acked-by: Linus Walleij< linusw@kernel.org>
>
> Thanks! I consider that for both patches. Gary, please send a v2 with all tags
> and elaborated commit messages and cover letter to address Mika's questions and
> concerns.
>
yeah, no problem to send patch V2 ASAP.
> --
> With Best Regards,
> Andy Shevchenko
>
>

^ permalink raw reply	[flat|nested] 16+ messages in thread

* Re: [PATCH 0/2] upboard pinctrl support for device id INTC1055
  2026-06-11 17:05       ` GaryWang
@ 2026-06-29 17:39         ` Andy Shevchenko
  2026-06-29 19:03           ` GaryWang
  0 siblings, 1 reply; 16+ messages in thread
From: Andy Shevchenko @ 2026-06-29 17:39 UTC (permalink / raw)
  To: GaryWang
  Cc: Linus Walleij, Mika Westerberg, Andy Shevchenko, Thomas Richard,
	Daniele Cleri, JunYingLai, Louis Chen, linux-gpio, linux-kernel

On Fri, Jun 12, 2026 at 01:05:35AM +0800, GaryWang wrote:
> On Fri, Jun 12, 2026 at 12:04 AM Andy Shevchenko
> <andriy.shevchenko@intel.com> wrote:
> > On Thu, Jun 11, 2026 at 02:30:00PM +0200, Linus Walleij wrote:
> > > On Wed, Jun 10, 2026 at 4:13 PM Andy Shevchenko
> > > <andriy.shevchenko@intel.com> wrote:
> > > > On Wed, Jun 10, 2026 at 04:34:22PM +0800, GaryWang wrote:

...

> > > > I can take both via Intel pin control tree, just waiting for Mika's and Linus' acks.
> > > > Then I can send PR to Linus to (try to) include this to v7.2-rc2 as it seems close
> > > > enough to the category of adding new HW support without breaking anything else.
> > >
> > > Excellent, thanks.
> > > Acked-by: Linus Walleij< linusw@kernel.org>
> >
> > Thanks! I consider that for both patches. Gary, please send a v2 with all tags
> > and elaborated commit messages and cover letter to address Mika's questions and
> > concerns.

> yeah, no problem to send patch V2 ASAP.

Did I miss v2? If not, now is a good time for it.

-- 
With Best Regards,
Andy Shevchenko



^ permalink raw reply	[flat|nested] 16+ messages in thread

* Re: [PATCH 0/2] upboard pinctrl support for device id INTC1055
  2026-06-29 17:39         ` Andy Shevchenko
@ 2026-06-29 19:03           ` GaryWang
  0 siblings, 0 replies; 16+ messages in thread
From: GaryWang @ 2026-06-29 19:03 UTC (permalink / raw)
  To: Andy Shevchenko
  Cc: Linus Walleij, Mika Westerberg, Andy Shevchenko, Thomas Richard,
	Daniele Cleri, JunYingLai, Louis Chen, linux-gpio, linux-kernel

On Tue, Jun 30, 2026 at 1:39 AM Andy Shevchenko
<andriy.shevchenko@intel.com> wrote:
>
> On Fri, Jun 12, 2026 at 01:05:35AM +0800, GaryWang wrote:
> > On Fri, Jun 12, 2026 at 12:04 AM Andy Shevchenko
> > <andriy.shevchenko@intel.com> wrote:
> > > On Thu, Jun 11, 2026 at 02:30:00PM +0200, Linus Walleij wrote:
> > > > On Wed, Jun 10, 2026 at 4:13 PM Andy Shevchenko
> > > > <andriy.shevchenko@intel.com> wrote:
> > > > > On Wed, Jun 10, 2026 at 04:34:22PM +0800, GaryWang wrote:
>
> ...
>
> > > > > I can take both via Intel pin control tree, just waiting for Mika's and Linus' acks.
> > > > > Then I can send PR to Linus to (try to) include this to v7.2-rc2 as it seems close
> > > > > enough to the category of adding new HW support without breaking anything else.
> > > >
> > > > Excellent, thanks.
> > > > Acked-by: Linus Walleij< linusw@kernel.org>
> > >
> > > Thanks! I consider that for both patches. Gary, please send a v2 with all tags
> > > and elaborated commit messages and cover letter to address Mika's questions and
> > > concerns.
>
> > yeah, no problem to send patch V2 ASAP.
>
> Did I miss v2? If not, now is a good time for it.
no, I am preparing V3 to fix n series processor and based on 7.2-rc1,
I'll send it after 2 days, thanks.
>
> --
> With Best Regards,
> Andy Shevchenko
>
>

^ permalink raw reply	[flat|nested] 16+ messages in thread

end of thread, other threads:[~2026-06-29 19:03 UTC | newest]

Thread overview: 16+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2026-06-10  8:34 [PATCH 0/2] upboard pinctrl support for device id INTC1055 GaryWang
2026-06-10  8:34 ` [PATCH 1/2] pinctrl: tigerlake: add some pin groups and functions for INTC1055 GaryWang
2026-06-10 14:24   ` Mika Westerberg
2026-06-10 14:37     ` Andy Shevchenko
2026-06-11  7:17       ` GaryWang
2026-06-11  7:40         ` Andy Shevchenko
2026-06-11  8:27           ` GaryWang
2026-06-10  8:34 ` [PATCH 2/2] pinctrl: upboard: add device id INTC1055 based UP boards support GaryWang
2026-06-11 12:29   ` Linus Walleij
2026-06-11 16:02     ` Andy Shevchenko
2026-06-10 14:13 ` [PATCH 0/2] upboard pinctrl support for device id INTC1055 Andy Shevchenko
2026-06-11 12:30   ` Linus Walleij
2026-06-11 16:04     ` Andy Shevchenko
2026-06-11 17:05       ` GaryWang
2026-06-29 17:39         ` Andy Shevchenko
2026-06-29 19:03           ` GaryWang

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®