From mboxrd@z Thu Jan 1 00:00:00 1970 From: Borislav Petkov Subject: Re: [RFC PATCH] edac: add support for ARM PL310 L2 cache parity Date: Wed, 9 Apr 2014 19:47:01 +0200 Message-ID: <20140409174701.GO6529@pd.tnic> References: <1393770760-32550-1-git-send-email-punnaia@xilinx.com> <20140409113246.GA8778@pd.tnic> <20140409151932.GK6529@pd.tnic> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Return-path: Content-Disposition: inline In-Reply-To: Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Punnaiah Choudary Cc: Rob Herring , Punnaiah Choudary Kalluri , Doug Thompson , "devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , "linux-doc-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , "linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org" , "linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , linux-edac-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Michal Simek , Rob Herring , Pawel Moll , Mark Rutland , Ian Campbell , Kumar Gala , Rob Landley , punnaiah choudary kalluri , punnaiah choudary kalluri List-Id: devicetree@vger.kernel.org On Wed, Apr 09, 2014 at 10:59:49PM +0530, Punnaiah Choudary wrote: > There is a driver file cache-l2x0.c under arch/arm/mm for pl310 cache > configuration and management. Russel king had suggested to use > single driver file for both pl310 edac implementation and cache-l2x0.c > Here is the thread > http://www.spinics.net/lists/arm-kernel/msg320407.html > > please provide your inputs Well, it is very simple - you don't really need an EDAC compilation unit just so that you can report errors - you can very well do that in your cache-l2x0.c and boom, problem solved. Simply move the irq handler and the parity error checker to that file. :-) -- Regards/Gruss, Boris. Sent from a fat crate under my desk. Formatting is fine. -- -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html