Re: [U-Boot] [PATCH] powerpc/8xxx: Display DIMM model

2011-02-09 Thread Kumar Gala

On Feb 4, 2011, at 3:58 PM, York Sun wrote:

 Beside displaying RDIMM or UDIMM, this patch adds display of the model numbers
 embedded in SPD.
 
 Signed-off-by: York Sun york...@freescale.com
 ---
 .../cpu/mpc8xxx/ddr/lc_common_dimm_params.c|   11 +++
 1 files changed, 7 insertions(+), 4 deletions(-)

applied to 85xx next

- k
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH] powerpc/8xxx: Display DIMM model

2011-02-04 Thread York Sun
Beside displaying RDIMM or UDIMM, this patch adds display of the model numbers
embedded in SPD.

Signed-off-by: York Sun york...@freescale.com
---
 .../cpu/mpc8xxx/ddr/lc_common_dimm_params.c|   11 +++
 1 files changed, 7 insertions(+), 4 deletions(-)

diff --git a/arch/powerpc/cpu/mpc8xxx/ddr/lc_common_dimm_params.c 
b/arch/powerpc/cpu/mpc8xxx/ddr/lc_common_dimm_params.c
index a58e5a9..684b2f4 100644
--- a/arch/powerpc/cpu/mpc8xxx/ddr/lc_common_dimm_params.c
+++ b/arch/powerpc/cpu/mpc8xxx/ddr/lc_common_dimm_params.c
@@ -207,10 +207,15 @@ compute_lowest_common_dimm_parameters(const dimm_params_t 
*dimm_params,
temp1 = temp2 = 0;
for (i = 0; i  number_of_dimms; i++) {
if (dimm_params[i].n_ranks) {
-   if (dimm_params[i].registered_dimm)
+   if (dimm_params[i].registered_dimm) {
temp1 = 1;
-   if (!dimm_params[i].registered_dimm)
+   printf(Detected RDIMM %s\n,
+   dimm_params[i].mpart);
+   } else {
temp2 = 1;
+   printf(Detected UDIMM %s\n,
+   dimm_params[i].mpart);
+   }
}
}
 
@@ -218,10 +223,8 @@ compute_lowest_common_dimm_parameters(const dimm_params_t 
*dimm_params,
outpdimm-all_DIMMs_unbuffered = 0;
if (temp1  !temp2) {
outpdimm-all_DIMMs_registered = 1;
-   printf(Detected RDIMM(s)\n);
} else if (!temp1  temp2) {
outpdimm-all_DIMMs_unbuffered = 1;
-   printf(Detected UDIMM(s)\n);
} else {
printf(ERROR:  Mix of registered buffered and unbuffered 
DIMMs detected!\n);
-- 
1.7.0.4


___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot