From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752375AbdHBSP6 convert rfc822-to-8bit (ORCPT ); Wed, 2 Aug 2017 14:15:58 -0400 Received: from mga14.intel.com ([192.55.52.115]:16575 "EHLO mga14.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751719AbdHBSP5 (ORCPT ); Wed, 2 Aug 2017 14:15:57 -0400 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.41,312,1498546800"; d="scan'208";a="135203782" From: "Gross, Mark" To: Arvind Yadav , "arnd@arndb.de" , "gregkh@linuxfoundation.org" CC: "linux-kernel@vger.kernel.org" Subject: RE: [PATCH 1/3] char: tlclk: constify attribute_group structures. Thread-Topic: [PATCH 1/3] char: tlclk: constify attribute_group structures. Thread-Index: AQHTC4E5w1JD9/3guE+5GH/MLjWNDqJxWzoQ Date: Wed, 2 Aug 2017 18:15:54 +0000 Message-ID: References: <1501672745-2951-1-git-send-email-arvind.yadav.cs@gmail.com> <1501672745-2951-2-git-send-email-arvind.yadav.cs@gmail.com> In-Reply-To: <1501672745-2951-2-git-send-email-arvind.yadav.cs@gmail.com> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: dlp-product: dlpe-windows dlp-version: 10.0.102.7 dlp-reaction: no-action x-originating-ip: [10.22.254.140] Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 8BIT MIME-Version: 1.0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org The driver doesn't seem to touch the name field after its defined it looks ok functionally. Note: I am not a fan of const declarations in the C language. It's too complicated when we get into casting and pointers to structures. I'm not sure there is a lot of value in being pedantic on a few users of attribute_group structures. I'm thinking this change should be all or nothing and change core function prototypes to create warnings if they are not used with properly defined const 's. Otherwise this is just a cosmetic change of minimal value to a subset of drivers. FWIW there does seem to be a good number (cscope finds 1700+) of attribute_group users across the kernel and a good number of them are not using const. If you go down this pedantic path I don't think you should at just 3 drivers. Reviewed-by: mark gross --mark > -----Original Message----- > From: Arvind Yadav [mailto:arvind.yadav.cs@gmail.com] > Sent: Wednesday, August 2, 2017 4:19 AM > To: arnd@arndb.de; gregkh@linuxfoundation.org; Gross, Mark > > Cc: linux-kernel@vger.kernel.org > Subject: [PATCH 1/3] char: tlclk: constify attribute_group structures. > > attribute_group are not supposed to change at runtime. All functions working > with attribute_group provided by work with const > attribute_group. So mark the non-const structs as const. > > Signed-off-by: Arvind Yadav > --- > drivers/char/tlclk.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/char/tlclk.c b/drivers/char/tlclk.c index 572a517..6210bff > 100644 > --- a/drivers/char/tlclk.c > +++ b/drivers/char/tlclk.c > @@ -766,7 +766,7 @@ static ssize_t store_reset (struct device *d, > NULL > }; > > -static struct attribute_group tlclk_attribute_group = { > +static const struct attribute_group tlclk_attribute_group = { > .name = NULL, /* put in device directory */ > .attrs = tlclk_sysfs_entries, > }; > -- > 1.9.1