From mboxrd@z Thu Jan 1 00:00:00 1970 From: Michal Simek Subject: Re: [PATCH 1/1] DEBUGFS: Fixing name for debug fs Date: Sun, 04 May 2008 10:28:32 +0200 Message-ID: <481D73B0.8020705@monstr.eu> References: <1209825314-8048-1-git-send-email-monstr@seznam.cz> <200805032307.04352.arnd@arndb.de> Reply-To: monstr@seznam.cz Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-15 Content-Transfer-Encoding: 7bit Return-path: Received: from smtp1.wifiinternet.cz ([89.31.47.1]:61722 "EHLO bor.wifiinternet.cz" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1753109AbYEDIz1 (ORCPT ); Sun, 4 May 2008 04:55:27 -0400 In-Reply-To: <200805032307.04352.arnd@arndb.de> Sender: linux-arch-owner@vger.kernel.org List-ID: To: Arnd Bergmann Cc: monstr@seznam.cz, paulus@samba.org, linux-kernel@vger.kernel.org, linux-arch@vger.kernel.org Hi Arnd, This is question up to PowerPC guys not to me. of_debugfs_root sounds good. M >> This change is made because powerpc_debugfs_root is arch >> specific name. > ... >> >> static int powerpc_debugfs_init(void) >> { >> - powerpc_debugfs_root = debugfs_create_dir("powerpc", NULL); >> + debugfs_root = debugfs_create_dir("powerpc", NULL); >> > > You left the name as "powerpc", which sort of defeats the purpose of your > patch. Files in debugfs are by definition not part of the ABI, so it's > actually possible to change this as well. How about making this "of" > instead of "powerpc"? > > Arnd <>< >