Date: Tue, 24 Feb 2015 14:38:36 -0400
From: Eduardo Valentin <edubezval@gmail.com>
To: Nishanth Menon <nm@ti.com>
Cc: Zhang Rui <rui.zhang@intel.com>, linux-kernel@vger.kernel.org,
        linux-pm@vger.kernel.org, Guenter Roeck <linux@roeck-us.net>
Subject: Re: [PATCH] thermal: Introduce dummy functions when thermal is not
 defined
Message-ID: <20150224183830.GE3448@developer.amazonguestwifi.org>
References: <1423877282-4589-1-git-send-email-nm@ti.com>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha1;
	protocol="application/pgp-signature"; boundary="LSp5EJdfMPwZcMS1"
Content-Disposition: inline
In-Reply-To: <1423877282-4589-1-git-send-email-nm@ti.com>
User-Agent: Mutt/1.5.22 (2013-10-16)
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 5196
Lines: 145


--LSp5EJdfMPwZcMS1
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

Rui,


On Fri, Feb 13, 2015 at 07:28:02PM -0600, Nishanth Menon wrote:
> When CONFIG_THERMAL is not enabled, it is better to introduce
> equivalent dummy functions in the exported header than to
> introduce #ifdeffery in drivers using the function.
>=20
> This will prevent issues such as that reported in:
> http://www.spinics.net/lists/linux-next/msg31573.html
>=20
> While at it switch over to IS_ENABLED for thermal macros
> to allow for thermal framework to be built as framework
> and relevant APIs be usable by relevant drivers as a result.
>=20
> Reported-by: Guenter Roeck <linux@roeck-us.net>
> Signed-off-by: Nishanth Menon <nm@ti.com>

I am collecting this one and queuing in my -fixes branch.

> ---
>=20
> based on next-20150213 tag.
>=20
> Note: there are a few "CHECK: Alignment should match open parenthesis"
> checkpatch warnings which I ignored since these are pure dummy
> functions anyways.
>=20
>  include/linux/thermal.h |   56 +++++++++++++++++++++++++++++++++++++++++=
++++--
>  1 file changed, 54 insertions(+), 2 deletions(-)
>=20
> diff --git a/include/linux/thermal.h b/include/linux/thermal.h
> index fc52e307efab..5eac316490ea 100644
> --- a/include/linux/thermal.h
> +++ b/include/linux/thermal.h
> @@ -314,6 +314,8 @@ void thermal_zone_of_sensor_unregister(struct device =
*dev,
>  }
> =20
>  #endif
> +
> +#if IS_ENABLED(CONFIG_THERMAL)
>  struct thermal_zone_device *thermal_zone_device_register(const char *, i=
nt, int,
>  		void *, struct thermal_zone_device_ops *,
>  		const struct thermal_zone_params *, int, int);
> @@ -340,8 +342,58 @@ struct thermal_instance *get_thermal_instance(struct=
 thermal_zone_device *,
>  		struct thermal_cooling_device *, int);
>  void thermal_cdev_update(struct thermal_cooling_device *);
>  void thermal_notify_framework(struct thermal_zone_device *, int);
> -
> -#ifdef CONFIG_NET
> +#else
> +static inline struct thermal_zone_device *thermal_zone_device_register(
> +	const char *type, int trips, int mask, void *devdata,
> +	struct thermal_zone_device_ops *ops,
> +	const struct thermal_zone_params *tzp,
> +	int passive_delay, int polling_delay)
> +{ return ERR_PTR(-ENODEV); }
> +static inline void thermal_zone_device_unregister(
> +	struct thermal_zone_device *tz)
> +{ }
> +static inline int thermal_zone_bind_cooling_device(
> +	struct thermal_zone_device *tz, int trip,
> +	struct thermal_cooling_device *cdev,
> +	unsigned long upper, unsigned long lower)
> +{ return -ENODEV; }
> +static inline int thermal_zone_unbind_cooling_device(
> +	struct thermal_zone_device *tz, int trip,
> +	struct thermal_cooling_device *cdev)
> +{ return -ENODEV; }
> +static inline void thermal_zone_device_update(struct thermal_zone_device=
 *tz)
> +{ }
> +static inline struct thermal_cooling_device *
> +thermal_cooling_device_register(char *type, void *devdata,
> +	const struct thermal_cooling_device_ops *ops)
> +{ return ERR_PTR(-ENODEV); }
> +static inline struct thermal_cooling_device *
> +thermal_of_cooling_device_register(struct device_node *np,
> +	char *type, void *devdata, const struct thermal_cooling_device_ops *ops)
> +{ return ERR_PTR(-ENODEV); }
> +static inline void thermal_cooling_device_unregister(
> +	struct thermal_cooling_device *cdev)
> +{ }
> +static inline struct thermal_zone_device *thermal_zone_get_zone_by_name(
> +		const char *name)
> +{ return ERR_PTR(-ENODEV); }
> +static inline int thermal_zone_get_temp(
> +		struct thermal_zone_device *tz, unsigned long *temp)
> +{ return -ENODEV; }
> +static inline int get_tz_trend(struct thermal_zone_device *tz, int trip)
> +{ return -ENODEV; }
> +static inline struct thermal_instance *
> +get_thermal_instance(struct thermal_zone_device *tz,
> +	struct thermal_cooling_device *cdev, int trip)
> +{ return ERR_PTR(-ENODEV); }
> +static inline void thermal_cdev_update(struct thermal_cooling_device *cd=
ev)
> +{ }
> +static inline void thermal_notify_framework(struct thermal_zone_device *=
tz,
> +	int trip)
> +{ }
> +#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
> --=20
> 1.7.9.5
>=20

--LSp5EJdfMPwZcMS1
Content-Type: application/pgp-signature; name="signature.asc"
Content-Description: Digital signature

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2

iQEcBAEBAgAGBQJU7MUeAAoJEMLUO4d9pOJWqDcH/j7Q14XIXlfPfH5WHh4scprH
87k5koO5007JKslWNQmFKaZcTAewa1oZ4LNxIJr9nGGACK2540zE7dMBl5tANmhE
YfbXbpvbpjz2wkimDDBTJiyK0uioRaSs2FccAgh/f4hxgNWMigbmlHcihPwTANXq
FXvPz5lKMFxtmiangRPjNTJsxdh7xPNwX5SR/t2M+xJDPbdYpTS1RQGpAiYUuLvO
5f1bUsOse6yYqv/VYs/T+Q1NiUWozMuw6nLVPRJ3Tj7yK2v6bzDJKgwcjeRVl6kd
aoKOd1H2BiKluGuz4BBUbzkjjbRnQpsoC/y2QJrYLbC8SyBoGDTi9Aa3qLCJB4Y=
=uyst
-----END PGP SIGNATURE-----

--LSp5EJdfMPwZcMS1--
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/