From mboxrd@z Thu Jan 1 00:00:00 1970 From: Wolfgang Denk Date: Wed, 27 Jul 2011 23:00:05 +0200 Subject: [U-Boot] [PATCH v2 6/7] ARMV7: OMAP: I2C driver: Write more than 1 byte at a time in i2c_write In-Reply-To: <4E2FC14F.6080907@matrix-vision.de> References: <1310720986-5474-7-git-send-email-michael.jones@matrix-vision.de> <1311001121-8129-1-git-send-email-michael.jones@matrix-vision.de> <4E2FAB26.6060208@denx.de> <4E2FC14F.6080907@matrix-vision.de> Message-ID: <20110727210005.45FD915794C3@gemini.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 Dear Michael Jones, In message <4E2FC14F.6080907@matrix-vision.de> you wrote: > > Question about cosmetics: the README says "In sources originating from > U-Boot a style corresponding to "Lindent -pcs" (adding spaces before > parameters to function calls) is actually used." Currently this is not This was the old (aka "my own") style. But I got overruled. I'm going to post a patch to fix the README. > uniform in this file, because checkpatch.pl doesn't like the spaces > between function names and '(' (and neither do I). Are there supposed to > be such spaces in U-Boot code? Or can we uniformly remove them in this file? Please remove them. Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de "A little knowledge is a dangerous thing." - Doug Gwyn