On Wednesday 01 July 2009 05:40:53 Alessio Centazzo wrote:

This patch fixes a debug compilation error for PPC4xx platforms, all other architectures are not affected by this change. The 'handler' pointer was undefined. The fix is exercised and has effect only if DEBUG is defined.

Unfortunately this still doesn't apply. Now I checked why this is the case (with such a "simple" patch). And it seems that you have a whitespace issue. Please see below.

Signed-off-by: Alessio Centazzo acpatin@yahoo.com

cpu/ppc4xx/uic.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/cpu/ppc4xx/uic.c b/cpu/ppc4xx/uic.c index a95d1cb..1a370b3 100644 --- a/cpu/ppc4xx/uic.c +++ b/cpu/ppc4xx/uic.c @@ -164,7 +164,7 @@ void pic_irq_enable(unsigned int vec) else if (vec >= 96) mtdcr(uic3er, mfdcr(uic3er) | UIC_MASK(vec));

Indentation done via spaces instead of tabs as done in the original file.

•   debug("Install interrupt for vector %d ==> %p\n", vec, handler);
  

•   debug("Enable interrupt vector %d\n", vec);
  

Again, indentation has to be done in tabs. Please fix and resubmit.

Thanks.

Best regards, Stefan

===================================================================== DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: +49-8142-66989-0 Fax: +49-8142-66989-80 Email: office@denx.de =====================================================================