Hi Max, On 5/18/21 9:21 PM, Max Gurtovoy wrote: > The vfio core driver is now taking refcount on the provider drivers, > remove redundant parent_module attribute from vfio_platform_device > structure. > > Signed-off-by: Max Gurtovoy <mgurtovoy@xxxxxxxxxx> Acked-by: Eric Auger <eric.auger@xxxxxxxxxx> Thanks Eric > --- > drivers/vfio/platform/vfio_amba.c | 1 - > drivers/vfio/platform/vfio_platform.c | 1 - > drivers/vfio/platform/vfio_platform_private.h | 1 - > 3 files changed, 3 deletions(-) > > diff --git a/drivers/vfio/platform/vfio_amba.c b/drivers/vfio/platform/vfio_amba.c > index f970eb2a999f..badfffea14fb 100644 > --- a/drivers/vfio/platform/vfio_amba.c > +++ b/drivers/vfio/platform/vfio_amba.c > @@ -59,7 +59,6 @@ static int vfio_amba_probe(struct amba_device *adev, const struct amba_id *id) > vdev->flags = VFIO_DEVICE_FLAGS_AMBA; > vdev->get_resource = get_amba_resource; > vdev->get_irq = get_amba_irq; > - vdev->parent_module = THIS_MODULE; > vdev->reset_required = false; > > ret = vfio_platform_probe_common(vdev, &adev->dev); > diff --git a/drivers/vfio/platform/vfio_platform.c b/drivers/vfio/platform/vfio_platform.c > index e4027799a154..68a1c87066d7 100644 > --- a/drivers/vfio/platform/vfio_platform.c > +++ b/drivers/vfio/platform/vfio_platform.c > @@ -50,7 +50,6 @@ static int vfio_platform_probe(struct platform_device *pdev) > vdev->flags = VFIO_DEVICE_FLAGS_PLATFORM; > vdev->get_resource = get_platform_resource; > vdev->get_irq = get_platform_irq; > - vdev->parent_module = THIS_MODULE; > vdev->reset_required = reset_required; > > ret = vfio_platform_probe_common(vdev, &pdev->dev); > diff --git a/drivers/vfio/platform/vfio_platform_private.h b/drivers/vfio/platform/vfio_platform_private.h > index a5ba82c8cbc3..dfb834c13659 100644 > --- a/drivers/vfio/platform/vfio_platform_private.h > +++ b/drivers/vfio/platform/vfio_platform_private.h > @@ -50,7 +50,6 @@ struct vfio_platform_device { > u32 num_irqs; > int refcnt; > struct mutex igate; > - struct module *parent_module; > const char *compat; > const char *acpihid; > struct module *reset_module; >