Hi, On Tue, May 28, 2019 at 10:24:32AM +0200, Rolf Eike Beer wrote: > > -#define CONST > > +#define CONST > > int module_frob_arch_sections(CONST Elf_Ehdr *hdr, > > CONST Elf_Shdr *sechdrs, > > CONST char *secstrings, > > @@ -619,7 +613,7 @@ int apply_relocate_add(Elf_Shdr *sechdrs, > > /* See note about special handling of SEGREL32 at > > * the beginning of this file. > > */ > > - *loc = fsel(val, addend); > > + *loc = fsel(val, addend); > > break; > > case R_PARISC_SECREL32: > > /* 32-bit section relative address. */ > > You are sneaking in unrelated whitespace fixes. I just want to let you know > that you got caught ;) Not contributing often to the linux kernel i'm not sure what the policy is. IMHO it's okay to fix whitespace errors when working on patches. If Helge says this is a problem i remove the whitespace changes. Regards Sven
