Linux GPIO subsystem development
 help / color / mirror / Atom feed
* [PATCH] gpio: sodaville: allow COMPILE_TEST builds
@ 2026-05-09  0:33 Rosen Penev
  2026-05-11 10:15 ` Bartosz Golaszewski
                   ` (2 more replies)
  0 siblings, 3 replies; 4+ messages in thread
From: Rosen Penev @ 2026-05-09  0:33 UTC (permalink / raw)
  To: linux-gpio
  Cc: Linus Walleij, Bartosz Golaszewski, Nathan Chancellor,
	Nick Desaulniers, Bill Wendling, Justin Stitt, open list,
	open list:CLANG/LLVM BUILD SUPPORT:Keyword:b(?i:clang|llvm)b

The Intel Sodaville GPIO driver uses PCI/MMIO and generic GPIO helpers, and
builds outside X86.  Move its Kconfig entry out of the X86-only port-I/O
menu and allow it to be selected with COMPILE_TEST.

Tested with:
make LLVM=1 ARCH=loongarch drivers/gpio/gpio-sodaville.o

Assisted-by: Codex:GPT-5.5
Signed-off-by: Rosen Penev <rosenp@gmail.com>
---
 drivers/gpio/Kconfig | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/drivers/gpio/Kconfig b/drivers/gpio/Kconfig
index f8e34b16fd99..9e1ed0f451b9 100644
--- a/drivers/gpio/Kconfig
+++ b/drivers/gpio/Kconfig
@@ -1908,16 +1908,16 @@ config GPIO_RDC321X
 	  Support for the RDC R321x SoC GPIOs over southbridge
 	  PCI configuration space.
 
+endmenu
+
 config GPIO_SODAVILLE
 	bool "Intel Sodaville GPIO support"
-	depends on X86 && OF
+	depends on (X86 || COMPILE_TEST) && OF
 	select GPIO_GENERIC
 	select GENERIC_IRQ_CHIP
 	help
 	  Say Y here to support Intel Sodaville GPIO.
 
-endmenu
-
 menu "SPI GPIO expanders"
 	depends on SPI_MASTER
 
-- 
2.54.0


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

* Re: [PATCH] gpio: sodaville: allow COMPILE_TEST builds
  2026-05-09  0:33 [PATCH] gpio: sodaville: allow COMPILE_TEST builds Rosen Penev
@ 2026-05-11 10:15 ` Bartosz Golaszewski
  2026-05-12 11:09 ` kernel test robot
  2026-05-12 11:42 ` kernel test robot
  2 siblings, 0 replies; 4+ messages in thread
From: Bartosz Golaszewski @ 2026-05-11 10:15 UTC (permalink / raw)
  To: Rosen Penev
  Cc: Linus Walleij, Bartosz Golaszewski, Nathan Chancellor,
	Nick Desaulniers, Bill Wendling, Justin Stitt, open list,
	open list:CLANG/LLVM BUILD SUPPORT:Keyword:b(?i:clang|llvm)b,
	linux-gpio

On Sat, 9 May 2026 02:33:27 +0200, Rosen Penev <rosenp@gmail.com> said:
> The Intel Sodaville GPIO driver uses PCI/MMIO and generic GPIO helpers, and
> builds outside X86.  Move its Kconfig entry out of the X86-only port-I/O
> menu and allow it to be selected with COMPILE_TEST.
>
> Tested with:
> make LLVM=1 ARCH=loongarch drivers/gpio/gpio-sodaville.o
>
> Assisted-by: Codex:GPT-5.5
> Signed-off-by: Rosen Penev <rosenp@gmail.com>
> ---
>  drivers/gpio/Kconfig | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/gpio/Kconfig b/drivers/gpio/Kconfig
> index f8e34b16fd99..9e1ed0f451b9 100644
> --- a/drivers/gpio/Kconfig
> +++ b/drivers/gpio/Kconfig
> @@ -1908,16 +1908,16 @@ config GPIO_RDC321X
>  	  Support for the RDC R321x SoC GPIOs over southbridge
>  	  PCI configuration space.
>
> +endmenu
> +
>  config GPIO_SODAVILLE
>  	bool "Intel Sodaville GPIO support"
> -	depends on X86 && OF
> +	depends on (X86 || COMPILE_TEST) && OF
>  	select GPIO_GENERIC
>  	select GENERIC_IRQ_CHIP
>  	help
>  	  Say Y here to support Intel Sodaville GPIO.
>
> -endmenu
> -
>  menu "SPI GPIO expanders"
>  	depends on SPI_MASTER
>
> --
> 2.54.0
>
>

I don't mind it but I'll wait for an Ack from Intel GPIO maintainers.

Bart

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

* Re: [PATCH] gpio: sodaville: allow COMPILE_TEST builds
  2026-05-09  0:33 [PATCH] gpio: sodaville: allow COMPILE_TEST builds Rosen Penev
  2026-05-11 10:15 ` Bartosz Golaszewski
@ 2026-05-12 11:09 ` kernel test robot
  2026-05-12 11:42 ` kernel test robot
  2 siblings, 0 replies; 4+ messages in thread
From: kernel test robot @ 2026-05-12 11:09 UTC (permalink / raw)
  To: Rosen Penev, linux-gpio

Hi Rosen,

kernel test robot noticed the following build errors:

[auto build test ERROR on brgl/gpio/for-next]
[also build test ERROR on linus/master v7.1-rc3 next-20260508]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch#_base_tree_information]

url:    https://github.com/intel-lab-lkp/linux/commits/Rosen-Penev/gpio-sodaville-allow-COMPILE_TEST-builds/20260512-125042
base:   https://git.kernel.org/pub/scm/linux/kernel/git/brgl/linux.git gpio/for-next
patch link:    https://lore.kernel.org/r/20260509003327.955942-1-rosenp%40gmail.com
patch subject: [PATCH] gpio: sodaville: allow COMPILE_TEST builds
config: sh-allmodconfig (https://download.01.org/0day-ci/archive/20260512/202605121826.ZJeoP3Ek-lkp@intel.com/config)
compiler: sh4-linux-gcc (GCC) 15.2.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20260512/202605121826.ZJeoP3Ek-lkp@intel.com/reproduce)

If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202605121826.ZJeoP3Ek-lkp@intel.com/

All errors (new ones prefixed by >>):

   drivers/gpio/gpio-sodaville.c:253:1: warning: data definition has no type or storage class
     253 | builtin_pci_driver(sdv_gpio_driver);
         | ^~~~~~~~~~~~~~~~~~
   drivers/gpio/gpio-sodaville.c:253:1: error: type defaults to 'int' in declaration of 'builtin_pci_driver' [-Wimplicit-int]
>> drivers/gpio/gpio-sodaville.c:253:1: error: parameter names (without types) in function declaration [-Wdeclaration-missing-parameter-type]
   drivers/gpio/gpio-sodaville.c:245:26: warning: 'sdv_gpio_driver' defined but not used [-Wunused-variable]
     245 | static struct pci_driver sdv_gpio_driver = {
         |                          ^~~~~~~~~~~~~~~


vim +253 drivers/gpio/gpio-sodaville.c

b43ab901d671e3e Sebastian Andrzej Siewior 2011-06-27  244  
b43ab901d671e3e Sebastian Andrzej Siewior 2011-06-27  245  static struct pci_driver sdv_gpio_driver = {
6a5ead91d45d091 Paul Gortmaker            2016-05-09  246  	.driver = {
6a5ead91d45d091 Paul Gortmaker            2016-05-09  247  		.suppress_bind_attrs = true,
6a5ead91d45d091 Paul Gortmaker            2016-05-09  248  	},
b43ab901d671e3e Sebastian Andrzej Siewior 2011-06-27  249  	.name = DRV_NAME,
b43ab901d671e3e Sebastian Andrzej Siewior 2011-06-27  250  	.id_table = sdv_gpio_pci_ids,
b43ab901d671e3e Sebastian Andrzej Siewior 2011-06-27  251  	.probe = sdv_gpio_probe,
b43ab901d671e3e Sebastian Andrzej Siewior 2011-06-27  252  };
6a5ead91d45d091 Paul Gortmaker            2016-05-09 @253  builtin_pci_driver(sdv_gpio_driver);

-- 
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki

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

* Re: [PATCH] gpio: sodaville: allow COMPILE_TEST builds
  2026-05-09  0:33 [PATCH] gpio: sodaville: allow COMPILE_TEST builds Rosen Penev
  2026-05-11 10:15 ` Bartosz Golaszewski
  2026-05-12 11:09 ` kernel test robot
@ 2026-05-12 11:42 ` kernel test robot
  2 siblings, 0 replies; 4+ messages in thread
From: kernel test robot @ 2026-05-12 11:42 UTC (permalink / raw)
  To: Rosen Penev, linux-gpio

Hi Rosen,

kernel test robot noticed the following build errors:

[auto build test ERROR on brgl/gpio/for-next]
[also build test ERROR on linus/master v7.1-rc3 next-20260508]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch#_base_tree_information]

url:    https://github.com/intel-lab-lkp/linux/commits/Rosen-Penev/gpio-sodaville-allow-COMPILE_TEST-builds/20260512-125042
base:   https://git.kernel.org/pub/scm/linux/kernel/git/brgl/linux.git gpio/for-next
patch link:    https://lore.kernel.org/r/20260509003327.955942-1-rosenp%40gmail.com
patch subject: [PATCH] gpio: sodaville: allow COMPILE_TEST builds
config: hexagon-allmodconfig (https://download.01.org/0day-ci/archive/20260512/202605121902.NsjonizV-lkp@intel.com/config)
compiler: clang version 17.0.6 (https://github.com/llvm/llvm-project 6009708b4367171ccdbf4b5905cb6a803753fe18)
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20260512/202605121902.NsjonizV-lkp@intel.com/reproduce)

If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202605121902.NsjonizV-lkp@intel.com/

All errors (new ones prefixed by >>):

>> drivers/gpio/gpio-sodaville.c:253:1: error: type specifier missing, defaults to 'int'; ISO C99 and later do not support implicit int [-Wimplicit-int]
     253 | builtin_pci_driver(sdv_gpio_driver);
         | ^
         | int
>> drivers/gpio/gpio-sodaville.c:253:20: error: a parameter list without types is only allowed in a function definition
     253 | builtin_pci_driver(sdv_gpio_driver);
         |                    ^
   2 errors generated.


vim +/int +253 drivers/gpio/gpio-sodaville.c

b43ab901d671e3 Sebastian Andrzej Siewior 2011-06-27  244  
b43ab901d671e3 Sebastian Andrzej Siewior 2011-06-27  245  static struct pci_driver sdv_gpio_driver = {
6a5ead91d45d09 Paul Gortmaker            2016-05-09  246  	.driver = {
6a5ead91d45d09 Paul Gortmaker            2016-05-09  247  		.suppress_bind_attrs = true,
6a5ead91d45d09 Paul Gortmaker            2016-05-09  248  	},
b43ab901d671e3 Sebastian Andrzej Siewior 2011-06-27  249  	.name = DRV_NAME,
b43ab901d671e3 Sebastian Andrzej Siewior 2011-06-27  250  	.id_table = sdv_gpio_pci_ids,
b43ab901d671e3 Sebastian Andrzej Siewior 2011-06-27  251  	.probe = sdv_gpio_probe,
b43ab901d671e3 Sebastian Andrzej Siewior 2011-06-27  252  };
6a5ead91d45d09 Paul Gortmaker            2016-05-09 @253  builtin_pci_driver(sdv_gpio_driver);

-- 
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki

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

end of thread, other threads:[~2026-05-12 11:43 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-05-09  0:33 [PATCH] gpio: sodaville: allow COMPILE_TEST builds Rosen Penev
2026-05-11 10:15 ` Bartosz Golaszewski
2026-05-12 11:09 ` kernel test robot
2026-05-12 11:42 ` kernel test robot

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