From patchwork Fri Oct 9 15:04:27 2015 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Viresh Kumar X-Patchwork-Id: 7362301 Return-Path: X-Original-To: patchwork-linux-pm@patchwork.kernel.org Delivered-To: patchwork-parsemail@patchwork1.web.kernel.org Received: from mail.kernel.org (mail.kernel.org [198.145.29.136]) by patchwork1.web.kernel.org (Postfix) with ESMTP id CF9749F1B9 for ; Fri, 9 Oct 2015 15:04:39 +0000 (UTC) Received: from mail.kernel.org (localhost [127.0.0.1]) by mail.kernel.org (Postfix) with ESMTP id A625C207A5 for ; Fri, 9 Oct 2015 15:04:38 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 77588207A0 for ; Fri, 9 Oct 2015 15:04:37 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752750AbbJIPEg (ORCPT ); Fri, 9 Oct 2015 11:04:36 -0400 Received: from mail-pa0-f49.google.com ([209.85.220.49]:36036 "EHLO mail-pa0-f49.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751563AbbJIPEf (ORCPT ); Fri, 9 Oct 2015 11:04:35 -0400 Received: by pablk4 with SMTP id lk4so89173170pab.3 for ; Fri, 09 Oct 2015 08:04:35 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:from:to:cc:subject:date:message-id; bh=8lvNHMLBiy93eyWTfWORezLvREiKelO8rqfRehoXYV8=; b=STuvkyN66pLTct7Jag6l1oXrl5d+0VNPv5n9Yj3AFzs3Tqu5BRBliS7m6Yu3gciKBC txXEEVmgIZC+10c/tV3i0WN4aZkOZS7h7nWnxnKhkVpl1Cf8MgIhjua0JUOZrT/Gw0M2 hAmFbqfakLP0iAgc6NZBH2U8VIJos5JoHy0b3Y05o/G2llcPDAfsJmcMiOCl/zq505Yl 3Wjh6nZh9olKyUs/366Jgdb5u98c3qHe4qJZ4ypQVUmxv2Hekp52twV9yQQI2iiGQqR+ EojO6dXeiRqw3uRIWE88pAyx1iJjGONk9TCSFJvQmjVwFFUlehLv+++JVkBM3WcF0+Z6 mkzw== X-Gm-Message-State: ALoCoQmnw/IWUZq5fGnBAy10NBpe7LsB7pqDWFibJh+ldbu05bzMRQb1xSP7OGQ4BKM5Qm/qo4KD X-Received: by 10.68.216.135 with SMTP id oq7mr7837420pbc.9.1444403074940; Fri, 09 Oct 2015 08:04:34 -0700 (PDT) Received: from localhost ([122.171.124.154]) by smtp.gmail.com with ESMTPSA id lo9sm2998861pab.19.2015.10.09.08.04.33 (version=TLSv1.2 cipher=RC4-SHA bits=128/128); Fri, 09 Oct 2015 08:04:34 -0700 (PDT) From: Viresh Kumar To: Greg Kroah-Hartman Cc: linaro-kernel@lists.linaro.org, Rafael Wysocki , linux-kernel@vger.kernel.org, arnd@arndb.de, sboyd@codeaurora.org, linux-pm@vger.kernel.org, linux@arm.linux.org.uk, stern@rowland.harvard.edu, Viresh Kumar Subject: [PATCH] debugfs: Add debugfs_create_ulong() Date: Fri, 9 Oct 2015 20:34:27 +0530 Message-Id: <3a48d2bb4320bba82f91354d7165c300ef8108bd.1444402709.git.viresh.kumar@linaro.org> X-Mailer: git-send-email 2.4.0 Sender: linux-pm-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-pm@vger.kernel.org X-Spam-Status: No, score=-6.9 required=5.0 tests=BAYES_00, RCVD_IN_DNSWL_HI, T_RP_MATCHES_RCVD, UNPARSEABLE_RELAY autolearn=ham version=3.3.1 X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on mail.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP Add debugfs_create_ulong() for the users of type 'unsigned long'. These will be 32 bits long on a 32 bit machine and 64 bits long on a 64 bit machine and so can't use the traditional debugfs_create_u64() or debugfs_create_u32() APIs. Signed-off-by: Viresh Kumar --- Hi Greg, I thought there were few supporters of this interface and I need it for the OPP debugfs stuff. So this is an attempt to move the discussion out of the closed thread, to an open one where others can join. And if all goes well, then you can apply it directly. Its tested with the OPP debugfs stuff :) fs/debugfs/file.c | 55 +++++++++++++++++++++++++++++++++++++++++++++++++ include/linux/debugfs.h | 4 ++++ 2 files changed, 59 insertions(+) diff --git a/fs/debugfs/file.c b/fs/debugfs/file.c index b70c20fae502..28a7fe6ea618 100644 --- a/fs/debugfs/file.c +++ b/fs/debugfs/file.c @@ -251,6 +251,60 @@ struct dentry *debugfs_create_u64(const char *name, umode_t mode, } EXPORT_SYMBOL_GPL(debugfs_create_u64); +static int debugfs_ulong_set(void *data, u64 val) +{ + *(unsigned long *)data = val; + return 0; +} + +static int debugfs_ulong_get(void *data, u64 *val) +{ + *val = *(unsigned long *)data; + return 0; +} +DEFINE_SIMPLE_ATTRIBUTE(fops_ulong, debugfs_ulong_get, debugfs_ulong_set, "%llu\n"); +DEFINE_SIMPLE_ATTRIBUTE(fops_ulong_ro, debugfs_ulong_get, NULL, "%llu\n"); +DEFINE_SIMPLE_ATTRIBUTE(fops_ulong_wo, NULL, debugfs_ulong_set, "%llu\n"); + +/** + * debugfs_create_ulong - create a debugfs file that is used to read and write + * an unsigned long value. + * @name: a pointer to a string containing the name of the file to create. + * @mode: the permission that the file should have + * @parent: a pointer to the parent dentry for this file. This should be a + * directory dentry if set. If this parameter is %NULL, then the + * file will be created in the root of the debugfs filesystem. + * @value: a pointer to the variable that the file should read to and write + * from. + * + * This function creates a file in debugfs with the given name that + * contains the value of the variable @value. If the @mode variable is so + * set, it can be read from, and written to. + * + * This function will return a pointer to a dentry if it succeeds. This + * pointer must be passed to the debugfs_remove() function when the file is + * to be removed (no automatic cleanup happens if your module is unloaded, + * you are responsible here.) If an error occurs, %NULL will be returned. + * + * If debugfs is not enabled in the kernel, the value -%ENODEV will be + * returned. It is not wise to check for this value, but rather, check for + * %NULL or !%NULL instead as to eliminate the need for #ifdef in the calling + * code. + */ +struct dentry *debugfs_create_ulong(const char *name, umode_t mode, + struct dentry *parent, unsigned long *value) +{ + /* if there are no write bits set, make read only */ + if (!(mode & S_IWUGO)) + return debugfs_create_file(name, mode, parent, value, &fops_ulong_ro); + /* if there are no read bits set, make write only */ + if (!(mode & S_IRUGO)) + return debugfs_create_file(name, mode, parent, value, &fops_ulong_wo); + + return debugfs_create_file(name, mode, parent, value, &fops_ulong); +} +EXPORT_SYMBOL_GPL(debugfs_create_ulong); + DEFINE_SIMPLE_ATTRIBUTE(fops_x8, debugfs_u8_get, debugfs_u8_set, "0x%02llx\n"); DEFINE_SIMPLE_ATTRIBUTE(fops_x8_ro, debugfs_u8_get, NULL, "0x%02llx\n"); DEFINE_SIMPLE_ATTRIBUTE(fops_x8_wo, NULL, debugfs_u8_set, "0x%02llx\n"); @@ -264,6 +318,7 @@ DEFINE_SIMPLE_ATTRIBUTE(fops_x32_ro, debugfs_u32_get, NULL, "0x%08llx\n"); DEFINE_SIMPLE_ATTRIBUTE(fops_x32_wo, NULL, debugfs_u32_set, "0x%08llx\n"); DEFINE_SIMPLE_ATTRIBUTE(fops_x64, debugfs_u64_get, debugfs_u64_set, "0x%016llx\n"); +DEFINE_SIMPLE_ATTRIBUTE(fops_xlong, debugfs_ulong_get, debugfs_ulong_set, "0x%016llx\n"); /* * debugfs_create_x{8,16,32,64} - create a debugfs file that is used to read and write an unsigned {8,16,32,64}-bit value diff --git a/include/linux/debugfs.h b/include/linux/debugfs.h index 8321fe3058d6..60b150248b54 100644 --- a/include/linux/debugfs.h +++ b/include/linux/debugfs.h @@ -79,6 +79,8 @@ struct dentry *debugfs_create_u32(const char *name, umode_t mode, struct dentry *parent, u32 *value); struct dentry *debugfs_create_u64(const char *name, umode_t mode, struct dentry *parent, u64 *value); +struct dentry *debugfs_create_ulong(const char *name, umode_t mode, + struct dentry *parent, unsigned long *value); struct dentry *debugfs_create_x8(const char *name, umode_t mode, struct dentry *parent, u8 *value); struct dentry *debugfs_create_x16(const char *name, umode_t mode, @@ -87,6 +89,8 @@ struct dentry *debugfs_create_x32(const char *name, umode_t mode, struct dentry *parent, u32 *value); struct dentry *debugfs_create_x64(const char *name, umode_t mode, struct dentry *parent, u64 *value); +struct dentry *debugfs_create_xlong(const char *name, umode_t mode, + struct dentry *parent, unsigned long *value); struct dentry *debugfs_create_size_t(const char *name, umode_t mode, struct dentry *parent, size_t *value); struct dentry *debugfs_create_atomic_t(const char *name, umode_t mode,