Re: [PATCH v7 3/7] powerpc/pseries: Add papr-indices char driver for ibm,get-indices

Hi Haren,

kernel test robot noticed the following build warnings:

[auto build test WARNING on powerpc/next]
[also build test WARNING on powerpc/fixes linus/master v6.14-rc6]
[cannot apply to next-20250307]
[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/Haren-Myneni/powerpc-pseries-Define-common-functions-for-RTAS-sequence-calls/20250310-054319
base:   https://git.kernel.org/pub/scm/linux/kernel/git/powerpc/linux.git next
patch link:    https://lore.kernel.org/r/20250309213916.762116-4-haren%40linux.ibm.com
patch subject: [PATCH v7 3/7] powerpc/pseries: Add papr-indices char driver for ibm,get-indices
config: powerpc64-randconfig-r072-20250311 (https://download.01.org/0day-ci/archive/20250311/202503111710.tsPk3Snj-lkp@intel.com/config)
compiler: clang version 21.0.0git (https://github.com/llvm/llvm-project e15545cad8297ec7555f26e5ae74a9f0511203e7)
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20250311/202503111710.tsPk3Snj-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 [off-list ref]
| Closes: https://lore.kernel.org/oe-kbuild-all/202503111710.tsPk3Snj-lkp@intel.com/ (local)

All warnings (new ones prefixed by >>):

quoted

arch/powerpc/platforms/pseries/papr-indices.c:124: warning: Function parameter or struct member 'seq' not described in 'indices_sequence_begin'

vim +124 arch/powerpc/platforms/pseries/papr-indices.c

   110	
   111	/*
   112	 * Internal indices sequence APIs. A sequence is a series of calls to
   113	 * ibm,get-indices for a given location code. The sequence ends when
   114	 * an error is encountered or all indices for the input has been
   115	 * returned.
   116	 */
   117	
   118	/**
   119	 * indices_sequence_begin() - Begin a indices retrieval sequence.
   120	 *
   121	 * Context: May sleep.
   122	 */
   123	static void indices_sequence_begin(struct papr_rtas_sequence *seq)
 > 124	{
   125		struct rtas_get_indices_params  *param;
   126	
   127		param = (struct rtas_get_indices_params *)seq->params;
   128		/*
   129		 * We could allocate the work area before acquiring the
   130		 * function lock, but that would allow concurrent requests to
   131		 * exhaust the limited work area pool for no benefit. So
   132		 * allocate the work area under the lock.
   133		 */
   134		mutex_lock(&rtas_ibm_get_indices_lock);
   135		param->work_area = rtas_work_area_alloc(RTAS_GET_INDICES_BUF_SIZE);
   136		param->next = 1;
   137		param->status = 0;
   138	}
   139	

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