On Wed, Oct 01, 2014 at 10:32:58AM -0500, Aravind Gopalakrishnan wrote: > >>+ if (dcsm & 0x3) { > >>+ /* LRDIMMs */ > >>+ edac_dbg(1, " DIMM type: LRDIMM %dx rank multiply;" > >>+ "CS = %d; all DIMMs support ECC: %s\n", > >>+ (dcsm & 0x3), cs, > >>+ (dclr & BIT(19)) ? "yes" : "no"); > >Why do we need to iterate over the DRAM CS sets? Just for the rank > >multiplier, apparently. We dump those normally in read_dct_base_mask(), > >though. > > It's not just for rank multiplier.. we find that it's LRDIMM only by > examining dcsm. Hence the iteration here.. So we can look only at the first DCSM, no? Or are there systems with different types of LRDIMMs on one DCT? -- Regards/Gruss, Boris. Sent from a fat crate under my desk. Formatting is fine. -- -- To unsubscribe from this list: send the line "unsubscribe linux-pci" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html