Thread (3 messages) 3 messages, 3 authors, 2021-09-16

Re: [rafael-pm:bleeding-edge 6/11] drivers/base/swnode.c:1120:17: error: too many arguments to function 'software_node_notify'

From: Heikki Krogerus <heikki.krogerus@linux.intel.com>
Date: 2021-09-16 06:58:32
Also in: linux-acpi, oe-kbuild-all
Subsystem: driver core, kobjects, debugfs and sysfs, software nodes and device properties, the rest · Maintainers: Greg Kroah-Hartman, "Rafael J. Wysocki", Danilo Krummrich, Linus Torvalds

On Thu, Sep 16, 2021 at 08:13:24AM +0800, kernel test robot wrote:
tree:   https://git.kernel.org/pub/scm/linux/kernel/git/rafael/linux-pm.git bleeding-edge
head:   9808623e1bba7d929b50865d7e2d7042cd4e5e7b
commit: 810fc8571f8b70a4263266d979f97edec2804574 [6/11] software node: balance refcount for managed sw nodes
config: arc-randconfig-r001-20210916 (attached as .config)
compiler: arc-elf-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://git.kernel.org/pub/scm/linux/kernel/git/rafael/linux-pm.git/commit/?id=810fc8571f8b70a4263266d979f97edec2804574
        git remote add rafael-pm https://git.kernel.org/pub/scm/linux/kernel/git/rafael/linux-pm.git
        git fetch --no-tags rafael-pm bleeding-edge
        git checkout 810fc8571f8b70a4263266d979f97edec2804574
        # save the attached .config to linux build tree
        mkdir build_dir
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.2.0 make.cross O=build_dir ARCH=arc SHELL=/bin/bash drivers/

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <redacted>

All errors (new ones prefixed by >>):

   drivers/base/swnode.c: In function 'device_create_managed_software_node':
quoted
quoted
drivers/base/swnode.c:1120:17: error: too many arguments to function 'software_node_notify'
    1120 |                 software_node_notify(dev, KOBJ_ADD);
         |                 ^~~~~~~~~~~~~~~~~~~~
   In file included from drivers/base/swnode.c:14:
   drivers/base/base.h:206:6: note: declared here
     206 | void software_node_notify(struct device *dev);
         |      ^~~~~~~~~~~~~~~~~~~~
Ah, that was modified in commit 384f5a857bae ("software nodes: Split
software_node_notify()").

So the patch is OK for stable, but for v5.15-rc1 it has to be changed:
diff --git a/drivers/base/swnode.c b/drivers/base/swnode.c
index 7bd0f3cfb7eb4..c46f6a8e14d23 100644
--- a/drivers/base/swnode.c
+++ b/drivers/base/swnode.c
@@ -1116,6 +1116,9 @@ int device_create_managed_software_node(struct device *dev,
        to_swnode(fwnode)->managed = true;
        set_secondary_fwnode(dev, fwnode);
 
+       if (device_is_registered(dev))
+               software_node_notify(dev);
+
        return 0;
 }
 EXPORT_SYMBOL_GPL(device_create_managed_software_node);

Rafael, how do want to handle this?

-- 
heikki
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help