All of lore.kernel.org
 help / color / mirror / Atom feed
* [avpatel:riscv_trace_acpi_support_v1 85/87] drivers/hwtracing/gtrace/gtrace-fwnode.c:17:5: warning: no previous prototype for function 'gtrace_of_parse_outconns'
@ 2026-10-02  2:51 kernel test robot
  0 siblings, 0 replies; only message in thread
From: kernel test robot @ 2026-10-02  2:51 UTC (permalink / raw)
  To: Sunil V L; +Cc: oe-kbuild-all, Anup Patel

tree:   https://github.com/avpatel/linux.git riscv_trace_acpi_support_v1
head:   dc5b9586f90cc15314085bdaa74661fc00d1688f
commit: 5c381ad05935a6c1e0108a97d1197952ca46201e [85/87] hwtracing: rvtrace: Move to fwnode_* APIs
config: sparc64-randconfig-r2106-20261002 (https://download.01.org/0day-ci/archive/20261002/202610020432.Z1GpmPkW-lkp@intel.com/config)
compiler: clang version 20.1.8 (https://github.com/llvm/llvm-project 87f0227cb60147a26a1eeb4fb06e3b505e9c7261)
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20261002/202610020432.Z1GpmPkW-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 <lkp@intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202610020432.Z1GpmPkW-lkp@intel.com/

All warnings (new ones prefixed by >>):

>> drivers/hwtracing/gtrace/gtrace-fwnode.c:17:5: warning: no previous prototype for function 'gtrace_of_parse_outconns' [-Wmissing-prototypes]
      17 | int gtrace_of_parse_outconns(struct gtrace_platform_data *pdata)
         |     ^
   drivers/hwtracing/gtrace/gtrace-fwnode.c:17:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
      17 | int gtrace_of_parse_outconns(struct gtrace_platform_data *pdata)
         | ^
         | static 
>> drivers/hwtracing/gtrace/gtrace-fwnode.c:105:5: warning: no previous prototype for function 'rvtrace_parse_inconns' [-Wmissing-prototypes]
     105 | int rvtrace_parse_inconns(struct gtrace_platform_data *pdata)
         |     ^
   drivers/hwtracing/gtrace/gtrace-fwnode.c:105:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
     105 | int rvtrace_parse_inconns(struct gtrace_platform_data *pdata)
         | ^
         | static 
   2 warnings generated.


vim +/gtrace_of_parse_outconns +17 drivers/hwtracing/gtrace/gtrace-fwnode.c

2533ea70f41f2e drivers/hwtracing/gtrace/gtrace-of.c     Anup Patel 2026-10-01   10  
2533ea70f41f2e drivers/hwtracing/gtrace/gtrace-of.c     Anup Patel 2026-10-01   11  /*
5c381ad05935a6 drivers/hwtracing/gtrace/gtrace-fwnode.c Sunil V L  2026-10-01   12   * Parse the "out-ports" graph of the component node and fill pdata->outconns.
2533ea70f41f2e drivers/hwtracing/gtrace/gtrace-of.c     Anup Patel 2026-10-01   13   * Return: 0 on success or when there are no output ports
2533ea70f41f2e drivers/hwtracing/gtrace/gtrace-of.c     Anup Patel 2026-10-01   14   *         -EPROBE_DEFER if a destination is not registered yet
2533ea70f41f2e drivers/hwtracing/gtrace/gtrace-of.c     Anup Patel 2026-10-01   15   *         negative error code otherwise.
2533ea70f41f2e drivers/hwtracing/gtrace/gtrace-of.c     Anup Patel 2026-10-01   16   */
2533ea70f41f2e drivers/hwtracing/gtrace/gtrace-of.c     Anup Patel 2026-10-01  @17  int gtrace_of_parse_outconns(struct gtrace_platform_data *pdata)
2533ea70f41f2e drivers/hwtracing/gtrace/gtrace-of.c     Anup Patel 2026-10-01   18  {
5c381ad05935a6 drivers/hwtracing/gtrace/gtrace-fwnode.c Sunil V L  2026-10-01   19  	struct fwnode_handle *parent, *ep_node, *rep_node, *rdev_node;
5c381ad05935a6 drivers/hwtracing/gtrace/gtrace-fwnode.c Sunil V L  2026-10-01   20  	struct fwnode_endpoint ep = { 0 };
5c381ad05935a6 drivers/hwtracing/gtrace/gtrace-fwnode.c Sunil V L  2026-10-01   21  	struct fwnode_endpoint rep = { 0 };
2533ea70f41f2e drivers/hwtracing/gtrace/gtrace-of.c     Anup Patel 2026-10-01   22  	struct gtrace_connection *conn;
2533ea70f41f2e drivers/hwtracing/gtrace/gtrace-of.c     Anup Patel 2026-10-01   23  	int ret = 0, i = 0;
2533ea70f41f2e drivers/hwtracing/gtrace/gtrace-of.c     Anup Patel 2026-10-01   24  
5c381ad05935a6 drivers/hwtracing/gtrace/gtrace-fwnode.c Sunil V L  2026-10-01   25  	parent = fwnode_get_named_child_node(dev_fwnode(pdata->dev), "out-ports");
2533ea70f41f2e drivers/hwtracing/gtrace/gtrace-of.c     Anup Patel 2026-10-01   26  	if (!parent)
2533ea70f41f2e drivers/hwtracing/gtrace/gtrace-of.c     Anup Patel 2026-10-01   27  		return 0;
2533ea70f41f2e drivers/hwtracing/gtrace/gtrace-of.c     Anup Patel 2026-10-01   28  
5c381ad05935a6 drivers/hwtracing/gtrace/gtrace-fwnode.c Sunil V L  2026-10-01   29  	pdata->nr_outconns = fwnode_graph_get_endpoint_count(parent, 0);
2533ea70f41f2e drivers/hwtracing/gtrace/gtrace-of.c     Anup Patel 2026-10-01   30  	pdata->outconns = devm_kcalloc(pdata->dev, pdata->nr_outconns,
2533ea70f41f2e drivers/hwtracing/gtrace/gtrace-of.c     Anup Patel 2026-10-01   31  				       sizeof(*pdata->outconns), GFP_KERNEL);
2533ea70f41f2e drivers/hwtracing/gtrace/gtrace-of.c     Anup Patel 2026-10-01   32  	if (!pdata->outconns) {
2533ea70f41f2e drivers/hwtracing/gtrace/gtrace-of.c     Anup Patel 2026-10-01   33  		ret = -ENOMEM;
2533ea70f41f2e drivers/hwtracing/gtrace/gtrace-of.c     Anup Patel 2026-10-01   34  		goto done;
2533ea70f41f2e drivers/hwtracing/gtrace/gtrace-of.c     Anup Patel 2026-10-01   35  	}
2533ea70f41f2e drivers/hwtracing/gtrace/gtrace-of.c     Anup Patel 2026-10-01   36  
5c381ad05935a6 drivers/hwtracing/gtrace/gtrace-fwnode.c Sunil V L  2026-10-01   37  	fwnode_graph_for_each_endpoint(parent, ep_node) {
2533ea70f41f2e drivers/hwtracing/gtrace/gtrace-of.c     Anup Patel 2026-10-01   38  		conn = devm_kzalloc(pdata->dev, sizeof(*conn), GFP_KERNEL);
2533ea70f41f2e drivers/hwtracing/gtrace/gtrace-of.c     Anup Patel 2026-10-01   39  		if (!conn) {
5c381ad05935a6 drivers/hwtracing/gtrace/gtrace-fwnode.c Sunil V L  2026-10-01   40  			fwnode_handle_put(ep_node);
2533ea70f41f2e drivers/hwtracing/gtrace/gtrace-of.c     Anup Patel 2026-10-01   41  			ret = -ENOMEM;
2533ea70f41f2e drivers/hwtracing/gtrace/gtrace-of.c     Anup Patel 2026-10-01   42  			break;
2533ea70f41f2e drivers/hwtracing/gtrace/gtrace-of.c     Anup Patel 2026-10-01   43  		}
2533ea70f41f2e drivers/hwtracing/gtrace/gtrace-of.c     Anup Patel 2026-10-01   44  
5c381ad05935a6 drivers/hwtracing/gtrace/gtrace-fwnode.c Sunil V L  2026-10-01   45  		ret = fwnode_graph_parse_endpoint(ep_node, &ep);
2533ea70f41f2e drivers/hwtracing/gtrace/gtrace-of.c     Anup Patel 2026-10-01   46  		if (ret) {
5c381ad05935a6 drivers/hwtracing/gtrace/gtrace-fwnode.c Sunil V L  2026-10-01   47  			fwnode_handle_put(ep_node);
2533ea70f41f2e drivers/hwtracing/gtrace/gtrace-of.c     Anup Patel 2026-10-01   48  			break;
2533ea70f41f2e drivers/hwtracing/gtrace/gtrace-of.c     Anup Patel 2026-10-01   49  		}
2533ea70f41f2e drivers/hwtracing/gtrace/gtrace-of.c     Anup Patel 2026-10-01   50  
5c381ad05935a6 drivers/hwtracing/gtrace/gtrace-fwnode.c Sunil V L  2026-10-01   51  		rep_node = fwnode_graph_get_remote_endpoint(ep_node);
2533ea70f41f2e drivers/hwtracing/gtrace/gtrace-of.c     Anup Patel 2026-10-01   52  		if (!rep_node) {
2533ea70f41f2e drivers/hwtracing/gtrace/gtrace-of.c     Anup Patel 2026-10-01   53  			ret = -ENODEV;
5c381ad05935a6 drivers/hwtracing/gtrace/gtrace-fwnode.c Sunil V L  2026-10-01   54  			fwnode_handle_put(ep_node);
2533ea70f41f2e drivers/hwtracing/gtrace/gtrace-of.c     Anup Patel 2026-10-01   55  			break;
2533ea70f41f2e drivers/hwtracing/gtrace/gtrace-of.c     Anup Patel 2026-10-01   56  		}
5c381ad05935a6 drivers/hwtracing/gtrace/gtrace-fwnode.c Sunil V L  2026-10-01   57  		rdev_node = fwnode_graph_get_port_parent(rep_node);
2533ea70f41f2e drivers/hwtracing/gtrace/gtrace-of.c     Anup Patel 2026-10-01   58  
5c381ad05935a6 drivers/hwtracing/gtrace/gtrace-fwnode.c Sunil V L  2026-10-01   59  		ret = fwnode_graph_parse_endpoint(rep_node, &rep);
2533ea70f41f2e drivers/hwtracing/gtrace/gtrace-of.c     Anup Patel 2026-10-01   60  		if (ret) {
5c381ad05935a6 drivers/hwtracing/gtrace/gtrace-fwnode.c Sunil V L  2026-10-01   61  			fwnode_handle_put(ep_node);
5c381ad05935a6 drivers/hwtracing/gtrace/gtrace-fwnode.c Sunil V L  2026-10-01   62  			fwnode_handle_put(rep_node);
5c381ad05935a6 drivers/hwtracing/gtrace/gtrace-fwnode.c Sunil V L  2026-10-01   63  			fwnode_handle_put(rdev_node);
2533ea70f41f2e drivers/hwtracing/gtrace/gtrace-of.c     Anup Patel 2026-10-01   64  			break;
2533ea70f41f2e drivers/hwtracing/gtrace/gtrace-of.c     Anup Patel 2026-10-01   65  		}
2533ea70f41f2e drivers/hwtracing/gtrace/gtrace-of.c     Anup Patel 2026-10-01   66  
2533ea70f41f2e drivers/hwtracing/gtrace/gtrace-of.c     Anup Patel 2026-10-01   67  		conn->src_port = ep.port;
2533ea70f41f2e drivers/hwtracing/gtrace/gtrace-of.c     Anup Patel 2026-10-01   68  		conn->src_fwnode = dev_fwnode(pdata->dev);
2533ea70f41f2e drivers/hwtracing/gtrace/gtrace-of.c     Anup Patel 2026-10-01   69  		/* The 'src_comp' is set by gtrace_register_component() */
2533ea70f41f2e drivers/hwtracing/gtrace/gtrace-of.c     Anup Patel 2026-10-01   70  		conn->src_comp = NULL;
2533ea70f41f2e drivers/hwtracing/gtrace/gtrace-of.c     Anup Patel 2026-10-01   71  		conn->dest_port = rep.port;
5c381ad05935a6 drivers/hwtracing/gtrace/gtrace-fwnode.c Sunil V L  2026-10-01   72  		conn->dest_fwnode = rdev_node;
2533ea70f41f2e drivers/hwtracing/gtrace/gtrace-of.c     Anup Patel 2026-10-01   73  		fwnode_handle_get(conn->dest_fwnode);
2533ea70f41f2e drivers/hwtracing/gtrace/gtrace-of.c     Anup Patel 2026-10-01   74  		conn->dest_comp = gtrace_find_by_fwnode(conn->dest_fwnode);
2533ea70f41f2e drivers/hwtracing/gtrace/gtrace-of.c     Anup Patel 2026-10-01   75  		if (!conn->dest_comp) {
2533ea70f41f2e drivers/hwtracing/gtrace/gtrace-of.c     Anup Patel 2026-10-01   76  			ret = -EPROBE_DEFER;
5c381ad05935a6 drivers/hwtracing/gtrace/gtrace-fwnode.c Sunil V L  2026-10-01   77  			fwnode_handle_put(ep_node);
5c381ad05935a6 drivers/hwtracing/gtrace/gtrace-fwnode.c Sunil V L  2026-10-01   78  			fwnode_handle_put(rep_node);
5c381ad05935a6 drivers/hwtracing/gtrace/gtrace-fwnode.c Sunil V L  2026-10-01   79  			fwnode_handle_put(rdev_node);
2533ea70f41f2e drivers/hwtracing/gtrace/gtrace-of.c     Anup Patel 2026-10-01   80  			break;
2533ea70f41f2e drivers/hwtracing/gtrace/gtrace-of.c     Anup Patel 2026-10-01   81  		}
2533ea70f41f2e drivers/hwtracing/gtrace/gtrace-of.c     Anup Patel 2026-10-01   82  
2533ea70f41f2e drivers/hwtracing/gtrace/gtrace-of.c     Anup Patel 2026-10-01   83  		pdata->outconns[i] = conn;
2533ea70f41f2e drivers/hwtracing/gtrace/gtrace-of.c     Anup Patel 2026-10-01   84  		i++;
2533ea70f41f2e drivers/hwtracing/gtrace/gtrace-of.c     Anup Patel 2026-10-01   85  	}
2533ea70f41f2e drivers/hwtracing/gtrace/gtrace-of.c     Anup Patel 2026-10-01   86  
2533ea70f41f2e drivers/hwtracing/gtrace/gtrace-of.c     Anup Patel 2026-10-01   87  done:
2533ea70f41f2e drivers/hwtracing/gtrace/gtrace-of.c     Anup Patel 2026-10-01   88  	if (ret) {
2533ea70f41f2e drivers/hwtracing/gtrace/gtrace-of.c     Anup Patel 2026-10-01   89  		for (i = 0; i < pdata->nr_outconns && pdata->outconns; i++) {
2533ea70f41f2e drivers/hwtracing/gtrace/gtrace-of.c     Anup Patel 2026-10-01   90  			conn = pdata->outconns[i];
2533ea70f41f2e drivers/hwtracing/gtrace/gtrace-of.c     Anup Patel 2026-10-01   91  			if (conn && conn->dest_fwnode)
2533ea70f41f2e drivers/hwtracing/gtrace/gtrace-of.c     Anup Patel 2026-10-01   92  				fwnode_handle_put(conn->dest_fwnode);
2533ea70f41f2e drivers/hwtracing/gtrace/gtrace-of.c     Anup Patel 2026-10-01   93  		}
2533ea70f41f2e drivers/hwtracing/gtrace/gtrace-of.c     Anup Patel 2026-10-01   94  	}
5c381ad05935a6 drivers/hwtracing/gtrace/gtrace-fwnode.c Sunil V L  2026-10-01   95  	fwnode_handle_put(parent);
2533ea70f41f2e drivers/hwtracing/gtrace/gtrace-of.c     Anup Patel 2026-10-01   96  	return ret;
2533ea70f41f2e drivers/hwtracing/gtrace/gtrace-of.c     Anup Patel 2026-10-01   97  }
5c381ad05935a6 drivers/hwtracing/gtrace/gtrace-fwnode.c Sunil V L  2026-10-01   98  EXPORT_SYMBOL_GPL(gtrace_parse_outconns);
2533ea70f41f2e drivers/hwtracing/gtrace/gtrace-of.c     Anup Patel 2026-10-01   99  
2533ea70f41f2e drivers/hwtracing/gtrace/gtrace-of.c     Anup Patel 2026-10-01  100  /*
5c381ad05935a6 drivers/hwtracing/gtrace/gtrace-fwnode.c Sunil V L  2026-10-01  101   * Parse the "out-ports" graph of the component node and allocate pdata->inconns.
2533ea70f41f2e drivers/hwtracing/gtrace/gtrace-of.c     Anup Patel 2026-10-01  102   *
2533ea70f41f2e drivers/hwtracing/gtrace/gtrace-of.c     Anup Patel 2026-10-01  103   * Return: 0 on success or when there are no input ports, -ENOMEM otherwise.
2533ea70f41f2e drivers/hwtracing/gtrace/gtrace-of.c     Anup Patel 2026-10-01  104   */
5c381ad05935a6 drivers/hwtracing/gtrace/gtrace-fwnode.c Sunil V L  2026-10-01 @105  int rvtrace_parse_inconns(struct gtrace_platform_data *pdata)

:::::: The code at line 17 was first introduced by commit
:::::: 2533ea70f41f2e1b6534c0bdfc90e83dad7bbf39 gtrace: Add RISC-V platform driver for the gtrace framework

:::::: TO: Anup Patel <anup.patel@oss.qualcomm.com>
:::::: CC: Anup Patel <anup@brainfault.org>

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

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2026-10-02  2:52 UTC | newest]

Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-10-02  2:51 [avpatel:riscv_trace_acpi_support_v1 85/87] drivers/hwtracing/gtrace/gtrace-fwnode.c:17:5: warning: no previous prototype for function 'gtrace_of_parse_outconns' kernel test robot

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.