llvm.lists.linux.dev archive mirror
 help / color / mirror / Atom feed
* Re: [RFC RESEND 1/3] mfd: Add support for UP board CPLD/FPGA
       [not found] <20230425152135.30745-2-larry.lai@yunjingtech.com>
@ 2023-05-02  4:56 ` kernel test robot
  2023-05-02 14:21 ` kernel test robot
  1 sibling, 0 replies; 2+ messages in thread
From: kernel test robot @ 2023-05-02  4:56 UTC (permalink / raw)
  To: larry.lai; +Cc: llvm, oe-kbuild-all

Hi larry.lai,

[This is a private test report for your RFC patch.]
kernel test robot noticed the following build warnings:

[auto build test WARNING on 4fe89d07dcc2804c8b562f6c7896a45643d34b2f]

url:    https://github.com/intel-lab-lkp/linux/commits/larry-lai/mfd-Add-support-for-UP-board-CPLD-FPGA/20230425-232744
base:   4fe89d07dcc2804c8b562f6c7896a45643d34b2f
patch link:    https://lore.kernel.org/r/20230425152135.30745-2-larry.lai%40yunjingtech.com
patch subject: [RFC RESEND 1/3] mfd: Add support for UP board CPLD/FPGA
config: i386-allmodconfig (https://download.01.org/0day-ci/archive/20230502/202305021237.3KjZn3Ga-lkp@intel.com/config)
compiler: clang version 14.0.6 (https://github.com/llvm/llvm-project f28c006a5895fc0e329fe15fead81e37457cb1d1)
reproduce (this is a W=1 build):
        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        # https://github.com/intel-lab-lkp/linux/commit/cb16ef420038f2bede2deb3878b529f0e6ea7729
        git remote add linux-review https://github.com/intel-lab-lkp/linux
        git fetch --no-tags linux-review larry-lai/mfd-Add-support-for-UP-board-CPLD-FPGA/20230425-232744
        git checkout cb16ef420038f2bede2deb3878b529f0e6ea7729
        # save the config file
        mkdir build_dir && cp config build_dir/.config
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir ARCH=i386 olddefconfig
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir ARCH=i386 SHELL=/bin/bash drivers/firmware/ drivers/hwmon/ drivers/mfd/ drivers/pinctrl/ drivers/staging/media/atomisp/

If you fix the issue, kindly add following tag where applicable
| Reported-by: kernel test robot <lkp@intel.com>
| Link: https://lore.kernel.org/oe-kbuild-all/202305021237.3KjZn3Ga-lkp@intel.com/

All warnings (new ones prefixed by >>):

>> drivers/mfd/upboard-fpga.c:448:49: warning: data argument not used by format string [-Wformat-extra-args]
                   dev_err(fpga->dev, "Failed to add GPIO LEDs", ret);
                                      ~~~~~~~~~~~~~~~~~~~~~~~~~  ^
   include/linux/dev_printk.h:144:65: note: expanded from macro 'dev_err'
           dev_printk_index_wrap(_dev_err, KERN_ERR, dev, dev_fmt(fmt), ##__VA_ARGS__)
                                                                  ~~~     ^
   include/linux/dev_printk.h:110:23: note: expanded from macro 'dev_printk_index_wrap'
                   _p_func(dev, fmt, ##__VA_ARGS__);                       \
                                ~~~    ^
   drivers/mfd/upboard-fpga.c:486:20: error: assigning to 'struct regmap_config *' from 'const struct regmap_config *const' discards qualifiers [-Werror,-Wincompatible-pointer-types-discards-qualifiers]
           fpga->cpld_config = fpga_data->cpld_config;
                             ^ ~~~~~~~~~~~~~~~~~~~~~~
   1 warning and 1 error generated.


vim +448 drivers/mfd/upboard-fpga.c

   383	
   384	/*
   385	 * MFD upboard-fpga is ACPI driver and can recognize the AANT ID from different
   386	 * kind of upboards. We get the LED GPIO initialized information from this
   387	 * then add led-upboard driver.
   388	 */
   389	int upboard_led_gpio_register(struct upboard_fpga *fpga)
   390	{
   391		struct gpio_led blue_led, yellow_led, green_led, red_led;
   392		struct gpio_desc *desc;
   393		static struct gpio_led upboard_gpio_leds[4];
   394		int leds = 0;
   395		static struct gpio_led_platform_data upboard_gpio_led_platform_data;
   396		static const struct mfd_cell upboard_gpio_led_cells[] = {
   397			MFD_CELL_BASIC("leds-gpio", NULL,
   398				       &upboard_gpio_led_platform_data,
   399				       sizeof(upboard_gpio_led_platform_data), 0)
   400		};
   401		int ret;
   402	
   403		desc = devm_gpiod_get(fpga->dev, "blue", GPIOD_OUT_LOW);
   404		if (!IS_ERR(desc)) {
   405			blue_led.name = "upboard:blue:";
   406			blue_led.gpio = desc_to_gpio(desc);
   407			blue_led.default_state = LEDS_GPIO_DEFSTATE_KEEP;
   408			upboard_gpio_leds[leds++] = blue_led;
   409			devm_gpiod_put(fpga->dev, desc);
   410		}
   411		desc = devm_gpiod_get(fpga->dev, "yellow", GPIOD_OUT_LOW);
   412		if (!IS_ERR(desc)) {
   413			yellow_led.name = "upboard:yellow:";
   414			yellow_led.gpio = desc_to_gpio(desc);
   415			yellow_led.default_state = LEDS_GPIO_DEFSTATE_KEEP;
   416			upboard_gpio_leds[leds++] = yellow_led;
   417			devm_gpiod_put(fpga->dev, desc);
   418		}
   419		desc = devm_gpiod_get(fpga->dev, "green", GPIOD_OUT_LOW);
   420		if (!IS_ERR(desc)) {
   421			green_led.name = "upboard:green:";
   422			green_led.gpio = desc_to_gpio(desc);
   423			green_led.default_state = LEDS_GPIO_DEFSTATE_KEEP;
   424			upboard_gpio_leds[leds++] = green_led;
   425			devm_gpiod_put(fpga->dev, desc);
   426		}
   427		desc = devm_gpiod_get(fpga->dev, "red", GPIOD_OUT_LOW);
   428		if (!IS_ERR(desc)) {
   429			red_led.name = "upboard:red:";
   430			red_led.gpio = desc_to_gpio(desc);
   431			red_led.default_state = LEDS_GPIO_DEFSTATE_KEEP;
   432			upboard_gpio_leds[leds++] = red_led;
   433			devm_gpiod_put(fpga->dev, desc);
   434		}
   435	
   436		/* no LEDs */
   437		if (leds == 0)
   438			return 0;
   439	
   440		upboard_gpio_led_platform_data.num_leds = leds;
   441		upboard_gpio_led_platform_data.leds = upboard_gpio_leds;
   442	
   443		ret = devm_mfd_add_devices(fpga->dev, PLATFORM_DEVID_AUTO,
   444					   upboard_gpio_led_cells,
   445					   ARRAY_SIZE(upboard_gpio_led_cells),
   446					   NULL, 0, NULL);
   447		if (ret) {
 > 448			dev_err(fpga->dev, "Failed to add GPIO LEDs", ret);
   449			return ret;
   450		}
   451	
   452		return 0;
   453	}
   454	

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

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

* Re: [RFC RESEND 1/3] mfd: Add support for UP board CPLD/FPGA
       [not found] <20230425152135.30745-2-larry.lai@yunjingtech.com>
  2023-05-02  4:56 ` [RFC RESEND 1/3] mfd: Add support for UP board CPLD/FPGA kernel test robot
@ 2023-05-02 14:21 ` kernel test robot
  1 sibling, 0 replies; 2+ messages in thread
From: kernel test robot @ 2023-05-02 14:21 UTC (permalink / raw)
  To: larry.lai; +Cc: llvm, oe-kbuild-all

Hi larry.lai,

[This is a private test report for your RFC patch.]
kernel test robot noticed the following build errors:

[auto build test ERROR on 4fe89d07dcc2804c8b562f6c7896a45643d34b2f]

url:    https://github.com/intel-lab-lkp/linux/commits/larry-lai/mfd-Add-support-for-UP-board-CPLD-FPGA/20230425-232744
base:   4fe89d07dcc2804c8b562f6c7896a45643d34b2f
patch link:    https://lore.kernel.org/r/20230425152135.30745-2-larry.lai%40yunjingtech.com
patch subject: [RFC RESEND 1/3] mfd: Add support for UP board CPLD/FPGA
config: i386-allmodconfig (https://download.01.org/0day-ci/archive/20230502/202305022231.zyZXagsH-lkp@intel.com/config)
compiler: clang version 14.0.6 (https://github.com/llvm/llvm-project f28c006a5895fc0e329fe15fead81e37457cb1d1)
reproduce (this is a W=1 build):
        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        # https://github.com/intel-lab-lkp/linux/commit/cb16ef420038f2bede2deb3878b529f0e6ea7729
        git remote add linux-review https://github.com/intel-lab-lkp/linux
        git fetch --no-tags linux-review larry-lai/mfd-Add-support-for-UP-board-CPLD-FPGA/20230425-232744
        git checkout cb16ef420038f2bede2deb3878b529f0e6ea7729
        # save the config file
        mkdir build_dir && cp config build_dir/.config
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir ARCH=i386 olddefconfig
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir ARCH=i386 SHELL=/bin/bash drivers/

If you fix the issue, kindly add following tag where applicable
| Reported-by: kernel test robot <lkp@intel.com>
| Link: https://lore.kernel.org/oe-kbuild-all/202305022231.zyZXagsH-lkp@intel.com/

All errors (new ones prefixed by >>):

   drivers/mfd/upboard-fpga.c:448:49: warning: data argument not used by format string [-Wformat-extra-args]
                   dev_err(fpga->dev, "Failed to add GPIO LEDs", ret);
                                      ~~~~~~~~~~~~~~~~~~~~~~~~~  ^
   include/linux/dev_printk.h:144:65: note: expanded from macro 'dev_err'
           dev_printk_index_wrap(_dev_err, KERN_ERR, dev, dev_fmt(fmt), ##__VA_ARGS__)
                                                                  ~~~     ^
   include/linux/dev_printk.h:110:23: note: expanded from macro 'dev_printk_index_wrap'
                   _p_func(dev, fmt, ##__VA_ARGS__);                       \
                                ~~~    ^
>> drivers/mfd/upboard-fpga.c:486:20: error: assigning to 'struct regmap_config *' from 'const struct regmap_config *const' discards qualifiers [-Werror,-Wincompatible-pointer-types-discards-qualifiers]
           fpga->cpld_config = fpga_data->cpld_config;
                             ^ ~~~~~~~~~~~~~~~~~~~~~~
   1 warning and 1 error generated.


vim +486 drivers/mfd/upboard-fpga.c

   465	
   466	static int __init upboard_fpga_probe(struct platform_device *pdev)
   467	{
   468		struct upboard_fpga *fpga;
   469		const struct acpi_device_id *id;
   470		const struct upboard_fpga_data *fpga_data;
   471		int ret;
   472		struct device *dev = &pdev->dev;
   473	
   474		id = acpi_match_device(upboard_fpga_acpi_match, dev);
   475		if (!id)
   476			return -ENODEV;
   477	
   478		fpga_data = (const struct upboard_fpga_data *) id->driver_data;
   479	
   480		fpga = devm_kzalloc(dev, sizeof(*fpga), GFP_KERNEL);
   481		if (!fpga)
   482			return -ENOMEM;
   483	
   484		platform_set_drvdata(pdev, fpga);
   485		fpga->regmap = devm_regmap_init(dev, NULL, fpga, fpga_data->cpld_config);
 > 486		fpga->cpld_config = fpga_data->cpld_config;
   487	
   488		if (IS_ERR(fpga->regmap))
   489			return PTR_ERR(fpga->regmap);
   490	
   491		ret = upboard_fpga_gpio_init(fpga);
   492		if (ret) {
   493			/*
   494			 * This is for compatiable with some upboards w/o FPGA firmware,
   495			 * so just showing debug info and do not return directly.
   496			 */
   497			dev_warn(dev, "Failed to initialize FPGA common GPIOs: %d", ret);
   498		} else {
   499			upboard_fpga_show_firmware_info(fpga);
   500		}
   501	
   502		/* register GPIO LEDs */
   503		ret = upboard_led_gpio_register(fpga);
   504		if (ret) {
   505			/*
   506			 * This is for compatiable with some upboards w/o LEDs.
   507			 */
   508			dev_warn(dev, "Failed to register LEDs: %d", ret);
   509		}
   510	
   511		return devm_mfd_add_devices(dev, PLATFORM_DEVID_AUTO,
   512					    fpga_data->cells,
   513					    fpga_data->ncells,
   514					    NULL, 0, NULL);
   515	}
   516	

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

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

end of thread, other threads:[~2023-05-02 14:22 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <20230425152135.30745-2-larry.lai@yunjingtech.com>
2023-05-02  4:56 ` [RFC RESEND 1/3] mfd: Add support for UP board CPLD/FPGA kernel test robot
2023-05-02 14:21 ` 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;
as well as URLs for NNTP newsgroup(s).