Re: [-mm patch] iwl-base.c bugfixes

["Tomas Winkler" <tomasw@gmail.com>]

On 8/28/07, Adrian Bunk <bunk@kernel.org> wrote:
> On Wed, Aug 22, 2007 at 02:06:48AM -0700, Andrew Morton wrote:
> >...
> > Changes since 2.6.23-rc2-mm2:
> >...
> >  git-wireless.patch
> >...
> >  git trees
> >...
>
> This patch fixes two obvious bugs.
>
> Signed-off-by: Adrian Bunk <bunk@kernel.org>
>
> ---
>
>  drivers/net/wireless/iwl-base.c |    4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> 600ffdc11b25ac0aee15271d1b2ce99a367efa31
> diff --git a/drivers/net/wireless/iwl-base.c b/drivers/net/wireless/iwl-base.c
> index b8293fe..f65c30e 100644
> --- a/drivers/net/wireless/iwl-base.c
> +++ b/drivers/net/wireless/iwl-base.c
> @@ -343,7 +343,7 @@ int iwl_tx_queue_init(struct iwl_priv *priv,
>          * command is very huge the system will not have two scan at the
>          * same time */
>         len = sizeof(struct iwl_cmd) * slots_num;
> -       if (txq_id == IWL_CMD_QUEUE_NUM);
> +       if (txq_id == IWL_CMD_QUEUE_NUM)
>                 len +=  IWL_MAX_SCAN_SIZE;
>         txq->cmd = pci_alloc_consistent(dev, len, &txq->dma_addr_cmd);
>         if (!txq->cmd)
> @@ -390,7 +390,7 @@ void iwl_tx_queue_free(struct iwl_priv *priv, struct iwl_tx_queue *txq)
>                 iwl_hw_txq_free_tfd(priv, txq);
>
>         len = sizeof(struct iwl_cmd) * q->n_window;
> -       if (q->id == IWL_CMD_QUEUE_NUM);
> +       if (q->id == IWL_CMD_QUEUE_NUM)
>                 len += IWL_MAX_SCAN_SIZE;
>
>         pci_free_consistent(dev, len, txq->cmd, txq->dma_addr_cmd);
>
>
Shame on me.
Thanks
> To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>
-
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/