Messages in this thread | | | From | Geert Uytterhoeven <> | Date | Mon, 4 Jul 2016 08:49:12 +0200 | Subject | Re: [PATCH] thermal: hwmon: EXPORT_SYMBOL for thermal hwmon sysfs |
| |
Hi Morimoto-san,
On Mon, Jul 4, 2016 at 2:31 AM, Kuninori Morimoto <kuninori.morimoto.gx@renesas.com> wrote: > From: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com> > > thermal_add_hwmon_sysfs()/thermal_remove_hwmon_sysfs() need > EXPORT_SYMBOL(). Otherwise we will have ERROR > >>> ERROR: "thermal_remove_hwmon_sysfs" [drivers/thermal/rcar_thermal.ko] undefined! >>> ERROR: "thermal_add_hwmon_sysfs" [drivers/thermal/rcar_thermal.ko] undefined! > > Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com> > --- > drivers/thermal/thermal_hwmon.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/drivers/thermal/thermal_hwmon.c b/drivers/thermal/thermal_hwmon.c > index 06fd2ed9..8eecba8 100644 > --- a/drivers/thermal/thermal_hwmon.c > +++ b/drivers/thermal/thermal_hwmon.c > @@ -232,6 +232,7 @@ int thermal_add_hwmon_sysfs(struct thermal_zone_device *tz) > > return result; > } > +EXPORT_SYMBOL(thermal_add_hwmon_sysfs);
EXPORT_SYMBOL_GPL? That seems to be prevalent in the thermal subsystem, except for a few cases in cpu_cooling.c and thermal_helpers.c.
> void thermal_remove_hwmon_sysfs(struct thermal_zone_device *tz) > { > @@ -270,3 +271,4 @@ void thermal_remove_hwmon_sysfs(struct thermal_zone_device *tz) > hwmon_device_unregister(hwmon->device); > kfree(hwmon); > } > +EXPORT_SYMBOL(thermal_remove_hwmon_sysfs);
EXPORT_SYMBOL_GPL?
Gr{oetje,eeting}s,
Geert
-- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds
| |