There's more: I said: > > @@ -370,12 +398,12 @@ static int synps_edac_init_csrows(struct mem_ctl_info *mci) > That function returns 0 unconditionally. Make it a void in a prepatch. But you've lumped this change together with a bunch more. Maybe my request wasn't clear so let me rephrase it: That function returns 0 unconditionally. Make it a void in a *separate* prepatch. Ok? -- Regards/Gruss, Boris. ECO tip #101: Trim your mails when you reply. --