From mboxrd@z Thu Jan 1 00:00:00 1970 From: Timur Tabi Date: Sat, 10 Feb 2007 20:34:09 -0600 Subject: [U-Boot-Users] [PATCH0/2] Re-do the patch for adding DO_SYNC in flash_write_cmd In-Reply-To: <200702100840.46587.sr@denx.de> References: <20070210010420.CD5CB353CAA@atlas.denx.de> <45CD72DF.8070409@orkun.us> <200702100840.46587.sr@denx.de> Message-ID: <45CE80A1.5050206@freescale.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Stefan Roese wrote: > Why not use > > #define sync() __asm__ __volatile__ ("sync" : : : "memory"); I like this idea, but is that semicolon supposed to be there? Shouldn't we make it an inline function instead of a macro?