Re: [RFC 01/28] clk: renesas: rzg2l: Add FOUTPOSTDIV clk support
by kernel test robot
Hi Biju,
[FYI, it's a private test report for your RFC patch.]
[auto build test WARNING on next-20220112]
[cannot apply to pinchartl-media/drm/du/next geert-renesas-drivers/renesas-clk geert-renesas-devel/next v5.16 v5.16-rc8 v5.16-rc7 v5.16]
[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/Biju-Das/Add-RZ-G2L-Display-supp...
base: 32ce2abb03cfae17a9eb42bd6b1b619b72f23f20
config: arm-randconfig-r021-20220113 (https://download.01.org/0day-ci/archive/20220113/202201131731.OhbX4Dpr-lk...)
compiler: clang version 14.0.0 (https://github.com/llvm/llvm-project d1021978b8e7e35dcc30201ca1731d64b5a602a8)
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# install arm cross compiling tool for clang build
# apt-get install binutils-arm-linux-gnueabi
# https://github.com/0day-ci/linux/commit/c0b834d1561a902d5f960a18e365d66fb...
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Biju-Das/Add-RZ-G2L-Display-support/20220113-014940
git checkout c0b834d1561a902d5f960a18e365d66fbeba0f95
# save the config file to linux build tree
mkdir build_dir
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir ARCH=arm SHELL=/bin/bash drivers/clk/renesas/
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
All warnings (new ones prefixed by >>):
>> drivers/clk/renesas/rzg2l-cpg.c:302:4: warning: integer literal is too large to be represented in type 'long', interpreting as 'unsigned long' per C89; this literal will have type 'long long' in C99 onwards [-Wc99-compat]
{ 3000000000, 1, 125, 0, 1, 1, 0, 0, 0, 0 }, /* 3000 MHz */
^
drivers/clk/renesas/rzg2l-cpg.c:317:4: warning: integer literal is too large to be represented in type 'long', interpreting as 'unsigned long' per C89; this literal will have type 'long long' in C99 onwards [-Wc99-compat]
{ 3000000000, 1, 125, 0, 1, 1, 0, 0, 0, 0 }, /* 3000 MHz */
^
2 warnings generated.
vim +302 drivers/clk/renesas/rzg2l-cpg.c
286
287 static const struct rzg2l_pll5_param dsi_mode_param[] = {
288 { 25175000, 1, 16, 13141593, 1, 1, 2, 3, 16, 0 }, /* VGA 25.175MHz */
289 { 25200000, 1, 16, 13421773, 1, 1, 2, 3, 16, 0 }, /* VGA 25.200MHz */
290 { 27000000, 1, 18, 0, 1, 1, 2, 3, 16, 0 }, /* 480p/576p 27.000MHz */
291 { 27027000, 1, 18, 301990, 1, 1, 2, 3, 16, 0 }, /* 480p 27.027MHz */
292 { 29605000, 1, 19, 12359216, 1, 1, 2, 3, 16, 0 }, /* WVGA 29.605MHz */
293 { 40000000, 2, 80, 0, 2, 1, 1, 2, 6, 0 }, /* SVGA 40.00MHz */
294 { 65000000, 1, 43, 5592405, 1, 1, 2, 3, 16, 0 }, /* XGA 65.00MHz */
295 { 71000000, 2, 71, 0, 1, 1, 1, 2, 6, 0 }, /* WXGA 1280x800 71.0MHz */
296 { 74176000, 1, 49, 7560932, 1, 1, 2, 3, 16, 0 }, /* 720p 74.176MHz */
297 { 74250000, 1, 49, 8388608, 1, 1, 2, 3, 16, 0 }, /* 720p 74.25MHz */
298 { 85500000, 2, 85, 8388608, 1, 1, 1, 2, 6, 0 }, /* FWXGA 1360x768 85.5MHz */
299 { 88750000, 2, 88, 12582912, 1, 1, 1, 2, 6, 1 }, /* WXGA+ 1440x900 88.75MHz */
300 { 108000000, 2, 108, 0, 1, 1, 1, 2, 6, 1 }, /* SXGA 108MHz */
301 { 148500000, 2, 148, 8388608, 1, 1, 1, 2, 6, 1 }, /* 1080p 148.5MHz */
> 302 { 3000000000, 1, 125, 0, 1, 1, 0, 0, 0, 0 }, /* 3000 MHz */
303 };
304
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
8 months, 1 week
Re: [PATCH v1 3/9] drivers: hv: dxgkrnl: Implement creation/destruction of GPU allocations/resources
by kernel test robot
Hi Iouri,
I love your patch! Perhaps something to improve:
[auto build test WARNING on linus/master]
[also build test WARNING on v5.16 next-20220113]
[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/Iouri-Tarassov/drivers-hv-dxgkrn...
base: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git e3084ed48fd6b661fe434da0cb36d7d6706cf27f
config: arm64-randconfig-r032-20220113 (https://download.01.org/0day-ci/archive/20220113/202201131642.1nercfCr-lk...)
compiler: clang version 14.0.0 (https://github.com/llvm/llvm-project d1021978b8e7e35dcc30201ca1731d64b5a602a8)
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# install arm64 cross compiling tool for clang build
# apt-get install binutils-aarch64-linux-gnu
# https://github.com/0day-ci/linux/commit/a2aa8c606c48a4e6bf8a7a51e2e4e5738...
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Iouri-Tarassov/drivers-hv-dxgkrnl-Driver-overview/20220113-035836
git checkout a2aa8c606c48a4e6bf8a7a51e2e4e5738e35da32
# save the config file to linux build tree
mkdir build_dir
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir ARCH=arm64 SHELL=/bin/bash drivers/hv/dxgkrnl/
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
All warnings (new ones prefixed by >>):
In file included from drivers/hv/dxgkrnl/ioctl.c:21:
drivers/hv/dxgkrnl/dxgvmbus.h:867:26: warning: implicit conversion from enumeration type 'enum dxgkvmb_commandtype' to different enumeration type 'enum dxgkvmb_commandtype_global' [-Wenum-conversion]
command->command_type = DXGK_VMBCOMMAND_INVALID;
~ ^~~~~~~~~~~~~~~~~~~~~~~
>> drivers/hv/dxgkrnl/ioctl.c:1245:5: warning: no previous prototype for function 'validate_alloc' [-Wmissing-prototypes]
int validate_alloc(struct dxgallocation *alloc0,
^
drivers/hv/dxgkrnl/ioctl.c:1245:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
int validate_alloc(struct dxgallocation *alloc0,
^
static
2 warnings generated.
--
In file included from drivers/hv/dxgkrnl/dxgvmbus.c:23:
drivers/hv/dxgkrnl/dxgvmbus.h:867:26: warning: implicit conversion from enumeration type 'enum dxgkvmb_commandtype' to different enumeration type 'enum dxgkvmb_commandtype_global' [-Wenum-conversion]
command->command_type = DXGK_VMBCOMMAND_INVALID;
~ ^~~~~~~~~~~~~~~~~~~~~~~
drivers/hv/dxgkrnl/dxgvmbus.c:151:5: warning: no previous prototype for function 'ntstatus2int' [-Wmissing-prototypes]
int ntstatus2int(struct ntstatus status)
^
drivers/hv/dxgkrnl/dxgvmbus.c:151:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
int ntstatus2int(struct ntstatus status)
^
static
drivers/hv/dxgkrnl/dxgvmbus.c:254:6: warning: no previous prototype for function 'process_inband_packet' [-Wmissing-prototypes]
void process_inband_packet(struct dxgvmbuschannel *channel,
^
drivers/hv/dxgkrnl/dxgvmbus.c:254:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
void process_inband_packet(struct dxgvmbuschannel *channel,
^
static
drivers/hv/dxgkrnl/dxgvmbus.c:272:6: warning: no previous prototype for function 'process_completion_packet' [-Wmissing-prototypes]
void process_completion_packet(struct dxgvmbuschannel *channel,
^
drivers/hv/dxgkrnl/dxgvmbus.c:272:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
void process_completion_packet(struct dxgvmbuschannel *channel,
^
static
drivers/hv/dxgkrnl/dxgvmbus.c:398:5: warning: no previous prototype for function 'dxgvmb_send_async_msg' [-Wmissing-prototypes]
int dxgvmb_send_async_msg(struct dxgvmbuschannel *channel,
^
drivers/hv/dxgkrnl/dxgvmbus.c:398:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
int dxgvmb_send_async_msg(struct dxgvmbuschannel *channel,
^
static
>> drivers/hv/dxgkrnl/dxgvmbus.c:909:5: warning: no previous prototype for function 'create_existing_sysmem' [-Wmissing-prototypes]
int create_existing_sysmem(struct dxgdevice *device,
^
drivers/hv/dxgkrnl/dxgvmbus.c:909:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
int create_existing_sysmem(struct dxgdevice *device,
^
static
drivers/hv/dxgkrnl/dxgvmbus.c:234:20: warning: unused function 'command_vm_to_host_init0' [-Wunused-function]
static inline void command_vm_to_host_init0(struct dxgkvmb_command_vm_to_host
^
7 warnings generated.
vim +/validate_alloc +1245 drivers/hv/dxgkrnl/ioctl.c
1244
> 1245 int validate_alloc(struct dxgallocation *alloc0,
1246 struct dxgallocation *alloc,
1247 struct dxgdevice *device,
1248 struct d3dkmthandle alloc_handle)
1249 {
1250 u32 fail_reason;
1251
1252 if (alloc == NULL) {
1253 fail_reason = 1;
1254 goto cleanup;
1255 }
1256 if (alloc->resource_owner != alloc0->resource_owner) {
1257 fail_reason = 2;
1258 goto cleanup;
1259 }
1260 if (alloc->resource_owner) {
1261 if (alloc->owner.resource != alloc0->owner.resource) {
1262 fail_reason = 3;
1263 goto cleanup;
1264 }
1265 if (alloc->owner.resource->device != device) {
1266 fail_reason = 4;
1267 goto cleanup;
1268 }
1269 if (alloc->owner.resource->shared_owner) {
1270 fail_reason = 5;
1271 goto cleanup;
1272 }
1273 } else {
1274 if (alloc->owner.device != device) {
1275 fail_reason = 6;
1276 goto cleanup;
1277 }
1278 }
1279 return 0;
1280 cleanup:
1281 pr_err("Alloc validation failed: reason: %d %x",
1282 fail_reason, alloc_handle.v);
1283 return -EINVAL;
1284 }
1285
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
8 months, 1 week
Re: [PATCH v4 3/3] m68k: virt: Remove LEGACY_TIMER_TICK
by kernel test robot
Hi Laurent,
Thank you for the patch! Perhaps something to improve:
[auto build test WARNING on geert-m68k/for-next]
[also build test WARNING on linux/master gerg-m68knommu/for-next linus/master v5.16 next-20220113]
[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/Laurent-Vivier/m68k-Add-Virtual-...
base: https://git.kernel.org/pub/scm/linux/kernel/git/geert/linux-m68k.git for-next
config: m68k-allyesconfig (https://download.01.org/0day-ci/archive/20220113/202201131603.TkF80Rn5-lk...)
compiler: m68k-linux-gcc (GCC) 11.2.0
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# https://github.com/0day-ci/linux/commit/f169b75339fd0f82fc1b5ed82611f70fc...
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Laurent-Vivier/m68k-Add-Virtual-M68k-Machine/20220113-054106
git checkout f169b75339fd0f82fc1b5ed82611f70fccc1401f
# save the config file to linux build tree
mkdir build_dir
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.2.0 make.cross O=build_dir ARCH=m68k SHELL=/bin/bash arch/m68k/virt/
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
All warnings (new ones prefixed by >>):
>> arch/m68k/virt/timer.c:45:46: warning: 'struct clock_event_device' declared inside parameter list will not be visible outside of this definition or declaration
45 | static int goldfish_timer_set_oneshot(struct clock_event_device *evt)
| ^~~~~~~~~~~~~~~~~~
arch/m68k/virt/timer.c:55:43: warning: 'struct clock_event_device' declared inside parameter list will not be visible outside of this definition or declaration
55 | static int goldfish_timer_shutdown(struct clock_event_device *evt)
| ^~~~~~~~~~~~~~~~~~
arch/m68k/virt/timer.c:63:45: warning: 'struct clock_event_device' declared inside parameter list will not be visible outside of this definition or declaration
63 | struct clock_event_device *evt)
| ^~~~~~~~~~~~~~~~~~
arch/m68k/virt/timer.c:79:8: error: variable 'goldfish_timer_clockevent' has initializer but incomplete type
79 | struct clock_event_device goldfish_timer_clockevent = {
| ^~~~~~~~~~~~~~~~~~
arch/m68k/virt/timer.c:80:10: error: 'struct clock_event_device' has no member named 'name'
80 | .name = "goldfish_timer",
| ^~~~
>> arch/m68k/virt/timer.c:80:35: warning: excess elements in struct initializer
80 | .name = "goldfish_timer",
| ^~~~~~~~~~~~~~~~
arch/m68k/virt/timer.c:80:35: note: (near initialization for 'goldfish_timer_clockevent')
arch/m68k/virt/timer.c:81:10: error: 'struct clock_event_device' has no member named 'features'
81 | .features = CLOCK_EVT_FEAT_ONESHOT,
| ^~~~~~~~
arch/m68k/virt/timer.c:81:35: error: 'CLOCK_EVT_FEAT_ONESHOT' undeclared here (not in a function)
81 | .features = CLOCK_EVT_FEAT_ONESHOT,
| ^~~~~~~~~~~~~~~~~~~~~~
arch/m68k/virt/timer.c:81:35: warning: excess elements in struct initializer
arch/m68k/virt/timer.c:81:35: note: (near initialization for 'goldfish_timer_clockevent')
arch/m68k/virt/timer.c:82:10: error: 'struct clock_event_device' has no member named 'set_state_shutdown'
82 | .set_state_shutdown = goldfish_timer_shutdown,
| ^~~~~~~~~~~~~~~~~~
arch/m68k/virt/timer.c:82:35: warning: excess elements in struct initializer
82 | .set_state_shutdown = goldfish_timer_shutdown,
| ^~~~~~~~~~~~~~~~~~~~~~~
arch/m68k/virt/timer.c:82:35: note: (near initialization for 'goldfish_timer_clockevent')
arch/m68k/virt/timer.c:83:10: error: 'struct clock_event_device' has no member named 'set_state_oneshot'
83 | .set_state_oneshot = goldfish_timer_set_oneshot,
| ^~~~~~~~~~~~~~~~~
arch/m68k/virt/timer.c:83:35: warning: excess elements in struct initializer
83 | .set_state_oneshot = goldfish_timer_set_oneshot,
| ^~~~~~~~~~~~~~~~~~~~~~~~~~
arch/m68k/virt/timer.c:83:35: note: (near initialization for 'goldfish_timer_clockevent')
arch/m68k/virt/timer.c:84:10: error: 'struct clock_event_device' has no member named 'set_next_event'
84 | .set_next_event = goldfish_timer_next_event,
| ^~~~~~~~~~~~~~
arch/m68k/virt/timer.c:84:35: warning: excess elements in struct initializer
84 | .set_next_event = goldfish_timer_next_event,
| ^~~~~~~~~~~~~~~~~~~~~~~~~
arch/m68k/virt/timer.c:84:35: note: (near initialization for 'goldfish_timer_clockevent')
arch/m68k/virt/timer.c:85:10: error: 'struct clock_event_device' has no member named 'shift'
85 | .shift = 32,
| ^~~~~
arch/m68k/virt/timer.c:85:35: warning: excess elements in struct initializer
85 | .shift = 32,
| ^~
arch/m68k/virt/timer.c:85:35: note: (near initialization for 'goldfish_timer_clockevent')
arch/m68k/virt/timer.c: In function 'golfish_timer_tick':
arch/m68k/virt/timer.c:92:12: error: invalid use of undefined type 'struct clock_event_device'
92 | evt->event_handler(evt);
| ^~
arch/m68k/virt/timer.c: In function 'virt_sched_init':
arch/m68k/virt/timer.c:110:9: error: implicit declaration of function 'clockevents_config_and_register' [-Werror=implicit-function-declaration]
110 | clockevents_config_and_register(&goldfish_timer_clockevent, NSEC_PER_SEC,
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
arch/m68k/virt/timer.c: At top level:
arch/m68k/virt/timer.c:79:27: error: storage size of 'goldfish_timer_clockevent' isn't known
79 | struct clock_event_device goldfish_timer_clockevent = {
| ^~~~~~~~~~~~~~~~~~~~~~~~~
cc1: some warnings being treated as errors
vim +45 arch/m68k/virt/timer.c
44
> 45 static int goldfish_timer_set_oneshot(struct clock_event_device *evt)
46 {
47 gf_timer->alarm_high = 0;
48 gf_timer->alarm_low = 0;
49
50 gf_timer->irq_enabled = 1;
51
52 return 0;
53 }
54
55 static int goldfish_timer_shutdown(struct clock_event_device *evt)
56 {
57 gf_timer->irq_enabled = 0;
58
59 return 0;
60 }
61
62 static int goldfish_timer_next_event(unsigned long delta,
63 struct clock_event_device *evt)
64 {
65 u64 now;
66
67 gf_timer->clear_interrupt = 1;
68
69 now = goldfish_timer_read(NULL);
70
71 now += delta;
72
73 gf_timer->alarm_high = upper_32_bits(now);
74 gf_timer->alarm_low = lower_32_bits(now);
75
76 return 0;
77 }
78
79 struct clock_event_device goldfish_timer_clockevent = {
> 80 .name = "goldfish_timer",
81 .features = CLOCK_EVT_FEAT_ONESHOT,
82 .set_state_shutdown = goldfish_timer_shutdown,
83 .set_state_oneshot = goldfish_timer_set_oneshot,
84 .set_next_event = goldfish_timer_next_event,
85 .shift = 32,
86 };
87
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
8 months, 1 week
[chrome-os:chromeos-5.4 9999/9999] drivers/bluetooth/btintel.c:1392:24: warning: format '%u' expects argument of type 'unsigned int', but argument 3 has type 'long unsigned int'
by kernel test robot
tree: https://chromium.googlesource.com/chromiumos/third_party/kernel chromeos-5.4
head: e49b215c996cd58949812f549dd65b0f6c7706bf
commit: e49b215c996cd58949812f549dd65b0f6c7706bf [9999/9999] CHROMIUM: Bluetooth: surface Intel telemetry events through mgmt
config: riscv-randconfig-r042-20220113 (https://download.01.org/0day-ci/archive/20220113/202201131612.vPCz3r4U-lk...)
compiler: riscv64-linux-gcc (GCC) 11.2.0
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
git remote add chrome-os https://chromium.googlesource.com/chromiumos/third_party/kernel
git fetch --no-tags chrome-os chromeos-5.4
git checkout e49b215c996cd58949812f549dd65b0f6c7706bf
# save the config file to linux build tree
mkdir build_dir
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.2.0 make.cross O=build_dir ARCH=riscv SHELL=/bin/bash
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
All warnings (new ones prefixed by >>):
In file included from drivers/bluetooth/btintel.c:14:
drivers/bluetooth/btintel.c: In function 'btintel_pull_quality_report_data':
>> drivers/bluetooth/btintel.c:1392:24: warning: format '%u' expects argument of type 'unsigned int', but argument 3 has type 'long unsigned int' [-Wformat=]
1392 | BT_ERR("Telemetry event length %d too short (at least %u)",
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
1393 | skb->len, sizeof(struct intel_tlv));
| ~~~~~~~~~~~~~~~~~~~~~~~~
| |
| long unsigned int
include/net/bluetooth/bluetooth.h:166:26: note: in definition of macro 'BT_ERR'
166 | bt_err(BT_PREFIX fmt "\n" BT_PREFIX_PARAM, ##__VA_ARGS__)
| ^~~
drivers/bluetooth/btintel.c:1392:72: note: format string is defined here
1392 | BT_ERR("Telemetry event length %d too short (at least %u)",
| ~^
| |
| unsigned int
| %lu
vim +1392 drivers/bluetooth/btintel.c
1385
1386 bool btintel_pull_quality_report_data(struct sk_buff *skb)
1387 {
1388 skb_pull(skb, sizeof(struct intel_prefix_evt_data));
1389
1390 /* A telemetry event contains at least one intel_tlv subevent. */
1391 if (skb->len < sizeof(struct intel_tlv)) {
> 1392 BT_ERR("Telemetry event length %d too short (at least %u)",
1393 skb->len, sizeof(struct intel_tlv));
1394 return false;
1395 }
1396
1397 return true;
1398 }
1399 EXPORT_SYMBOL_GPL(btintel_pull_quality_report_data);
1400
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
8 months, 1 week