From mboxrd@z Thu Jan 1 00:00:00 1970 From: Shinya Kuribayashi Date: Fri, 12 Aug 2011 11:43:04 +0900 Subject: [U-Boot] [PATCH v3] mips32: fix wrong loop bound in flush_cache() In-Reply-To: <1312960276-3421-1-git-send-email-saturdaycoder@gmail.com> References: <1312960276-3421-1-git-send-email-saturdaycoder@gmail.com> Message-ID: <4E449338.9060007@pobox.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hi, On 08/10/2011 04:11 PM, Yao Cheng wrote: > The issue is found when calling flush_cache() with zero "size" argument. > The bound of loop is miscalculated in this case and flush_cache() enters a wrong flushing loop. > To fix this issue I skipped the operations when "size" is found to be zero. It feels like the last sentence is somewhat redundant (the diff output speaks for itself), so omitted. Other than that, the patch is ok and queued up to u-boot-mips/master. If no MIPS patches come up within a couple of weeks, I'll make a pull request. Thanks for your report,