On Wed, 22 Nov 2023, David E. Box wrote: > Instead of checking for a NULL parent argument in intel_vsec_add_aux() and > then assigning it to the probed device, remove this check and just pass the > device in the call. Since this function is exported, return -EINVAL if the > parent is not specified. > > Signed-off-by: David E. Box <david.e.box@xxxxxxxxxxxxxxx> > --- > V5 - New patch > > drivers/platform/x86/intel/vsec.c | 8 ++++---- > 1 file changed, 4 insertions(+), 4 deletions(-) > > diff --git a/drivers/platform/x86/intel/vsec.c b/drivers/platform/x86/intel/vsec.c > index 7dc3650f2757..7a717183c58b 100644 > --- a/drivers/platform/x86/intel/vsec.c > +++ b/drivers/platform/x86/intel/vsec.c > @@ -103,6 +103,9 @@ int intel_vsec_add_aux(struct pci_dev *pdev, struct device *parent, > struct auxiliary_device *auxdev = &intel_vsec_dev->auxdev; > int ret, id; > > + if (!parent) > + return -EINVAL; > + > mutex_lock(&vsec_ida_lock); > id = ida_alloc(intel_vsec_dev->ida, GFP_KERNEL); > mutex_unlock(&vsec_ida_lock); > @@ -124,9 +127,6 @@ int intel_vsec_add_aux(struct pci_dev *pdev, struct device *parent, > return ret; > } > > - if (!parent) > - parent = &pdev->dev; > - > auxdev->id = id; > auxdev->name = name; > auxdev->dev.parent = parent; > @@ -212,7 +212,7 @@ static int intel_vsec_add_dev(struct pci_dev *pdev, struct intel_vsec_header *he > * Pass the ownership of intel_vsec_dev and resource within it to > * intel_vsec_add_aux() > */ > - return intel_vsec_add_aux(pdev, NULL, no_free_ptr(intel_vsec_dev), > + return intel_vsec_add_aux(pdev, &pdev->dev, no_free_ptr(intel_vsec_dev), > intel_vsec_name(header->id)); > } Reviewed-by: Ilpo Järvinen <ilpo.jarvinen@xxxxxxxxxxxxxxx> -- i.
