From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1032384AbbKFHvv (ORCPT ); Fri, 6 Nov 2015 02:51:51 -0500 Received: from mx07-00178001.pphosted.com ([62.209.51.94]:17705 "EHLO mx07-00178001.pphosted.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1031994AbbKFHvt (ORCPT ); Fri, 6 Nov 2015 02:51:49 -0500 Subject: Re: [PATCH 0/2] [media] c8sectpfe: Deletion of a few unnecessary checks To: SF Markus Elfring , Mauro Carvalho Chehab , Patrice Chotard , Srinivas Kandagatla , , , References: <5307CAA2.8060406@users.sourceforge.net> <530A086E.8010901@users.sourceforge.net> <530A72AA.3000601@users.sourceforge.net> <530B5FB6.6010207@users.sourceforge.net> <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> <563BA3CC.4040709@users.sourceforge.net> CC: LKML , , Julia Lawall From: Maxime Coquelin Message-ID: <563C5BCD.806@st.com> Date: Fri, 6 Nov 2015 08:50:37 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.3.0 MIME-Version: 1.0 In-Reply-To: <563BA3CC.4040709@users.sourceforge.net> Content-Type: text/plain; charset="windows-1252"; format=flowed Content-Transfer-Encoding: 7bit X-Originating-IP: [10.201.23.80] X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10432:5.15.21,1.0.33,0.0.0000 definitions=2015-11-06_05:2015-11-05,2015-11-06,1970-01-01 signatures=0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Markus, On 11/05/2015 07:45 PM, SF Markus Elfring wrote: > From: Markus Elfring > Date: Thu, 5 Nov 2015 19:39:32 +0100 > > Another update suggestion was taken into account after a patch was applied > from static source code analysis. > > Markus Elfring (2): > Delete unnecessary checks before two function calls > Combine three checks into a single if block > > drivers/media/platform/sti/c8sectpfe/c8sectpfe-common.c | 16 ++++++++-------- > 1 file changed, 8 insertions(+), 8 deletions(-) > For the series: Acked-by: Maxime Coquelin Thanks! Maxime