RE: [PATCH v7 3/3] aerdrv: Cleanup log output for CPER based AER

2012-12-05 Thread Ortiz, Lance E
> > Here it is, you're initializing 'prefix' here although it is being > used in the previous patch. You should concentrate the whole prefix > initialization and passing in one patch so that there are no breakages. Yes I probably could have broken up the patches in a cleaner way. Thanks. > > A

Re: [PATCH v7 3/3] aerdrv: Cleanup log output for CPER based AER

2012-12-05 Thread Borislav Petkov
On Tue, Dec 04, 2012 at 02:03:18PM -0700, Lance Ortiz wrote: > These changes make cper_print_aer more consistent with aer_print_error > which is called in the AER interrupt case. The string in the variable > 'prefix' is printed at the beginning of each print statement in > cper_print_aer(). The pre

Re: [PATCH v7 3/3] aerdrv: Cleanup log output for CPER based AER

2012-12-05 Thread Mauro Carvalho Chehab
Em Tue, 04 Dec 2012 14:03:18 -0700 Lance Ortiz escreveu: Hmm... I did a reply to v6 of this patch, but i pressed the wrong button and it went only to Joe. Excuse-me for that. Let me resend the comments to everybody: On Tue, 2012-12-04 at 16:33 -0200, Mauro Carvalho Chehab wrote: > Em Tue, 04 Dec