From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-6.6 required=3.0 tests=DKIM_INVALID,DKIM_SIGNED, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI,SIGNED_OFF_BY, SPF_HELO_NONE,SPF_PASS autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 4F50EC10F14 for ; Thu, 3 Oct 2019 18:23:48 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 2A6772086A for ; Thu, 3 Oct 2019 18:23:48 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=fail reason="key not found in DNS" (0-bit key) header.d=codeaurora.org header.i=@codeaurora.org header.b="IKwrWf6R"; dkim=fail reason="key not found in DNS" (0-bit key) header.d=codeaurora.org header.i=@codeaurora.org header.b="RZh3OlGe" Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S2389062AbfJCSXr (ORCPT ); Thu, 3 Oct 2019 14:23:47 -0400 Received: from smtp.codeaurora.org ([198.145.29.96]:58072 "EHLO smtp.codeaurora.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S2388954AbfJCSXr (ORCPT ); Thu, 3 Oct 2019 14:23:47 -0400 Received: by smtp.codeaurora.org (Postfix, from userid 1000) id 19DCD6013C; Thu, 3 Oct 2019 18:23:46 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=codeaurora.org; s=default; t=1570127026; bh=RDN4iFNhQcAMgHoGWyKu66A34JYMo/wGVBxKCXxZUKM=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=IKwrWf6RH/tcSo+6t4M0G0qKv5ha6PfHFKYw6mZB4WazVMu13nOi4EBhXcvaChCie oWv3/xD76Zev6yEwGsEVU6++YKG5d/+YuHORjfvNo7J2Ob7ye9XqoWv/irj0WvLdZ4 QG/L9bRQdSD8rrCTAlj09XIIsG8BcEOxxCiC/EJo= Received: from mail.codeaurora.org (localhost.localdomain [127.0.0.1]) by smtp.codeaurora.org (Postfix) with ESMTP id 2C2BB6016D; Thu, 3 Oct 2019 18:23:45 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=codeaurora.org; s=default; t=1570127025; bh=RDN4iFNhQcAMgHoGWyKu66A34JYMo/wGVBxKCXxZUKM=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=RZh3OlGe1epEWCab1HjE4eBR7BC4MPil0dOYLh4k+hoA60ykZFKpBk2zsVYkwmuSz +8GES5FFyLDAIY7w7J8et4ohI5n5kXZfPQSNuPTXSL9rCWuv5tfHaNFVWVEphte3iT ftD5kDQIZN4BAy5DWU1ol4hRzsbcH8Pp8P3GuD0Q= MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII; format=flowed Content-Transfer-Encoding: 7bit Date: Thu, 03 Oct 2019 11:23:45 -0700 From: mnalajal@codeaurora.org To: Greg KH Cc: rafael@kernel.org, linux-arm-msm@vger.kernel.org, linux-kernel@vger.kernel.org, bjorn.andersson@linaro.org Subject: Re: [PATCH] base: soc: Handle custom soc information sysfs entries In-Reply-To: <20191003070610.GC1814133@kroah.com> References: <1570061174-4918-1-git-send-email-mnalajal@codeaurora.org> <20191003070610.GC1814133@kroah.com> Message-ID: <0d219d344cea82b5f6c1ab23341de25b@codeaurora.org> X-Sender: mnalajal@codeaurora.org User-Agent: Roundcube Webmail/1.2.5 Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 2019-10-03 00:06, Greg KH wrote: > On Wed, Oct 02, 2019 at 05:06:14PM -0700, Murali Nalajala wrote: >> Soc framework exposed sysfs entries are not sufficient for some >> of the h/w platforms. Currently there is no interface where soc >> drivers can expose further information about their SoCs via soc >> framework. This change address this limitation where clients can >> pass their custom entries as attribute group and soc framework >> would expose them as sysfs properties. >> >> Signed-off-by: Murali Nalajala >> --- >> drivers/base/soc.c | 26 ++++++++++++++++++-------- >> include/linux/sys_soc.h | 1 + >> 2 files changed, 19 insertions(+), 8 deletions(-) >> >> diff --git a/drivers/base/soc.c b/drivers/base/soc.c >> index 7c0c5ca..ec70a58 100644 >> --- a/drivers/base/soc.c >> +++ b/drivers/base/soc.c >> @@ -15,6 +15,8 @@ >> #include >> #include >> >> +#define NUM_ATTR_GROUPS 3 >> + >> static DEFINE_IDA(soc_ida); >> >> static ssize_t soc_info_get(struct device *dev, >> @@ -104,11 +106,6 @@ static ssize_t soc_info_get(struct device *dev, >> .is_visible = soc_attribute_mode, >> }; >> >> -static const struct attribute_group *soc_attr_groups[] = { >> - &soc_attr_group, >> - NULL, >> -}; >> - >> static void soc_release(struct device *dev) >> { >> struct soc_device *soc_dev = container_of(dev, struct soc_device, >> dev); >> @@ -121,6 +118,7 @@ static void soc_release(struct device *dev) >> struct soc_device *soc_device_register(struct soc_device_attribute >> *soc_dev_attr) >> { >> struct soc_device *soc_dev; >> + const struct attribute_group **soc_attr_groups = NULL; >> int ret; >> >> if (!soc_bus_type.p) { >> @@ -136,10 +134,20 @@ struct soc_device *soc_device_register(struct >> soc_device_attribute *soc_dev_attr >> goto out1; >> } >> >> + soc_attr_groups = kzalloc(sizeof(*soc_attr_groups) * >> + NUM_ATTR_GROUPS, GFP_KERNEL); >> + if (!soc_attr_groups) { >> + ret = -ENOMEM; >> + goto out2; >> + } >> + soc_attr_groups[0] = &soc_attr_group; >> + soc_attr_groups[1] = soc_dev_attr->custom_attr_group; >> + soc_attr_groups[2] = NULL; >> + >> /* Fetch a unique (reclaimable) SOC ID. */ >> ret = ida_simple_get(&soc_ida, 0, 0, GFP_KERNEL); >> if (ret < 0) >> - goto out2; >> + goto out3; >> soc_dev->soc_dev_num = ret; >> >> soc_dev->attr = soc_dev_attr; >> @@ -151,14 +159,16 @@ struct soc_device *soc_device_register(struct >> soc_device_attribute *soc_dev_attr >> >> ret = device_register(&soc_dev->dev); >> if (ret) >> - goto out3; >> + goto out4; >> >> return soc_dev; >> >> -out3: >> +out4: >> ida_simple_remove(&soc_ida, soc_dev->soc_dev_num); >> put_device(&soc_dev->dev); >> soc_dev = NULL; >> +out3: >> + kfree(soc_attr_groups); >> out2: >> kfree(soc_dev); >> out1: >> diff --git a/include/linux/sys_soc.h b/include/linux/sys_soc.h >> index 48ceea8..d9b3cf0 100644 >> --- a/include/linux/sys_soc.h >> +++ b/include/linux/sys_soc.h >> @@ -15,6 +15,7 @@ struct soc_device_attribute { >> const char *serial_number; >> const char *soc_id; >> const void *data; >> + const struct attribute_group *custom_attr_group; > > Shouldn't you make this: > const struct attribute_group **soc_groups; > > to match up with the rest of the way the driver core works? Assumption is, soc drivers send their custom attribute group and soc framework has already soc_attr_group" (basic info exposed). With my changes i am combining these two groups and passing to "device_register()". I do not think soc drivers have a requirement where they can pass various groups rather one single group attribute. > > thanks, > > greg k-h