[PATCH] cmd: mem: fix range of bitflip test

Stefan Roese sr at denx.de
Wed Sep 9 17:13:49 CEST 2020


Hi Ralph,

On 09.09.20 17:07, Ralph Siemsen wrote:
> Hi Stefan,
> 
> On Wed, Sep 09, 2020 at 03:53:08PM +0200, Stefan Roese wrote:
>> Hi Ralph,
>>
>> On 09.09.20 15:49, Ralph Siemsen wrote:
>>>
>>> Very good, I will send a separate patch that adds a Kconfig option.
> 
> As it turns out, doing a separate patch for this gets messy, and also 
> would introduce a dependency between the patches. Will do v3 instead.

Okay.

>>> Small procedural question: Patchwork is showing state=new for both 
>>> versions of the mtest fix [1]. Is this normal, or did I miss some 
>>> step when posting the v2 patch?
>>
>> The only thing you missed, was adding my RB tag to v2. I did send a new
>> RB again to this mail, so patchwork will collect it automatically.
> 
> Thanks, I was not entirely clear what the policy was for this. 
> Presumably there is a way for a reviewer to 'revoke' the RB in case they 
> disagree with subsequent changes?

Usually the RB is only added, when not too many further changes are
made by the committer. There is no strict rule here AFAIK.

>> You could mark your v1 as superseded in patchwork though.
> 
> Seems somebody beat me to it, v1 is now marked as superseded.

Tom is quite active here. ;)

Thanks,
Stefan


More information about the U-Boot mailing list