From mboxrd@z Thu Jan 1 00:00:00 1970 From: Wolfgang Denk Date: Sat, 10 Feb 2007 02:04:20 +0100 Subject: [U-Boot-Users] [PATCH0/2] Re-do the patch for adding DO_SYNC in flash_write_cmd In-Reply-To: Your message of "Fri, 09 Feb 2007 14:48:03 EST." <1171050483.3932.31.camel@udp097531uds.am.freescale.net> Message-ID: <20070210010420.CD5CB353CAA@atlas.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 In message <1171050483.3932.31.camel@udp097531uds.am.freescale.net> you wrote: > SYNC is defined as > " #define SYNC \ > sync; \ > isync > " in include/ppc_asm.tmpl, > > and can not be used by .c file, am I right? :-). In fact it is used by > the start.S file for 85xx/83xx/8xx/4xx/5xxx/74xx_7xx. > > We need to define SYNC as asm("sync;"). Or, to be sure, ""sync;isync" Where is the problem? Which code includes include/ppc_asm.tmpl ? Why cannot we have the same definition once for C and again for assembler? Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, HRB 165235 Munich, CEO: Wolfgang Denk Office: Kirchenstr. 5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de You can do this in a number of ways. IBM chose to do all of them. Why do you find that funny? -- D. Taylor, Computer Science 350