Messages in this thread | | | Date | Fri, 30 Mar 2012 17:38:59 +0200 | From | Borislav Petkov <> | Subject | Re: [PATCH 01/13] edac: Create a dimm struct and move the labels into it |
| |
On Fri, Mar 30, 2012 at 10:26:13AM -0300, Mauro Carvalho Chehab wrote: > > There's a double loop above this one which iterates over the same > > things: rows and then channels in each row. So merge that loop with the > > one above instead of repeating it here. > > The first loop will disappear on the patch I'm writing right now with the > edac_mc_alloc() changes to per-csrow/per-dimm/per-channel kzalloc().
Nevertheless, please fix it as suggested.
-- Regards/Gruss, Boris.
Advanced Micro Devices GmbH Einsteinring 24, 85609 Dornach GM: Alberto Bozzo Reg: Dornach, Landkreis Muenchen HRB Nr. 43632 WEEE Registernr: 129 19551
| |