Subject: Re: [PATCH 2/2] powerpc/powernv: Use common code for the symbol_map export

From: kbuild test robot <[email-masked]>
Date: Fri, 4 Oct 2019 17:23:19 +0800

Hi Oliver,

I love your patch! Yet something to improve:

[auto build test ERROR on powerpc/next]
[cannot apply to v5.4-rc1 next-20191004]
[if your patch is applied to the wrong git tree, please drop us a note to help
improve the system. BTW, we also suggest to use '--base' option to specify the
base tree in git format-patch, please see https://stackoverflow.com/a/37406982]

url:    https://github.com/0day-ci/linux/commits/Oliver-O-Halloran/powerpc-powernv-Rework-exports-to-support-subnodes/20191004-165257
base:   https://git.kernel.org/pub/scm/linux/kernel/git/powerpc/linux.git next
config: powerpc-allyesconfig (attached as .config)
compiler: powerpc64-linux-gcc (GCC) 7.4.0
reproduce:
        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        # save the attached .config to linux build tree
        GCC_VERSION=7.4.0 make.cross ARCH=powerpc 

If you fix the issue, kindly add following tag
Reported-by: kbuild test robot <[email-masked]>

All errors (new ones prefixed by >>):

   arch/powerpc/platforms/powernv/opal.c: In function 'opal_export_attrs':
>> arch/powerpc/platforms/powernv/opal.c:817:2: error: 'rc' undeclared (first use in this function); did you mean 'rq'?
     rc = opal_add_one_export(opal_kobj, "symbol_map",
     ^~
     rq
   arch/powerpc/platforms/powernv/opal.c:817:2: note: each undeclared identifier is reported only once for each function it appears in

vim +817 arch/powerpc/platforms/powernv/opal.c

   787	
   788	/*
   789	 * opal_export_attrs: creates a sysfs node for each property listed in
   790	 * the device-tree under /ibm,opal/firmware/exports/
   791	 * All new sysfs nodes are created under /opal/exports/.
   792	 * This allows for reserved memory regions (e.g. HDAT) to be read.
   793	 * The new sysfs nodes are only readable by root.
   794	 */
   795	static void opal_export_attrs(void)
   796	{
   797		struct device_node *np;
   798		struct kobject *kobj;
   799	
   800		np = of_find_node_by_path("/ibm,opal/firmware/exports");
   801		if (!np)
   802			return;
   803	
   804		/* Create new 'exports' directory - /sys/firmware/opal/exports */
   805		kobj = kobject_create_and_add("exports", opal_kobj);
   806		if (!kobj) {
   807			pr_warn("kobject_create_and_add() of exports failed\n");
   808			return;
   809		}
   810	
   811		opal_add_exported_attrs(np, kobj);
   812	
   813		/*
   814		 * NB: symbol_map existed before the generic export interface so it
   815		 * lives under the top level opal_kobj.
   816		 */
 > 817		rc = opal_add_one_export(opal_kobj, "symbol_map",
   818					 np->parent, "symbol-map");
   819		if (rc)
   820			pr_warn("Error %d creating OPAL symbols file\n", rc);
   821	
   822		of_node_put(np);
   823	}
   824	

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation