Re: [PATCH v2 1/5] drivers: usb: chipidea: Add qoriq platform driver

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




Hi,

[auto build test WARNING on phy/next]
[also build test WARNING on v4.7-rc6 next-20160708]
[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/Rajesh-Bhagat/drivers-usb-chipidea-Add-qoriq-platform-driver/20160709-130557
base:   https://git.kernel.org/pub/scm/linux/kernel/git/kishon/linux-phy.git next
config: i386-allmodconfig (attached as .config)
compiler: gcc-6 (Debian 6.1.1-1) 6.1.1 20160430
reproduce:
        # save the attached .config to linux build tree
        make ARCH=i386 

All warnings (new ones prefixed by >>):

   In file included from include/linux/printk.h:289:0,
                    from include/linux/kernel.h:13,
                    from include/linux/list.h:8,
                    from include/linux/module.h:9,
                    from drivers/usb/chipidea/ci_hdrc_qoriq.c:12:
   drivers/usb/chipidea/ci_hdrc_qoriq.c: In function 'ci_hdrc_qoriq_usb_setup':
>> drivers/usb/chipidea/ci_hdrc_qoriq.c:85:15: warning: format '%llx' expects argument of type 'long long unsigned int', but argument 4 has type 'resource_size_t {aka unsigned int}' [-Wformat=]
     dev_dbg(dev, "res->start %llx, resource_size(res) %llx\n", res->start,
                  ^
   include/linux/dynamic_debug.h:86:39: note: in definition of macro 'dynamic_dev_dbg'
      __dynamic_dev_dbg(&descriptor, dev, fmt, \
                                          ^~~
>> drivers/usb/chipidea/ci_hdrc_qoriq.c:85:2: note: in expansion of macro 'dev_dbg'
     dev_dbg(dev, "res->start %llx, resource_size(res) %llx\n", res->start,
     ^~~~~~~
   drivers/usb/chipidea/ci_hdrc_qoriq.c:85:15: warning: format '%llx' expects argument of type 'long long unsigned int', but argument 5 has type 'resource_size_t {aka unsigned int}' [-Wformat=]
     dev_dbg(dev, "res->start %llx, resource_size(res) %llx\n", res->start,
                  ^
   include/linux/dynamic_debug.h:86:39: note: in definition of macro 'dynamic_dev_dbg'
      __dynamic_dev_dbg(&descriptor, dev, fmt, \
                                          ^~~
>> drivers/usb/chipidea/ci_hdrc_qoriq.c:85:2: note: in expansion of macro 'dev_dbg'
     dev_dbg(dev, "res->start %llx, resource_size(res) %llx\n", res->start,
     ^~~~~~~

vim +85 drivers/usb/chipidea/ci_hdrc_qoriq.c

     6	 *
     7	 * This program is free software; you can redistribute it and/or modify
     8	 * it under the terms of the GNU General Public License version 2 as
     9	 * published by the Free Software Foundation.
    10	 *
    11	 */
  > 12	#include <linux/module.h>
    13	#include <linux/of_platform.h>
    14	#include <linux/of_gpio.h>
    15	#include <linux/platform_device.h>
    16	#include <linux/pm_runtime.h>
    17	#include <linux/dma-mapping.h>
    18	#include <linux/usb/of.h>
    19	#include <linux/usb/chipidea.h>
    20	#include <linux/clk.h>
    21	
    22	#include "ci.h"
    23	#include "ci_hdrc_qoriq.h"
    24	
    25	struct ci_hdrc_qoriq_data {
    26		struct phy *phy;
    27		struct clk *clk;
    28		void __iomem *qoriq_regs;
    29		struct platform_device *ci_pdev;
    30		enum usb_phy_interface phy_mode;
    31	};
    32	
    33	/*
    34	 * clock helper functions
    35	 */
    36	static int ci_hdrc_qoriq_get_clks(struct platform_device *pdev)
    37	{
    38		int ret;
    39		struct device *dev = &pdev->dev;
    40		struct ci_hdrc_qoriq_data *data = platform_get_drvdata(pdev);
    41	
    42		data->clk = devm_clk_get(dev, "usb2-clock");
    43		if (IS_ERR(data->clk)) {
    44			dev_err(dev, "failed to get clk, err=%ld\n",
    45						PTR_ERR(data->clk));
    46				return ret;
    47		}
    48		return 0;
    49	}
    50	
    51	static int ci_hdrc_qoriq_prepare_enable_clks(struct platform_device *pdev)
    52	{
    53		int ret;
    54		struct device *dev = &pdev->dev;
    55		struct ci_hdrc_qoriq_data *data = platform_get_drvdata(pdev);
    56	
    57		ret = clk_prepare_enable(data->clk);
    58		if (ret) {
    59			dev_err(dev, "failed to prepare/enable clk, err=%d\n", ret);
    60			return ret;
    61		}
    62		return 0;
    63	}
    64	
    65	static void ci_hdrc_qoriq_disable_unprepare_clks(struct platform_device *pdev)
    66	{
    67		struct ci_hdrc_qoriq_data *data = platform_get_drvdata(pdev);
    68	
    69		clk_disable_unprepare(data->clk);
    70	}
    71	
    72	static int ci_hdrc_qoriq_usb_setup(struct platform_device *pdev)
    73	{
    74		u32 reg;
    75		struct resource *res;
    76		struct device *dev = &pdev->dev;
    77		struct ci_hdrc_qoriq_data *data = platform_get_drvdata(pdev);
    78	
    79		res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
    80		if (!res) {
    81			dev_err(dev, "failed to get I/O memory\n");
    82			return -ENOENT;
    83		}
    84	
  > 85		dev_dbg(dev, "res->start %llx, resource_size(res) %llx\n", res->start,
    86			resource_size(res));
    87		data->qoriq_regs = devm_ioremap(dev, res->start, resource_size(res));
    88		if (IS_ERR(data->qoriq_regs)) {

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation

Attachment: .config.gz
Description: Binary data


[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]
  Powered by Linux