From: Bartosz Golaszewski <brgl@bgdev.pl>
To: kbuild-all@lists.01.org
Subject: Re: [PATCH v11 3/6] gpio: sim: new testing module
Date: Wed, 01 Dec 2021 09:59:16 +0100 [thread overview]
Message-ID: <CAMRc=MdWei2REeEeSEovb9P0FLa8i8X5Wn8viSsJWWzWbXtOhA@mail.gmail.com> (raw)
In-Reply-To: <202112011018.3gANFz4h-lkp@intel.com>
[-- Attachment #1: Type: text/plain, Size: 3725 bytes --]
On Wed, Dec 1, 2021 at 3:56 AM kernel test robot <lkp@intel.com> wrote:
>
> Hi Bartosz,
>
> I love your patch! Perhaps something to improve:
>
> [auto build test WARNING on v5.16-rc3]
> [cannot apply to shuah-kselftest/next linusw-gpio/for-next next-20211130]
> [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]
>
> url: https://github.com/0day-ci/linux/commits/Bartosz-Golaszewski/gpio-sim-configfs-based-GPIO-simulator/20211130-234338
> base: d58071a8a76d779eedab38033ae4c821c30295a5
> config: sh-allmodconfig (https://download.01.org/0day-ci/archive/20211201/202112011018.3gANFz4h-lkp(a)intel.com/config)
> compiler: sh4-linux-gcc (GCC) 11.2.0
> 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/0day-ci/linux/commit/1e50c35dae10799b1e2bbed56f68cfbac59bea08
> git remote add linux-review https://github.com/0day-ci/linux
> git fetch --no-tags linux-review Bartosz-Golaszewski/gpio-sim-configfs-based-GPIO-simulator/20211130-234338
> git checkout 1e50c35dae10799b1e2bbed56f68cfbac59bea08
> # save the config file to linux build tree
> mkdir build_dir
> COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.2.0 make.cross O=build_dir ARCH=sh SHELL=/bin/bash drivers/ security//
>
> If you fix the issue, kindly add following tag as appropriate
> Reported-by: kernel test robot <lkp@intel.com>
>
> All warnings (new ones prefixed by >>):
>
> >> drivers/gpio/gpio-sim.c:807:23: warning: no previous prototype for 'gpio_sim_make_bank_swnode' [-Wmissing-prototypes]
> 807 | struct fwnode_handle *gpio_sim_make_bank_swnode(struct gpio_sim_bank *bank,
> | ^~~~~~~~~~~~~~~~~~~~~~~~~
>
>
> vim +/gpio_sim_make_bank_swnode +807 drivers/gpio/gpio-sim.c
>
> 806
> > 807 struct fwnode_handle *gpio_sim_make_bank_swnode(struct gpio_sim_bank *bank,
> 808 struct fwnode_handle *parent)
> 809 {
Will statify it in v12.
Bart
> 810 struct property_entry properties[GPIO_SIM_PROP_MAX];
> 811 unsigned int prop_idx = 0, line_names_size = 0;
> 812 struct fwnode_handle *swnode;
> 813 char **line_names;
> 814
> 815 memset(properties, 0, sizeof(properties));
> 816
> 817 properties[prop_idx++] = PROPERTY_ENTRY_U32("ngpios", bank->num_lines);
> 818
> 819 if (bank->label)
> 820 properties[prop_idx++] = PROPERTY_ENTRY_STRING("gpio-sim,label",
> 821 bank->label);
> 822
> 823 line_names = gpio_sim_make_line_names(bank, &line_names_size);
> 824 if (IS_ERR(line_names))
> 825 return ERR_CAST(line_names);
> 826
> 827 if (line_names)
> 828 properties[prop_idx++] = PROPERTY_ENTRY_STRING_ARRAY_LEN(
> 829 "gpio-line-names",
> 830 line_names, line_names_size);
> 831
> 832 swnode = fwnode_create_software_node(properties, parent);
> 833 kfree(line_names);
> 834 return swnode;
> 835 }
> 836
>
> ---
> 0-DAY CI Kernel Test Service, Intel Corporation
> https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org
next prev parent reply other threads:[~2021-12-01 8:59 UTC|newest]
Thread overview: 34+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-11-30 15:41 [PATCH v11 0/6] gpio-sim: configfs-based GPIO simulator Bartosz Golaszewski
2021-11-30 15:41 ` [PATCH v11 1/6] gpiolib: provide gpiod_remove_hogs() Bartosz Golaszewski
2021-11-30 15:41 ` [PATCH v11 2/6] gpiolib: allow to specify the firmware node in struct gpio_chip Bartosz Golaszewski
2021-11-30 16:14 ` Andy Shevchenko
2021-11-30 16:19 ` Andy Shevchenko
2021-11-30 16:55 ` Andy Shevchenko
2021-11-30 18:32 ` Bartosz Golaszewski
2021-11-30 20:31 ` Andy Shevchenko
2021-11-30 20:25 ` Bartosz Golaszewski
2021-11-30 20:59 ` Andy Shevchenko
2021-11-30 21:04 ` Bartosz Golaszewski
2021-12-01 13:11 ` Bartosz Golaszewski
2021-12-01 13:39 ` Andy Shevchenko
2021-12-01 13:53 ` Bartosz Golaszewski
2021-12-01 14:28 ` Andy Shevchenko
2021-12-01 14:33 ` Andy Shevchenko
2021-12-01 14:36 ` Bartosz Golaszewski
2021-12-01 14:54 ` Andy Shevchenko
2021-12-02 10:57 ` Andy Shevchenko
2021-12-02 11:24 ` Bartosz Golaszewski
2021-12-02 11:35 ` Andy Shevchenko
2021-12-02 11:37 ` Andy Shevchenko
2021-12-02 13:06 ` Bartosz Golaszewski
2021-12-02 13:44 ` Andy Shevchenko
2021-12-02 13:52 ` Bartosz Golaszewski
2021-12-02 15:40 ` Andy Shevchenko
2021-12-02 17:00 ` Bartosz Golaszewski
2021-12-02 17:29 ` Andy Shevchenko
2021-11-30 15:41 ` [PATCH v11 3/6] gpio: sim: new testing module Bartosz Golaszewski
2021-12-01 2:55 ` kernel test robot
2021-12-01 8:59 ` Bartosz Golaszewski [this message]
2021-11-30 15:41 ` [PATCH v11 4/6] selftests: gpio: provide a helper for reading chip info Bartosz Golaszewski
2021-11-30 15:41 ` [PATCH v11 5/6] selftests: gpio: add a helper for reading GPIO line names Bartosz Golaszewski
2021-11-30 15:41 ` [PATCH v11 6/6] selftests: gpio: add test cases for gpio-sim Bartosz Golaszewski
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='CAMRc=MdWei2REeEeSEovb9P0FLa8i8X5Wn8viSsJWWzWbXtOhA@mail.gmail.com' \
--to=brgl@bgdev.pl \
--cc=kbuild-all@lists.01.org \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.