From mboxrd@z Thu Jan 1 00:00:00 1970 From: Albert ARIBAUD Date: Fri, 6 Jul 2012 17:26:17 +0200 Subject: [U-Boot] [PATCH] armv7: Fix to mistake clean the memory space In-Reply-To: <4FF6F065.8080208@gmail.com> References: <1341577211-5563-1-git-send-email-bocui107@gmail.com> <4FF6E6EA.8070401@gmail.com> <4FF6ED0C.10803@gmail.com> <4FF6EEAF.2020503@gmail.com> <4FF6F065.8080208@gmail.com> Message-ID: <20120706172617.4bc0dfbb@aari01-12> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hi Zhong Hongbo, On Fri, 06 Jul 2012 22:04:21 +0800, Zhong Hongbo wrote: > On 07/06/2012 09:57 PM, Andreas Bie?mann wrote: > > Dear Zhong Hongbo, > > > > On 06.07.2012 15:50, Zhong Hongbo wrote: > >> On 07/06/2012 09:23 PM, Andreas Bie?mann wrote: > >>> On 06.07.2012 14:20, Zhong Hongbo wrote: > >>>> From: Zhong Hongbo > >>>> > >>>> In currently, when __bss_start is equal to __bss_end__, > >>>> The bss loop will clear all the things in memory space. > >>>> > >>>> But just only when __bss_end__ greater than __bss_start__, > >>>> we do the clear bss section operation. > >>>> > >>>> Signed-off-by: Hongbo Zhong > >>>> Signed-off-by: Andreas Bie?mann > >>> > >>> well this is wrong, I would maybe ack it > >> > >> Ok > > > > After sending I doubt my statement was correct, but I found a source > > http://www.denx.de/wiki/view/U-Boot/Patches#Review_Process_Git_Tags > > he, Ye, I think we(You, Albert and me) are involved in the development > of the patch. It could be said of any reviewer that they are "involved in the development of the patch", yet not all reviewers's Signed-off-by is added to evey patch they review. There is a Reviewed-by tag for reviewers. I personally do not consider it would make sense to have my Signed-off-by on this patch. Amicalement, -- Albert.