From: kbuild test robot <lkp@intel.com> To: Ulf Hansson <ulf.hansson@linaro.org> Cc: kbuild-all@01.org, Kishon Vijay Abraham I <kishon@ti.com>, linux-kernel@vger.kernel.org, "Rafael J . Wysocki" <rjw@rjwysocki.net>, linux-pm@vger.kernel.org, Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com>, Geert Uytterhoeven <geert@linux-m68k.org>, linux-renesas-soc@vger.kernel.org, Ulf Hansson <ulf.hansson@linaro.org> Subject: Re: [PATCH v2 2/3] phy: core: Drop unused runtime PM APIs Date: Sat, 23 Dec 2017 18:08:41 +0800 [thread overview] Message-ID: <201712231728.94eR1hbd%fengguang.wu@intel.com> (raw) In-Reply-To: <1513778960-10073-3-git-send-email-ulf.hansson@linaro.org> [-- Attachment #1: Type: text/plain, Size: 3518 bytes --] Hi Ulf, I love your patch! Yet something to improve: [auto build test ERROR on phy/next] [also build test ERROR on v4.15-rc4 next-20171222] [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/Ulf-Hansson/phy-core-Re-work-runtime-PM-deployment-and-fix-an-issue/20171223-170432 base: https://git.kernel.org/pub/scm/linux/kernel/git/kishon/linux-phy.git next config: x86_64-acpi-redef (attached as .config) compiler: gcc-7 (Debian 7.2.0-12) 7.2.1 20171025 reproduce: # save the attached .config to linux build tree make ARCH=x86_64 All errors (new ones prefixed by >>): drivers//ata/libahci.c: In function 'ahci_rpm_get_port': >> drivers//ata/libahci.c:239:9: error: implicit declaration of function 'pm_runtime_get_sync'; did you mean 'ktime_get_ns'? [-Werror=implicit-function-declaration] return pm_runtime_get_sync(ap->dev); ^~~~~~~~~~~~~~~~~~~ ktime_get_ns drivers//ata/libahci.c: In function 'ahci_rpm_put_port': >> drivers//ata/libahci.c:251:2: error: implicit declaration of function 'pm_runtime_put'; did you mean 'of_node_put'? [-Werror=implicit-function-declaration] pm_runtime_put(ap->dev); ^~~~~~~~~~~~~~ of_node_put cc1: some warnings being treated as errors vim +251 drivers//ata/libahci.c 365cfa1e Anton Vorontsov 2010-03-28 228 bb03c640 Mika Westerberg 2016-02-18 229 /** bb03c640 Mika Westerberg 2016-02-18 230 * ahci_rpm_get_port - Make sure the port is powered on bb03c640 Mika Westerberg 2016-02-18 231 * @ap: Port to power on bb03c640 Mika Westerberg 2016-02-18 232 * bb03c640 Mika Westerberg 2016-02-18 233 * Whenever there is need to access the AHCI host registers outside of bb03c640 Mika Westerberg 2016-02-18 234 * normal execution paths, call this function to make sure the host is bb03c640 Mika Westerberg 2016-02-18 235 * actually powered on. bb03c640 Mika Westerberg 2016-02-18 236 */ bb03c640 Mika Westerberg 2016-02-18 237 static int ahci_rpm_get_port(struct ata_port *ap) bb03c640 Mika Westerberg 2016-02-18 238 { bb03c640 Mika Westerberg 2016-02-18 @239 return pm_runtime_get_sync(ap->dev); bb03c640 Mika Westerberg 2016-02-18 240 } bb03c640 Mika Westerberg 2016-02-18 241 bb03c640 Mika Westerberg 2016-02-18 242 /** bb03c640 Mika Westerberg 2016-02-18 243 * ahci_rpm_put_port - Undoes ahci_rpm_get_port() bb03c640 Mika Westerberg 2016-02-18 244 * @ap: Port to power down bb03c640 Mika Westerberg 2016-02-18 245 * bb03c640 Mika Westerberg 2016-02-18 246 * Undoes ahci_rpm_get_port() and possibly powers down the AHCI host bb03c640 Mika Westerberg 2016-02-18 247 * if it has no more active users. bb03c640 Mika Westerberg 2016-02-18 248 */ bb03c640 Mika Westerberg 2016-02-18 249 static void ahci_rpm_put_port(struct ata_port *ap) bb03c640 Mika Westerberg 2016-02-18 250 { bb03c640 Mika Westerberg 2016-02-18 @251 pm_runtime_put(ap->dev); bb03c640 Mika Westerberg 2016-02-18 252 } bb03c640 Mika Westerberg 2016-02-18 253 :::::: The code at line 251 was first introduced by commit :::::: bb03c640697155639b2e15e2aaa4c10f60bf0d5e ahci: Add functions to manage runtime PM of AHCI ports :::::: TO: Mika Westerberg <mika.westerberg@linux.intel.com> :::::: CC: Tejun Heo <tj@kernel.org> --- 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: 30686 bytes --]
WARNING: multiple messages have this Message-ID (diff)
From: kbuild test robot <lkp@intel.com> Cc: kbuild-all@01.org, Kishon Vijay Abraham I <kishon@ti.com>, linux-kernel@vger.kernel.org, "Rafael J . Wysocki" <rjw@rjwysocki.net>, linux-pm@vger.kernel.org, Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com>, Geert Uytterhoeven <geert@linux-m68k.org>, linux-renesas-soc@vger.kernel.org, Ulf Hansson <ulf.hansson@linaro.org> Subject: Re: [PATCH v2 2/3] phy: core: Drop unused runtime PM APIs Date: Sat, 23 Dec 2017 18:08:41 +0800 [thread overview] Message-ID: <201712231728.94eR1hbd%fengguang.wu@intel.com> (raw) In-Reply-To: <1513778960-10073-3-git-send-email-ulf.hansson@linaro.org> [-- Attachment #1: Type: text/plain, Size: 3518 bytes --] Hi Ulf, I love your patch! Yet something to improve: [auto build test ERROR on phy/next] [also build test ERROR on v4.15-rc4 next-20171222] [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/Ulf-Hansson/phy-core-Re-work-runtime-PM-deployment-and-fix-an-issue/20171223-170432 base: https://git.kernel.org/pub/scm/linux/kernel/git/kishon/linux-phy.git next config: x86_64-acpi-redef (attached as .config) compiler: gcc-7 (Debian 7.2.0-12) 7.2.1 20171025 reproduce: # save the attached .config to linux build tree make ARCH=x86_64 All errors (new ones prefixed by >>): drivers//ata/libahci.c: In function 'ahci_rpm_get_port': >> drivers//ata/libahci.c:239:9: error: implicit declaration of function 'pm_runtime_get_sync'; did you mean 'ktime_get_ns'? [-Werror=implicit-function-declaration] return pm_runtime_get_sync(ap->dev); ^~~~~~~~~~~~~~~~~~~ ktime_get_ns drivers//ata/libahci.c: In function 'ahci_rpm_put_port': >> drivers//ata/libahci.c:251:2: error: implicit declaration of function 'pm_runtime_put'; did you mean 'of_node_put'? [-Werror=implicit-function-declaration] pm_runtime_put(ap->dev); ^~~~~~~~~~~~~~ of_node_put cc1: some warnings being treated as errors vim +251 drivers//ata/libahci.c 365cfa1e Anton Vorontsov 2010-03-28 228 bb03c640 Mika Westerberg 2016-02-18 229 /** bb03c640 Mika Westerberg 2016-02-18 230 * ahci_rpm_get_port - Make sure the port is powered on bb03c640 Mika Westerberg 2016-02-18 231 * @ap: Port to power on bb03c640 Mika Westerberg 2016-02-18 232 * bb03c640 Mika Westerberg 2016-02-18 233 * Whenever there is need to access the AHCI host registers outside of bb03c640 Mika Westerberg 2016-02-18 234 * normal execution paths, call this function to make sure the host is bb03c640 Mika Westerberg 2016-02-18 235 * actually powered on. bb03c640 Mika Westerberg 2016-02-18 236 */ bb03c640 Mika Westerberg 2016-02-18 237 static int ahci_rpm_get_port(struct ata_port *ap) bb03c640 Mika Westerberg 2016-02-18 238 { bb03c640 Mika Westerberg 2016-02-18 @239 return pm_runtime_get_sync(ap->dev); bb03c640 Mika Westerberg 2016-02-18 240 } bb03c640 Mika Westerberg 2016-02-18 241 bb03c640 Mika Westerberg 2016-02-18 242 /** bb03c640 Mika Westerberg 2016-02-18 243 * ahci_rpm_put_port - Undoes ahci_rpm_get_port() bb03c640 Mika Westerberg 2016-02-18 244 * @ap: Port to power down bb03c640 Mika Westerberg 2016-02-18 245 * bb03c640 Mika Westerberg 2016-02-18 246 * Undoes ahci_rpm_get_port() and possibly powers down the AHCI host bb03c640 Mika Westerberg 2016-02-18 247 * if it has no more active users. bb03c640 Mika Westerberg 2016-02-18 248 */ bb03c640 Mika Westerberg 2016-02-18 249 static void ahci_rpm_put_port(struct ata_port *ap) bb03c640 Mika Westerberg 2016-02-18 250 { bb03c640 Mika Westerberg 2016-02-18 @251 pm_runtime_put(ap->dev); bb03c640 Mika Westerberg 2016-02-18 252 } bb03c640 Mika Westerberg 2016-02-18 253 :::::: The code at line 251 was first introduced by commit :::::: bb03c640697155639b2e15e2aaa4c10f60bf0d5e ahci: Add functions to manage runtime PM of AHCI ports :::::: TO: Mika Westerberg <mika.westerberg@linux.intel.com> :::::: CC: Tejun Heo <tj@kernel.org> --- 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: 30686 bytes --]
next prev parent reply other threads:[~2017-12-23 10:09 UTC|newest] Thread overview: 26+ messages / expand[flat|nested] mbox.gz Atom feed top 2017-12-20 14:09 [PATCH v2 0/3] phy: core: Re-work runtime PM deployment and fix an issue Ulf Hansson 2017-12-20 14:09 ` [PATCH v2 1/3] phy: core: Move runtime PM reference counting to the parent device Ulf Hansson 2017-12-21 1:39 ` Rafael J. Wysocki 2017-12-21 10:50 ` Ulf Hansson 2017-12-23 1:35 ` Rafael J. Wysocki 2017-12-23 1:50 ` Rafael J. Wysocki 2017-12-23 12:37 ` Ulf Hansson 2017-12-23 12:47 ` Rafael J. Wysocki 2017-12-23 12:39 ` Rafael J. Wysocki 2017-12-23 15:09 ` Ulf Hansson 2017-12-24 12:00 ` Rafael J. Wysocki 2018-01-02 13:28 ` Ulf Hansson 2017-12-20 14:09 ` [PATCH v2 2/3] phy: core: Drop unused runtime PM APIs Ulf Hansson 2017-12-21 10:33 ` Yoshihiro Shimoda 2017-12-21 10:33 ` Yoshihiro Shimoda 2017-12-21 10:57 ` Ulf Hansson 2017-12-21 10:57 ` Ulf Hansson 2017-12-21 12:24 ` Yoshihiro Shimoda 2017-12-21 12:24 ` Yoshihiro Shimoda 2017-12-21 14:23 ` Ulf Hansson 2017-12-21 14:23 ` Ulf Hansson 2017-12-23 9:55 ` kbuild test robot 2017-12-23 9:55 ` kbuild test robot 2017-12-23 10:08 ` kbuild test robot [this message] 2017-12-23 10:08 ` kbuild test robot 2017-12-20 14:09 ` [PATCH v2 3/3] phy: core: Update the runtime PM section in the docs to reflect changes Ulf Hansson
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=201712231728.94eR1hbd%fengguang.wu@intel.com \ --to=lkp@intel.com \ --cc=geert@linux-m68k.org \ --cc=kbuild-all@01.org \ --cc=kishon@ti.com \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-pm@vger.kernel.org \ --cc=linux-renesas-soc@vger.kernel.org \ --cc=rjw@rjwysocki.net \ --cc=ulf.hansson@linaro.org \ --cc=yoshihiro.shimoda.uh@renesas.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: linkBe 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.