lkml.org 
[lkml]   [2009]   [May]   [4]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
DateMon, 04 May 2009 18:59:14 -0300
FromMauro Carvalho Chehab <>
SubjectRe: [PATCH 09/21] amd64_edac: assign DRAM chip select base and mask in a family-specific way
Borislav Petkov escreveu:
> + for (cs = 0; cs < CHIPSELECT_COUNT; cs++) {
> + reg = K8_DCSB0 + (cs * 4);
> + err = pci_read_config_dword(pvt->dram_f2_ctl, reg,
> + &pvt->dcsb0[cs]);
> + if (err != 0)
> + debugf0("%s() Reading K8_DCSB0[%d] failed\n",
> + __func__, cs);
> +
> + debugf0(" DCSB0[%d]=0x%08x reg: F2x%x\n",
> + cs, pvt->dcsb0[cs], reg);
>

Hmm... I suspect that there's a missing else before the debugf0(). If
you got an error while reading it, you shouldn't be showing the results.
> +
> + /* If DCT are NOT ganged, then read in DCT1's base */
> + if (boot_cpu_data.x86 >= 0x10 && !dct_ganging_enabled(pvt)) {
> + reg = F10_DCSB1 + (cs * 4);
> + err = pci_read_config_dword(pvt->dram_f2_ctl, reg,
> + &pvt->dcsb1[cs]);
> + if (err != 0)
> + debugf0("%s() Reading F10_DCSB1[%d] failed\n",
> + __func__, cs);
> + debugf0(" DCSB1[%d]=0x%08x reg: F2x%x\n",
> + cs, pvt->dcsb1[cs], reg);
>
The same issue here: if you got an error while reading it, you shouldn't
be showing the results.

Cheers,
Mauro.


\
 
 \ /
  Last update: 2009-05-05 00:03    [from the cache]
©2003-2010