On 12/11/19 6:47 PM, Simon Glass wrote: > - * void relocate_code (addr_moni) > + * void relocate_code(addr_moni) Is this really necessary? I think you're bloating your patch with these comment changes. Your patch would be half the size without these changes, so I don't think this really qualifies as "while we are here".