Re: [PATCH v4 2/2] hv_utils: implement Hyper-V PTP source

From: kbuild test robot
Date: Thu Jan 19 2017 - 13:21:04 EST


Hi Vitaly,

[auto build test WARNING on linus/master]
[also build test WARNING on v4.10-rc4 next-20170119]
[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/Vitaly-Kuznetsov/hv_util-adjust-system-time-smoothly/20170120-011342
config: x86_64-rhel (attached as .config)
compiler: gcc-6 (Debian 6.2.0-3) 6.2.0 20160901
reproduce:
# save the attached .config to linux build tree
make ARCH=x86_64

All warnings (new ones prefixed by >>):

drivers/hv/hv_util.c: In function 'hv_timesync_init':
>> drivers/hv/hv_util.c:561:36: warning: passing argument 1 of 'ptp_clock_register' discards 'const' qualifier from pointer target type [-Wdiscarded-qualifiers]
hv_ptp_clock = ptp_clock_register(&ptp_hyperv_info, NULL);
^
In file included from drivers/hv/hv_util.c:30:0:
include/linux/ptp_clock_kernel.h:172:26: note: expected 'struct ptp_clock_info *' but argument is of type 'const struct ptp_clock_info *'
extern struct ptp_clock *ptp_clock_register(struct ptp_clock_info *info,
^~~~~~~~~~~~~~~~~~

vim +561 drivers/hv/hv_util.c

545 .gettime64 = hv_ptp_gettime,
546 .settime64 = hv_ptp_settime,
547 .owner = THIS_MODULE,
548 };
549
550 static struct ptp_clock *hv_ptp_clock;
551
552 static int hv_timesync_init(struct hv_util_service *srv)
553 {
554 INIT_WORK(&wrk.work, hv_set_host_time);
555
556 /*
557 * ptp_clock_register() returns NULL when CONFIG_PTP_1588_CLOCK is
558 * disabled but the driver is still useful without the PTP device
559 * as it still handles the ICTIMESYNCFLAG_SYNC case.
560 */
> 561 hv_ptp_clock = ptp_clock_register(&ptp_hyperv_info, NULL);
562 if (IS_ERR_OR_NULL(hv_ptp_clock)) {
563 pr_err("cannot register PTP clock: %ld\n",
564 PTR_ERR(hv_ptp_clock));
565 hv_ptp_clock = NULL;
566 }
567
568 return 0;
569 }

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

Attachment: .config.gz
Description: application/gzip