LWN.net Logo

Add INT mode support for EDAC drivers on Maple

From:  Harry Ciao <qingtao.cao@windriver.com>
To:  benh@kernel.crashing.org, bluesmoke-devel@lists.sourceforge.net
Subject:  [v1 PATCH 0/4] Add INT mode support for EDAC drivers on Maple
Date:  Mon, 18 May 2009 18:04:56 +0800
Message-ID:  <1242641100-15324-1-git-send-email-qingtao.cao@windriver.com>
Cc:  linux-kernel@vger.kernel.org, linuxppc-dev@ozlabs.org
Archive-link:  Article, Thread


Hi Ben,

This is the v2 patches that have integrated your suggestions to
remove the refcount for a hwriq2virq mapping as long as we don't
dispose it, the changes are mostly within the 1/4 patch where the
unnecessary refcount and irqmap structure are removed, and callings
to edac_put_mpic_irq() are also removed from the rest of patches.

Since there are 3 EDAC modules on Maple will use one same copy of
code to create hwirq2virq mappings, I perfer to preserve it in
edac_mpic_irq.c.

How do feel about the assumption that MPIC will latch INT 0 pin for 
the NMI Reqeust Messages whose vector is == 0? This is the thing that
I have least confidence in, so far all I can get is the brief
introduction in CPC925 user manual that "This interrupt vector is used
to set a corresponding interrupt latch", P111, so I think it imples that
vector==0 will latch pin #0.

Many thanks for all your comments!

Best regards,

Harry
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Copyright © 2009, Eklektix, Inc.
Comments and public postings are copyrighted by their creators.
Linux is a registered trademark of Linus Torvalds