Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751936AbdIFGBB (ORCPT ); Wed, 6 Sep 2017 02:01:01 -0400 Received: from esa5.microchip.iphmx.com ([216.71.150.166]:21003 "EHLO esa5.microchip.iphmx.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751646AbdIFGAz (ORCPT ); Wed, 6 Sep 2017 02:00:55 -0400 X-IronPort-AV: E=Sophos;i="5.41,482,1498546800"; d="scan'208";a="4450378" Date: Wed, 6 Sep 2017 07:59:06 +0200 From: Ludovic Desroches To: "Yang, Wenyou" CC: SF Markus Elfring , , Ludovic Desroches , Mauro Carvalho Chehab , Songjun Wu , LKML , Subject: Re: [PATCH 0/6] [media] Atmel: Adjustments for seven function implementations Message-ID: <20170906055906.pzhqxor7goybrdif@rfolt0960.corp.atmel.com> References: <88d0739c-fdc1-9d7d-fe53-b7c2eeed1849@users.sourceforge.net> <30c46f61-5c4f-9f75-e8b5-fab77fe1e11f@Microchip.com> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Disposition: inline In-Reply-To: <30c46f61-5c4f-9f75-e8b5-fab77fe1e11f@Microchip.com> User-Agent: NeoMutt/20170609 (1.8.3) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1144 Lines: 35 On Wed, Sep 06, 2017 at 08:58:26AM +0800, Yang, Wenyou wrote: > Hi, > > > On 2017/9/5 4:04, SF Markus Elfring wrote: > > From: Markus Elfring > > Date: Mon, 4 Sep 2017 21:44:55 +0200 > > > > A few update suggestions were taken into account > > from static source code analysis. > Thank you for your patches. > > You can add my Acked-by for the patch series. > > Acked-by: Wenyou Yang Acked-by: Ludovic Desroches > > > > > Markus Elfring (6): > > Delete an error message for a failed memory allocation in isc_formats_init() > > Improve a size determination in isc_formats_init() > > Adjust three checks for null pointers > > Delete an error message for a failed memory allocation in two functions > > Improve three size determinations > > Adjust a null pointer check in three functions > > > > drivers/media/platform/atmel/atmel-isc.c | 12 +++++------- > > drivers/media/platform/atmel/atmel-isi.c | 20 ++++++++------------ > > 2 files changed, 13 insertions(+), 19 deletions(-) > > > > Best Regards, > Wenyou Yang