Below patch removes the following warning during compilation.
drivers/net/wireless/libertas/cmd.c:826: warning: unused variable 'old_channel'
Signed-off-by : Manish Katiyar <[email protected]>
---
drivers/net/wireless/libertas/cmd.c | 2 ++
1 files changed, 2 insertions(+), 0 deletions(-)
diff --git a/drivers/net/wireless/libertas/cmd.c
b/drivers/net/wireless/libertas/cmd.c
index a912fb6..297696d 100644
--- a/drivers/net/wireless/libertas/cmd.c
+++ b/drivers/net/wireless/libertas/cmd.c
@@ -823,7 +823,9 @@ int lbs_update_channel(struct lbs_private *priv)
int lbs_set_channel(struct lbs_private *priv, u8 channel)
{
struct cmd_ds_802_11_rf_channel cmd;
+#ifdef DEBUG
u8 old_channel = priv->curbssparams.channel;
+#endif
int ret = 0;
lbs_deb_enter(LBS_DEB_CMD);
--
1.5.4.3
Thanks -
Manish
On Mon, Oct 13, 2008 at 4:22 PM, Manish Katiyar <[email protected]> wrote:
> Below patch removes the following warning during compilation.
>
> drivers/net/wireless/libertas/cmd.c:826: warning: unused variable 'old_channel'
Any ack/nack on this ??
Thanks -
Manish
>
>
> Signed-off-by : Manish Katiyar <[email protected]>
> ---
> drivers/net/wireless/libertas/cmd.c | 2 ++
> 1 files changed, 2 insertions(+), 0 deletions(-)
>
> diff --git a/drivers/net/wireless/libertas/cmd.c
> b/drivers/net/wireless/libertas/cmd.c
> index a912fb6..297696d 100644
> --- a/drivers/net/wireless/libertas/cmd.c
> +++ b/drivers/net/wireless/libertas/cmd.c
> @@ -823,7 +823,9 @@ int lbs_update_channel(struct lbs_private *priv)
> int lbs_set_channel(struct lbs_private *priv, u8 channel)
> {
> struct cmd_ds_802_11_rf_channel cmd;
> +#ifdef DEBUG
> u8 old_channel = priv->curbssparams.channel;
> +#endif
> int ret = 0;
>
> lbs_deb_enter(LBS_DEB_CMD);
> --
> 1.5.4.3
>
>
> Thanks -
> Manish
>
On Tue, Oct 14, 2008 at 10:06 PM, Dan Williams <[email protected]> wrote:
> On Mon, 2008-10-13 at 16:22 +0530, Manish Katiyar wrote:
>> Below patch removes the following warning during compilation.
>>
>> drivers/net/wireless/libertas/cmd.c:826: warning: unused variable 'old_channel'
>>
>>
>> Signed-off-by : Manish Katiyar <[email protected]>
>
> Acked-by: Dan Williams <[email protected]>
Hi Dan,
A newbie question. Will you merge this patch or will I have to send it
to Andrew again acked by you ?
Thanks -
Manish
>
>> ---
>> drivers/net/wireless/libertas/cmd.c | 2 ++
>> 1 files changed, 2 insertions(+), 0 deletions(-)
>>
>> diff --git a/drivers/net/wireless/libertas/cmd.c
>> b/drivers/net/wireless/libertas/cmd.c
>> index a912fb6..297696d 100644
>> --- a/drivers/net/wireless/libertas/cmd.c
>> +++ b/drivers/net/wireless/libertas/cmd.c
>> @@ -823,7 +823,9 @@ int lbs_update_channel(struct lbs_private *priv)
>> int lbs_set_channel(struct lbs_private *priv, u8 channel)
>> {
>> struct cmd_ds_802_11_rf_channel cmd;
>> +#ifdef DEBUG
>> u8 old_channel = priv->curbssparams.channel;
>> +#endif
>> int ret = 0;
>>
>> lbs_deb_enter(LBS_DEB_CMD);
>
>
On Wed, Oct 15, 2008 at 2:17 PM, Johannes Berg
<[email protected]> wrote:
> [dropping libertas list, subscribers only]
>
> On Wed, 2008-10-15 at 13:40 +0530, Manish Katiyar wrote:
>> On Tue, Oct 14, 2008 at 10:06 PM, Dan Williams <[email protected]> wrote:
>> > On Mon, 2008-10-13 at 16:22 +0530, Manish Katiyar wrote:
>> >> Below patch removes the following warning during compilation.
>> >>
>> >> drivers/net/wireless/libertas/cmd.c:826: warning: unused variable 'old_channel'
>> >>
>> >>
>> >> Signed-off-by : Manish Katiyar <[email protected]>
>> >
>> > Acked-by: Dan Williams <[email protected]>
>>
>> Hi Dan,
>>
>> A newbie question. Will you merge this patch or will I have to send it
>> to Andrew again acked by you ?
>
> John has already applied it, Andrew isn't typically involved with
> wireless.
Thanks a lot John.
>
> johannes
>
On Wed, Oct 15, 2008 at 02:31:37PM +0530, Manish Katiyar wrote:
> On Wed, Oct 15, 2008 at 2:17 PM, Johannes Berg
> <[email protected]> wrote:
> > [dropping libertas list, subscribers only]
> >
> > On Wed, 2008-10-15 at 13:40 +0530, Manish Katiyar wrote:
> >> On Tue, Oct 14, 2008 at 10:06 PM, Dan Williams <[email protected]> wrote:
> >> > On Mon, 2008-10-13 at 16:22 +0530, Manish Katiyar wrote:
> >> >> Below patch removes the following warning during compilation.
> >> >>
> >> >> drivers/net/wireless/libertas/cmd.c:826: warning: unused variable 'old_channel'
> >> >>
> >> >>
> >> >> Signed-off-by : Manish Katiyar <[email protected]>
> >> >
> >> > Acked-by: Dan Williams <[email protected]>
> >>
> >> Hi Dan,
> >>
> >> A newbie question. Will you merge this patch or will I have to send it
> >> to Andrew again acked by you ?
> >
> > John has already applied it, Andrew isn't typically involved with
> > wireless.
>
> Thanks a lot John.
No, thank you! :-)
--
John W. Linville Linux should be at the core
[email protected] of your literate lifestyle.
[dropping libertas list, subscribers only]
On Wed, 2008-10-15 at 13:40 +0530, Manish Katiyar wrote:
> On Tue, Oct 14, 2008 at 10:06 PM, Dan Williams <[email protected]> wrote:
> > On Mon, 2008-10-13 at 16:22 +0530, Manish Katiyar wrote:
> >> Below patch removes the following warning during compilation.
> >>
> >> drivers/net/wireless/libertas/cmd.c:826: warning: unused variable 'old_channel'
> >>
> >>
> >> Signed-off-by : Manish Katiyar <[email protected]>
> >
> > Acked-by: Dan Williams <[email protected]>
>
> Hi Dan,
>
> A newbie question. Will you merge this patch or will I have to send it
> to Andrew again acked by you ?
John has already applied it, Andrew isn't typically involved with
wireless.
johannes
On Mon, 2008-10-13 at 16:22 +0530, Manish Katiyar wrote:
> Below patch removes the following warning during compilation.
>
> drivers/net/wireless/libertas/cmd.c:826: warning: unused variable 'old_channel'
>
>
> Signed-off-by : Manish Katiyar <[email protected]>
Acked-by: Dan Williams <[email protected]>
> ---
> drivers/net/wireless/libertas/cmd.c | 2 ++
> 1 files changed, 2 insertions(+), 0 deletions(-)
>
> diff --git a/drivers/net/wireless/libertas/cmd.c
> b/drivers/net/wireless/libertas/cmd.c
> index a912fb6..297696d 100644
> --- a/drivers/net/wireless/libertas/cmd.c
> +++ b/drivers/net/wireless/libertas/cmd.c
> @@ -823,7 +823,9 @@ int lbs_update_channel(struct lbs_private *priv)
> int lbs_set_channel(struct lbs_private *priv, u8 channel)
> {
> struct cmd_ds_802_11_rf_channel cmd;
> +#ifdef DEBUG
> u8 old_channel = priv->curbssparams.channel;
> +#endif
> int ret = 0;
>
> lbs_deb_enter(LBS_DEB_CMD);