drivers/media/v4l2-core/v4l2-ioctl.c:303:28: warning: taking address of packed member 'pixelformat' of class or structure 'v4l2_pix_format_mplane' may result in an unaligned pointer value
by kernel test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: 8096acd7442e613fad0354fc8dfdb2003cceea0b
commit: e927e1e0f0dd3e353d5556503a71484008692c82 v4l: ioctl: Use %p4cc printk modifier to print FourCC codes
date: 5 months ago
config: mips-randconfig-r004-20210714 (attached as .config)
compiler: clang version 13.0.0 (https://github.com/llvm/llvm-project 8d69635ed9ecf36fd0ca85906bfde17949671cbe)
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 mips cross compiling tool for clang build
# apt-get install binutils-mips-linux-gnu
# https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit...
git remote add linus https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
git fetch --no-tags linus master
git checkout e927e1e0f0dd3e353d5556503a71484008692c82
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=mips
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/media/v4l2-core/v4l2-ioctl.c:303:28: warning: taking address of packed member 'pixelformat' of class or structure 'v4l2_pix_format_mplane' may result in an unaligned pointer value [-Waddress-of-packed-member]
mp->width, mp->height, &mp->pixelformat,
^~~~~~~~~~~~~~~
include/linux/printk.h:385:26: note: expanded from macro 'pr_cont'
printk(KERN_CONT fmt, ##__VA_ARGS__)
^~~~~~~~~~~
>> drivers/media/v4l2-core/v4l2-ioctl.c:347:37: warning: taking address of packed member 'pixelformat' of class or structure 'v4l2_sdr_format' may result in an unaligned pointer value [-Waddress-of-packed-member]
pr_cont(", pixelformat=%p4cc\n", &sdr->pixelformat);
^~~~~~~~~~~~~~~~
include/linux/printk.h:385:26: note: expanded from macro 'pr_cont'
printk(KERN_CONT fmt, ##__VA_ARGS__)
^~~~~~~~~~~
>> drivers/media/v4l2-core/v4l2-ioctl.c:353:5: warning: taking address of packed member 'dataformat' of class or structure 'v4l2_meta_format' may result in an unaligned pointer value [-Waddress-of-packed-member]
&meta->dataformat, meta->buffersize);
^~~~~~~~~~~~~~~~
include/linux/printk.h:385:26: note: expanded from macro 'pr_cont'
printk(KERN_CONT fmt, ##__VA_ARGS__)
^~~~~~~~~~~
3 warnings generated.
vim +303 drivers/media/v4l2-core/v4l2-ioctl.c
273
274 static void v4l_print_format(const void *arg, bool write_only)
275 {
276 const struct v4l2_format *p = arg;
277 const struct v4l2_pix_format *pix;
278 const struct v4l2_pix_format_mplane *mp;
279 const struct v4l2_vbi_format *vbi;
280 const struct v4l2_sliced_vbi_format *sliced;
281 const struct v4l2_window *win;
282 const struct v4l2_sdr_format *sdr;
283 const struct v4l2_meta_format *meta;
284 u32 planes;
285 unsigned i;
286
287 pr_cont("type=%s", prt_names(p->type, v4l2_type_names));
288 switch (p->type) {
289 case V4L2_BUF_TYPE_VIDEO_CAPTURE:
290 case V4L2_BUF_TYPE_VIDEO_OUTPUT:
291 pix = &p->fmt.pix;
292 pr_cont(", width=%u, height=%u, pixelformat=%p4cc, field=%s, bytesperline=%u, sizeimage=%u, colorspace=%d, flags=0x%x, ycbcr_enc=%u, quantization=%u, xfer_func=%u\n",
293 pix->width, pix->height, &pix->pixelformat,
294 prt_names(pix->field, v4l2_field_names),
295 pix->bytesperline, pix->sizeimage,
296 pix->colorspace, pix->flags, pix->ycbcr_enc,
297 pix->quantization, pix->xfer_func);
298 break;
299 case V4L2_BUF_TYPE_VIDEO_CAPTURE_MPLANE:
300 case V4L2_BUF_TYPE_VIDEO_OUTPUT_MPLANE:
301 mp = &p->fmt.pix_mp;
302 pr_cont(", width=%u, height=%u, format=%p4cc, field=%s, colorspace=%d, num_planes=%u, flags=0x%x, ycbcr_enc=%u, quantization=%u, xfer_func=%u\n",
> 303 mp->width, mp->height, &mp->pixelformat,
304 prt_names(mp->field, v4l2_field_names),
305 mp->colorspace, mp->num_planes, mp->flags,
306 mp->ycbcr_enc, mp->quantization, mp->xfer_func);
307 planes = min_t(u32, mp->num_planes, VIDEO_MAX_PLANES);
308 for (i = 0; i < planes; i++)
309 printk(KERN_DEBUG "plane %u: bytesperline=%u sizeimage=%u\n", i,
310 mp->plane_fmt[i].bytesperline,
311 mp->plane_fmt[i].sizeimage);
312 break;
313 case V4L2_BUF_TYPE_VIDEO_OVERLAY:
314 case V4L2_BUF_TYPE_VIDEO_OUTPUT_OVERLAY:
315 win = &p->fmt.win;
316 /* Note: we can't print the clip list here since the clips
317 * pointer is a userspace pointer, not a kernelspace
318 * pointer. */
319 pr_cont(", wxh=%dx%d, x,y=%d,%d, field=%s, chromakey=0x%08x, clipcount=%u, clips=%p, bitmap=%p, global_alpha=0x%02x\n",
320 win->w.width, win->w.height, win->w.left, win->w.top,
321 prt_names(win->field, v4l2_field_names),
322 win->chromakey, win->clipcount, win->clips,
323 win->bitmap, win->global_alpha);
324 break;
325 case V4L2_BUF_TYPE_VBI_CAPTURE:
326 case V4L2_BUF_TYPE_VBI_OUTPUT:
327 vbi = &p->fmt.vbi;
328 pr_cont(", sampling_rate=%u, offset=%u, samples_per_line=%u, sample_format=%p4cc, start=%u,%u, count=%u,%u\n",
329 vbi->sampling_rate, vbi->offset,
330 vbi->samples_per_line, &vbi->sample_format,
331 vbi->start[0], vbi->start[1],
332 vbi->count[0], vbi->count[1]);
333 break;
334 case V4L2_BUF_TYPE_SLICED_VBI_CAPTURE:
335 case V4L2_BUF_TYPE_SLICED_VBI_OUTPUT:
336 sliced = &p->fmt.sliced;
337 pr_cont(", service_set=0x%08x, io_size=%d\n",
338 sliced->service_set, sliced->io_size);
339 for (i = 0; i < 24; i++)
340 printk(KERN_DEBUG "line[%02u]=0x%04x, 0x%04x\n", i,
341 sliced->service_lines[0][i],
342 sliced->service_lines[1][i]);
343 break;
344 case V4L2_BUF_TYPE_SDR_CAPTURE:
345 case V4L2_BUF_TYPE_SDR_OUTPUT:
346 sdr = &p->fmt.sdr;
> 347 pr_cont(", pixelformat=%p4cc\n", &sdr->pixelformat);
348 break;
349 case V4L2_BUF_TYPE_META_CAPTURE:
350 case V4L2_BUF_TYPE_META_OUTPUT:
351 meta = &p->fmt.meta;
352 pr_cont(", dataformat=%p4cc, buffersize=%u\n",
> 353 &meta->dataformat, meta->buffersize);
354 break;
355 }
356 }
357
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
3 weeks, 6 days
arch/powerpc/kernel/time.c:801:5: error: no previous prototype for 'update_persistent_clock64'
by kernel test robot
tree: https://github.com/0day-ci/linux/commits/UPDATE-20210801-003033/Carlos-Bi...
head: fb6876d6185db9db41efb455500de67da0937a36
commit: fb6876d6185db9db41efb455500de67da0937a36 include: linux: Reorganize timekeeping and ktime headers
date: 3 hours ago
config: powerpc-bluestone_defconfig (attached as .config)
compiler: powerpc-linux-gcc (GCC) 10.3.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/fb6876d6185db9db41efb455500de67da...
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review UPDATE-20210801-003033/Carlos-Bilbao/include-linux-Reorganize-timekeeping-and-ktime-headers/20210722-233301
git checkout fb6876d6185db9db41efb455500de67da0937a36
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-10.3.0 make.cross ARCH=powerpc
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
All errors (new ones prefixed by >>):
>> arch/powerpc/kernel/time.c:801:5: error: no previous prototype for 'update_persistent_clock64' [-Werror=missing-prototypes]
801 | int update_persistent_clock64(struct timespec64 now)
| ^~~~~~~~~~~~~~~~~~~~~~~~~
>> arch/powerpc/kernel/time.c:840:6: error: no previous prototype for 'read_persistent_clock64' [-Werror=missing-prototypes]
840 | void read_persistent_clock64(struct timespec64 *ts)
| ^~~~~~~~~~~~~~~~~~~~~~~
cc1: all warnings being treated as errors
vim +/update_persistent_clock64 +801 arch/powerpc/kernel/time.c
10f7e7c15e6ce4 arch/ppc64/kernel/time.c Arnd Bergmann 2005-06-23 800
5235afa89a246c arch/powerpc/kernel/time.c Arnd Bergmann 2018-04-23 @801 int update_persistent_clock64(struct timespec64 now)
^1da177e4c3f41 arch/ppc64/kernel/time.c Linus Torvalds 2005-04-16 802 {
f2783c15007468 arch/powerpc/kernel/time.c Paul Mackerras 2005-10-20 803 struct rtc_time tm;
f2783c15007468 arch/powerpc/kernel/time.c Paul Mackerras 2005-10-20 804
aa3be5f32db137 arch/powerpc/kernel/time.c Tony Breeds 2007-09-21 805 if (!ppc_md.set_rtc_time)
023f333a99cee9 arch/powerpc/kernel/time.c Jason Gunthorpe 2012-12-17 806 return -ENODEV;
aa3be5f32db137 arch/powerpc/kernel/time.c Tony Breeds 2007-09-21 807
5235afa89a246c arch/powerpc/kernel/time.c Arnd Bergmann 2018-04-23 808 rtc_time64_to_tm(now.tv_sec + 1 + timezone_offset, &tm);
aa3be5f32db137 arch/powerpc/kernel/time.c Tony Breeds 2007-09-21 809
aa3be5f32db137 arch/powerpc/kernel/time.c Tony Breeds 2007-09-21 810 return ppc_md.set_rtc_time(&tm);
aa3be5f32db137 arch/powerpc/kernel/time.c Tony Breeds 2007-09-21 811 }
aa3be5f32db137 arch/powerpc/kernel/time.c Tony Breeds 2007-09-21 812
5bfd643583b2e2 arch/powerpc/kernel/time.c Arnd Bergmann 2018-04-23 813 static void __read_persistent_clock(struct timespec64 *ts)
aa3be5f32db137 arch/powerpc/kernel/time.c Tony Breeds 2007-09-21 814 {
aa3be5f32db137 arch/powerpc/kernel/time.c Tony Breeds 2007-09-21 815 struct rtc_time tm;
aa3be5f32db137 arch/powerpc/kernel/time.c Tony Breeds 2007-09-21 816 static int first = 1;
aa3be5f32db137 arch/powerpc/kernel/time.c Tony Breeds 2007-09-21 817
d90246cd8e0141 arch/powerpc/kernel/time.c Martin Schwidefsky 2009-08-22 818 ts->tv_nsec = 0;
aa3be5f32db137 arch/powerpc/kernel/time.c Tony Breeds 2007-09-21 819 /* XXX this is a litle fragile but will work okay in the short term */
aa3be5f32db137 arch/powerpc/kernel/time.c Tony Breeds 2007-09-21 820 if (first) {
aa3be5f32db137 arch/powerpc/kernel/time.c Tony Breeds 2007-09-21 821 first = 0;
aa3be5f32db137 arch/powerpc/kernel/time.c Tony Breeds 2007-09-21 822 if (ppc_md.time_init)
aa3be5f32db137 arch/powerpc/kernel/time.c Tony Breeds 2007-09-21 823 timezone_offset = ppc_md.time_init();
aa3be5f32db137 arch/powerpc/kernel/time.c Tony Breeds 2007-09-21 824
aa3be5f32db137 arch/powerpc/kernel/time.c Tony Breeds 2007-09-21 825 /* get_boot_time() isn't guaranteed to be safe to call late */
d90246cd8e0141 arch/powerpc/kernel/time.c Martin Schwidefsky 2009-08-22 826 if (ppc_md.get_boot_time) {
d90246cd8e0141 arch/powerpc/kernel/time.c Martin Schwidefsky 2009-08-22 827 ts->tv_sec = ppc_md.get_boot_time() - timezone_offset;
d90246cd8e0141 arch/powerpc/kernel/time.c Martin Schwidefsky 2009-08-22 828 return;
d90246cd8e0141 arch/powerpc/kernel/time.c Martin Schwidefsky 2009-08-22 829 }
d90246cd8e0141 arch/powerpc/kernel/time.c Martin Schwidefsky 2009-08-22 830 }
d90246cd8e0141 arch/powerpc/kernel/time.c Martin Schwidefsky 2009-08-22 831 if (!ppc_md.get_rtc_time) {
d90246cd8e0141 arch/powerpc/kernel/time.c Martin Schwidefsky 2009-08-22 832 ts->tv_sec = 0;
d90246cd8e0141 arch/powerpc/kernel/time.c Martin Schwidefsky 2009-08-22 833 return;
aa3be5f32db137 arch/powerpc/kernel/time.c Tony Breeds 2007-09-21 834 }
f2783c15007468 arch/powerpc/kernel/time.c Paul Mackerras 2005-10-20 835 ppc_md.get_rtc_time(&tm);
978d7eb31d44de arch/powerpc/kernel/time.c Benjamin Herrenschmidt 2009-11-01 836
5bfd643583b2e2 arch/powerpc/kernel/time.c Arnd Bergmann 2018-04-23 837 ts->tv_sec = rtc_tm_to_time64(&tm);
f2783c15007468 arch/powerpc/kernel/time.c Paul Mackerras 2005-10-20 838 }
f2783c15007468 arch/powerpc/kernel/time.c Paul Mackerras 2005-10-20 839
5bfd643583b2e2 arch/powerpc/kernel/time.c Arnd Bergmann 2018-04-23 @840 void read_persistent_clock64(struct timespec64 *ts)
978d7eb31d44de arch/powerpc/kernel/time.c Benjamin Herrenschmidt 2009-11-01 841 {
978d7eb31d44de arch/powerpc/kernel/time.c Benjamin Herrenschmidt 2009-11-01 842 __read_persistent_clock(ts);
978d7eb31d44de arch/powerpc/kernel/time.c Benjamin Herrenschmidt 2009-11-01 843
978d7eb31d44de arch/powerpc/kernel/time.c Benjamin Herrenschmidt 2009-11-01 844 /* Sanitize it in case real time clock is set below EPOCH */
978d7eb31d44de arch/powerpc/kernel/time.c Benjamin Herrenschmidt 2009-11-01 845 if (ts->tv_sec < 0) {
978d7eb31d44de arch/powerpc/kernel/time.c Benjamin Herrenschmidt 2009-11-01 846 ts->tv_sec = 0;
978d7eb31d44de arch/powerpc/kernel/time.c Benjamin Herrenschmidt 2009-11-01 847 ts->tv_nsec = 0;
978d7eb31d44de arch/powerpc/kernel/time.c Benjamin Herrenschmidt 2009-11-01 848 }
978d7eb31d44de arch/powerpc/kernel/time.c Benjamin Herrenschmidt 2009-11-01 849
978d7eb31d44de arch/powerpc/kernel/time.c Benjamin Herrenschmidt 2009-11-01 850 }
978d7eb31d44de arch/powerpc/kernel/time.c Benjamin Herrenschmidt 2009-11-01 851
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
1 month, 2 weeks
Re: [PATCH 1/1] can: esd: add support for esd GmbH PCIe/402 CAN interface family
by kernel test robot
Hi "Stefan,
Thank you for the patch! Yet something to improve:
[auto build test ERROR on 8dad5561c13ade87238d9de6dd410b43f7562447]
url: https://github.com/0day-ci/linux/commits/Stefan-M-tje/can-esd-add-support...
base: 8dad5561c13ade87238d9de6dd410b43f7562447
config: i386-randconfig-a004-20210728 (attached as .config)
compiler: gcc-10 (Ubuntu 10.3.0-1ubuntu1~20.04) 10.3.0
reproduce (this is a W=1 build):
# https://github.com/0day-ci/linux/commit/5110edee0d2180efeaa1236b265270984...
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Stefan-M-tje/can-esd-add-support-for-esd-GmbH-PCIe-402-CAN-interface-family/20210729-044114
git checkout 5110edee0d2180efeaa1236b2652709848bdd6fc
# save the attached .config to linux build tree
make W=1 ARCH=i386
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
All errors (new ones prefixed by >>, old ones prefixed by <<):
>> ERROR: modpost: "register_candev" [drivers/net/can/esd/esd_402_pci.ko] undefined!
>> ERROR: modpost: "alloc_can_err_skb" [drivers/net/can/esd/esd_402_pci.ko] undefined!
>> ERROR: modpost: "can_bus_off" [drivers/net/can/esd/esd_402_pci.ko] undefined!
>> ERROR: modpost: "close_candev" [drivers/net/can/esd/esd_402_pci.ko] undefined!
>> ERROR: modpost: "alloc_candev_mqs" [drivers/net/can/esd/esd_402_pci.ko] undefined!
ERROR: modpost: "free_candev" [drivers/net/can/esd/esd_402_pci.ko] undefined!
ERROR: modpost: "can_change_mtu" [drivers/net/can/esd/esd_402_pci.ko] undefined!
ERROR: modpost: "can_change_state" [drivers/net/can/esd/esd_402_pci.ko] undefined!
ERROR: modpost: "unregister_candev" [drivers/net/can/esd/esd_402_pci.ko] undefined!
ERROR: modpost: "alloc_can_skb" [drivers/net/can/esd/esd_402_pci.ko] undefined!
WARNING: modpost: suppressed 4 unresolved symbol warnings because there were too many)
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
1 month, 2 weeks
[PATCH v4 0/3] Add ilitek ili9341 panel driver
by dillon.minfei＠gmail.com
From: Dillon Min <dillon.minfei(a)gmail.com>
Since the st,sf-tc240t-9370-t dts binding already exist in stm32f429-disco.dts
but, the panel driver didn't get accepted from mainline. it's time to submit
patch fot it.
This driver can support two different interface by different dts bindings:
- spi+dpi, use spi to configure register, dpi for graphic data.
st,sf-tc240t-9370-t
- only spi, just like tiny/ili9341.c (actually, this part is copy from tiny)
adafruit,yx240qv29
I was submited the first patch last year, you can find it at [1].
this patch has one major difference from that one, which is replace the low
level communication way, from spi_sync() to mipi_dbi_{command,
command_stackbuf}() interface, referred from Linus's patch [2].
both the two dpi/dbi interface was tested on stm32f429-disco board, if anyone
want to verify this patch, you need apply the clk patch for this board first,
you can get it from [3].
[1] "drm/panel: Add ilitek ili9341 panel driver"
https://lore.kernel.org/lkml/1590378348-8115-7-git-send-email-dillon.minf...
[2] "drm/panel: s6e63m0: Switch to DBI abstraction for SPI"
https://lore.kernel.org/dri-devel/20210611214243.669892-1-linus.walleij@l...
[3]
https://lore.kernel.org/lkml/1590378348-8115-6-git-send-email-dillon.minf...
v4:
- fix m68k-allmodconfig build error which reported by lkp, thanks.
- add Copyright 2018 David Lechner <david(a)lechnology.com>.
v3 link:
https://lore.kernel.org/lkml/1627013203-23099-1-git-send-email-dillon.min...
v3:
- add Fixes tags.
- collect reviewed-by tags from linus and jagan.
- replace DRM_ERROR() with dev_err() or drm_err().
- remove kernel-doc markers from struct ili9341_config{}.
- reorder include headers.
- remove the struct device *dev from struct ili9341{}.
- restructure the ili9341_probe() function, add two ili9341_{dbi,dpi)_probe()
to make it more readable according to jagan's suggestion, thanks.
for the full drm driver exist in drm/panel need Sam and Laurent's feedback.
so, not cover this part at this time, will be update later.
v2 link:
https://lore.kernel.org/lkml/1626853288-31223-1-git-send-email-dillon.min...
v2:
- replace vcc regulator to bulk regulators in driver, from linus suggestion.
- fix dtbs_check warnings on ili9341 dts binding check.
- add bulk regulation node in ilitek,ili9341.yaml.
v1 link:
https://lore.kernel.org/lkml/1626430843-23823-1-git-send-email-dillon.min...
Dillon Min (3):
dt-bindings: display: panel: Add ilitek ili9341 panel bindings
ARM: dts: stm32: fix dtbs_check warning on ili9341 dts binding
drm/panel: Add ilitek ili9341 panel driver
.../bindings/display/panel/ilitek,ili9341.yaml | 78 ++
arch/arm/boot/dts/stm32f429-disco.dts | 2 +-
drivers/gpu/drm/panel/Kconfig | 12 +
drivers/gpu/drm/panel/Makefile | 1 +
drivers/gpu/drm/panel/panel-ilitek-ili9341.c | 792 +++++++++++++++++++++
5 files changed, 884 insertions(+), 1 deletion(-)
create mode 100644 Documentation/devicetree/bindings/display/panel/ilitek,ili9341.yaml
create mode 100644 drivers/gpu/drm/panel/panel-ilitek-ili9341.c
--
1.9.1
1 month, 2 weeks