Message ID | 20240327-module-owner-virtio-v1-16-0feffab77d99@linaro.org (mailing list archive) |
---|---|
State | Superseded |
Headers | show |
Series | virtio: store owner from modules with register_virtio_driver() | expand |
Context | Check | Description |
---|---|---|
tedd_an/pre-ci_am | success | Success |
tedd_an/CheckPatch | success | CheckPatch PASS |
tedd_an/GitLint | success | Gitlint PASS |
tedd_an/SubjectPrefix | fail | "Bluetooth: " prefix is not specified in the subject |
tedd_an/IncrementalBuild | success | Incremental Build PASS |
On Wed, Mar 27, 2024 at 01:41:09PM +0100, Krzysztof Kozlowski wrote: >virtio core already sets the .owner, so driver does not need to. > >Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> > >--- > >Depends on the first patch. >--- > net/vmw_vsock/virtio_transport.c | 1 - > 1 file changed, 1 deletion(-) Acked-by: Stefano Garzarella <sgarzare@redhat.com> Nit: you can use "vsock/virtio: " as prefix for the commit title. > >diff --git a/net/vmw_vsock/virtio_transport.c b/net/vmw_vsock/virtio_transport.c >index 1748268e0694..13f42a62b034 100644 >--- a/net/vmw_vsock/virtio_transport.c >+++ b/net/vmw_vsock/virtio_transport.c >@@ -858,7 +858,6 @@ static struct virtio_driver virtio_vsock_driver = { > .feature_table = features, > .feature_table_size = ARRAY_SIZE(features), > .driver.name = KBUILD_MODNAME, >- .driver.owner = THIS_MODULE, > .id_table = id_table, > .probe = virtio_vsock_probe, > .remove = virtio_vsock_remove, > >-- >2.34.1 >
diff --git a/net/vmw_vsock/virtio_transport.c b/net/vmw_vsock/virtio_transport.c index 1748268e0694..13f42a62b034 100644 --- a/net/vmw_vsock/virtio_transport.c +++ b/net/vmw_vsock/virtio_transport.c @@ -858,7 +858,6 @@ static struct virtio_driver virtio_vsock_driver = { .feature_table = features, .feature_table_size = ARRAY_SIZE(features), .driver.name = KBUILD_MODNAME, - .driver.owner = THIS_MODULE, .id_table = id_table, .probe = virtio_vsock_probe, .remove = virtio_vsock_remove,
virtio core already sets the .owner, so driver does not need to. Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> --- Depends on the first patch. --- net/vmw_vsock/virtio_transport.c | 1 - 1 file changed, 1 deletion(-)