From: kernel test robot <lkp@intel.com>
To: Zhiyong Tao <zhiyong.tao@mediatek.com>,
robh+dt@kernel.org, linus.walleij@linaro.org,
mark.rutland@arm.com, matthias.bgg@gmail.com,
sean.wang@kernel.org
Cc: llvm@lists.linux.dev, kbuild-all@lists.01.org,
srv_heupstream@mediatek.com, zhiyong.tao@mediatek.com,
hui.liu@mediatek.com, eddie.huang@mediatek.com,
light.hsieh@mediatek.com
Subject: Re: [PATCH v11 4/4] pinctrl: mediatek: add rsel setting on MT8195
Date: Mon, 30 Aug 2021 11:33:08 +0800 [thread overview]
Message-ID: <202108301159.uf0SdXgB-lkp@intel.com> (raw)
In-Reply-To: <20210830003603.31864-5-zhiyong.tao@mediatek.com>
[-- Attachment #1: Type: text/plain, Size: 3198 bytes --]
Hi Zhiyong,
Thank you for the patch! Perhaps something to improve:
[auto build test WARNING on robh/for-next]
[also build test WARNING on v5.14 next-20210827]
[cannot apply to pinctrl/devel mediatek/for-next]
[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]
url: https://github.com/0day-ci/linux/commits/Zhiyong-Tao/Mediatek-pinctrl-patch-on-mt8195/20210830-083752
base: https://git.kernel.org/pub/scm/linux/kernel/git/robh/linux.git for-next
config: arm64-buildonly-randconfig-r006-20210829 (attached as .config)
compiler: clang version 14.0.0 (https://github.com/llvm/llvm-project 4b1fde8a2b681dad2ce0c082a5d6422caa06b0bc)
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 arm64 cross compiling tool for clang build
# apt-get install binutils-aarch64-linux-gnu
# https://github.com/0day-ci/linux/commit/eec1945e55fba60a9ef8b426b21d12c0a2aa1ac7
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Zhiyong-Tao/Mediatek-pinctrl-patch-on-mt8195/20210830-083752
git checkout eec1945e55fba60a9ef8b426b21d12c0a2aa1ac7
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=arm64
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/pinctrl/mediatek/pinctrl-mtk-common-v2.c:723:3: warning: variable 'rsel_val' is uninitialized when used here [-Wuninitialized]
rsel_val -= MTK_PULL_SET_RSEL_000;
^~~~~~~~
drivers/pinctrl/mediatek/pinctrl-mtk-common-v2.c:709:19: note: initialize the variable 'rsel_val' to silence this warning
int err, rsel_val;
^
= 0
1 warning generated.
vim +/rsel_val +723 drivers/pinctrl/mediatek/pinctrl-mtk-common-v2.c
704
705 static int mtk_pinconf_bias_set_rsel(struct mtk_pinctrl *hw,
706 const struct mtk_pin_desc *desc,
707 u32 pullup, u32 arg)
708 {
709 int err, rsel_val;
710
711 if (hw->soc->pin_rsel) {
712 /* find pin rsel_index from pin_rsel array*/
713 err = mtk_hw_pin_rsel_lookup(hw, desc, pullup, arg, &rsel_val);
714 if (err)
715 goto out;
716 } else {
717 if (arg < MTK_PULL_SET_RSEL_000 ||
718 arg > MTK_PULL_SET_RSEL_111) {
719 err = -EINVAL;
720 goto out;
721 }
722
> 723 rsel_val -= MTK_PULL_SET_RSEL_000;
724 }
725
726 err = mtk_hw_set_value(hw, desc, PINCTRL_PIN_REG_RSEL, rsel_val);
727 if (err)
728 goto out;
729
730 err = mtk_pinconf_bias_set_pu_pd(hw, desc, pullup, MTK_ENABLE);
731
732 out:
733 return err;
734 }
735
---
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: 44984 bytes --]
WARNING: multiple messages have this Message-ID (diff)
From: kernel test robot <lkp@intel.com>
To: kbuild-all@lists.01.org
Subject: Re: [PATCH v11 4/4] pinctrl: mediatek: add rsel setting on MT8195
Date: Mon, 30 Aug 2021 11:33:08 +0800 [thread overview]
Message-ID: <202108301159.uf0SdXgB-lkp@intel.com> (raw)
In-Reply-To: <20210830003603.31864-5-zhiyong.tao@mediatek.com>
[-- Attachment #1: Type: text/plain, Size: 3280 bytes --]
Hi Zhiyong,
Thank you for the patch! Perhaps something to improve:
[auto build test WARNING on robh/for-next]
[also build test WARNING on v5.14 next-20210827]
[cannot apply to pinctrl/devel mediatek/for-next]
[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]
url: https://github.com/0day-ci/linux/commits/Zhiyong-Tao/Mediatek-pinctrl-patch-on-mt8195/20210830-083752
base: https://git.kernel.org/pub/scm/linux/kernel/git/robh/linux.git for-next
config: arm64-buildonly-randconfig-r006-20210829 (attached as .config)
compiler: clang version 14.0.0 (https://github.com/llvm/llvm-project 4b1fde8a2b681dad2ce0c082a5d6422caa06b0bc)
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 arm64 cross compiling tool for clang build
# apt-get install binutils-aarch64-linux-gnu
# https://github.com/0day-ci/linux/commit/eec1945e55fba60a9ef8b426b21d12c0a2aa1ac7
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Zhiyong-Tao/Mediatek-pinctrl-patch-on-mt8195/20210830-083752
git checkout eec1945e55fba60a9ef8b426b21d12c0a2aa1ac7
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=arm64
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/pinctrl/mediatek/pinctrl-mtk-common-v2.c:723:3: warning: variable 'rsel_val' is uninitialized when used here [-Wuninitialized]
rsel_val -= MTK_PULL_SET_RSEL_000;
^~~~~~~~
drivers/pinctrl/mediatek/pinctrl-mtk-common-v2.c:709:19: note: initialize the variable 'rsel_val' to silence this warning
int err, rsel_val;
^
= 0
1 warning generated.
vim +/rsel_val +723 drivers/pinctrl/mediatek/pinctrl-mtk-common-v2.c
704
705 static int mtk_pinconf_bias_set_rsel(struct mtk_pinctrl *hw,
706 const struct mtk_pin_desc *desc,
707 u32 pullup, u32 arg)
708 {
709 int err, rsel_val;
710
711 if (hw->soc->pin_rsel) {
712 /* find pin rsel_index from pin_rsel array*/
713 err = mtk_hw_pin_rsel_lookup(hw, desc, pullup, arg, &rsel_val);
714 if (err)
715 goto out;
716 } else {
717 if (arg < MTK_PULL_SET_RSEL_000 ||
718 arg > MTK_PULL_SET_RSEL_111) {
719 err = -EINVAL;
720 goto out;
721 }
722
> 723 rsel_val -= MTK_PULL_SET_RSEL_000;
724 }
725
726 err = mtk_hw_set_value(hw, desc, PINCTRL_PIN_REG_RSEL, rsel_val);
727 if (err)
728 goto out;
729
730 err = mtk_pinconf_bias_set_pu_pd(hw, desc, pullup, MTK_ENABLE);
731
732 out:
733 return err;
734 }
735
---
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: 44984 bytes --]
next prev parent reply other threads:[~2021-08-30 3:34 UTC|newest]
Thread overview: 78+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-08-30 0:35 [PATCH v11 0/4] Mediatek pinctrl patch on mt8195 Zhiyong Tao
2021-08-30 0:35 ` Zhiyong Tao
2021-08-30 0:35 ` Zhiyong Tao
2021-08-30 0:36 ` [PATCH v11 1/4] dt-bindings: pinctrl: mt8195: add rsel define Zhiyong Tao
2021-08-30 0:36 ` Zhiyong Tao
2021-08-30 0:36 ` Zhiyong Tao
2021-08-31 22:09 ` Rob Herring
2021-08-31 22:09 ` Rob Herring
2021-08-31 22:09 ` Rob Herring
2021-09-01 4:35 ` Chen-Yu Tsai
2021-09-01 4:35 ` Chen-Yu Tsai
2021-09-01 4:35 ` Chen-Yu Tsai
2021-09-02 2:54 ` zhiyong.tao
2021-09-02 2:54 ` zhiyong.tao
2021-09-02 3:35 ` Chen-Yu Tsai
2021-09-02 3:35 ` Chen-Yu Tsai
2021-09-02 3:35 ` Chen-Yu Tsai
2021-09-04 8:40 ` zhiyong.tao
2021-09-04 8:40 ` zhiyong.tao
2021-09-06 8:20 ` Chen-Yu Tsai
2021-09-06 8:20 ` Chen-Yu Tsai
2021-09-06 8:20 ` Chen-Yu Tsai
2021-09-09 6:46 ` zhiyong.tao
2021-09-09 6:46 ` zhiyong.tao
2021-09-14 12:27 ` zhiyong.tao
2021-09-14 12:27 ` zhiyong.tao
2021-09-15 3:25 ` Chen-Yu Tsai
2021-09-15 3:25 ` Chen-Yu Tsai
2021-09-15 3:25 ` Chen-Yu Tsai
2021-09-15 5:01 ` Light Hsieh (謝明燈)
2021-08-30 0:36 ` [PATCH v11 2/4] dt-bindings: pinctrl: mt8195: change pull up/down description Zhiyong Tao
2021-08-30 0:36 ` Zhiyong Tao
2021-08-30 0:36 ` Zhiyong Tao
2021-08-31 22:13 ` Rob Herring
2021-08-31 22:13 ` Rob Herring
2021-08-31 22:13 ` Rob Herring
2021-09-01 2:00 ` zhiyong.tao
2021-09-01 2:00 ` zhiyong.tao
2021-09-01 10:21 ` Chen-Yu Tsai
2021-09-01 10:21 ` Chen-Yu Tsai
2021-09-01 10:21 ` Chen-Yu Tsai
2021-09-02 3:31 ` zhiyong.tao
2021-09-02 3:31 ` zhiyong.tao
2021-08-30 0:36 ` [PATCH v11 3/4] pinctrl: mediatek: mt8195: Add pm_ops Zhiyong Tao
2021-08-30 0:36 ` Zhiyong Tao
2021-08-30 0:36 ` Zhiyong Tao
2021-08-31 7:53 ` Chen-Yu Tsai
2021-08-31 7:53 ` Chen-Yu Tsai
2021-08-31 7:53 ` Chen-Yu Tsai
2021-08-31 9:11 ` zhiyong.tao
2021-08-31 9:11 ` zhiyong.tao
2021-08-30 0:36 ` [PATCH v11 4/4] pinctrl: mediatek: add rsel setting on MT8195 Zhiyong Tao
2021-08-30 0:36 ` Zhiyong Tao
2021-08-30 0:36 ` Zhiyong Tao
2021-08-30 3:33 ` kernel test robot [this message]
2021-08-30 3:33 ` kernel test robot
2021-09-01 10:10 ` Chen-Yu Tsai
2021-09-01 10:10 ` Chen-Yu Tsai
2021-09-01 10:10 ` Chen-Yu Tsai
2021-09-06 3:17 ` zhiyong.tao
2021-09-06 3:17 ` zhiyong.tao
2021-09-06 10:09 ` Chen-Yu Tsai
2021-09-06 10:09 ` Chen-Yu Tsai
2021-09-06 10:09 ` Chen-Yu Tsai
2021-09-09 7:46 ` zhiyong.tao
2021-09-09 7:46 ` zhiyong.tao
2021-09-16 9:31 ` zhiyong.tao
2021-09-16 9:31 ` zhiyong.tao
2021-09-16 9:40 ` Chen-Yu Tsai
2021-09-16 9:40 ` Chen-Yu Tsai
2021-09-16 9:40 ` Chen-Yu Tsai
2021-09-17 2:10 ` zhiyong.tao
2021-09-17 2:10 ` zhiyong.tao
2021-09-19 19:33 ` Linus Walleij
2021-09-19 19:33 ` Linus Walleij
2021-09-19 19:33 ` Linus Walleij
2021-09-22 1:47 ` zhiyong.tao
2021-09-22 1:47 ` zhiyong.tao
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=202108301159.uf0SdXgB-lkp@intel.com \
--to=lkp@intel.com \
--cc=eddie.huang@mediatek.com \
--cc=hui.liu@mediatek.com \
--cc=kbuild-all@lists.01.org \
--cc=light.hsieh@mediatek.com \
--cc=linus.walleij@linaro.org \
--cc=llvm@lists.linux.dev \
--cc=mark.rutland@arm.com \
--cc=matthias.bgg@gmail.com \
--cc=robh+dt@kernel.org \
--cc=sean.wang@kernel.org \
--cc=srv_heupstream@mediatek.com \
--cc=zhiyong.tao@mediatek.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 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.