From: kbuild test robot <lkp@intel.com>
To: sean.wang@mediatek.com
Cc: kbuild-all@01.org, robh+dt@kernel.org, mark.rutland@arm.com,
sre@kernel.org, lee.jones@linaro.org, a.zummo@towertech.it,
alexandre.belloni@free-electrons.com, eddie.huang@mediatek.com,
devicetree@vger.kernel.org, linux-rtc@vger.kernel.org,
linux-pm@vger.kernel.org, linux-mediatek@lists.infradead.org,
linux-kernel@vger.kernel.org, Sean Wang <sean.wang@mediatek.com>
Subject: Re: [PATCH v1 12/16] rtc: mediatek: cleanup header files to include
Date: Sun, 25 Mar 2018 13:21:59 +0800 [thread overview]
Message-ID: <201803251308.YtMqvlLL%fengguang.wu@intel.com> (raw)
In-Reply-To: <6e4f709162d57c6e81dfca0996f88358080bf411.1521794177.git.sean.wang@mediatek.com>
[-- Attachment #1: Type: text/plain, Size: 5005 bytes --]
Hi Sean,
I love your patch! Yet something to improve:
[auto build test ERROR on abelloni/rtc-next]
[also build test ERROR on v4.16-rc6]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]
url: https://github.com/0day-ci/linux/commits/sean-wang-mediatek-com/Add-support-to-MT6323-RTC-and-its-power-device/20180325-104529
base: https://git.kernel.org/pub/scm/linux/kernel/git/abelloni/linux.git rtc-next
config: tile-allmodconfig (attached as .config)
compiler: tilegx-linux-gcc (GCC) 7.2.0
reproduce:
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# save the attached .config to linux build tree
make.cross ARCH=tile
All errors (new ones prefixed by >>):
drivers//rtc/rtc-mt6397.c: In function 'mtk_rtc_probe':
>> drivers//rtc/rtc-mt6397.c:270:13: error: implicit declaration of function 'irq_create_mapping'; did you mean 'page_mapping'? [-Werror=implicit-function-declaration]
rtc->irq = irq_create_mapping(mt6397_chip->irq_domain, res->start);
^~~~~~~~~~~~~~~~~~
page_mapping
cc1: some warnings being treated as errors
vim +270 drivers//rtc/rtc-mt6397.c
fc2979118 Tianping Fang 2015-05-06 254
fc2979118 Tianping Fang 2015-05-06 255 static int mtk_rtc_probe(struct platform_device *pdev)
fc2979118 Tianping Fang 2015-05-06 256 {
fc2979118 Tianping Fang 2015-05-06 257 struct resource *res;
fc2979118 Tianping Fang 2015-05-06 258 struct mt6397_chip *mt6397_chip = dev_get_drvdata(pdev->dev.parent);
fc2979118 Tianping Fang 2015-05-06 259 struct mt6397_rtc *rtc;
fc2979118 Tianping Fang 2015-05-06 260 int ret;
fc2979118 Tianping Fang 2015-05-06 261
fc2979118 Tianping Fang 2015-05-06 262 rtc = devm_kzalloc(&pdev->dev, sizeof(struct mt6397_rtc), GFP_KERNEL);
fc2979118 Tianping Fang 2015-05-06 263 if (!rtc)
fc2979118 Tianping Fang 2015-05-06 264 return -ENOMEM;
fc2979118 Tianping Fang 2015-05-06 265
fc2979118 Tianping Fang 2015-05-06 266 res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
fc2979118 Tianping Fang 2015-05-06 267 rtc->addr_base = res->start;
fc2979118 Tianping Fang 2015-05-06 268
fc2979118 Tianping Fang 2015-05-06 269 res = platform_get_resource(pdev, IORESOURCE_IRQ, 0);
fc2979118 Tianping Fang 2015-05-06 @270 rtc->irq = irq_create_mapping(mt6397_chip->irq_domain, res->start);
fc2979118 Tianping Fang 2015-05-06 271 if (rtc->irq <= 0)
fc2979118 Tianping Fang 2015-05-06 272 return -EINVAL;
fc2979118 Tianping Fang 2015-05-06 273
fc2979118 Tianping Fang 2015-05-06 274 rtc->regmap = mt6397_chip->regmap;
fc2979118 Tianping Fang 2015-05-06 275 rtc->dev = &pdev->dev;
fc2979118 Tianping Fang 2015-05-06 276 mutex_init(&rtc->lock);
fc2979118 Tianping Fang 2015-05-06 277
fc2979118 Tianping Fang 2015-05-06 278 platform_set_drvdata(pdev, rtc);
fc2979118 Tianping Fang 2015-05-06 279
66c231b40 Sean Wang 2018-03-23 280 ret = devm_request_threaded_irq(&pdev->dev, rtc->irq, NULL,
fc2979118 Tianping Fang 2015-05-06 281 mtk_rtc_irq_handler_thread,
fc2979118 Tianping Fang 2015-05-06 282 IRQF_ONESHOT | IRQF_TRIGGER_HIGH,
fc2979118 Tianping Fang 2015-05-06 283 "mt6397-rtc", rtc);
fc2979118 Tianping Fang 2015-05-06 284 if (ret) {
fc2979118 Tianping Fang 2015-05-06 285 dev_err(&pdev->dev, "Failed to request alarm IRQ: %d: %d\n",
fc2979118 Tianping Fang 2015-05-06 286 rtc->irq, ret);
63044753b Sean Wang 2018-03-23 287 return ret;
fc2979118 Tianping Fang 2015-05-06 288 }
fc2979118 Tianping Fang 2015-05-06 289
baeca4495 Wei-Ning Huang 2015-07-02 290 device_init_wakeup(&pdev->dev, 1);
baeca4495 Wei-Ning Huang 2015-07-02 291
66c231b40 Sean Wang 2018-03-23 292 rtc->rtc_dev = devm_rtc_device_register(&pdev->dev, "mt6397-rtc",
fc2979118 Tianping Fang 2015-05-06 293 &mtk_rtc_ops, THIS_MODULE);
fc2979118 Tianping Fang 2015-05-06 294 if (IS_ERR(rtc->rtc_dev)) {
fc2979118 Tianping Fang 2015-05-06 295 dev_err(&pdev->dev, "register rtc device failed\n");
fc2979118 Tianping Fang 2015-05-06 296 ret = PTR_ERR(rtc->rtc_dev);
fc2979118 Tianping Fang 2015-05-06 297 return ret;
fc2979118 Tianping Fang 2015-05-06 298 }
fc2979118 Tianping Fang 2015-05-06 299
89a68f3c0 Sean Wang 2018-03-23 300 return devm_of_platform_populate(&pdev->dev);
fc2979118 Tianping Fang 2015-05-06 301 }
fc2979118 Tianping Fang 2015-05-06 302
:::::: The code at line 270 was first introduced by commit
:::::: fc2979118f3f5193475cb53d5df7bdaa7e358a42 rtc: mediatek: Add MT6397 RTC driver
:::::: TO: Tianping Fang <tianping.fang@mediatek.com>
:::::: CC: Alexandre Belloni <alexandre.belloni@free-electrons.com>
---
0-DAY kernel test infrastructure Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all Intel Corporation
[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 51610 bytes --]
WARNING: multiple messages have this Message-ID (diff)
From: kbuild test robot <lkp@intel.com>
Cc: kbuild-all@01.org, robh+dt@kernel.org, mark.rutland@arm.com,
sre@kernel.org, lee.jones@linaro.org, a.zummo@towertech.it,
alexandre.belloni@free-electrons.com, eddie.huang@mediatek.com,
devicetree@vger.kernel.org, linux-rtc@vger.kernel.org,
linux-pm@vger.kernel.org, linux-mediatek@lists.infradead.org,
linux-kernel@vger.kernel.org, Sean Wang <sean.wang@mediatek.com>
Subject: Re: [PATCH v1 12/16] rtc: mediatek: cleanup header files to include
Date: Sun, 25 Mar 2018 13:21:59 +0800 [thread overview]
Message-ID: <201803251308.YtMqvlLL%fengguang.wu@intel.com> (raw)
In-Reply-To: <6e4f709162d57c6e81dfca0996f88358080bf411.1521794177.git.sean.wang@mediatek.com>
[-- Attachment #1: Type: text/plain, Size: 5005 bytes --]
Hi Sean,
I love your patch! Yet something to improve:
[auto build test ERROR on abelloni/rtc-next]
[also build test ERROR on v4.16-rc6]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]
url: https://github.com/0day-ci/linux/commits/sean-wang-mediatek-com/Add-support-to-MT6323-RTC-and-its-power-device/20180325-104529
base: https://git.kernel.org/pub/scm/linux/kernel/git/abelloni/linux.git rtc-next
config: tile-allmodconfig (attached as .config)
compiler: tilegx-linux-gcc (GCC) 7.2.0
reproduce:
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# save the attached .config to linux build tree
make.cross ARCH=tile
All errors (new ones prefixed by >>):
drivers//rtc/rtc-mt6397.c: In function 'mtk_rtc_probe':
>> drivers//rtc/rtc-mt6397.c:270:13: error: implicit declaration of function 'irq_create_mapping'; did you mean 'page_mapping'? [-Werror=implicit-function-declaration]
rtc->irq = irq_create_mapping(mt6397_chip->irq_domain, res->start);
^~~~~~~~~~~~~~~~~~
page_mapping
cc1: some warnings being treated as errors
vim +270 drivers//rtc/rtc-mt6397.c
fc2979118 Tianping Fang 2015-05-06 254
fc2979118 Tianping Fang 2015-05-06 255 static int mtk_rtc_probe(struct platform_device *pdev)
fc2979118 Tianping Fang 2015-05-06 256 {
fc2979118 Tianping Fang 2015-05-06 257 struct resource *res;
fc2979118 Tianping Fang 2015-05-06 258 struct mt6397_chip *mt6397_chip = dev_get_drvdata(pdev->dev.parent);
fc2979118 Tianping Fang 2015-05-06 259 struct mt6397_rtc *rtc;
fc2979118 Tianping Fang 2015-05-06 260 int ret;
fc2979118 Tianping Fang 2015-05-06 261
fc2979118 Tianping Fang 2015-05-06 262 rtc = devm_kzalloc(&pdev->dev, sizeof(struct mt6397_rtc), GFP_KERNEL);
fc2979118 Tianping Fang 2015-05-06 263 if (!rtc)
fc2979118 Tianping Fang 2015-05-06 264 return -ENOMEM;
fc2979118 Tianping Fang 2015-05-06 265
fc2979118 Tianping Fang 2015-05-06 266 res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
fc2979118 Tianping Fang 2015-05-06 267 rtc->addr_base = res->start;
fc2979118 Tianping Fang 2015-05-06 268
fc2979118 Tianping Fang 2015-05-06 269 res = platform_get_resource(pdev, IORESOURCE_IRQ, 0);
fc2979118 Tianping Fang 2015-05-06 @270 rtc->irq = irq_create_mapping(mt6397_chip->irq_domain, res->start);
fc2979118 Tianping Fang 2015-05-06 271 if (rtc->irq <= 0)
fc2979118 Tianping Fang 2015-05-06 272 return -EINVAL;
fc2979118 Tianping Fang 2015-05-06 273
fc2979118 Tianping Fang 2015-05-06 274 rtc->regmap = mt6397_chip->regmap;
fc2979118 Tianping Fang 2015-05-06 275 rtc->dev = &pdev->dev;
fc2979118 Tianping Fang 2015-05-06 276 mutex_init(&rtc->lock);
fc2979118 Tianping Fang 2015-05-06 277
fc2979118 Tianping Fang 2015-05-06 278 platform_set_drvdata(pdev, rtc);
fc2979118 Tianping Fang 2015-05-06 279
66c231b40 Sean Wang 2018-03-23 280 ret = devm_request_threaded_irq(&pdev->dev, rtc->irq, NULL,
fc2979118 Tianping Fang 2015-05-06 281 mtk_rtc_irq_handler_thread,
fc2979118 Tianping Fang 2015-05-06 282 IRQF_ONESHOT | IRQF_TRIGGER_HIGH,
fc2979118 Tianping Fang 2015-05-06 283 "mt6397-rtc", rtc);
fc2979118 Tianping Fang 2015-05-06 284 if (ret) {
fc2979118 Tianping Fang 2015-05-06 285 dev_err(&pdev->dev, "Failed to request alarm IRQ: %d: %d\n",
fc2979118 Tianping Fang 2015-05-06 286 rtc->irq, ret);
63044753b Sean Wang 2018-03-23 287 return ret;
fc2979118 Tianping Fang 2015-05-06 288 }
fc2979118 Tianping Fang 2015-05-06 289
baeca4495 Wei-Ning Huang 2015-07-02 290 device_init_wakeup(&pdev->dev, 1);
baeca4495 Wei-Ning Huang 2015-07-02 291
66c231b40 Sean Wang 2018-03-23 292 rtc->rtc_dev = devm_rtc_device_register(&pdev->dev, "mt6397-rtc",
fc2979118 Tianping Fang 2015-05-06 293 &mtk_rtc_ops, THIS_MODULE);
fc2979118 Tianping Fang 2015-05-06 294 if (IS_ERR(rtc->rtc_dev)) {
fc2979118 Tianping Fang 2015-05-06 295 dev_err(&pdev->dev, "register rtc device failed\n");
fc2979118 Tianping Fang 2015-05-06 296 ret = PTR_ERR(rtc->rtc_dev);
fc2979118 Tianping Fang 2015-05-06 297 return ret;
fc2979118 Tianping Fang 2015-05-06 298 }
fc2979118 Tianping Fang 2015-05-06 299
89a68f3c0 Sean Wang 2018-03-23 300 return devm_of_platform_populate(&pdev->dev);
fc2979118 Tianping Fang 2015-05-06 301 }
fc2979118 Tianping Fang 2015-05-06 302
:::::: The code at line 270 was first introduced by commit
:::::: fc2979118f3f5193475cb53d5df7bdaa7e358a42 rtc: mediatek: Add MT6397 RTC driver
:::::: TO: Tianping Fang <tianping.fang@mediatek.com>
:::::: CC: Alexandre Belloni <alexandre.belloni@free-electrons.com>
---
0-DAY kernel test infrastructure Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all Intel Corporation
[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 51610 bytes --]
next prev parent reply other threads:[~2018-03-25 5:22 UTC|newest]
Thread overview: 79+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-03-23 9:14 [PATCH v1 00/16] Add support to MT6323 RTC and its power device sean.wang
2018-03-23 9:14 ` sean.wang
2018-03-23 9:14 ` [PATCH v1 01/16] dt-bindings: power: reset: mediatek: add bindings for " sean.wang
2018-03-23 9:14 ` sean.wang
2018-03-26 22:24 ` Rob Herring
2018-03-27 3:21 ` Sean Wang
2018-03-27 3:21 ` Sean Wang
2018-03-23 9:14 ` [PATCH v1 02/16] dt-bindings: rtc: mediatek: add bindings for PMIC RTC sean.wang
2018-03-23 9:14 ` sean.wang-NuS5LvNUpcJWk0Htik3J/w
2018-03-23 9:41 ` Alexandre Belloni
2018-03-23 10:15 ` Alexandre Belloni
2018-03-23 10:15 ` Alexandre Belloni
2018-03-24 19:36 ` Sean Wang
2018-03-24 19:36 ` Sean Wang
2018-03-27 15:18 ` Alexandre Belloni
2018-03-28 3:53 ` Sean Wang
2018-03-28 3:53 ` Sean Wang
2018-03-28 9:19 ` Alexandre Belloni
2018-03-23 9:15 ` [PATCH v1 03/16] dt-bindings: mfd: mediatek: add a description for MT6323 RTC sean.wang
2018-03-23 9:15 ` sean.wang-NuS5LvNUpcJWk0Htik3J/w
2018-03-26 22:24 ` Rob Herring
2018-03-28 11:13 ` Lee Jones
2018-03-23 9:15 ` [PATCH v1 04/16] mfd: mt6397: add MT6323 RTC support into MT6397 driver sean.wang
2018-03-23 9:15 ` sean.wang-NuS5LvNUpcJWk0Htik3J/w
2018-03-28 11:15 ` Lee Jones
2018-03-28 11:15 ` Lee Jones
2018-03-23 9:15 ` [PATCH v1 05/16] rtc: mediatek: add MT6323 support to RTC driver sean.wang
2018-03-23 9:15 ` sean.wang-NuS5LvNUpcJWk0Htik3J/w
2018-03-23 10:01 ` Alexandre Belloni
2018-03-24 7:06 ` Sean Wang
2018-03-24 7:06 ` Sean Wang
2018-03-23 9:15 ` [PATCH v1 06/16] rtc: mediatek: remove unnecessary parentheses sean.wang
2018-03-23 9:15 ` sean.wang
2018-03-23 10:21 ` Alexandre Belloni
2018-03-24 7:14 ` Sean Wang
2018-03-24 7:14 ` Sean Wang
2018-03-24 18:53 ` Alexandre Belloni
2018-03-24 19:21 ` Sean Wang
2018-03-24 19:21 ` Sean Wang
2018-03-23 9:15 ` [PATCH v1 07/16] rtc: mediatek: replace a poll with regmap_read_poll_timeout sean.wang
2018-03-23 9:15 ` sean.wang-NuS5LvNUpcJWk0Htik3J/w
2018-03-23 9:15 ` [PATCH v1 08/16] rtc: mediatek: remove unnecessary irq_dispose_mapping sean.wang
2018-03-23 9:15 ` sean.wang-NuS5LvNUpcJWk0Htik3J/w
2018-03-23 10:38 ` Alexandre Belloni
2018-03-26 2:22 ` Sean Wang
2018-03-26 2:22 ` Sean Wang
2018-03-23 9:15 ` [PATCH v1 09/16] rtc: mediatek: convert to use device managed functions sean.wang
2018-03-23 9:15 ` sean.wang-NuS5LvNUpcJWk0Htik3J/w
2018-03-23 10:50 ` Alexandre Belloni
2018-03-26 4:07 ` Sean Wang
2018-03-26 4:07 ` Sean Wang
2018-03-27 15:07 ` Alexandre Belloni
2018-03-27 15:07 ` Alexandre Belloni
2018-03-23 9:15 ` [PATCH v1 10/16] rtc: mediatek: add devm_of_platform_populate sean.wang
2018-03-23 9:15 ` sean.wang-NuS5LvNUpcJWk0Htik3J/w
2018-03-23 9:15 ` [PATCH v1 11/16] rtc: mediatek: move the declaration into a globally visible header file sean.wang
2018-03-23 9:15 ` sean.wang-NuS5LvNUpcJWk0Htik3J/w
2018-03-23 9:57 ` Alexandre Belloni
2018-03-23 9:57 ` Alexandre Belloni
2018-03-24 7:31 ` Sean Wang
2018-03-24 7:31 ` Sean Wang
2018-03-24 18:54 ` Alexandre Belloni
2018-03-24 20:00 ` Fabio Estevam
2018-03-25 3:13 ` Sean Wang
2018-03-25 3:13 ` Sean Wang
2018-03-23 9:15 ` [PATCH v1 12/16] rtc: mediatek: cleanup header files to include sean.wang
2018-03-23 9:15 ` sean.wang
2018-03-25 4:17 ` kbuild test robot
2018-03-25 4:17 ` kbuild test robot
2018-03-25 5:21 ` kbuild test robot [this message]
2018-03-25 5:21 ` kbuild test robot
2018-03-23 9:15 ` [PATCH v1 13/16] rtc: mediatek: update license converting to using SPDX identifiers sean.wang
2018-03-23 9:15 ` sean.wang-NuS5LvNUpcJWk0Htik3J/w
2018-03-23 9:15 ` [PATCH v1 14/16] power: reset: mediatek: add a power-off driver using PMIC RTC device sean.wang
2018-03-23 9:15 ` sean.wang
2018-03-23 9:15 ` [PATCH v1 15/16] MAINTAINERS: update entry for ARM/Mediatek RTC DRIVER sean.wang
2018-03-23 9:15 ` sean.wang-NuS5LvNUpcJWk0Htik3J/w
2018-03-23 9:15 ` [PATCH v1 16/16] MAINTAINERS: add an entry for MediaTek board level shutdown driver sean.wang
2018-03-23 9:15 ` sean.wang-NuS5LvNUpcJWk0Htik3J/w
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=201803251308.YtMqvlLL%fengguang.wu@intel.com \
--to=lkp@intel.com \
--cc=a.zummo@towertech.it \
--cc=alexandre.belloni@free-electrons.com \
--cc=devicetree@vger.kernel.org \
--cc=eddie.huang@mediatek.com \
--cc=kbuild-all@01.org \
--cc=lee.jones@linaro.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mediatek@lists.infradead.org \
--cc=linux-pm@vger.kernel.org \
--cc=linux-rtc@vger.kernel.org \
--cc=mark.rutland@arm.com \
--cc=robh+dt@kernel.org \
--cc=sean.wang@mediatek.com \
--cc=sre@kernel.org \
/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.