✗ Fi.CI.CHECKPATCH: warning for mdev based hardware virtio offloading support

From: Patchwork <patchwork@emeril.freedesktop.org>
To: Jason Wang <jasowang@redhat.com>
Cc: intel-gfx@lists.freedesktop.org
Subject: ✗ Fi.CI.CHECKPATCH: warning for mdev based hardware virtio offloading support
Date: Mon, 18 Nov 2019 13:35:05 -0000	[thread overview]
Message-ID: <157408410594.2240.2784308451446609317@emeril.freedesktop.org> (raw)
In-Reply-To: <20191118105923.7991-1-jasowang@redhat.com>

== Series Details ==

Series: mdev based hardware virtio offloading support
URL   : https://patchwork.freedesktop.org/series/69621/
State : warning

== Summary ==

$ dim checkpatch origin/drm-tip
67f3ea3cda70 mdev: make mdev bus agnostic
-:648: CHECK:LINE_SPACING: Please don't use multiple blank lines
#648: FILE: drivers/vfio/mdev/mdev_private.h:42:
+
+

-:651: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'bus' - possible side-effects?
#651: FILE: drivers/vfio/mdev/mdev_private.h:44:
+#define dev_is_mdev(d, bus)	((d)->bus == bus)

-:651: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'bus' may be better as '(bus)' to avoid precedence issues
#651: FILE: drivers/vfio/mdev/mdev_private.h:44:
+#define dev_is_mdev(d, bus)	((d)->bus == bus)

-:656: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating?
#656: 
new file mode 100644

total: 0 errors, 1 warnings, 3 checks, 776 lines checked
f69d8d9902fd mdev: split out VFIO bus specific parent ops
-:285: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'mdev' - possible side-effects?
#285: FILE: drivers/vfio/mdev/mdev_vfio.c:20:
+#define to_vfio_mdev_device(mdev) container_of(mdev, \
+					       struct mdev_vfio_device, mdev)

total: 0 errors, 0 warnings, 1 checks, 585 lines checked
8544fcd816bc mdev: move to drivers/
-:57: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating?
#57: 
new file mode 100644

total: 0 errors, 1 warnings, 0 checks, 92 lines checked
fe3f935905d1 mdev: introduce mediated virtio bus
-:57: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating?
#57: 
new file mode 100644

-:91: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'mdev' - possible side-effects?
#91: FILE: drivers/mdev/virtio.c:30:
+#define to_mdev_virtio(mdev) container_of(mdev, \
+					  struct mdev_virtio_device, mdev)

-:405: CHECK:LINE_SPACING: Please don't use multiple blank lines
#405: FILE: scripts/mod/file2alias.c:1348:
+
+

total: 0 errors, 1 warnings, 2 checks, 361 lines checked
bbb2f0eac428 virtio: introduce a mdev based transport
-:52: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating?
#52: 
new file mode 100644

-:111: WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#111: FILE: drivers/virtio/virtio_mdev.c:55:
+static void virtio_mdev_get(struct virtio_device *vdev, unsigned offset,

-:112: WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#112: FILE: drivers/virtio/virtio_mdev.c:56:
+			    void *buf, unsigned len)

-:120: WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#120: FILE: drivers/virtio/virtio_mdev.c:64:
+static void virtio_mdev_set(struct virtio_device *vdev, unsigned offset,

-:121: WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#121: FILE: drivers/virtio/virtio_mdev.c:65:
+			    const void *buf, unsigned len)

-:134: CHECK:LINE_SPACING: Please don't use multiple blank lines
#134: FILE: drivers/virtio/virtio_mdev.c:78:
+
+

-:302: WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#302: FILE: drivers/virtio/virtio_mdev.c:246:
+static int virtio_mdev_find_vqs(struct virtio_device *vdev, unsigned nvqs,

total: 0 errors, 6 warnings, 1 checks, 443 lines checked
8a37245a6260 docs: sample driver to demonstrate how to implement virtio-mdev framework
-:62: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating?
#62: 
new file mode 100644

total: 0 errors, 1 warnings, 0 checks, 717 lines checked

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx