Re: [PATCH 3/4] smiapp: Get clock rate if it's not available through DT

kbuild test robot
2017-02-13T18:27:36Z

[-- Attachment #1: Type: text/plain, Size: 2015 bytes --]

Hi Sakari,

[auto build test WARNING on linus/master]
[also build test WARNING on v4.10-rc8]
[cannot apply to linuxtv-media/master next-20170213]
[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/Sakari-Ailus/smiapp-cleanups-clock-control-changes/20170214-010429
config: i386-allmodconfig (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=i386

All warnings (new ones prefixed by >>):

drivers/media/i2c/smiapp/smiapp-core.c: In function 'smiapp_probe':
>> drivers/media/i2c/smiapp/smiapp-core.c:2908:41: warning: format '%lu' expects argument of type 'long unsigned int', but argument 3 has type 'uint32_t {aka unsigned int}' [-Wformat=]
"can't set clock freq, asked for %lu but got %lu\n",
^

vim +2908 drivers/media/i2c/smiapp/smiapp-core.c

2892 if (sensor->ext_clk) {
2893 if (sensor->hwcfg->ext_clk) {
2894 unsigned long rate;
2895
2896 rval = clk_set_rate(sensor->ext_clk,
2897 sensor->hwcfg->ext_clk);
2898 if (rval < 0) {
2899 dev_err(&client->dev,
2900 "unable to set clock freq to %u\n",
2901 sensor->hwcfg->ext_clk);
2902 return rval;
2903 }
2904
2905 rate = clk_get_rate(sensor->ext_clk);
2906 if (rate != sensor->hwcfg->ext_clk) {
2907 dev_err(&client->dev,
> 2908 "can't set clock freq, asked for %lu but got %lu\n",
2909 sensor->hwcfg->ext_clk, rate);
2910 return rval;
2911 }
2912 } else {
2913 sensor->hwcfg->ext_clk = clk_get_rate(sensor->ext_clk);
2914 dev_dbg(&client->dev, "obtained clock freq %u\n",
2915 sensor->hwcfg->ext_clk);
2916 }

---
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: 57940 bytes --]