From patchwork Thu Oct 17 12:27:33 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Amit Kucheria X-Patchwork-Id: 176580 Delivered-To: patch@linaro.org Received: by 2002:a92:7e96:0:0:0:0:0 with SMTP id q22csp849359ill; Thu, 17 Oct 2019 05:27:50 -0700 (PDT) X-Google-Smtp-Source: APXvYqwtuV9eJ9UhpP22LDD31XRyppdFT0cxBvY3kr5F/0FBIemvr3vG+TPrCGMuCId68/qKpNcF X-Received: by 2002:a17:906:c443:: with SMTP id ck3mr3296120ejb.0.1571315270099; Thu, 17 Oct 2019 05:27:50 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1571315270; cv=none; d=google.com; s=arc-20160816; b=iAtvuxJ/uGum7XVJhNi/QgesSPjkks+0qoOZ+4EpURjHXCV+paYl3Y942SkqxLvynT 6yXkzYrUtBuLNqVu2VvigkqqDrvgI9CCH1+KGGUKmqeK0b5rLysM22T3bXVjMHnTAvFM rY7cHRT/6E72cZRDakw+ZCSvhOEUXPeVDIIrkBqUh/YPMRRsAGB917sokNsBUziN6MPu 2juiR9ktoNc1bfu3roRmUO3dLSM1Hwn53jE5Qnqi5qsTe5hfIMp5XOLpITXGbwh4qomE OoyjfqyJkPMsfZImlxauVdTKGDbIyiGi19eXZZqX1oE+406CfyOuF1m7qU7HF1LfkXtj zdLg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:references:in-reply-to:references :in-reply-to:message-id:date:subject:cc:to:from:dkim-signature; bh=ILr9joUKye34Z5XuMa3tOX2NbCDB31U4gUj4EK9AjAc=; b=L0jLTtdMQ7LQ4ym0pKHCEKmKvllcmt5Ndz+6+ORRjasU88PynRCubJKKI/Jz+XZpwr ODDo6+LwMOmNKSbg4uy9oHQ9rCvEZVm7hz1h193TrzmW9jeUSjDmbut3AUByXYUBB+sE nGkMoPQTdR/TV20dgqayZAlZd2gE6fe7Vb43SZv2crFK2yP3GiWwv038yuoPdyPiI8sY Ba+dSwfrnEO+/orMdS7E7AM4J2dZhpszzUPc5xJ4csGEhFYNgrMKRbWfqAgA7xjeoqPg Fs4mS/Wj6pM9eX1UOfmijN6XMVsU+BADnSmbLH3wl77rkXxbrvmeMIfMau7osO5qfND3 AA2A== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=LFUgaeMd; spf=pass (google.com: best guess record for domain of linux-arm-msm-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-arm-msm-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id f17si1862648edf.328.2019.10.17.05.27.49; Thu, 17 Oct 2019 05:27:50 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-arm-msm-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=LFUgaeMd; spf=pass (google.com: best guess record for domain of linux-arm-msm-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-arm-msm-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S2502321AbfJQM1t (ORCPT + 15 others); Thu, 17 Oct 2019 08:27:49 -0400 Received: from mail-pf1-f196.google.com ([209.85.210.196]:45027 "EHLO mail-pf1-f196.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S2406117AbfJQM1s (ORCPT ); Thu, 17 Oct 2019 08:27:48 -0400 Received: by mail-pf1-f196.google.com with SMTP id q21so1553910pfn.11 for ; Thu, 17 Oct 2019 05:27:48 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=from:to:cc:subject:date:message-id:in-reply-to:references :in-reply-to:references; bh=ILr9joUKye34Z5XuMa3tOX2NbCDB31U4gUj4EK9AjAc=; b=LFUgaeMdjVeFcNKERe9xHFpUZYWPcsn8/0qej7QjTdu5UabxWgRjnT6NzE3yRrQb6i LII1mzhB362FPr3U6rvDWoFg2f/Ll4TxW4TRRPhwIQRA161dxfolDo7NUFRjWLIfMIEa IzBLeDnElE12GG44AQ9OP95rnWvjhVto1pY4aqXrWR2pAWGNy85FFXsRXXqcCXThQuXd dBWszGjkGGA3X4B3G2qLLGnKthf1JvgwwCV9DI/pgLmGfJHiwrvy5sPsucbnfYOQzgA9 4QXz7LhTzFXowaAPOxY2PnXfBkM4DEo2kzvS7LaLk3+aVYy7JE91a2H3FEeul8C7oe0r gSVw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references:in-reply-to:references; bh=ILr9joUKye34Z5XuMa3tOX2NbCDB31U4gUj4EK9AjAc=; b=e6lLohWn5FSXEiB1/ZglxmQsVX63OyqCq4058TWWZeA0+WMsxCUaSnZiy3buLC1RU7 HRglALcMX6G2bn2EyUR1GtVKgK4KPrtj+2VJGrSVhcWUOVey65ptYp103Dr2C2iRMIiA RdEsGprFZMGSuK/6N4+qJbJ8rVkqZvJlKPrhQFFTXAgCaqJJYvHsBzU9HRNLcFskds4K wo+ib8IZ7Yy8H441XPIIhu1y31pjBReY656uxq2b9FZarTkmHe7kMV7HISGE+x18QdZq XHpntJcVJndW+WWiR7pa69XInCCZ7ji9XSMrg2HRneZNFGPh7cOsI3bewcX3UJwgkcaT r4Iw== X-Gm-Message-State: APjAAAVNglq3YXGAffcDDGAgy0aNlIJ0aJxHxoZAhTQ8SfUFl5Lp3vcW 05rfI1Mw4nPdD3AhXru+ohUAUw== X-Received: by 2002:a17:90a:aa97:: with SMTP id l23mr4002710pjq.7.1571315267761; Thu, 17 Oct 2019 05:27:47 -0700 (PDT) Received: from localhost ([49.248.54.231]) by smtp.gmail.com with ESMTPSA id ep10sm15468221pjb.2.2019.10.17.05.27.46 (version=TLS1_2 cipher=ECDHE-RSA-CHACHA20-POLY1305 bits=256/256); Thu, 17 Oct 2019 05:27:47 -0700 (PDT) From: Amit Kucheria To: linux-kernel@vger.kernel.org, linux-arm-msm@vger.kernel.org, daniel.lezcano@linaro.org, viresh.kumar@linaro.org, sudeep.holla@arm.com, bjorn.andersson@linaro.org, edubezval@gmail.com, agross@kernel.org, tdas@codeaurora.org, swboyd@chromium.org, ilina@codeaurora.org, "Rafael J. Wysocki" , Amit Kucheria , Zhang Rui Cc: linux-doc@vger.kernel.org, linux-pm@vger.kernel.org Subject: [PATCH v3 1/6] thermal: Remove netlink support Date: Thu, 17 Oct 2019 17:57:33 +0530 Message-Id: X-Mailer: git-send-email 2.17.1 In-Reply-To: References: In-Reply-To: References: Sender: linux-arm-msm-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-arm-msm@vger.kernel.org There are no users of netlink messages for thermal inside the kernel. Remove the code and adjust the documentation. Signed-off-by: Amit Kucheria --- .../driver-api/thermal/sysfs-api.rst | 26 +---- drivers/thermal/thermal_core.c | 107 +----------------- include/linux/thermal.h | 11 -- 3 files changed, 10 insertions(+), 134 deletions(-) -- 2.17.1 diff --git a/Documentation/driver-api/thermal/sysfs-api.rst b/Documentation/driver-api/thermal/sysfs-api.rst index fab2c9b36d08..b40b1f839148 100644 --- a/Documentation/driver-api/thermal/sysfs-api.rst +++ b/Documentation/driver-api/thermal/sysfs-api.rst @@ -725,24 +725,10 @@ method, the sys I/F structure will be built like this:: |---temp1_input: 37000 |---temp1_crit: 100000 -4. Event Notification +4. Export Symbol APIs ===================== -The framework includes a simple notification mechanism, in the form of a -netlink event. Netlink socket initialization is done during the _init_ -of the framework. Drivers which intend to use the notification mechanism -just need to call thermal_generate_netlink_event() with two arguments viz -(originator, event). The originator is a pointer to struct thermal_zone_device -from where the event has been originated. An integer which represents the -thermal zone device will be used in the message to identify the zone. The -event will be one of:{THERMAL_AUX0, THERMAL_AUX1, THERMAL_CRITICAL, -THERMAL_DEV_FAULT}. Notification can be sent when the current temperature -crosses any of the configured thresholds. - -5. Export Symbol APIs -===================== - -5.1. get_tz_trend +4.1. get_tz_trend ----------------- This function returns the trend of a thermal zone, i.e the rate of change @@ -751,14 +737,14 @@ are supposed to implement the callback. If they don't, the thermal framework calculated the trend by comparing the previous and the current temperature values. -5.2. get_thermal_instance +4.2. get_thermal_instance ------------------------- This function returns the thermal_instance corresponding to a given {thermal_zone, cooling_device, trip_point} combination. Returns NULL if such an instance does not exist. -5.3. thermal_notify_framework +4.3. thermal_notify_framework ----------------------------- This function handles the trip events from sensor drivers. It starts @@ -768,14 +754,14 @@ and does actual throttling for other trip points i.e ACTIVE and PASSIVE. The throttling policy is based on the configured platform data; if no platform data is provided, this uses the step_wise throttling policy. -5.4. thermal_cdev_update +4.4. thermal_cdev_update ------------------------ This function serves as an arbitrator to set the state of a cooling device. It sets the cooling device to the deepest cooling state if possible. -6. thermal_emergency_poweroff +5. thermal_emergency_poweroff ============================= On an event of critical trip temperature crossing. Thermal framework diff --git a/drivers/thermal/thermal_core.c b/drivers/thermal/thermal_core.c index d4481cc8958f..d21b754baee2 100644 --- a/drivers/thermal/thermal_core.c +++ b/drivers/thermal/thermal_core.c @@ -19,8 +19,6 @@ #include #include #include -#include -#include #include #define CREATE_TRACE_POINTS @@ -1464,97 +1462,6 @@ struct thermal_zone_device *thermal_zone_get_zone_by_name(const char *name) } EXPORT_SYMBOL_GPL(thermal_zone_get_zone_by_name); -#ifdef CONFIG_NET -static const struct genl_multicast_group thermal_event_mcgrps[] = { - { .name = THERMAL_GENL_MCAST_GROUP_NAME, }, -}; - -static struct genl_family thermal_event_genl_family __ro_after_init = { - .module = THIS_MODULE, - .name = THERMAL_GENL_FAMILY_NAME, - .version = THERMAL_GENL_VERSION, - .maxattr = THERMAL_GENL_ATTR_MAX, - .mcgrps = thermal_event_mcgrps, - .n_mcgrps = ARRAY_SIZE(thermal_event_mcgrps), -}; - -int thermal_generate_netlink_event(struct thermal_zone_device *tz, - enum events event) -{ - struct sk_buff *skb; - struct nlattr *attr; - struct thermal_genl_event *thermal_event; - void *msg_header; - int size; - int result; - static unsigned int thermal_event_seqnum; - - if (!tz) - return -EINVAL; - - /* allocate memory */ - size = nla_total_size(sizeof(struct thermal_genl_event)) + - nla_total_size(0); - - skb = genlmsg_new(size, GFP_ATOMIC); - if (!skb) - return -ENOMEM; - - /* add the genetlink message header */ - msg_header = genlmsg_put(skb, 0, thermal_event_seqnum++, - &thermal_event_genl_family, 0, - THERMAL_GENL_CMD_EVENT); - if (!msg_header) { - nlmsg_free(skb); - return -ENOMEM; - } - - /* fill the data */ - attr = nla_reserve(skb, THERMAL_GENL_ATTR_EVENT, - sizeof(struct thermal_genl_event)); - - if (!attr) { - nlmsg_free(skb); - return -EINVAL; - } - - thermal_event = nla_data(attr); - if (!thermal_event) { - nlmsg_free(skb); - return -EINVAL; - } - - memset(thermal_event, 0, sizeof(struct thermal_genl_event)); - - thermal_event->orig = tz->id; - thermal_event->event = event; - - /* send multicast genetlink message */ - genlmsg_end(skb, msg_header); - - result = genlmsg_multicast(&thermal_event_genl_family, skb, 0, - 0, GFP_ATOMIC); - if (result) - dev_err(&tz->device, "Failed to send netlink event:%d", result); - - return result; -} -EXPORT_SYMBOL_GPL(thermal_generate_netlink_event); - -static int __init genetlink_init(void) -{ - return genl_register_family(&thermal_event_genl_family); -} - -static void genetlink_exit(void) -{ - genl_unregister_family(&thermal_event_genl_family); -} -#else /* !CONFIG_NET */ -static inline int genetlink_init(void) { return 0; } -static inline void genetlink_exit(void) {} -#endif /* !CONFIG_NET */ - static int thermal_pm_notify(struct notifier_block *nb, unsigned long mode, void *_unused) { @@ -1601,19 +1508,15 @@ static int __init thermal_init(void) mutex_init(&poweroff_lock); result = thermal_register_governors(); if (result) - goto error; + goto init_exit; result = class_register(&thermal_class); if (result) goto unregister_governors; - result = genetlink_init(); - if (result) - goto unregister_class; - result = of_parse_thermal_zones(); if (result) - goto exit_netlink; + goto exit_zone_parse; result = register_pm_notifier(&thermal_pm_nb); if (result) @@ -1622,13 +1525,11 @@ static int __init thermal_init(void) return 0; -exit_netlink: - genetlink_exit(); -unregister_class: +exit_zone_parse: class_unregister(&thermal_class); unregister_governors: thermal_unregister_governors(); -error: +init_exit: ida_destroy(&thermal_tz_ida); ida_destroy(&thermal_cdev_ida); mutex_destroy(&thermal_list_lock); diff --git a/include/linux/thermal.h b/include/linux/thermal.h index e45659c75920..d9111aebb97d 100644 --- a/include/linux/thermal.h +++ b/include/linux/thermal.h @@ -544,15 +544,4 @@ static inline void thermal_notify_framework(struct thermal_zone_device *tz, { } #endif /* CONFIG_THERMAL */ -#if defined(CONFIG_NET) && IS_ENABLED(CONFIG_THERMAL) -extern int thermal_generate_netlink_event(struct thermal_zone_device *tz, - enum events event); -#else -static inline int thermal_generate_netlink_event(struct thermal_zone_device *tz, - enum events event) -{ - return 0; -} -#endif - #endif /* __THERMAL_H__ */