| Message ID | 1599468981-17301-6-git-send-email-guohanjun@huawei.com |
|---|---|
| State | New |
| Headers | show |
| Series | Cleanups for ACPI APD driver | expand |
On 2020/9/7 19:19, kernel test robot wrote: > Hi Hanjun, > > I love your patch! Perhaps something to improve: > > [auto build test WARNING on pm/linux-next] > [also build test WARNING on v5.9-rc4 next-20200903] > [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/Hanjun-Guo/Cleanups-for-ACPI-APD-driver/20200907-170548 > base: https://git.kernel.org/pub/scm/linux/kernel/git/rafael/linux-pm.git linux-next > config: x86_64-randconfig-s022-20200907 (attached as .config) > compiler: gcc-9 (Debian 9.3.0-15) 9.3.0 > reproduce: > # apt-get install sparse > # sparse version: v0.6.2-191-g10164920-dirty > # save the attached .config to linux build tree > make W=1 C=1 CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__' 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/acpi/acpi_apd.c:250:13: warning: no previous prototype for 'acpi_apd_init' [-Wmissing-prototypes] > 250 | void __init acpi_apd_init(void) > | ^~~~~~~~~~~~~ Hmm, I didn't get this locally, I think it's because acpi_apd_init() is added in the head file internal.h, but I removed the internal.h inclusion for head file, I will add it back. Thanks Hanjun
diff --git a/drivers/acpi/acpi_apd.c b/drivers/acpi/acpi_apd.c index 561d9ba..5d50ca7 100644 --- a/drivers/acpi/acpi_apd.c +++ b/drivers/acpi/acpi_apd.c @@ -7,17 +7,13 @@ * Wu, Jeff <Jeff.Wu@amd.com> */ -#include <linux/clk-provider.h> -#include <linux/platform_data/clk-fch.h> -#include <linux/platform_device.h> -#include <linux/pm_domain.h> -#include <linux/clkdev.h> #include <linux/acpi.h> +#include <linux/clkdev.h> +#include <linux/clk-provider.h> #include <linux/err.h> #include <linux/io.h> -#include <linux/pm.h> - -#include "internal.h" +#include <linux/platform_data/clk-fch.h> +#include <linux/platform_device.h> struct apd_private_data;
Make the including head files to be alphabetical order, and remove the not needed head file inclusion. Signed-off-by: Hanjun Guo <guohanjun@huawei.com> --- drivers/acpi/acpi_apd.c | 12 ++++-------- 1 file changed, 4 insertions(+), 8 deletions(-)