* [PATCH 3.14] power_supply: don't export power_supply_notifier
@ 2014-02-21 13:19 Johannes Berg
2014-02-25 11:37 ` Jenny Tc
0 siblings, 1 reply; 2+ messages in thread
From: Johannes Berg @ 2014-02-21 13:19 UTC (permalink / raw)
To: Jenny TC
Cc: Pali Rohár, linux-kernel, Anton Vorontsov, luca, johannes,
Johannes Berg
From: Johannes Berg <johannes.berg@intel.com>
Since there are registration/unregistration functions and the
invocation is in the core code, there's no need to export the
notifier chain head, make it static instead.
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
---
This should probably go into 3.14 so it's not exported in the release.
---
drivers/power/power_supply_core.c | 3 +--
include/linux/power_supply.h | 1 -
2 files changed, 1 insertion(+), 3 deletions(-)
diff --git a/drivers/power/power_supply_core.c b/drivers/power/power_supply_core.c
index 26606641fe44..31a105235c3e 100644
--- a/drivers/power/power_supply_core.c
+++ b/drivers/power/power_supply_core.c
@@ -25,8 +25,7 @@
struct class *power_supply_class;
EXPORT_SYMBOL_GPL(power_supply_class);
-ATOMIC_NOTIFIER_HEAD(power_supply_notifier);
-EXPORT_SYMBOL_GPL(power_supply_notifier);
+static ATOMIC_NOTIFIER_HEAD(power_supply_notifier);
static struct device_type power_supply_dev_type;
diff --git a/include/linux/power_supply.h b/include/linux/power_supply.h
index c9dc4e09854c..80968ee25940 100644
--- a/include/linux/power_supply.h
+++ b/include/linux/power_supply.h
@@ -240,7 +240,6 @@ struct power_supply_info {
int use_for_apm;
};
-extern struct atomic_notifier_head power_supply_notifier;
extern int power_supply_reg_notifier(struct notifier_block *nb);
extern void power_supply_unreg_notifier(struct notifier_block *nb);
extern struct power_supply *power_supply_get_by_name(const char *name);
--
1.8.5.3
^ permalink raw reply [flat|nested] 2+ messages in thread* Re: [PATCH 3.14] power_supply: don't export power_supply_notifier
2014-02-21 13:19 [PATCH 3.14] power_supply: don't export power_supply_notifier Johannes Berg
@ 2014-02-25 11:37 ` Jenny Tc
0 siblings, 0 replies; 2+ messages in thread
From: Jenny Tc @ 2014-02-25 11:37 UTC (permalink / raw)
To: Johannes Berg
Cc: Pali Rohár, linux-kernel, Anton Vorontsov, luca, Johannes Berg
On Fri, Feb 21, 2014 at 02:19:28PM +0100, Johannes Berg wrote:
> From: Johannes Berg <johannes.berg@intel.com>
>
> Since there are registration/unregistration functions and the
> invocation is in the core code, there's no need to export the
> notifier chain head, make it static instead.
This is a generic notifier which can be used outside power_supply_core.c
to post psy events as in patch
power_supply-Introduce-generic-psy-charging-driver.patch. So the notifier chain
need to be exported.
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2014-02-25 11:37 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-02-21 13:19 [PATCH 3.14] power_supply: don't export power_supply_notifier Johannes Berg
2014-02-25 11:37 ` Jenny Tc
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®