Re: [PATCH 1/2] staging: android: ion: Deletion of unnecessary checks before two function calls

Thierry Reding
2014-12-02T09:59:29Z

[-- Attachment #1: Type: text/plain, Size: 736 bytes --]

On Sun, Nov 23, 2014 at 07:39:23PM +0100, SF Markus Elfring wrote:
> From: Markus Elfring <elfring@users.sourceforge.net>
> Date: Sun, 23 Nov 2014 18:48:15 +0100
>
> The functions ion_heap_destroy() and vfree() perform also input
> parameter validation. Thus the test around the call is not needed.
>
> This issue was detected by using the Coccinelle software.
>
> Signed-off-by: Markus Elfring <elfring@users.sourceforge.net>
> ---
> drivers/staging/android/ion/ion.c | 6 ++----
> drivers/staging/android/ion/ion_dummy_driver.c | 6 ++----
> drivers/staging/android/ion/tegra/tegra_ion.c | 6 ++----
> 3 files changed, 6 insertions(+), 12 deletions(-)

Acked-by: Thierry Reding <treding@nvidia.com>

[-- Attachment #2: Type: application/pgp-signature, Size: 819 bytes --]