Re: [PATCH] accel/qaic: Drop redundant vfree() null check in sahara
On 11/17/2024 1:26 PM, Jeffrey Hugo wrote: The documentation for vfree() says that passing in NULL is ok. Therefore we can drop the null check as redundant. Reported-by: kernel test robot Closes: https://lore.kernel.org/oe-kbuild-all/202410301732.abf5md4e-...@intel.com/ Signed-off-by: Jeffrey Hugo Applied to drm-misc-next -Jeff
Re: [PATCH] accel/qaic: Drop redundant vfree() null check in sahara
On 11/17/2024 12:26 PM, Jeffrey Hugo wrote: The documentation for vfree() says that passing in NULL is ok. Therefore we can drop the null check as redundant. Reported-by: kernel test robot Closes: https://lore.kernel.org/oe-kbuild-all/202410301732.abf5md4e-...@intel.com/ Signed-off-by: Jeffrey Hugo Reviewed-by: Carl Vanderlip
Re: [PATCH] accel/qaic: Drop redundant vfree() null check in sahara
On 11/17/2024 8:26 PM, Jeffrey Hugo wrote: > The documentation for vfree() says that passing in NULL is ok. Therefore > we can drop the null check as redundant. > > Reported-by: kernel test robot > Closes: > https://lore.kernel.org/oe-kbuild-all/202410301732.abf5md4e-...@intel.com/ > Signed-off-by: Jeffrey Hugo Reviewed-by: Youssef Samir
Re: [PATCH] accel/qaic: Drop redundant vfree() null check in sahara
On Sun, Nov 17, 2024 at 01:26:29PM -0700, Jeffrey Hugo wrote: > The documentation for vfree() says that passing in NULL is ok. Therefore > we can drop the null check as redundant. > > Reported-by: kernel test robot > Closes: > https://lore.kernel.org/oe-kbuild-all/202410301732.abf5md4e-...@intel.com/ > Signed-off-by: Jeffrey Hugo > --- > drivers/accel/qaic/sahara.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > Reviewed-by: Dmitry Baryshkov -- With best wishes Dmitry
[PATCH] accel/qaic: Drop redundant vfree() null check in sahara
The documentation for vfree() says that passing in NULL is ok. Therefore we can drop the null check as redundant. Reported-by: kernel test robot Closes: https://lore.kernel.org/oe-kbuild-all/202410301732.abf5md4e-...@intel.com/ Signed-off-by: Jeffrey Hugo --- drivers/accel/qaic/sahara.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/accel/qaic/sahara.c b/drivers/accel/qaic/sahara.c index 6d772143d612..21d58aed0deb 100644 --- a/drivers/accel/qaic/sahara.c +++ b/drivers/accel/qaic/sahara.c @@ -772,8 +772,7 @@ static void sahara_mhi_remove(struct mhi_device *mhi_dev) cancel_work_sync(&context->fw_work); cancel_work_sync(&context->dump_work); - if (context->mem_dump) - vfree(context->mem_dump); + vfree(context->mem_dump); sahara_release_image(context); mhi_unprepare_from_transfer(mhi_dev); } -- 2.34.1
