From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stefan Roese Date: Tue, 18 Oct 2011 16:23:07 +0200 Subject: [U-Boot] [PATCH] gpio: Add PCA9698 40-bit I2C I/O port In-Reply-To: <1317719635-2792-1-git-send-email-eibach@gdsys.de> References: <1317719635-2792-1-git-send-email-eibach@gdsys.de> Message-ID: <201110181623.07484.sr@denx.de> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hi Dirk, On Tuesday 04 October 2011 11:13:51 eibach at gdsys.de wrote: > From: Dirk Eibach Checkpatch output: ERROR: memset size is 3rd argument, not the second. #239: FILE: drivers/gpio/pca9698.c:120: + memset(data, sizeof(data), 0); total: 1 errors, 0 warnings, 139 lines checked So, this now really is a good checkpatch feature. I wasn't ware, that it checked for such bugs as well. Nice. :) Please fix and resubmit. Thanks, Stefan -- DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-0 Fax: (+49)-8142-66989-80 Email: office at denx.de