From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Date: Mon, 16 Nov 2015 16:30:19 +0100 From: Heiko Carstens Subject: Re: [PATCH] s390: Delete unnecessary checks before the function call "debug_unregister" Message-ID: <20151116153019.GD4069@osiris> References: <530C5E18.1020800@users.sourceforge.net> <530CD2C4.4050903@users.sourceforge.net> <530CF8FF.8080600@users.sourceforge.net> <530DD06F.4090703@users.sourceforge.net> <5317A59D.4@users.sourceforge.net> <5649E4F5.5060604@users.sourceforge.net> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <5649E4F5.5060604@users.sourceforge.net> Sender: linux-kernel-owner@vger.kernel.org List-Archive: List-Post: To: SF Markus Elfring Cc: Ingo Tuchscherer , Martin Schwidefsky , Peter Oberparleiter , Sebastian Ott , linux-s390@vger.kernel.org, LKML , kernel-janitors@vger.kernel.org, Julia Lawall List-ID: On Mon, Nov 16, 2015 at 03:15:17PM +0100, SF Markus Elfring wrote: > From: Markus Elfring > Date: Mon, 16 Nov 2015 14:45:40 +0100 > > The debug_unregister() function performs also input parameter validation. > Thus the test around the calls is not needed. > > This issue was detected by using the Coccinelle software. > > Signed-off-by: Markus Elfring > --- > drivers/s390/cio/chsc_sch.c | 3 +-- > drivers/s390/cio/cio.c | 9 +++------ > drivers/s390/cio/qdio_debug.c | 6 ++---- > drivers/s390/crypto/zcrypt_api.c | 6 ++---- > 4 files changed, 8 insertions(+), 16 deletions(-) Applied, thanks!