Messages in this thread | | | Date | Mon, 1 Jun 2020 13:57:21 -0600 | From | Mathieu Poirier <> | Subject | Re: [PATCH v1] coresight: Drop double check for ACPI companion device |
| |
Hi Andy,
On Fri, May 29, 2020 at 04:32:10PM +0300, Andy Shevchenko wrote: > acpi_dev_get_resources() does perform the NULL pointer check against > ACPI companion device which is given as function parameter. Thus, > there is no need to duplicate this check in the caller. > > Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com> > --- > drivers/hwtracing/coresight/coresight-stm.c | 2 -- > 1 file changed, 2 deletions(-) > > diff --git a/drivers/hwtracing/coresight/coresight-stm.c b/drivers/hwtracing/coresight/coresight-stm.c > index b908ca104645..673d2f56ed1e 100644 > --- a/drivers/hwtracing/coresight/coresight-stm.c > +++ b/drivers/hwtracing/coresight/coresight-stm.c > @@ -727,8 +727,6 @@ static int acpi_stm_get_stimulus_area(struct device *dev, struct resource *res) > > struct acpi_device *adev = ACPI_COMPANION(dev); > > - if (!adev) > - return -ENODEV; > rc = acpi_dev_get_resources(adev, &res_list, NULL, NULL); > if (rc < 0) > return rc;
I have applied your patch.
Thanks, Mathieu
> -- > 2.26.2 >
| |