On Mon, 2007-09-03 at 01:41 +0530, Satyam Sharma wrote:
quoted text
> net/wireless/sysfs.c:108: warning: =E2=80=98wiphy_uevent=E2=80=99 defined=
 but not used
quoted text
>=20
> when CONFIG_HOTPLUG=3Dn is because the only usage site of this function
> is #ifdef'ed as such, so let's #ifdef the definition also.
>=20
> Signed-off-by: Satyam Sharma <satyam@infradead.org>

Looks good to me. The new style seems to be to do
#ifdef SOMETHING
code
#else
#define wiphy_uevent NULL
#endif

and then using it unconditionally, but I'm fine with both.

Acked-by: Johannes Berg <johannes@sipsolutions.net>

quoted text
> ---
>=20
>  net/wireless/sysfs.c |    2 ++
>  1 file changed, 2 insertions(+)
>=20
> --- linux-2.6.23-rc4-mm1/net/wireless/sysfs.c=E2=80=BEfix	2007-09-02 20:0=
6:11.000000000 +0530
quoted text
> +++ linux-2.6.23-rc4-mm1/net/wireless/sysfs.c	2007-09-02 20:07:01.0000000=
00 +0530
quoted text
> @@ -104,11 +104,13 @@ static void wiphy_dev_release(struct dev
>  	cfg80211_dev_free(rdev);
>  }
> =20
> +#ifdef CONFIG_HOTPLUG
>  static int wiphy_uevent(struct device *dev, struct kobj_uevent_env *env)
>  {
>  	/* TODO, we probably need stuff here */
>  	return 0;
>  }
> +#endif
> =20
>  struct class ieee80211_class =3D {
>  	.name =3D "ieee80211",