From mboxrd@z Thu Jan 1 00:00:00 1970 From: Scott Wood Date: Wed, 13 Aug 2014 15:32:18 -0500 Subject: [U-Boot] [PATCH 3/4] mtd: nand: add Freescale NFC driver In-Reply-To: <7b41b54fe3f11a943b0ec63f2e44f573@agner.ch> References: <4ea124ab817db6e3dee2067aadd6db14643990f5.1407312577.git.stefan@agner.ch> <1407796426.7427.100.camel@snotra.buserror.net> <1380dbe3110cbf9359220a78d69e1896@agner.ch> <1407881820.7427.137.camel@snotra.buserror.net> <87wqadl3h6.fsf@nbsps.com> <7b41b54fe3f11a943b0ec63f2e44f573@agner.ch> Message-ID: <1407961938.7427.161.camel@snotra.buserror.net> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Wed, 2014-08-13 at 13:20 +0200, Stefan Agner wrote: > IMHO we should use the raw_writel only and "hand optimize" for functions > which are used often. For the initialization/configuration functions, > there is little value to save some register access. raw_writel() is itself something that should only be used for hand-optimized sections. For non-performance-critical code you should use normal writel() so that you don't need to worry about manually adding I/O barriers. -Scott