* [android-common:android-mainline 1/6] drivers/tty/serial/msm_geni_serial_earlycon.c:307:6: warning: variable 'rx_stale' set but not used
@ 2020-10-02 2:14 kernel test robot
0 siblings, 0 replies; only message in thread
From: kernel test robot @ 2020-10-02 2:14 UTC (permalink / raw)
To: kbuild-all
[-- Attachment #1: Type: text/plain, Size: 3741 bytes --]
tree: https://android.googlesource.com/kernel/common android-mainline
head: a44b31f34c58e6e9dca649a262e0c9986e757ed8
commit: e1be87019fc97f736f2729a2a4198d250d28633b [1/6] ANDROID: serial: msm_geni_serial_earlycon: Add Earlycon support
config: i386-allyesconfig (attached as .config)
compiler: gcc-9 (Debian 9.3.0-15) 9.3.0
reproduce (this is a W=1 build):
git remote add android-common https://android.googlesource.com/kernel/common
git fetch --no-tags android-common android-mainline
git checkout e1be87019fc97f736f2729a2a4198d250d28633b
# save the attached .config to linux build tree
make W=1 ARCH=i386
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>
All warnings (new ones prefixed by >>):
drivers/tty/serial/msm_geni_serial_earlycon.c: In function 'msm_geni_serial_earlycon_setup':
>> drivers/tty/serial/msm_geni_serial_earlycon.c:307:6: warning: variable 'rx_stale' set but not used [-Wunused-but-set-variable]
307 | u32 rx_stale = 0;
| ^~~~~~~~
>> drivers/tty/serial/msm_geni_serial_earlycon.c:305:6: warning: variable 'rx_parity_cfg' set but not used [-Wunused-but-set-variable]
305 | u32 rx_parity_cfg = 0;
| ^~~~~~~~~~~~~
>> drivers/tty/serial/msm_geni_serial_earlycon.c:304:6: warning: variable 'rx_trans_cfg' set but not used [-Wunused-but-set-variable]
304 | u32 rx_trans_cfg = 0;
| ^~~~~~~~~~~~
vim +/rx_stale +307 drivers/tty/serial/msm_geni_serial_earlycon.c
296
297 static int __init
298 msm_geni_serial_earlycon_setup(struct earlycon_device *dev,
299 const char *opt)
300 {
301 int ret = 0;
302 u32 tx_trans_cfg = 0;
303 u32 tx_parity_cfg = 0;
> 304 u32 rx_trans_cfg = 0;
> 305 u32 rx_parity_cfg = 0;
306 u32 stop_bit = 0;
> 307 u32 rx_stale = 0;
308 u32 bits_per_char = 0;
309 unsigned long cfg0, cfg1;
310 struct uart_port *uport = &dev->port;
311
312 if (!uport->membase) {
313 ret = -ENOMEM;
314 goto exit;
315 }
316
317 if (get_se_proto_earlycon(uport->membase) != UART_PROTOCOL) {
318 ret = -ENXIO;
319 goto exit;
320 }
321
322 /*
323 * Ignore Flow control.
324 * Disable Tx Parity.
325 * Don't check Parity during Rx.
326 * Disable Rx Parity.
327 * n = 8.
328 * Stop bit = 0.
329 * Stale timeout in bit-time (3 chars worth).
330 */
331 tx_trans_cfg |= UART_CTS_MASK;
332 tx_parity_cfg = 0;
333 rx_trans_cfg = 0;
334 rx_parity_cfg = 0;
335 bits_per_char = 0x8;
336 stop_bit = 0;
337 rx_stale = 0x18;
338
339 msm_geni_serial_poll_abort_tx(uport);
340
341 se_get_packing_config_earlycon(8, 1, false, &cfg0, &cfg1);
342
343 se_io_init_earlycon(uport->membase);
344
345 geni_se_select_fifo_mode_earlycon(uport->membase);
346 writel_relaxed(cfg0, uport->membase + SE_GENI_TX_PACKING_CFG0);
347 writel_relaxed(cfg1, uport->membase + SE_GENI_TX_PACKING_CFG1);
348 writel_relaxed(tx_trans_cfg, uport->membase + SE_UART_TX_TRANS_CFG);
349 writel_relaxed(tx_parity_cfg, uport->membase + SE_UART_TX_PARITY_CFG);
350 writel_relaxed(bits_per_char, uport->membase + SE_UART_TX_WORD_LEN);
351 writel_relaxed(stop_bit, uport->membase + SE_UART_TX_STOP_BIT_LEN);
352
353 dev->con->write = msm_geni_serial_early_console_write;
354 dev->con->setup = NULL;
355 exit:
356 return ret;
357 }
358
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org
[-- Attachment #2: config.gz --]
[-- Type: application/gzip, Size: 74974 bytes --]
^ permalink raw reply [flat|nested] only message in thread
only message in thread, other threads:[~2020-10-02 2:14 UTC | newest]
Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-10-02 2:14 [android-common:android-mainline 1/6] drivers/tty/serial/msm_geni_serial_earlycon.c:307:6: warning: variable 'rx_stale' set but not used 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.