From: kernel test robot <lkp@intel.com>
To: glider <glider@google.com>
Cc: kbuild-all@lists.01.org, clang-built-linux@googlegroups.com,
linux-kernel@vger.kernel.org, "Kees Cook" <keescook@chromium.org>,
"Maciej Żenczykowski" <maze@google.com>
Subject: drivers/media/i2c/mt9t112.c:670:12: warning: stack frame size of 8344 bytes in function 'mt9t112_init_camera'
Date: Sat, 17 Oct 2020 13:45:13 +0800 [thread overview]
Message-ID: <202010171308.ROWY5cjm-lkp@intel.com> (raw)
[-- Attachment #1: Type: text/plain, Size: 5436 bytes --]
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: 071a0578b0ce0b0e543d1e38ee6926b9cc21c198
commit: f0fe00d4972a8cd4b98cc2c29758615e4d51cdfe security: allow using Clang's zero initialization for stack variables
date: 4 months ago
config: x86_64-randconfig-a012-20201017 (attached as .config)
compiler: clang version 12.0.0 (https://github.com/llvm/llvm-project efd02c1548ee458d59063f6393e94e972b5c3d50)
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# install x86_64 cross compiling tool for clang build
# apt-get install binutils-x86-64-linux-gnu
# https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=f0fe00d4972a8cd4b98cc2c29758615e4d51cdfe
git remote add linus https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
git fetch --no-tags linus master
git checkout f0fe00d4972a8cd4b98cc2c29758615e4d51cdfe
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=x86_64
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/media/i2c/mt9t112.c:670:12: warning: stack frame size of 8344 bytes in function 'mt9t112_init_camera' [-Wframe-larger-than=]
static int mt9t112_init_camera(const struct i2c_client *client)
^
1 warning generated.
vim +/mt9t112_init_camera +670 drivers/media/i2c/mt9t112.c
7641b04421954ea Jacopo Mondi 2018-03-12 669
7641b04421954ea Jacopo Mondi 2018-03-12 @670 static int mt9t112_init_camera(const struct i2c_client *client)
7641b04421954ea Jacopo Mondi 2018-03-12 671 {
7641b04421954ea Jacopo Mondi 2018-03-12 672 int ret;
7641b04421954ea Jacopo Mondi 2018-03-12 673
7641b04421954ea Jacopo Mondi 2018-03-12 674 ECHECKER(ret, mt9t112_reset(client));
7641b04421954ea Jacopo Mondi 2018-03-12 675 ECHECKER(ret, mt9t112_init_pll(client));
7641b04421954ea Jacopo Mondi 2018-03-12 676 ECHECKER(ret, mt9t112_init_setting(client));
7641b04421954ea Jacopo Mondi 2018-03-12 677 ECHECKER(ret, mt9t112_auto_focus_setting(client));
7641b04421954ea Jacopo Mondi 2018-03-12 678
7641b04421954ea Jacopo Mondi 2018-03-12 679 mt9t112_reg_mask_set(ret, client, 0x0018, 0x0004, 0);
7641b04421954ea Jacopo Mondi 2018-03-12 680
6a26f141bf6200a Jacopo Mondi 2018-03-12 681 /* Analog setting B.*/
7641b04421954ea Jacopo Mondi 2018-03-12 682 mt9t112_reg_write(ret, client, 0x3084, 0x2409);
7641b04421954ea Jacopo Mondi 2018-03-12 683 mt9t112_reg_write(ret, client, 0x3092, 0x0A49);
7641b04421954ea Jacopo Mondi 2018-03-12 684 mt9t112_reg_write(ret, client, 0x3094, 0x4949);
7641b04421954ea Jacopo Mondi 2018-03-12 685 mt9t112_reg_write(ret, client, 0x3096, 0x4950);
7641b04421954ea Jacopo Mondi 2018-03-12 686
7641b04421954ea Jacopo Mondi 2018-03-12 687 /*
6a26f141bf6200a Jacopo Mondi 2018-03-12 688 * Disable adaptive clock.
7641b04421954ea Jacopo Mondi 2018-03-12 689 * PRI_A_CONFIG_JPEG_OB_TX_CONTROL_VAR
7641b04421954ea Jacopo Mondi 2018-03-12 690 * PRI_B_CONFIG_JPEG_OB_TX_CONTROL_VAR
7641b04421954ea Jacopo Mondi 2018-03-12 691 */
7641b04421954ea Jacopo Mondi 2018-03-12 692 mt9t112_mcu_write(ret, client, VAR(26, 160), 0x0A2E);
7641b04421954ea Jacopo Mondi 2018-03-12 693 mt9t112_mcu_write(ret, client, VAR(27, 160), 0x0A2E);
7641b04421954ea Jacopo Mondi 2018-03-12 694
6a26f141bf6200a Jacopo Mondi 2018-03-12 695 /*
6a26f141bf6200a Jacopo Mondi 2018-03-12 696 * Configure Status in Status_before_length Format and enable header.
6a26f141bf6200a Jacopo Mondi 2018-03-12 697 * PRI_B_CONFIG_JPEG_OB_TX_CONTROL_VAR
6a26f141bf6200a Jacopo Mondi 2018-03-12 698 */
7641b04421954ea Jacopo Mondi 2018-03-12 699 mt9t112_mcu_write(ret, client, VAR(27, 144), 0x0CB4);
7641b04421954ea Jacopo Mondi 2018-03-12 700
6a26f141bf6200a Jacopo Mondi 2018-03-12 701 /*
6a26f141bf6200a Jacopo Mondi 2018-03-12 702 * Enable JPEG in context B.
6a26f141bf6200a Jacopo Mondi 2018-03-12 703 * PRI_B_CONFIG_JPEG_OB_TX_CONTROL_VAR
6a26f141bf6200a Jacopo Mondi 2018-03-12 704 */
7641b04421954ea Jacopo Mondi 2018-03-12 705 mt9t112_mcu_write(ret, client, VAR8(27, 142), 0x01);
7641b04421954ea Jacopo Mondi 2018-03-12 706
6a26f141bf6200a Jacopo Mondi 2018-03-12 707 /* Disable Dac_TXLO. */
7641b04421954ea Jacopo Mondi 2018-03-12 708 mt9t112_reg_write(ret, client, 0x316C, 0x350F);
7641b04421954ea Jacopo Mondi 2018-03-12 709
6a26f141bf6200a Jacopo Mondi 2018-03-12 710 /* Set max slew rates. */
7641b04421954ea Jacopo Mondi 2018-03-12 711 mt9t112_reg_write(ret, client, 0x1E, 0x777);
7641b04421954ea Jacopo Mondi 2018-03-12 712
7641b04421954ea Jacopo Mondi 2018-03-12 713 return ret;
7641b04421954ea Jacopo Mondi 2018-03-12 714 }
7641b04421954ea Jacopo Mondi 2018-03-12 715
:::::: The code at line 670 was first introduced by commit
:::::: 7641b04421954ea092b8fa7edbf904c3852875b5 media: i2c: Copy mt9t112 soc_camera sensor driver
:::::: TO: Jacopo Mondi <jacopo+renesas@jmondi.org>
:::::: CC: Mauro Carvalho Chehab <mchehab@s-opensource.com>
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 35695 bytes --]
reply other threads:[~2020-10-17 5:46 UTC|newest]
Thread overview: [no followups] expand[flat|nested] mbox.gz Atom feed
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=202010171308.ROWY5cjm-lkp@intel.com \
--to=lkp@intel.com \
--cc=clang-built-linux@googlegroups.com \
--cc=glider@google.com \
--cc=kbuild-all@lists.01.org \
--cc=keescook@chromium.org \
--cc=linux-kernel@vger.kernel.org \
--cc=maze@google.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).