[android-common:android16-6.12-kminext 1/1] drivers/remoteproc/da8xx_remoteproc.c:77: warning: Function parameter or struct member 'dsp_reset' not described in 'da8xx_rproc'
kernel test robot <[email protected]>
| Newsgroups | dev.linux.lists.oe-kbuild-all |
|---|---|
| Message-ID | <[email protected]> |
tree: https://android.googlesource.com/kernel/common android16-6.12-kminext head: 7273d1bead29045f912c5442899d11b6075869bf commit: dfc80ed249b618f2cf48481adc8aff0d8ef444b4 [1/1] ARM: davinci: da850: fix selecting ARCH_DAVINCI_DA8XX config: arm-randconfig-001-20260826 (https://download.01.org/0day-ci/archive/20260826/[email protected]/config) compiler: clang version 18.1.8 (https://github.com/llvm/llvm-project 3b5b5c1ec4a3095ab096dd780e84d7ab81f3d7ff) reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20260826/[email protected]/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 <[email protected]> | Closes: https://lore.kernel.org/oe-kbuild-all/[email protected]/ All warnings (new ones prefixed by >>): >> drivers/remoteproc/da8xx_remoteproc.c:77: warning: Function parameter or struct member 'dsp_reset' not described in 'da8xx_rproc' >> drivers/remoteproc/da8xx_remoteproc.c:86: warning: Function parameter or struct member 'irq' not described in 'handle_event' >> drivers/remoteproc/da8xx_remoteproc.c:86: warning: Function parameter or struct member 'p' not described in 'handle_event' >> drivers/remoteproc/da8xx_remoteproc.c:106: warning: Function parameter or struct member 'irq' not described in 'da8xx_rproc_callback' >> drivers/remoteproc/da8xx_remoteproc.c:106: warning: Function parameter or struct member 'p' not described in 'da8xx_rproc_callback' vim +77 drivers/remoteproc/da8xx_remoteproc.c 59b2355fc90e4a4 Suman Anna 2017-08-01 53 13be5432d8721d8 Robert Tivy 2013-04-09 54 /** 13be5432d8721d8 Robert Tivy 2013-04-09 55 * struct da8xx_rproc - da8xx remote processor instance state 13be5432d8721d8 Robert Tivy 2013-04-09 56 * @rproc: rproc handle 59b2355fc90e4a4 Suman Anna 2017-08-01 57 * @mem: internal memory regions data 59b2355fc90e4a4 Suman Anna 2017-08-01 58 * @num_mems: number of internal memory regions 13be5432d8721d8 Robert Tivy 2013-04-09 59 * @dsp_clk: placeholder for platform's DSP clk 13be5432d8721d8 Robert Tivy 2013-04-09 60 * @ack_fxn: chip-specific ack function for ack'ing irq 13be5432d8721d8 Robert Tivy 2013-04-09 61 * @irq_data: ack_fxn function parameter 13be5432d8721d8 Robert Tivy 2013-04-09 62 * @chipsig: virt ptr to DSP interrupt registers (CHIPSIG & CHIPSIG_CLR) 13be5432d8721d8 Robert Tivy 2013-04-09 63 * @bootreg: virt ptr to DSP boot address register (HOST1CFG) 13be5432d8721d8 Robert Tivy 2013-04-09 64 * @irq: irq # used by this instance 13be5432d8721d8 Robert Tivy 2013-04-09 65 */ 13be5432d8721d8 Robert Tivy 2013-04-09 66 struct da8xx_rproc { 13be5432d8721d8 Robert Tivy 2013-04-09 67 struct rproc *rproc; 59b2355fc90e4a4 Suman Anna 2017-08-01 68 struct da8xx_rproc_mem *mem; 59b2355fc90e4a4 Suman Anna 2017-08-01 69 int num_mems; 13be5432d8721d8 Robert Tivy 2013-04-09 70 struct clk *dsp_clk; b2201ee554a5811 Bartosz Golaszewski 2018-06-21 71 struct reset_control *dsp_reset; 13be5432d8721d8 Robert Tivy 2013-04-09 72 void (*ack_fxn)(struct irq_data *data); 13be5432d8721d8 Robert Tivy 2013-04-09 73 struct irq_data *irq_data; 13be5432d8721d8 Robert Tivy 2013-04-09 74 void __iomem *chipsig; 13be5432d8721d8 Robert Tivy 2013-04-09 75 void __iomem *bootreg; 13be5432d8721d8 Robert Tivy 2013-04-09 76 int irq; 13be5432d8721d8 Robert Tivy 2013-04-09 @77 }; 13be5432d8721d8 Robert Tivy 2013-04-09 78 13be5432d8721d8 Robert Tivy 2013-04-09 79 /** 13be5432d8721d8 Robert Tivy 2013-04-09 80 * handle_event() - inbound virtqueue message workqueue function 13be5432d8721d8 Robert Tivy 2013-04-09 81 * 13be5432d8721d8 Robert Tivy 2013-04-09 82 * This function is registered as a kernel thread and is scheduled by the 13be5432d8721d8 Robert Tivy 2013-04-09 83 * kernel handler. 13be5432d8721d8 Robert Tivy 2013-04-09 84 */ 13be5432d8721d8 Robert Tivy 2013-04-09 85 static irqreturn_t handle_event(int irq, void *p) 13be5432d8721d8 Robert Tivy 2013-04-09 @86 { 8666071391f473b Yu Zhe 2023-03-28 87 struct rproc *rproc = p; 13be5432d8721d8 Robert Tivy 2013-04-09 88 13be5432d8721d8 Robert Tivy 2013-04-09 89 /* Process incoming buffers on all our vrings */ 13be5432d8721d8 Robert Tivy 2013-04-09 90 rproc_vq_interrupt(rproc, 0); 13be5432d8721d8 Robert Tivy 2013-04-09 91 rproc_vq_interrupt(rproc, 1); 13be5432d8721d8 Robert Tivy 2013-04-09 92 13be5432d8721d8 Robert Tivy 2013-04-09 93 return IRQ_HANDLED; 13be5432d8721d8 Robert Tivy 2013-04-09 94 } 13be5432d8721d8 Robert Tivy 2013-04-09 95 13be5432d8721d8 Robert Tivy 2013-04-09 96 /** 13be5432d8721d8 Robert Tivy 2013-04-09 97 * da8xx_rproc_callback() - inbound virtqueue message handler 13be5432d8721d8 Robert Tivy 2013-04-09 98 * 13be5432d8721d8 Robert Tivy 2013-04-09 99 * This handler is invoked directly by the kernel whenever the remote 13be5432d8721d8 Robert Tivy 2013-04-09 100 * core (DSP) has modified the state of a virtqueue. There is no 13be5432d8721d8 Robert Tivy 2013-04-09 101 * "payload" message indicating the virtqueue index as is the case with 13be5432d8721d8 Robert Tivy 2013-04-09 102 * mailbox-based implementations on OMAP4. As such, this handler "polls" 13be5432d8721d8 Robert Tivy 2013-04-09 103 * each known virtqueue index for every invocation. 13be5432d8721d8 Robert Tivy 2013-04-09 104 */ 13be5432d8721d8 Robert Tivy 2013-04-09 105 static irqreturn_t da8xx_rproc_callback(int irq, void *p) 13be5432d8721d8 Robert Tivy 2013-04-09 @106 { 8666071391f473b Yu Zhe 2023-03-28 107 struct rproc *rproc = p; 8666071391f473b Yu Zhe 2023-03-28 108 struct da8xx_rproc *drproc = rproc->priv; 13be5432d8721d8 Robert Tivy 2013-04-09 109 u32 chipsig; 13be5432d8721d8 Robert Tivy 2013-04-09 110 13be5432d8721d8 Robert Tivy 2013-04-09 111 chipsig = readl(drproc->chipsig); 13be5432d8721d8 Robert Tivy 2013-04-09 112 if (chipsig & SYSCFG_CHIPSIG0) { 13be5432d8721d8 Robert Tivy 2013-04-09 113 /* Clear interrupt level source */ 13be5432d8721d8 Robert Tivy 2013-04-09 114 writel(SYSCFG_CHIPSIG0, drproc->chipsig + 4); 13be5432d8721d8 Robert Tivy 2013-04-09 115 13be5432d8721d8 Robert Tivy 2013-04-09 116 /* 13be5432d8721d8 Robert Tivy 2013-04-09 117 * ACK intr to AINTC. 13be5432d8721d8 Robert Tivy 2013-04-09 118 * 13be5432d8721d8 Robert Tivy 2013-04-09 119 * It has already been ack'ed by the kernel before calling 13be5432d8721d8 Robert Tivy 2013-04-09 120 * this function, but since the ARM<->DSP interrupts in the 13be5432d8721d8 Robert Tivy 2013-04-09 121 * CHIPSIG register are "level" instead of "pulse" variety, 13be5432d8721d8 Robert Tivy 2013-04-09 122 * we need to ack it after taking down the level else we'll 13be5432d8721d8 Robert Tivy 2013-04-09 123 * be called again immediately after returning. 13be5432d8721d8 Robert Tivy 2013-04-09 124 */ 13be5432d8721d8 Robert Tivy 2013-04-09 125 drproc->ack_fxn(drproc->irq_data); 13be5432d8721d8 Robert Tivy 2013-04-09 126 13be5432d8721d8 Robert Tivy 2013-04-09 127 return IRQ_WAKE_THREAD; 13be5432d8721d8 Robert Tivy 2013-04-09 128 } 13be5432d8721d8 Robert Tivy 2013-04-09 129 13be5432d8721d8 Robert Tivy 2013-04-09 130 return IRQ_HANDLED; 13be5432d8721d8 Robert Tivy 2013-04-09 131 } 13be5432d8721d8 Robert Tivy 2013-04-09 132 :::::: The code at line 77 was first introduced by commit :::::: 13be5432d8721d89cadae105663761f45f427842 remoteproc/davinci: add a remoteproc driver for OMAP-L13x DSP :::::: TO: Robert Tivy <[email protected]> :::::: CC: Ohad Ben-Cohen <[email protected]> -- 0-DAY CI Kernel Test Service https://github.com/intel/lkp-tests/wiki